Display order rearrangement of galleries on the Galleries page doesn't work properly. Galleries can be moved via drag & drop but saving of the rearranged state doesn't work . Rearrangement of pictures in gallery works fine.

CommentFileSizeAuthor
#2 collection-sorting-broken-1228094-2.patch740 bytesMoloc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

b_ruben’s picture

Component: User interface » Code
Assigned: b_ruben » Unassigned
Moloc’s picture

Title: Gallery rearranging media not working properly » Gallery rearranging is not stored
Version: 7.x-1.0-beta6 » 7.x-1.0-beta7
Status: Active » Needs review
FileSize
740 bytes

After drag & drop rearranging the galleries, the state should be saved. Instead of saving, a JavaScript error is generated:

$.deparam is undefined
http://sandbox003/sites/all/modules/media_gallery/media_gallery.dragdrop.js?lwv7v1
Line 66

deparam is defined in the bbq library of jquery, which is loaded in the gallery view (to sort the media), but not in the collection view (to sort the galleries). Adding the missing library solves this issue.

b-netix’s picture

how can i add this patch? In witch file/line?

myke’s picture

Under this line (currently line 17):

drupal_add_library('system', 'ui.sortable');

Add this line:

drupal_add_library('system', 'jquery.bbq');

And it'll work perfectly. Now we just need a way to move galleries to other pages!

-Myke

elgandoz’s picture

Same here, but my error was: Uncaught TypeError: Cannot call method 'querystring' of undefined.
The fix above works perfectly.
Thank you

PS: Why the hell it's not included? In the previous versions that was ok...

scorchio’s picture

Status: Needs review » Reviewed & tested by the community

Obvious fix working perfectly. It would be nice to get this commited as soon as possible.

lsolesen’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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