I wanted freelinking to display the image preview if the node type being linked to was an image, so I wrote a patch for 1.7 to do that. I'd love to see it included in the official version if that's practical.

Note that the user has the option to turn it off (and it's off by default) so it should be a safe drop-in replacement for the current version.

This of course requires the Image module for it to actually display the image, but if the module is not installed, there shouldn't be any problem.

CommentFileSizeAuthor
freelinking.module.patch2.88 KBbartk

Comments

toemaz’s picture

toemaz commented

Tip: always develop against the latest HEAD version. In this case I suppose it is 1.x-dev.

bartk’s picture

bartk commented

Yes, in retrospect, that would have probably made things a bit easier. I just had the code there and started hacking away at it without really considering that. :)

kevin.gomes’s picture

kevin.gomes commented

Thanks!! this was helpful.

druvision’s picture

druvision commented

following

Grayside’s picture

Grayside commented
Version: 6.x-1.7 » 6.x-1.x-dev

More reviews and a patch reroll would convince me to commit this, despite the feature freeze on 1.x.

juampynr’s picture

juampynr commented
Version: 6.x-1.x-dev » 7.x-3.x-dev
Status: Needs review » Closed (won't fix)

Closing. Obsolete.