Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
afer mass sorting images in the gallery, all images has new 'node published' date. I think only 'node updated' date should be changed.
Now it messes my "latest image" view completely and I have to change publishing date manually! :)
Comment | File | Size | Author |
---|---|---|---|
#6 | 748054.patch | 648 bytes | justintime |
Comments
Comment #1
Daugilas CreditAttribution: Daugilas commentedBy the way, it also changes author of the picture. So if I upload images with 'regular user' and gallery is sorted by 'moderator', after sorting all images are submited by 'moderator'!
Comment #2
justintime CreditAttribution: justintime commentedI'll take a look at this one.
Comment #3
dddave CreditAttribution: dddave commentedAt least #2 is a dupe of #550994: Admin editing gallery changes user ID.
Comment #4
justintime CreditAttribution: justintime commentedI think there's a good chance that the patch posted in #550994: Admin editing gallery changes user ID will also fix this bug. I'm going to mark it as needs review so someone can test it out.
Comment #5
dddave CreditAttribution: dddave commentedSorry, but the date changes when I sort the images. Not for all images in a gallery but for all those shuffled around.
Comment #6
justintime CreditAttribution: justintime commentedO.M.G.
That has to be the most time I've spent on one line of code *ever*. Very deep within the bowels of node.module, I found the problem. The fix is simple, we just need to call node_object_prepare() on the node before saving it.
The attached patch should fix the problem.
Comment #7
dddave CreditAttribution: dddave commentedFix confirmed.
I really admire people with the ability to read code AND understand it. Btw is a new dev on its way? Would like to have the "My Galleries" fix back without patching again. Lost it when I used a clean NG to test some other patches... ;)
Comment #8
justintime CreditAttribution: justintime commentedCommitted. I'll do a mass-merge of the code changes I've been making in HEAD back to the 2.x branch in just a bit.