Silent cleanup of picklist items to align with MULTIVALUED_DELIMITER

cidgoh · Jan 3, 2025 · 00d026f · 00d026f
1 parent 96e7dd7
commit 00d026f
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/lib/DataHarmonizer.js b/lib/DataHarmonizer.js
@@ -2781,9 +2781,19 @@ class DataHarmonizer {
 
           if (datatype === 'Provenance') {
             checkProvenance(cellChanges, fullVersion, cellVal, row, col);
-          } else if (field.flatVocabulary) {
+          } 
+          // Clean up capitalization and spacing.  This applies to both single 
+          // and multivalued
+          else if (field.flatVocabulary) {
             if (field.multivalued === true) {
-              const [, update] = validateValsAgainstVocab(cellVal, field);
+              let [, update] = validateValsAgainstVocab(cellVal, field);
+              if (!update) {
+                // Inefficient 2nd check: problem is cellVal containing only
+                // caps or non-trimmed items doesn't trigger quiet update
+                const update_val = formatMultivaluedValue(parseMultivaluedValue(cellVal));
+                if (cellVal != update_val)
+                  update = update_val;
+              }
               if (update) {
                 data[row][col] = update;
                 cellChanges.push([row, col, update, 'thisChange']);