From ce7e490507620f8d712c5edf0a12cb0010ef0839 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Fri, 16 Nov 2018 15:52:29 -0500
Subject: [PATCH] place holds continued

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 .../src/app/staff/catalog/hold/hold.component.html | 15 +++--
 .../src/app/staff/catalog/hold/hold.component.ts   | 74 +++++++++++++++++-----
 .../app/staff/catalog/result/record.component.ts   |  3 +-
 3 files changed, 70 insertions(+), 22 deletions(-)

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 f0677be137..7f403ffd81 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
@@ -6,6 +6,7 @@
     <div class="col-lg-3">
       <div class="form-check">
         <input class="form-check-input" type="radio" 
+          (change)="holdForChanged()"
           name="holdFor" value="patron" [(ngModel)]="holdFor"/>
         <label class="form-check-label" i18n>
           Place hold for patron by barcode:
@@ -27,6 +28,7 @@
     <div class="col-lg-3">
       <div class="form-check">
         <input class="form-check-input" type="radio" 
+          (change)="holdForChanged()"
           name="holdFor" value="staff" [(ngModel)]="holdFor"/>
         <label class="form-check-label" i18n>
           Place hold for this staff account: {{requestor.usrname()}}
@@ -94,13 +96,18 @@
   </div>
 </form>
 
-<h4 class="p-3" i18n>Placing Hold(s) On Records...</h4>
-
+<div class="row font-weight-bold mt-2 ml-1 mr-1">
+  <div class="col-lg-10" i18n>Placing hold(s) on record(s)</div>
+  <div class="col-lg-2" i18n>Holds Status</div>
+</div>
 <ng-container *ngFor="let recId of recordIds">
-  <div class="row mt-2 ml-2 mr-2">
-    <div class="col-lg-12">
+  <div class="row mt-2 ml-1 mr-1">
+    <div class="col-lg-10">
       <eg-bib-summary [recordId]="recId" [expand]="false"></eg-bib-summary> 
     </div>
+    <div class="col-lg-2">
+      <div class="alert alert-info">Hold Pending</div>
+    </div>
   </div>
 </ng-container>
 
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 4d86428336..f31dd35f89 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
@@ -23,12 +23,13 @@ export class HoldComponent implements OnInit {
     pickupLib: number;
     notifyEmail: boolean;
     notifyPhone: boolean;
+    notifySms: boolean;
     phoneValue: string;
     suspend: boolean;
     activeDate: string;
     recordIds: number[];
 
-    barcodeInFlight: string;
+    currentUserBarcode: string;
 
     constructor(
         private router: Router,
@@ -52,7 +53,7 @@ export class HoldComponent implements OnInit {
         }
 
         this.holdFor = 'patron';
-        this.requestor = this.user = this.auth.user();
+        this.requestor = this.auth.user();
         this.pickupLib = this.auth.user().ws_ou();
         this.findRecords();
 
@@ -72,10 +73,12 @@ export class HoldComponent implements OnInit {
         console.log('placing hold for ' + this.holdFor);
 
         if (this.holdFor === 'patron') {
-            // Patron lookup occurs via onchange on the barcode
-            // input or via the patron search form.
+            if (this.userBarcode) {
+                this.userBarcodeChanged();
+            }
         } else {
-            this.user = this.requestor;
+            this.currentUserBarcode = '_'; // just for preventing dupes
+            this.getUser(this.requestor.id());
         }
     }
 
@@ -84,17 +87,17 @@ export class HoldComponent implements OnInit {
     }
 
     userBarcodeChanged() {
-        this.user = null;
 
+        // Avoid simultaneous or duplicate lookups
+        if (this.userBarcode === this.currentUserBarcode) { 
+            return; 
+        }
+
+        this.resetForm();
         if (!this.userBarcode) { return; }
 
-        // Avoid simultaneous lookups caused by firing the
-        // keyup handler and change handler in quick succession.
-        // keyup handler applied so barcode scanner will result
-        // in immediate lookup.  change handler for humans 
-        // copy/paste'ing then tabbing through the form or off-clicking.
-        if (this.userBarcode === this.barcodeInFlight) { return; }
-        this.barcodeInFlight = this.userBarcode;
+        this.user = null;
+        this.currentUserBarcode = this.userBarcode;
 
         this.net.request(
             'open-ils.actor',
@@ -117,12 +120,49 @@ export class HoldComponent implements OnInit {
         });
     }
 
+    resetForm() {
+        this.notifyEmail = true;
+        this.notifyPhone = true;
+        this.phoneValue = '';
+        this.pickupLib = this.requestor.ws_ou();
+    }
+
     getUser(id: number) {
-        // TODO fetch user settings for PU lib, etc.
-        this.pcrud.retrieve('au', id).subscribe(user => {
+        this.pcrud.retrieve('au', id, {flesh: 1, flesh_fields: {au: ['settings']}})
+        .subscribe(user => {
             this.user = user;
-            this.notifyPhone = user.day_phone() || user.evening_phone();
-            this.barcodeInFlight = null;
+            this.applyUserSettings();
+        });
+    }
+
+    applyUserSettings() {
+        if (!this.user || !this.user.settings()) { return; }
+
+        // Start with defaults.
+        this.phoneValue = this.user.day_phone() || this.user.evening_phone();
+
+        // Default to work org if placing holds for staff.
+        if (this.user.id() !== this.requestor.id()) {
+            this.pickupLib = this.user.home_ou();
+        }
+
+        this.user.settings().forEach(setting => {
+            const name = setting.name();
+            const value = setting.value();
+
+            if (value === '' || value === null) { return; }
+
+            switch(name) {
+                case 'opac.hold_notify':
+                    this.notifyPhone = Boolean(value.match(/phone/));
+                    this.notifyEmail = Boolean(value.match(/email/));
+                    this.notifySms = Boolean(value.match(/sms/));
+                    break;
+
+                case 'opac.default_pickup_location':
+                    this.pickupLib = value; 
+                    break;
+            }
         });
     }
 }
diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
index cb6567f6bd..b99a4a1b05 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
@@ -64,7 +64,8 @@ export class ResultRecordComponent implements OnInit, OnDestroy {
     }
 
     placeHold(): void {
-        alert('Placing hold on bib ' + this.summary.id);
+        this.router.navigate(['/staff/catalog/hold/T'], 
+            {queryParams: {target: this.summary.id}});
     }
 
     addToList(): void {
-- 
2.11.0