From: Bill Erickson <berickxx@gmail.com>
Date: Mon, 6 Jul 2020 15:03:21 +0000 (-0400)
Subject: LPXXX Angular Volcopy
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=fc80ad54f28fad59822d26c5f6c67bc7028aeb0e;p=working%2FEvergreen.git

LPXXX Angular Volcopy

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

diff --git a/Open-ILS/src/eg2/src/app/staff/cat/volcopy/config.component.html b/Open-ILS/src/eg2/src/app/staff/cat/volcopy/config.component.html
index 8125e57a1d..031561a806 100644
--- a/Open-ILS/src/eg2/src/app/staff/cat/volcopy/config.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/cat/volcopy/config.component.html
@@ -141,15 +141,15 @@
         </li>
         <li class="list-group-item">
           <div class="form-check form-check-inline">
-            <label class="mr-2" for="statcat_filter" i18n>
-              Stat Cat Library Filter
-            </label>
             <eg-org-select
               domId="statcat_filter"
               placeholder="Stat Cat Filter..." i18n-placeholder
               [initialOrgId]="volcopy.defaults.values.statcat_filter"
               (onChange)="volcopy.defaults.values.statcat_filter = $event ? $event.id() : null">
             </eg-org-select>
+            <label class="ml-2" for="statcat_filter" i18n>
+              Default Stat Cat Library Filter
+            </label>
           </div>
         </li>
       </ul>
diff --git a/Open-ILS/src/eg2/src/app/staff/cat/volcopy/copy-attrs.component.ts b/Open-ILS/src/eg2/src/app/staff/cat/volcopy/copy-attrs.component.ts
index 1b2d109dd9..6bdda44263 100644
--- a/Open-ILS/src/eg2/src/app/staff/cat/volcopy/copy-attrs.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/cat/volcopy/copy-attrs.component.ts
@@ -336,11 +336,19 @@ export class CopyAttrsComponent implements OnInit, AfterViewInit {
 
         const entryId = this.statCatValues[catId];
 
+        if (!entryId || !this.volcopy.statCatEntryMap[entryId]) {
+            console.warn(
+                `Attempt to apply stat cat value which does not exist.
+                This is likely the result of a stale copy template.
+                stat_cat=${catId} entry=${entryId}`);
+
+            return;
+        }
+
         this.context.copyList().forEach(copy => {
 
             let entry = copy.stat_cat_entries()
                 .filter(e => e.stat_cat() === catId)[0];
-                console.log('0', entry);
 
             if (entry) {
                 if (entry.id() === entryId) {