Update Merger.php to compare strict type for checkboxes where a dummy-string is being...
authorwfan <wfan@wikimedia.org>
Thu, 16 Nov 2023 00:04:07 +0000 (16:04 -0800)
committerwfan <wfan@wikimedia.org>
Thu, 16 Nov 2023 00:04:07 +0000 (16:04 -0800)
CRM/Dedupe/Merger.php

index d4cb2f7c616b37f02a3c7292b9e73c57029d816f..08852897a8154a04b41d690c9016efee22193df9 100644 (file)
@@ -1561,8 +1561,8 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
         $value[1] = NULL;
       }
 
-      // Display a checkbox to migrate, only if the values are different
-      if ($value != $main[$field]) {
+      // Display a checkbox to migrate, only if the values are different, should check type also for true value
+      if ($value !== $main[$field]) {
         // Don't check source if main is empty, because the source of the other contact is not the source of the merged contact
         $isChecked = ($field === 'source') ? FALSE : (!isset($main[$field]) || $main[$field] === '');
         $elements[] = [