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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna 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 NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
metatag-n2281833-2.patch998 bytes

This needs some testing.

haysuess’s picture

haysuess CreditAttribution: haysuess commented

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna 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.