Standardise Type on includedGroups parameter

[eileenmcnaughton]

Overview

Standardise Type on includedGroups parameter

Before

I had to read through the function to determine that passing NULL or [] for $includedGroups had the same outcome (because it is only used if !empty($includedGroups)

After

Parameter standardised as an array, NULL deprecated

Technical Details

The deprecation notice might cause some test fails, which I will address if so

Comments

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[demeritcowboy]

• General standards
  • [r-explain] NULL
  • [r-user] NULL
  • [r-doc] array
  • [r-run] (array) NULL
• Developer standards
  • [r-tech] 'null'
  • [r-code] array
  • [r-maint] NULL
  • [r-test] NULL