fix to bin update not working on numeric age unit change

Author: ddooley

lib/DataHarmonizer.js

@@ -2435,6 +2435,12 @@ class DataHarmonizer {
    */
 
   /**
+   * fieldChangeRules() is triggered when user changes a value on a cell,
+   * before that change is committed.
+   *  
+   * IMPORTANT: Value of cell user is changing is in change[3], not in
+   * this.hot.getDataAtCell(row, col) - the getDataAtCell() is old value.
+   * 
    * Iterate through rules set up for named columns
    * Like matrixFieldChangeRules but this is triggered by a single change
    * by a user edit on a field cell. This creates complexity for fields that
@@ -2481,13 +2487,13 @@ class DataHarmonizer {
           // Don't attempt to reformat x/y/z dates here.
           const dateGranularity = this.hot.getDataAtCell(row, col + 1).split(' ')[0];
           // previously had to block x/y/z with change[3].indexOf('/') === -1 &&
+          // ISSUE: not multilingual?
           if (dateGranularity === 'year' || dateGranularity === 'month') {
             change[3] = this.setDateChange(dateGranularity, change[3]);
           }
           return;
         }
 
-        /* Deprecating this. Users need to change unit to trigger bin
         // Match <field> [field]_unit [field]_bin
         const nextNextName =
           fields.length > col + 2 ? fields[col + 2].name : null;
@@ -2497,7 +2503,6 @@ class DataHarmonizer {
           this.binChangeTest(matrix, col, fields, 2, triggered_changes);
           return;
         }
-        */
       }
 
       // On [field] here, where next column is "[field]_bin". 
@@ -2513,7 +2518,7 @@ class DataHarmonizer {
           console.log("on", prevName + '_unit')
           // trigger reevaluation of bin from field
           matrixRow[col - 1] = this.hot.getDataAtCell(row, col - 1);
-          matrixRow[col] = this.hot.getDataAtCell(row, col);
+          matrixRow[col] = change[3];
           this.binChangeTest(matrix, col - 1, fields, 2, triggered_changes);
           return;
         }