I've been receiving the following respective warnings in drush_hosting_task() whenever verifying any site, platform, or server:
Parameter 1 to hosting_hosting_site_context_options() expected to be a reference, value given task.hosting.inc:130
Parameter 1 to hosting_hosting_platform_context_options() expected to be a reference, value given task.hosting.inc:130
Parameter 1 to hosting_hosting_server_context_options() expected to be a reference, value given task.hosting.inc:130
I've verified that on just-installed instance of Aegir, the frontend-triggered verify tasks generate these warnings (even on the verify tasks that pre-populate the task queue created by the hostmaster-install drush command).
If additional Aegir modules, like hosting_site_data or hosting_http_basic_auth, are enabled, you get additional similar warnings, for example, for hosting_http_basic_auth_hosting_site_context_options() and hosting_site_data_hosting_site_context_options(), respectively.
Comment | File | Size | Author |
---|---|---|---|
#2 | hosting-hook_hosting_TASK_OBJECT_context_options-2846897-2.patch | 606 bytes | JamesAn |
Comments
Comment #2
JamesAn CreditAttribution: JamesAn as a volunteer commentedThe following patch follows a PHP.net documentation example found on the call_user_func() page demonstrating how to pass a variable by reference using call_user_func_array() instead.
All occurrences of the warning appear to be resolved with this small change.
Comment #4
helmo CreditAttribution: helmo at Initfour websolutions commentedThanks, committed.