Call-time pass-by-reference has been deprecated

Heine - September 19, 2008 - 14:09
Project:AHAH helper
Version:6.x-2.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:reviewed & tested by the community
Description

Simple patch removes extra &.

AttachmentSize
ahah_helper_callbyref.patch4.03 KB

#1

ppblaauw - October 2, 2008 - 00:55
Status:active» reviewed & tested by the community

Applied this patch.

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

#2

Wim Leers - November 19, 2008 - 16:25
Status:reviewed & tested by the community» fixed

Thanks Heine! :)

Committed.

#3

System Message - December 3, 2008 - 16:35
Status:fixed» closed

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

#4

greg.harvey - July 8, 2009 - 17:12
Version:6.x-1.0» 6.x-2.0
Status:closed» 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

greg.harvey - July 8, 2009 - 17:15
Status:active» needs review

Patch attached.

AttachmentSize
310724-ahah_helper_2.x_callbyref.patch 0 bytes

#6

greg.harvey - July 8, 2009 - 17:17

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

alexpott - July 14, 2009 - 13:03

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

Thanks!

#8

Rok Žlender - August 19, 2009 - 20:55
Status:needs review» reviewed & tested by the community

Worked fine for me too.

#9

Roger Saner - August 20, 2009 - 19:57

Works for me too.

#10

boombatower - September 30, 2009 - 01:15

Ping.

#11

prosk - October 16, 2009 - 09:55

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

Thank you !

#12

greg.harvey - October 16, 2009 - 10:01

#6 *is* for the D6 release.

#13

svendecabooter - October 21, 2009 - 09:14

Patch in #6 does indeed fix the problem.

#14

jefkin - November 5, 2009 - 14:48

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

 
 

Drupal is a registered trademark of Dries Buytaert.