By http://drupal.org/node/64279#comment-145200 the declaration of image_view should not contain &$node:

function image_view($node, $teaser = 0, $page = 0) {

CommentFileSizeAuthor
#3 image_view_126817.patch659 bytesdrewish

Comments

drewish’s picture

drewish commented
Status: Active » Postponed (maintainer needs more info)

well that's my comment and i think it is correct way of doing it. are you experiencing any incorrect behavior as a result of it or just saying it's not right?

alladjex’s picture

alladjex commented

I just mention it for the implemantator of the image.modulle that he should change the declaration of function image_view.

Personally I think your comment is correct. I use it yesterday in my private module. Everything seems to be ok.

drewish’s picture

drewish commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new image_view_126817.patch659 bytes

well here's a patch for it.

drewish’s picture

drewish commented
Status: Needs review » Reviewed & tested by the community

this accidentally got committed to 5 or HEAD (i don't remember which). i guess it should get committed to the other...

drewish’s picture

drewish commented
Status: Reviewed & tested by the community » Fixed

commited this.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)