CRM-20157 Code cleanup - remove unused & unuseful variables in dedupe classes
authoreileen <emcnaughton@wikimedia.org>
Wed, 22 Feb 2017 04:30:30 +0000 (17:30 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 27 Feb 2017 21:58:13 +0000 (10:58 +1300)
CRM/Core/BAO/PrevNextCache.php
CRM/Dedupe/Merger.php

index ed1e367bd41d28c9e36183937e45361abc1fdd99..b6e19a8c6b4158a7f008157290cba33cf9e1b0b6 100644 (file)
@@ -282,7 +282,7 @@ FROM   civicrm_prevnext_cache pn
 
       if (!empty($select)) {
         $extraData = array();
-        foreach ($select as $dfield => $sfield) {
+        foreach ($select as $sfield) {
           $extraData[$sfield]  = $dao->$sfield;
         }
         $main[$count] = array(
index 1905a85fdc0c4669c47669b95db1749e4046f0b6..14a18d4775b91624951057d1d1c14e6b3d5cb20d 100644 (file)
@@ -1165,7 +1165,7 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
         $values = civicrm_api3($blockName, 'get', $searchParams);
         if ($values['count']) {
           $cnt = 0;
-          foreach ($values['values'] as $index => $value) {
+          foreach ($values['values'] as $value) {
             $locations[$moniker][$blockName][$cnt] = $value;
             // Fix address display
             if ($blockName == 'address') {
@@ -1519,10 +1519,9 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
         CRM_Core_DAO::$_nullObject, NULL, -1
       );
     }
-    $cgTree = &$treeCache[$migrationInfo['main_details']['contact_type']];
 
     $cFields = array();
-    foreach ($cgTree as $key => $group) {
+    foreach ($treeCache[$migrationInfo['main_details']['contact_type']] as $key => $group) {
       if (!isset($group['fields'])) {
         continue;
       }