Remove IF as always true
authoreileen <emcnaughton@wikimedia.org>
Thu, 24 Oct 2019 02:40:06 +0000 (15:40 +1300)
committereileen <emcnaughton@wikimedia.org>
Thu, 24 Oct 2019 02:40:06 +0000 (15:40 +1300)
CRM/Dedupe/Merger.php

index e513265d807da200ba7bf739747f30e72fec1858..5cdab873aff0fa3bf86bb48c5ed07d8d5c638e57 100644 (file)
@@ -1095,34 +1095,32 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
       $rows["move_$field"]['other'] = self::getFieldValueAndLabel($field, $other)['label'];
 
       foreach (['other' => $other] as $moniker => $contact) {
-        if ($moniker == 'other') {
-          $value = self::getFieldValueAndLabel($field, $other)['value'];
-          //CRM-14334
-          if ($value === NULL || $value == '') {
-            $value = 'null';
-          }
-          if ($value === 0 or $value === '0') {
-            $value = $qfZeroBug;
-          }
-          if (is_array($value) && empty($value[1])) {
-            $value[1] = NULL;
-          }
+        $value = self::getFieldValueAndLabel($field, $other)['value'];
+        //CRM-14334
+        if ($value === NULL || $value == '') {
+          $value = 'null';
+        }
+        if ($value === 0 or $value === '0') {
+          $value = $qfZeroBug;
+        }
+        if (is_array($value) && empty($value[1])) {
+          $value[1] = NULL;
+        }
 
-          // Display a checkbox to migrate, only if the values are different
-          if ($value != $main[$field]) {
-            $elements[] = [
-              'advcheckbox',
-              "move_$field",
-              NULL,
-              NULL,
-              NULL,
-              $value,
-            ];
-          }
+        // Display a checkbox to migrate, only if the values are different
+        if ($value != $main[$field]) {
+          $elements[] = [
+            'advcheckbox',
+            "move_$field",
+            NULL,
+            NULL,
+            NULL,
+            $value,
+          ];
+        }
 
-          $migrationInfo["move_$field"] = $value;
+        $migrationInfo["move_$field"] = $value;
         }
-      }
       $rows["move_$field"]['title'] = $fields[$field]['title'];
     }