(NFC) Correct type hints for bad null default values

[braders]

Overview

Correct type hints for bad null default values.

I've been trying to get rid of cases where the phpdoc has been defaulted to null, despite an actual value being expected in most cases (e.g. @param null $title, where $title is not always expected to be null). These were added in bulk based on the default argument several years ago.

Before

Bad types in PHPDoc comments.

After

Fewer bad docs (there are still a few other tricker examples of this which should be tackled seperately).

Comments

I've just been searching for @param null and working through the list. In some cases the default value is null but I've not included null in the PHPDoc because in practice a proper value is always expected. These could be tidied up at a later date (e.g. to change the default value).

[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]

👍