#2358995: Remove usage of drupal_wrap_mail() removed the usages of drupal_wrap_mail(). In this issue, let's remove the function itself.

CommentFileSizeAuthor
#5 2359071-5.patch1.19 KBrpayanm
#1 remove_drupal_wrap_mail-2359071-1.patch1.16 KBJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
FileSize
remove_drupal_wrap_mail-2359071-1.patch1.16 KB
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm commented
Related issues: +#2358995: Remove usage of drupal_wrap_mail()
ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk commented
Status: Postponed » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: remove_drupal_wrap_mail-2359071-1.patch, failed testing.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm commented
Status: Needs work » Needs review
FileSize
2359071-5.patch1.19 KB
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Status: Needs review » Reviewed & tested by the community

Looks good!

  • alexpott committed c1b74a3 on 8.0.x 
    Issue #2359071 by JeroenT, rpayanm: Remove drupal_wrap_mail
ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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