Not sure when it happened, but somewhere in the upgrades of the 6.0 alpha's of the Image module, now my galleries only show 10 images, no matter how many images are in the gallery or what I set in the "images per page" setting in admin/settings/image/image_gallery. Any ideas?

CommentFileSizeAuthor
#13 561854-image_gallery-admin-note.patch2.52 KBjoachim

Comments

Hetta’s picture

Status: Active » Postponed (maintainer needs more info)

Do you have views installed? If so, what are the settings for the views image gallery?

deshilachado’s picture

Status: Postponed (maintainer needs more info) » Active

same problem here
after running update.php there were a lot of messages, no erros and now all galleries display ten images and no pager.
there are always nine images in three rows & three columns and then one image below, centered.

deshilachado’s picture

sorry, i didn't mean to change the status.
before you said it, i didn't know image is using views for the gallery
i went to http://example.com/admin/build/views/edit/image_gallery
and changed the items_per_page from 10 to 0 (unlimited)
this works for me, thanks a lot

deshilachado’s picture

but stil it is strange that under
http://example.com/admin/settings/image/image_gallery
one can set a number of images per gallery but this settings doesn't change anything, as far as i can see

Hetta’s picture

The settings under image_gallery work if you get rid of views. You can, if you so like, submit a patch adding some help text to the image_gallery admin page.

deshilachado’s picture

ok, good to know.
in case anyone wants to read more about image and views: #405456: Galleries made with views

right now i'm at work, but if noone is faster than me i'll try to find the time this weekend to write that help text (allthough i know it will take me hours to re-learn how to write a patch...)

joachim’s picture

Ok, to fix I would say we need:

- message on upgrade that galleries now made with views (can update functions use dsm?)
- note on the IG settings page. this can be cribbed from D5. Totally forgot about that!

I am away from teh nets till conference -- a patch would be great!

dugh’s picture

See same issue

My images are all munged together overlapping. Will work with views

shelleyp’s picture

The change also has caused problems with the display, and you have to remember to add paging to get beyond the first set of images.

All in all, a rather disruptive update. Can you give us some clues about what we need to do, both to the view and to the styles to make the display similar to what we had before this update?

joachim’s picture

It should look about the same.
Though the note about paging -- yes, should have paging built in. Please file another bug for this!

joachim’s picture

> All in all, a rather disruptive update.

Big changes yes, but this is image module getting to where it needs to be. Anything in an alpha release is subject to change :)

shelleyp’s picture

"changes yes, but this is image module getting to where it needs to be. Anything in an alpha release is subject to change :)"

Oh sure, I understand. I'll find a bug for the paging, but it's easy enough to modify in the view.

I have found serious styling issues, too. Images overlapping each other in the gallery. Is there anything on differences in layout between the view and the original gallery that may be different?

joachim’s picture

Title: problem with image gallery only displaying 10 images » Notify on admin page that gallery is being made with views
Status: Active » Fixed
StatusFileSize
new2.52 KB

Committed the attached patch.

This adds a note to the gallery settings page to explain what's going on, and removes the settings while the view is enabled, as they will have no effect.

Thanks for reporting this :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.