CRM-14334 merge records: allow empty value to overwrite existing value
authorBrian Shaughnessy <brian@lcdservices.biz>
Mon, 7 Apr 2014 19:11:49 +0000 (15:11 -0400)
committerBrian Shaughnessy <brian@lcdservices.biz>
Mon, 7 Apr 2014 19:11:49 +0000 (15:11 -0400)
CRM/Dedupe/Merger.php

index dd046c7640dc18c9226ca81d0e7c3089eb54e66e..3faa547bd407e72c7adbf14326b298712b024792 100644 (file)
@@ -890,7 +890,8 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
         }
         $rows["move_$field"][$moniker] = $label;
         if ($moniker == 'other') {
-          if ($value === NULL) {
+          //CRM-14334
+          if ($value === NULL || $value == '') {
             $value = 'null';
           }
           if ($value === 0 or $value === '0') {
@@ -1185,9 +1186,10 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
       if ($value == $qfZeroBug) {
         $value = '0';
       }
-      if ((in_array(substr($key, 5), CRM_Dedupe_Merger::getContactFields()) or
-          substr($key, 0, 12) == 'move_custom_'
-        ) and $value != NULL) {
+      if ((in_array(substr($key, 5), CRM_Dedupe_Merger::getContactFields()) ||
+        substr($key, 0, 12) == 'move_custom_') &&
+        $value != NULL
+      ) {
         $submitted[substr($key, 5)] = $value;
       }
       elseif (substr($key, 0, 14) == 'move_location_' and $value != NULL) {