Widget does not show up

Edit file linkimagefield_widget.inc at line number 180.

Edit function theme_linkimagefield_widget(&$element) {

Remove the & so that it looks like function theme_linkimagefield_widget($element) {

That works for me. :)

I second that this #1 edit at line 180 works. It sorted my problem. Thank you tgrondin!

That worked great. Is it fair to assume this easy-to-do, minor tweak (that causes the module not to work at all) has at least made it to the latest dev snapshot? Shouldn't it be committed if it's making it non-functioning? Thanks for a great module though, for real. Not sure why this isn't just an optional setting in imagefield...that would be better than a separate module. Has that ever been explored/considered with those folks?

Great module, my site using this module is http://www.primastar-mandiri.com/project-reference.html, thank so much for release tis module

Isn't the & versus no & on line 180 a php version issue?

I believe that php 5.3.x passes argument by value (no &) versus php 5.2.x passes arguments by reference (& needed)

So it could be that the reason we need to remove the & is because we have php 5.3.x. I would be curious to know if someone runs php 5.2.x would need to remove the &.

Is there any patch to correct this issue or must I change manually the code as tgrondin suggested ?

I did the same mentioned in #1 and it is working now. Please inform when it is solved in releases.
Thanks

It works
Thanx

Thanks tgrondin for finding a simple one-character fix for this issue. I can confirm that my linkimage widgets stopped showing up when my web host updated the PHP version to 5.3 recently, so this problem was quite vexing. I hope there will be a new release soon that includes this fix.

that fixes it for me too, thank you :)

works fine for me too

Right way.
Thanks!

Marked #1350854: Link Image not showing in node edit as duplicate of this issue.

Here is a patch that contains the fix from #1. It also updates the documentation of this function.

Thanks a lot. Patch works for me fine :-)

Removed headers on patch that blocked me from applying the patch.

@robaminima: That should not be necessary, these headers contain metadata and are fully compatible with git and the standard patch tool. If your software does not support this then this is a bug in your software and you should report it to the developers.

@pfrenssen Patch works like a charm. Thanks!

Wish I found this sooner. Works.
Is this going to get rolled?

Patch worked for me

My ISP, Site5, just changed my site to PHP 5.3 (shared hosting -- I don't maintain the PHP myself). I'm just posting here in case anyone else runs into this issue to mention that the patch worked for me. I simply went to the linkimagefield directory, found linkimagefield_widget.inc and changed

'function theme_imagefield_widget(&$element)'

to

'function theme_imagefield_widget($element)'

That is, I took out the & .

Thanks pfrenssen, et al!

Can this very simple change be committed? The patch has been well reviewed and works.

Development or support is not planned for D6. All D6-related issues are marked as outdated in a bunch.

Everyone can apply the patch above (not tested by the maintainers, tested by the community) to their D6 projects.
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!