LP1904036 Patron 'Done' button clears hold recipient

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

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/catalog.service.ts b/Open-ILS/src/eg2/src/app/staff/catalog/catalog.service.ts
index 51a92ed..24d32df 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/catalog/catalog.service.ts
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/catalog.service.ts
@@ -89,6 +89,9 @@ export class StaffCatalogService {
                 this.holdForUser = user;
                 this.holdForChange.emit();
             });
+        } else {
+            // In case the session item was cleared from another component.
+            this.clearHoldPatron();
         }
 
         this.searchContext.org = this.org; // service, not searchOrg

diff --git a/Open-ILS/src/eg2/src/app/staff/circ/patron/checkout.component.ts b/Open-ILS/src/eg2/src/app/staff/circ/patron/checkout.component.ts
index f8a1fe3..7d40790 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/circ/patron/checkout.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/circ/patron/checkout.component.ts
@@ -394,8 +394,10 @@ export class CheckoutComponent implements OnInit, AfterViewInit {
     }
 
     doneRedirect() {
+        // Clear the assumed hold recipient since we're done with
+        // this patron.
+        this.store.removeLoginSessionItem('eg.circ.patron_hold_target');
         this.router.navigate(['/staff/circ/patron/bcsearch']);
     }
-
 }