Closed (outdated)
Project:
Link Image Field
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Aug 2009 at 05:47 UTC
Updated:
24 Oct 2025 at 11:45 UTC
Jump to comment: Most recent
Comments
Comment #1
johnfyoung commentedgood catch - I have work to do today...I'll try to upload a fix later. Removing the & is exactly what you need to do. Looks like I just copied the function prototype for imagefield_widget_process.
Comment #2
jackhutton commentedWarning: Call-time pass-by-reference has been deprecated in /home/content/s/w/2/sw2010/html/beta3/sites/all/modules/linkimagefield/linkimagefield_widget.inc on line 134
this is the error message i get..and immediately removed the module...
i'll review your fix above..but , could really use this module if it worked..
thanks
Comment #3
jackhutton commentedi modified lines 132 and 134 to remove the & as noted above..and so far, no error message..
I've been able to use the link image widget w.out problem after the fix above.. thanks alot.. very useful.. and the error message was/is offputting..
eggonbeagle..
Comment #4
johnfyoung commentedactually, you should only remove the & from line 134. That is the call-time pass-by-reference. I'm uploading the fix now, though.
Comment #5
jackhutton commentedhmmm
should i add the ampersand back on to line 132..after the fact? its running fine, seemingly..
thanks for the response..
Comment #6
johnfyoung commentedthe function prototype on line 132 allows the
$form_statevariable to be modified in this function and the changes will stick after this function returns. As long as one doesn't make a change to$form_state, it shouldn't matter. This is a standard method for handling the form's state in Drupal's Form API, i.e., developers expect to receive a modifiable reference in these types of functions.Line 134, however, is not a function protoype but rather an actual function call and PHP no longer allows explicitly passing a reference into a function. This & is there because I copied the function prototype from the
imagefield_widget_processfunction and neglected to remove that &.Comment #7
jackhutton commentedthanks for that explanation.
i've added it back in on line 132..
thank you again.
Comment #9
ch_masson commentedI am using ver 6.x-1.x-dev and I got the same error message on line 89 of linkimagefield_widget.inc
According to thsi thread, this issue was fixed.
Or could it be a php version issue? I am using php 5.2.11
Thanks,
Comment #10
ch_masson commentedComment #11
sickness29 commentedDevelopment or support is not planned for D6. All D6-related issues are marked as outdated in a bunch.
If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.
Thanks!