Remove outputHeader as a param for writeCSVFile as it is always true

[eileenmcnaughton]

Overview

Removes a meaningless parameter

Before

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

After

public static function writeCSVFile($fileName, $header, $rows)

Technical Details

The calls to this function do not pass the last parameter

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).

[eileenmcnaughton]

test this please

[demeritcowboy]

This looks ok to me just running a test since it's been a week, which is ages in civi-time. It's possible there's an extension that uses it. I've never downloaded the "fetch universe" - this might be a good one to grep writeCSVFile on and see what comes up. Jenkins test this please.

[mattwire]

I don't quite have the universe locally but I'm close - and I can't see it used anywhere so marking merge on pass based on @demeritcowboy review.