Remove unused parameter
authoreileen <emcnaughton@wikimedia.org>
Mon, 25 Nov 2019 03:46:59 +0000 (16:46 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 25 Nov 2019 21:40:33 +0000 (10:40 +1300)
This parameter is always TRUE so it adds no value, removing

CRM/Core/Report/Excel.php
CRM/Export/BAO/ExportProcessor.php

index dc0e11f28012ead6efcb2790e233aa148e25792f..46c4943a9379fcc6c415fba0330af8a5a0a6f806 100644 (file)
@@ -27,21 +27,17 @@ class CRM_Core_Report_Excel {
    * @param array $rows
    *   result set rows.
    * @param string $titleHeader
-   * @param bool $print
-   *   Should the output be printed.
    * @param bool $outputHeader
    *
    * @return mixed
    *   empty if output is printed, else output
    *
    */
-  public static function makeCSVTable($header, $rows, $titleHeader = NULL, $print = TRUE, $outputHeader = TRUE) {
+  public static function makeCSVTable($header, $rows, $titleHeader = NULL, $outputHeader = TRUE) {
     if ($titleHeader) {
       echo $titleHeader;
     }
 
-    $result = '';
-
     $config = CRM_Core_Config::singleton();
     $seperator = $config->fieldSeparator;
     $enclosed = '"';
@@ -59,12 +55,7 @@ class CRM_Core_Report_Excel {
       // need to add PMA_exportOutputHandler functionality out here, rather than
       // doing it the moronic way of assembling a buffer
       $out = trim(substr($schema_insert, 0, -1)) . $add_character;
-      if ($print) {
-        echo $out;
-      }
-      else {
-        $result .= $out;
-      }
+      echo $out;
     }
 
     $fields_cnt = count($header);
@@ -110,19 +101,7 @@ class CRM_Core_Report_Excel {
       // end for
 
       $out = $schema_insert . $add_character;
-      if ($print) {
-        echo $out;
-      }
-      else {
-        $result .= $out;
-      }
-    }
-
-    if ($print) {
-      return;
-    }
-    else {
-      return $result;
+      echo $out;
     }
   }
 
@@ -189,7 +168,7 @@ class CRM_Core_Report_Excel {
     }
 
     if (!empty($rows)) {
-      return self::makeCSVTable($header, $rows, $titleHeader, TRUE, $outputHeader);
+      return self::makeCSVTable($header, $rows, $titleHeader, $outputHeader);
     }
   }
 
index cba408b5662084ff878adc85ed8dd5eb0ddc0805..5fda71b013b9d24b3cb81c0eb1a731ba2e500a4e 100644 (file)
@@ -2387,7 +2387,7 @@ LIMIT $offset, $limit
       'csv',
       FALSE
     );
-    CRM_Core_Report_Excel::makeCSVTable($headerRows, [], NULL, TRUE, TRUE);
+    CRM_Core_Report_Excel::makeCSVTable($headerRows, [], NULL, TRUE);
   }
 
   /**