From ae12e3c0f20d3d1c8d046ed8d03784f7ae816f08 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Thu, 24 Sep 2020 15:53:16 -0400
Subject: [PATCH] LP1888723 Add/Manage item notes from catalog holdings

Adds an entry point for adding and managing Item Notes from the Angular
staff catalog holdings view.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 .../app/staff/catalog/record/holdings.component.html |  8 +++++++-
 .../app/staff/catalog/record/holdings.component.ts   | 20 +++++++++++++++++++-
 2 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.html b/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.html
index c1355c1936..31b81e35be 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.html
@@ -48,6 +48,7 @@
 <eg-mark-missing-dialog #markMissingDialog></eg-mark-missing-dialog>
 <eg-copy-alerts-dialog #copyAlertsDialog></eg-copy-alerts-dialog>
 <eg-copy-tags-dialog #copyTagsDialog></eg-copy-tags-dialog>
+<eg-copy-notes-dialog #copyNotesDialog></eg-copy-notes-dialog>
 <eg-replace-barcode-dialog #replaceBarcode></eg-replace-barcode-dialog>
 <eg-delete-holding-dialog #deleteHolding></eg-delete-holding-dialog>
 <eg-bucket-dialog #bucketDialog></eg-bucket-dialog>
@@ -127,6 +128,11 @@
     </eg-grid-toolbar-action>
 
     <eg-grid-toolbar-action
+      i18n-group group="Add" i18n-label label="Add/Manage Item Notes"
+      (onClick)="openItemNotes($event)">
+    </eg-grid-toolbar-action>
+
+    <eg-grid-toolbar-action
       i18n-group group="Add" i18n-label label="Add Items To Bucket"
       (onClick)="openBucketDialog($event)">
     </eg-grid-toolbar-action>
@@ -167,7 +173,7 @@
     
     <eg-grid-toolbar-action
       i18n-group group="Edit" i18n-label label="Edit Item Alerts"
-      (onClick)="openItemNotes($event, 'manage')">
+      (onClick)="openItemAlerts($event, 'manage')">
     </eg-grid-toolbar-action>
 
     <eg-grid-toolbar-action
diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.ts b/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.ts
index 1df6abde05..5f17c5740c 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/record/holdings.component.ts
@@ -25,6 +25,8 @@ import {CopyAlertsDialogComponent
     } from '@eg/staff/share/holdings/copy-alerts-dialog.component';
 import {CopyTagsDialogComponent
     } from '@eg/staff/share/holdings/copy-tags-dialog.component';
+import {CopyNotesDialogComponent
+    } from '@eg/staff/share/holdings/copy-notes-dialog.component';
 import {ReplaceBarcodeDialogComponent
     } from '@eg/staff/share/holdings/replace-barcode-dialog.component';
 import {DeleteHoldingDialogComponent
@@ -110,6 +112,8 @@ export class HoldingsMaintenanceComponent implements OnInit {
         private copyAlertsDialog: CopyAlertsDialogComponent;
     @ViewChild('copyTagsDialog', {static: false})
         private copyTagsDialog: CopyTagsDialogComponent;
+    @ViewChild('copyNotesDialog', {static: false})
+        private copyNotesDialog: CopyNotesDialogComponent;
     @ViewChild('replaceBarcode', { static: true })
         private replaceBarcode: ReplaceBarcodeDialogComponent;
     @ViewChild('deleteHolding', { static: true })
@@ -888,7 +892,7 @@ export class HoldingsMaintenanceComponent implements OnInit {
         }
     }
 
-    openItemNotes(rows: HoldingsEntry[], mode: string) {
+    openItemAlerts(rows: HoldingsEntry[], mode: string) {
         const copyIds = this.selectedCopyIds(rows);
         if (copyIds.length === 0) { return; }
 
@@ -917,6 +921,20 @@ export class HoldingsMaintenanceComponent implements OnInit {
         );
     }
 
+    openItemNotes(rows: HoldingsEntry[]) {
+        const copyIds = this.selectedCopyIds(rows);
+        if (copyIds.length === 0) { return; }
+
+        this.copyNotesDialog.copyIds = copyIds;
+        this.copyNotesDialog.open({size: 'lg'}).subscribe(
+            modified => {
+                if (modified) {
+                    this.hardRefresh();
+                }
+            }
+        );
+    }
+
     openReplaceBarcodeDialog(rows: HoldingsEntry[]) {
         const ids = this.selectedCopyIds(rows);
         if (ids.length === 0) { return; }
-- 
2.11.0