[REF] don't handle non-array, pass by reference

[eileenmcnaughton]

Overview

Pretty trivial cleanup

Before

Squirrels dancing

After

Squirrels more subdued

Technical Details

This is only called from one place & it is definitely coming in as an array & using the return values.

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]

One of my squirrels tells me this is "eek ch ch ee eek". I have no idea what that means since I don't speak squirrel but agree with the technical detail assessment above.

I did a couple simple tests of the form and seems good.

Since I assumed this was related to other work involving force=1, I did one test with that (with sort_name), and it sets the form field value. It doesn't return the right results but it didn't before either.

[eileenmcnaughton]

@seamuslee001 does that count as a solid review?

[seamuslee001]

Makes sense merging as per @demeritcowboy 's review