I believe this #1721522: Array key not recognized when apostrophe is present goes a little deeper. We were having the following problem:

1. Other is chosen and value with ampersand is entered. Node is saved.
2. Node is edited and the entered value with the ampersand is not selected because while the displayed value &, the option value is &

Patch follows.

CommentFileSizeAuthor
#2 select_or_other-values_with_ampersand-2568353-1.patch673 bytesmicnap
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

micnap created an issue. See original summary.

micnap’s picture

micnap CreditAttribution: micnap at Mediacurrent commented
FileSize
select_or_other-values_with_ampersand-2568353-1.patch673 bytes
darksnow’s picture

darksnow CreditAttribution: darksnow commented
Status: Active » Needs review

This patch works fine for me and I can't see any side effects from such a simple patch.

Great work micnap

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo commented
Status: Needs review » Needs work

I guess the patch is simple enough, but I would also like to have test coverage for this.

micnap’s picture

micnap CreditAttribution: micnap at Mediacurrent commented
Issue summary: View changes
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo as a volunteer commented
Assigned: Unassigned » legolasbo
Issue tags: +Quick fix

  • legolasbo committed c08b116 on 7.x-3.x authored by micnap 
    Issue #2568353 by micnap, legolasbo: Other values with ampersand not...
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo as a volunteer commented
Status: Needs work » Fixed

Added tests and committed, thanks for the patch!

Status: Fixed » Closed (fixed)

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