Download & Extend
AHAH helper » Issues

Call-time pass-by-reference has been deprecated

Posted by Heine on September 19, 2008 at 2:09pm
Project:AHAH helper
Version:6.x-2.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Simple patch removes extra &.

AttachmentSize
ahah_helper_callbyref.patch4.03 KB
Login or register to post comments

Comments

#1

Posted by ppblaauw on October 2, 2008 at 12:55am
Status:active» reviewed & tested by the community

Applied this patch.

No "Call-time pass-by-reference has been deprecated" errors anymore.

#2

Posted by Wim Leers on November 19, 2008 at 4:25pm
Status:reviewed & tested by the community» fixed

Thanks Heine! :)

Committed.

#3

Posted by System Message on December 3, 2008 at 4:35pm
Status:fixed» closed (fixed)

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

#4

Posted by greg.harvey on July 8, 2009 at 5:12pm
Version:6.x-1.0» 6.x-2.0
Status:closed (fixed)» active

Getting this again in 6.x-2.0:

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 /home/gharvey/workspace/cmspros/trunk/www/sites/all/modules/contrib/ahah_helper/ahah_helper.module on line 255

#5

Posted by greg.harvey on July 8, 2009 at 5:15pm
Status:active» needs review

Patch attached.

AttachmentSize
310724-ahah_helper_2.x_callbyref.patch 0 bytes

#6

Posted by greg.harvey on July 8, 2009 at 5:17pm

Ugh, ignore #5 - forgot to paste the code in to the file from my clipboard. Patch attached HERE.

AttachmentSize
310724-ahah_helper_2.x_callbyref.patch 456 bytes

#7

Posted by alexpott on July 14, 2009 at 1:03pm

Was getting the same warnings and applied patch in #6 - which works great.

Thanks!

#8

Posted by Rok Žlender on August 19, 2009 at 8:55pm
Status:needs review» reviewed & tested by the community

Worked fine for me too.

#9

Posted by Roger Saner on August 20, 2009 at 7:57pm

Works for me too.

#10

Posted by boombatower on September 30, 2009 at 1:15am

Ping.

#11

Posted by prosk on October 16, 2009 at 9:55am

Is it possible to apply the patch on the d6 release?

Thank you !

#12

Posted by greg.harvey on October 16, 2009 at 10:01am

#6 *is* for the D6 release.

#13

Posted by svendecabooter on October 21, 2009 at 9:14am

Patch in #6 does indeed fix the problem.

#14

Posted by jefkin on November 5, 2009 at 2:48pm

Months have passed, with no complaints, (#6 works for me btw) why not push this through?

#15

Posted by the_real_codecowboy on January 14, 2010 at 5:08pm

#6 path works for me too - does not seem to be in latest release

#16

Posted by BradM on April 7, 2010 at 3:25pm

Confirming patch in #6 needs to be applied to the latest release.

#17

Posted by Cristhian on May 18, 2010 at 4:42pm
Status:reviewed & tested by the community» fixed

Applied to 6.x-2.x-dev.

#18

Posted by System Message on June 1, 2010 at 4:50pm
Status:fixed» closed (fixed)

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

#19

Posted by OliverColeman on January 19, 2011 at 11:19pm
Version:6.x-2.0» 6.x-2.1

The fix doesn't seem to have made it into 6.x-2.1.

#20

Posted by OliverColeman on January 19, 2011 at 11:20pm
Status:closed (fixed)» active

#21

Posted by Manuel Garcia on February 15, 2011 at 12:35pm

Definetly stil happening in 2.1,

#22

Posted by Manuel Garcia on February 15, 2011 at 12:37pm
Status:active» fixed

Just checked, this is fixed in the lates dev version, so I assume when the next release is out it will be fixed.

#23

Posted by System Message on March 1, 2011 at 12:41pm
Status:fixed» closed (fixed)

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

nobody click here