The recently added support for multiple tags (#1305402: Single image URL for image field with multiple images) has a problem - the values end up being alphabetically sorted before they're displayed, so the desired order is changed.

CommentFileSizeAuthor
#2 metatag-n2281833-2.patch998 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Ok, so they aren't strictly output in alphabetical order.

Input:

  1. http://d7.dev/sites/default/files/_gallery_photos/LOLwalrus%20-%20Mah%20...
  2. http://d7.dev/sites/default/files/_gallery_photos/cthulhu.jpg
  3. http://d7.dev/sites/default/files/_gallery_photos/jalapeno-all-the-thing...
  4. http://d7.dev/sites/default/files/_gallery_photos/onedoesnotsimplyhackte...

Output:

  1. http://d7.dev/sites/default/files/_gallery_photos/LOLwalrus%20-%20Mah%20...
  2. http://d7.dev/sites/default/files/_gallery_photos/cthulhu.jpg
  3. http://d7.dev/sites/default/files/_gallery_photos/onedoesnotsimplyhackte...
  4. http://d7.dev/sites/default/files/_gallery_photos/jalapeno-all-the-thing...

Uhm..

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new metatag-n2281833-2.patch998 bytes

This needs some testing.

haysuess’s picture

haysuess commented

Beautiful, can confirm patch in #2 works perfectly. Thanks! :)

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

@hayseuss: Thanks for reviewing the patch, glad it worked correctly. I've committed this fix.

  • Commit 6673070 on 7.x-1.x by DamienMcKenna: 
    Issue #2281833 by DamienMcKenna: Ensure multi-item values are output in...

Status: Fixed » Closed (fixed)

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