Problem/Motivation

After #2896055: From header containing UTF-8 characters appears scrambled in emails landed #2717965: Site name is not UTF-8 encoded in email headers also introduced a $shorten parameter for Unicode::mimeHeaderEncode() which it also uses for the From header: https://www.drupal.org/project/drupal/issues/2717965#comment-12320178

Webform should do this too I guess.

CommentFileSizeAuthor
#2 2937761-2.patch1.39 KBcasey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

casey created an issue. See original summary.

casey’s picture

casey CreditAttribution: casey at SWIS commented
Status: Active » Needs review
FileSize
2937761-2.patch1.39 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2937761-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented

The patch makes complete sense to me. I am going retest the patch.

  • jrockowitz committed 5116a13 on 8.x-5.x authored by casey 
    Issue #2937761 by casey, jrockowitz: Use shorten parameter of Unicode::...
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs work » Needs review

Yep, the testbot was having a random failure and the patch was fine. Thanks.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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