From: Bill Erickson <berickxx@gmail.com>
Date: Wed, 30 Jun 2021 17:46:45 +0000 (-0400)
Subject: LP1934164 Grid context/duration date minor refactoring
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=13705eb22b771373df12075186de57596c474992;p=working%2FEvergreen.git

LP1934164 Grid context/duration date minor refactoring

Move some repeated code into a shared method.

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

diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid.ts b/Open-ILS/src/eg2/src/app/share/grid/grid.ts
index fdcc81bea6..ca3a766dc1 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid.ts
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid.ts
@@ -933,14 +933,16 @@ export class GridContext {
         ).length > 0;
     }
 
-    getRowColumnValue(row: any, col: GridColumn): string {
-        let val;
-
+    getRowColumnBareValue(row: any, col: GridColumn): any {
         if (col.name in row) {
-            val = this.getObjectFieldValue(row, col.name);
+            return this.getObjectFieldValue(row, col.name);
         } else if (col.path) {
-            val = this.nestedItemFieldValue(row, col);
+            return this.nestedItemFieldValue(row, col);
         }
+    }
+
+    getRowColumnValue(row: any, col: GridColumn): any {
+        const val = this.getRowColumnBareValue(row, col);
 
         if (col.datatype === 'bool') {
             // Avoid string-ifying bools so we can use an <eg-bool/>
@@ -948,18 +950,13 @@ export class GridContext {
             return val;
         }
 
-        // Get the value of
         let interval;
         const intField = col.dateOnlyIntervalField;
         if (intField) {
-            if (intField in row) {
-                interval = this.getObjectFieldValue(row, intField);
-            } else  {
-                // find the referenced column
-                const intCol = this.columnSet.columns.filter(c => c.path === intField)[0];
-                if (intCol) {
-                    interval = this.nestedItemFieldValue(row, intCol);
-                }
+            const intCol =
+                this.columnSet.columns.filter(c => c.path === intField)[0];
+            if (intCol) {
+                interval = this.getRowColumnBareValue(row, intCol);
             }
         }