From: Bill Erickson <berickxx@gmail.com>
Date: Tue, 28 Jul 2020 15:48:33 +0000 (-0400)
Subject: LP1889128 Staffcat hold form reset option
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=4dd6401ada3489574ca7fff0d47fa8eac08db539;p=working%2FEvergreen.git

LP1889128 Staffcat hold form reset option

Adds a Reset button to the staff catalog holds form so subsequent holds
may be placed for the same items but for different patrons.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.html b/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.html
index 714e3f10cd..2ee4e8825f 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.html
@@ -166,16 +166,16 @@
               </eg-combobox>
             </div>
           </li>
+          <li class="list-group-item">
+            <button class="btn btn-success" (click)="placeHolds()" 
+              [disabled]="!user || placeHoldsClicked" i18n>Place Hold(s)</button>
+
+            <button class="btn btn-outline-dark ml-2" (click)="reset()" i18n>Reset</button>
+          </li>
         </ul><!-- col -->
       </div><!-- row -->
     </div><!--card -->
   </div><!-- col -->
-  <div class="row mt-2">
-    <div class="col-lg-3">
-      <button class="btn btn-success" (click)="placeHolds()" 
-        [disabled]="!user || placeHoldsClicked" i18n>Place Hold(s)</button>
-    </div>
-  </div>
 </form>
 
 <div class="row"><div class="col-lg-12"><hr/></div></div>
diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.ts b/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.ts
index f5648d1424..83272c9367 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/hold/hold.component.ts
@@ -101,8 +101,10 @@ export class HoldComponent implements OnInit {
         this.smsCarriers = [];
     }
 
-    ngOnInit() {
+    reset() {
 
+        this.user = null;
+        this.userBarcode = null;
         this.holdType = this.route.snapshot.params['type'];
         this.holdTargets = this.route.snapshot.queryParams['target'];
         this.holdFor = this.route.snapshot.queryParams['holdFor'] || 'patron';
@@ -129,11 +131,19 @@ export class HoldComponent implements OnInit {
             return ctx;
         });
 
+        this.resetForm();
+
         if (this.holdFor === 'staff' || this.userBarcode) {
             this.holdForChanged();
         }
 
         this.getTargetMeta();
+        this.placeHoldsClicked = false;
+    }
+
+    ngOnInit() {
+
+        this.reset();
 
         this.org.settings(['sms.enable', 'circ.holds.max_duplicate_holds'])
         .then(sets => {