Problem/Motivation

Be liberal about what you accept

The change in #2993049: Customer email promotion condition does a case sensitive e-mail comparison meant that is an OrderEmail condition was not configured via form then it has to make the email lowercase.

Proposed resolution

Don't change input - do a case insensitive multi-byte safe comparison. Why multibyte - well https://googleblog.blogspot.com/2014/08/a-first-step-toward-more-global-... is thing.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3033054-2.patch922 bytesalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott as a volunteer and commented
Status: Active » Needs review
FileSize
3033054-2.patch922 bytes

  • bojanz committed 0d35a37 on 8.x-2.x authored by alexpott 
    Issue #3033054 by alexpott: OrderEmail should obey Postel's principle
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Needs review » Fixed

Thanks Alex, this is much more elegant.

Status: Fixed » Closed (fixed)

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