[REF] Remove titleHeader var

[eileenmcnaughton]

Overview

Removes $titleHeader param that is never passed in, and, logically, it would break the csv if it was

Before

 
 public static function writeCSVFile($fileName, $header, $rows, $titleHeader = NULL, $outputHeader = TRUE) {

• called from 3 places but only 1 passes more than the first 3 params & titleHeader is set to NULL

public static function makeCSVTable($header, $rows, $titleHeader = NULL, $outputHeader = TRUE) {

• called from 2 places. One is writeCSVFile which only passes NULL for $titleHeader and the other place was passing in NULL

After

public static function writeCSVFile($fileName, $header, $rows, $outputHeader = TRUE) {

 public static function makeCSVTable($header, $rows, $outputHeader = TRUE) {

Technical Details

Minor code cleanup - no impact other than readability

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[colemanw]

Makes sense.