(towards CRM-19492) Do not assign variables by reference
authoreileen <emcnaughton@wikimedia.org>
Thu, 10 Nov 2016 03:49:24 +0000 (16:49 +1300)
committerJKingsnorth <john@johnkingsnorth.co.uk>
Mon, 27 Feb 2017 09:44:49 +0000 (09:44 +0000)
They are not used as references & this just adds confusion.

CRM/Contact/Form/Merge.php

index 503acfcb9d9bab52f00346a84e2735cde6a13b64..cb529b5027b1f1c2dcd5819fdc855110e90d97f2 100644 (file)
@@ -95,7 +95,7 @@ class CRM_Contact_Form_Merge extends CRM_Core_Form {
     }
     $this->assign('browseUrl', $browseUrl);
     if ($browseUrl) {
-      $session->pushUserContext($browseUrl);
+      CRM_Core_Session::singleton()->pushUserContext($browseUrl);
     }
 
     $cacheKey = CRM_Dedupe_Merger::getMergeCacheKeyString($this->_rgid, $gid);
@@ -170,8 +170,8 @@ class CRM_Contact_Form_Merge extends CRM_Core_Form {
     $session = CRM_Core_Session::singleton();
 
     $rowsElementsAndInfo = CRM_Dedupe_Merger::getRowsElementsAndInfo($cid, $oid);
-    $main = $this->_mainDetails = &$rowsElementsAndInfo['main_details'];
-    $other = $this->_otherDetails = &$rowsElementsAndInfo['other_details'];
+    $main = $this->_mainDetails = $rowsElementsAndInfo['main_details'];
+    $other = $this->_otherDetails = $rowsElementsAndInfo['other_details'];
 
     if ($main['contact_id'] != $cid) {
       CRM_Core_Error::fatal(ts('The main contact record does not exist'));