Closed (fixed)
Project:
Prepopulate
Version:
6.x-2.0
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Apr 2006 at 16:18 UTC
Updated:
18 Aug 2010 at 20:36 UTC
[error] PHP 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 [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. However, future versions may not support this any longer. in d:\\path_to_drupal\\modules\\prepopulate\\prepopulate.module on line 59
Seems pass by reference has been deprecated in php5.
Could you please modify it to work with pass by value.
I tried. But could not get working. am not good at playing with such things in php.
Comments
Comment #1
eafarris commentedFrom the php documentation it appears that the ampersand is optional in php5. Could you try changing all instances of "&$form" to simply "$form" and see if that does it? I do not yet have a test environment with php5.
Comment #2
zigma commentedI tried removing all ampersand. It does not throw any error. But it does not initialize the forms.
Comment #3
eafarris commentedChanged all call-time pass-by-reference to accept-by-reference. Should do the trick. Committed to CVS rather than DRUPAL-4-7, apparently I didn't branch the module properly, so please use the CVS version for now.
Comment #4
(not verified) commentedComment #5
mikey_p commentedGetting this on line 46:
_prepopulate_request_walk(&$form, &$request);
Why is this still there?
Comment #6
jbrauer commentedThat doesn't appear on line 46 in any of the current versions of CVS. If there is such a bug in the current version please post a link to the file in CVS with the line number.
Comment #7
mikey_p commentedCame from the patch at #360613-26: Prepopulating CCK Autocomplete Nodereference field