pass-by-reference
Chani - July 20, 2008 - 07:05
| Project: | CCK Redirection |
| Version: | 6.x-1.1 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
When I installed this module, I got a php error:
Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of template_preprocess_page(). 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/.raven/cema/teegopher.com/public_html/modules/cck_redirection/cck_redirection.module on line 465
#1
confirming this is still an error in 6.4 using 6.x-1.1
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 template_preprocess_page(). 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 /var/www/vhosts/domain.com/httpdocs/7325433/sites/all/modules/cck_redirection/cck_redirection.module on line 465
#2
Ditto on the confirmation, easy fix though: remove the & in front of $vars on line 465 of cck_redirection.module. :)
#3
Proct0t: Thank you very much for the fix!
Project-Team, please fix this in the new release!!
#4
#5
Yup.... Patched from DRUPAL-6--1.
#6
Uhh, forgot the patch.
#7
Here it be without .project.
#8
#9
Don't need to (and probably shouldn't) remove the & in the function declaration. Passing variables by reference in a function declaration is perfectly OK, it's passing by reference in a function call that's deprecated.
#10
i just got this error and applied the fix in #2.
is this going to be fixed in a future release?
thanks!
#11
The fix is in CVS DRUPAL-6--1 but not in the 6.x-1.1 release.
#12
Hello Robin !!! Thanks for updating CVS HEAD, it now contains this fix I think. I don't know if I should mark this as fixed or not, since there's no -dev branch yet.
Guix
#13
Corrected in 6.x-1.2.
Robin
#14
Automatically closed -- issue fixed for 2 weeks with no activity.