Export - don't pass arrays as reference when they are not changed
authoreileen <emcnaughton@wikimedia.org>
Mon, 30 Jul 2018 11:11:48 +0000 (23:11 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 30 Jul 2018 11:11:48 +0000 (23:11 +1200)
CRM/Export/BAO/Export.php

index c6bd9c9279e73cbf73077df869a91fd51fa15f78..3db3c72f98e0d7bedc02e15e9324514818b76ed9 100644 (file)
@@ -796,7 +796,7 @@ INSERT INTO {$componentTable} SELECT distinct gc.contact_id FROM civicrm_group_c
    * @param $details
    * @param $sqlColumns
    */
-  public static function writeDetailsToTable($tableName, &$details, &$sqlColumns) {
+  public static function writeDetailsToTable($tableName, $details, $sqlColumns) {
     if (empty($details)) {
       return;
     }
@@ -813,10 +813,10 @@ FROM   $tableName
 
     $sqlClause = array();
 
-    foreach ($details as $dontCare => $row) {
+    foreach ($details as $row) {
       $id++;
       $valueString = array($id);
-      foreach ($row as $dontCare => $value) {
+      foreach ($row as $value) {
         if (empty($value)) {
           $valueString[] = "''";
         }
@@ -843,7 +843,7 @@ VALUES $sqlValueString
    *
    * @return string
    */
-  public static function createTempTable(&$sqlColumns) {
+  public static function createTempTable($sqlColumns) {
     //creating a temporary table for the search result that need be exported
     $exportTempTable = CRM_Utils_SQL_TempTable::build()->setDurable()->setCategory('export')->getName();