Not sure where they come from, but I am seeing the following PHP
warnings in httpd log, whenever I visit the index page.
I did not see those with beta11.
Nothing strange in watchdog.
I am running PHP5

Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of subscriptions_content_nodeapi(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. in subscriptions_content.module on line 524

Comments

salvis’s picture

Status: Active » Needs review

Yes, thanks. Just remove the ampersands on lines 524 and 525 of subscriptions_content.module.

crocodyl’s picture

Thanks, removing the ampersands (this symbol: &) did the trick. I will post my full error below so that others can easily find this page on a keyword search when upgrading to the newest version of the subscriptions module in Drupal 5.

Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of subscriptions_content_nodeapi(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in yoursite.com/modules/subscriptions/subscriptions_content.module on line 524

Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of subscriptions_content_nodeapi(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in yoursite.com/modules/subscriptions/subscriptions_content.module on line 525
gustav’s picture

Why is this marked "patch (code needs review)"? I can't see a patch.

salvis’s picture

Status: Needs review » Active

Sorry, I mis-used the flag to get feedback on whether the proposed edits fixed the problem.

develcuy’s picture

I'm seeing this bug in og issue queue and fivestar... is this related to: #243394: Warning: Call-time pass-by-reference has been deprecated; ? a problem with particular server configuration maybe?

skizzo’s picture

Removing the ampersand fixed the problem for me too.
I am also running og.module (but no fivestar.module).

salvis’s picture

@develCuy: I'm not sure what you mean. Yes, it's very likely the same syntax error in different modules, and old PHP versions may still accept it because it used to be legal syntax. Other than that there's no connection.

develcuy’s picture

@salvis, Yes, I refer to that relation: PHP syntax. I was consternated if this is a bug in core, but now it seems to be PHP. Thank you.

Blessings!

salvis’s picture

Version: 5.x-2.0-beta12 » 5.x-2.0-rc1
Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.