Closed (fixed)
Project:
ecard
Version:
6.x-1.1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
6 Jun 2008 at 02:49 UTC
Updated:
27 Feb 2009 at 19:10 UTC
Hello,
I just installed ecard-6.x-1.1 and when I activate the module, I get this message at the top of all my pages:
Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of validate_emails(). 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/.westberry/scain/scottcain.net/modules/ecard/ecard.module on line 307
Where line 307 is this:
$emails=validate_emails($form_id,&$form_state);
I tried adding this line to my .htaccess file:
php_value allow_call_time_pass_reference 1
but it doesn't appear to due any good. My site is hosted at Dreamhost and I have php 5. Any suggestions?
Thanks,
Scott
Comments
Comment #1
scain commentedThis is actually a problem with the way Dreamhost works--the .htaccess file is ignored(!), so you need to do some funny stuff with a custom php.ini file. It is described on the Dreamhost wiki: http://wiki.dreamhost.com/PHP.ini and modifying the php.ini file to change allow_call_time_pass_reference to 1 in it.
Scott
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #3
rootworkI wouldn't describe this as a problem with the server. If I understand what's happening correctly, the module is using a PHP-depreciated function. The solution isn't "modify your server settings" (though that's one workaround for now), it's to change the module so that it operates using current PHP functions.
For the module developer, you can consider it a feature request rather than a bug if you like, but I think this is pretty critical to high uptake of this module. On a purely practical level, if you're going to tell everyone who hosts on DreamHost (lots of people) that the first step to enable this module is "modify your server settings" then I don't think it's going to get very widespread use...
Comment #4
kars-t commentedYes this seems to be deprecated. I try to fix it with the next D6 release. Please be patiant a little longer :)
Comment #5
rootworkThanks Kars-T. I see there's a slightly more recent dev version; has it been fixed in that one? If not please let us know when the dev version fixes this -- I'd be willing to start using a dev version with a fix, even if it's otherwise rough around the edges.
Comment #6
kars-t commentedHi its not in the current dev but I am at it. A quick fix would be
But I think I will have a new dev in the next hours.
Comment #7
kars-t commentedLatest dev fixes this. Please test it.