Running php 5.2.2 on Dreamhost. Please let me know if you need any additional info. Here are the errors:

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 834

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 1417

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 1455

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 1507

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 1513

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of [runtime function name](). 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 /home/drupal/sites/all/modules/webfm/webfm.module on line 1578

CommentFileSizeAuthor
#3 webfm.module.patch2.4 KBandremolnar

Comments

jmlane’s picture

jmlane commented

I am also running on Dreamhost with PHP 5.2.2 and getting this same error after upgrading to 5.x-2.7.

I haven't really had a chance to check out the code and see what the problem is exactly. I may check this out later if I get a chance.

Any thoughts about this?

pyutaros’s picture

pyutaros commented

Well, I've downgraded to 2.6 until I have more time to look at this error. I think the solution will lie in enabling the said function in my php.ini. If you aren't running Drupal on your own customized version of PHP, you may want to check this article out from the Dreamhost wiki. http://wiki.dreamhost.com/PHP.ini I already have used this to increase my php max upload limit, and will hopefully be using it to disable Zend Optimizer, so I can use Devel Themer. The other way to approach this would be to submit a patch to the module that uses a non-deprecated function to perform what this function is doing. That's just a wee more complex, so, I'm going to try the php.ini first. I'll let you know what I find when I have more time to look at this.
Thanks,
Jonathan

andremolnar’s picture

andremolnar commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new webfm.module.patch2.4 KB

Here is a patch until a new release is made.

Ye Olde Fashioned Code got committed by mistake.

robmilne’s picture

robmilne commented
Status: Reviewed & tested by the community » Active

Mea culpa. This is what happens when crossing between languages too often. I mistakenly put ampersands before the arg of the call (ala C# ref) where in PHP it is only required before the arg in the function definition. My test environment is too forgiving and didn't trap this error. I guess version 2.8 will be rolling out earlier than expected.

robmilne’s picture

robmilne commented
Status: Active » Fixed

Fixed in HEAD.

pyutaros’s picture

pyutaros commented

Patch works excellent on my test site. I will wait for 2.8 on my production site.
Thanks,
Jonathan

robmilne’s picture

robmilne commented

No waiting - It's ready now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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