The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi,
I am running wamp 2.0
Server Configuration
Apache Version :
2.2.8
PHP Version :
5.2.6
Not sure if its just me. But I get the following.
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 D:\<site>\sites\all\modules\signup\modules\signup_confirm_email\signup_confirm_email.module on line 36
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 D:\<site>\sites\all\modules\signup\modules\signup_confirm_email\signup_confirm_email.module on line 36
cheers
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 550938-1.signup_confirm_email_call_by_reference.patch | 830 bytes | dww |
Comments
Comment #1
dwwNope, not just you. That's a (cut and paste) bug. ;) Try this on for size.
Comment #2
juicytoo commentedThanks Derek.
Comment #3
dww@juicytoo: Does that mean you applied the patch and it worked? ;)
Comment #4
dwwFYI: I'm incredibly reluctant to answer support requests from people who don't contribute back in the issue queue in some way. ;). Now's your chance to earn some credibility and appreciation from me.
Cheers,
-Derek
Comment #5
juicytoo commentedHi Derek,
Yes, The patch worked.
Thankyou.
cheers
Comment #6
dwwGreat, thanks. For future reference, it's not actually "fixed" until I commit the patch to drupal.org's CVS repository. ;) I'll do that soon, now that I know it works...
Comment #7
alexiscott commentedI just tested it, and can also confirm that the patch works, that is I no longer get the error "Call-time pass-by-reference has been deprecated"
Comment #8
dwwCommitted to HEAD and DRUPAL-6--1. Thanks for testing, folks.
Comment #9
rmmcclay commentedThe patch worked for me as well. Thank you.