LP1929741 Sanity check distrib. formula values

No fields within a distribution formula are required, so confirm a set
of values exists before calling Object.keys() on an undefined value.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

diff --git a/Open-ILS/src/eg2/src/app/staff/acq/lineitem/copies.component.ts b/Open-ILS/src/eg2/src/app/staff/acq/lineitem/copies.component.ts
index efc64c6..3ad72df 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/acq/lineitem/copies.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/acq/lineitem/copies.component.ts
@@ -171,9 +171,14 @@ export class LineitemCopiesComponent implements OnInit, AfterViewInit {
     // are not required to go fetch them en masse / en duplicato.
     fetchFormulaValues(): Promise<any> {
 
-        const funds = Object.keys(this.formulaValues.fund);
-        const mods = Object.keys(this.formulaValues.circ_modifier);
-        const locs = Object.keys(this.formulaValues.location);
+        const funds = this.formulaValues.fund ?
+            Object.keys(this.formulaValues.fund) : [];
+
+        const mods = this.formulaValues.circ_modifier ?
+            Object.keys(this.formulaValues.circ_modifier) : [];
+
+        const locs = this.formulaValues.location ?
+            Object.keys(this.formulaValues.location) : [];
 
         let promise = Promise.resolve();