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

CommentFileSizeAuthor
#7 284888.patch627 bytesrobloach
#6 284888.patch1.04 KBrobloach

Comments

drupal999’s picture

drupal999 commented

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

Proct0t’s picture

Proct0t commented

Ditto on the confirmation, easy fix though: remove the & in front of $vars on line 465 of cck_redirection.module. :)

kalesco’s picture

kalesco commented

Proct0t: Thank you very much for the fix!

Project-Team, please fix this in the new release!!

choster’s picture

choster commented
Category: support » bug
robloach’s picture

robloach
he/him
commented
Status: Active » Needs review

Yup.... Patched from DRUPAL-6--1.

robloach’s picture

robloach
he/him
commented
StatusFileSize
new 284888.patch1.04 KB

Uhh, forgot the patch.

robloach’s picture

robloach
he/him
commented
StatusFileSize
new 284888.patch627 bytes

Here it be without .project.

choster’s picture

choster commented
Assigned: Chani » Unassigned
Status: Needs review » Reviewed & tested by the community
Ogredude’s picture

Ogredude commented

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.

chadd’s picture

chadd commented

i just got this error and applied the fix in #2.
is this going to be fixed in a future release?
thanks!

guillaumeduveau’s picture

guillaumeduveau
Location Toulouse
commented

The fix is in CVS DRUPAL-6--1 but not in the 6.x-1.1 release.

guillaumeduveau’s picture

guillaumeduveau
Location Toulouse
commented

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

robin monks’s picture

robin monks commented
Status: Reviewed & tested by the community » Fixed

Corrected in 6.x-1.2.

Robin

Status: Fixed » Closed (fixed)

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