Remove never-passed parameter from now unshared function

[eileenmcnaughton]

Overview

Remove never-passed parameter from now unshared parameter

Before

Function is not only used from this one place (+ a couple of tests) but still accepts a param this function never passed

After

param $formDates removed

Technical Details

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

[demeritcowboy]

This looks ok to me. Civimobile-api looks like it calls this function but they don't pass formDate.

[eileenmcnaughton]

@demeritcowboy that's pretty terrifying that they call this function - I was going to move it but I'll just copy it & deprecate the original then