The default view image_gallery defines (image_gallery) has no URL set although the documentation says that this is required for views that provide a page.
Eg, from the views edit form:
Enter the URL to use for this view in the form of 'dir/dir'. Do not begin or end the URL with a /. Example: 'view/tracker'. This is required if providing a page view. You can also add $arg as a placeholder for arguments passed in the URL, for example 'user/$arg/tracker' or 'node/$arg/related'. Note that any arguments listed here will be required, even if they are listed as optional below. You do not need to list arguments at the end of the path. Do not try to use URLs such as taxonomy/term/$arg.
This bug in image_gallery was found by analysing a bug for nodewords: see #294475: array-flip not array-flipping. That issue has been fixed in nodewords, but this is still a bug in image_gallery I believe. Patch attached.
Comment | File | Size | Author |
---|---|---|---|
#1 | image_gallery_295293.patch | 1002 bytes | drewish |
image_gallery.default_views.patch.txt | 404 bytes | Robrecht Jacques |
Comments
Comment #1
drewish CreditAttribution: drewish commentedIt doesn't have a title either so I added one of those and committed the attached. Thanks.
Comment #2
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #3
joachim CreditAttribution: joachim commentedThis patch was totally wrong, and broke galleries.
The gallery view is used as an embedded view by our own menu callback at /image.
Giving the view a path causes the view to override our path, and so no child galleries are shown.
This patch needs to be rolled back.
I'm not sure I understand the bug in nodewords. Having a path is not a required field for views that provide a page, as the page settings are used for when the view is embedded.
We could put in an unused path, like "views/image" (that's what I do on my production sites so I can see what a view is producing). Or nodewords could filter its array before trying to flip it.
Comment #4
joachim CreditAttribution: joachim commented@sun: I'd like to roll this back ASAP.
Comment #5
sunRemoved $view->url.
Comment #6
joachim CreditAttribution: joachim commentedMarking this as by design, as the image gallery view is intended only for embedding.