It's only necessary for the image_gallery.module to set menu location when a single image is on display. This does not make sense for teasers.

In addition, this breaks views menus for those that include the teaser of an image node as discussed:

http://drupal.org/node/151978

To fix, simply change the test on line 404 of image_gallery.module to:

      if ($page && $node->type == 'image' && !$teaser) {
CommentFileSizeAuthor
#4 image_gallery_162496.module.patch837 bytesdrewish
#3 image.162496.patch749 bytesraintonr

Comments

raintonr’s picture

raintonr commented
Status: Active » Needs review

Sorry, status should be patched for review of course given the snippet I included.

drewish’s picture

drewish commented

seems like a logical change. would you mind rolling a patch? it's the easiest way for me to review and commit changes. see http://drupal.org/patch/create if you need information on how to do so.

raintonr’s picture

raintonr commented
StatusFileSize
new image.162496.patch749 bytes

No worries... patch attached.

drewish’s picture

drewish commented
Status: Needs review » Fixed
StatusFileSize
new image_gallery_162496.module.patch837 bytes

great. made one small change, put !$teaser next to $page. committed to HEAD and DRUPAL-5

Anonymous’s picture

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