#2358991: Remove usage of drupal_mail() removed the usages of drupal_mail, new let's remove it!

CommentFileSizeAuthor
#4 remove_drupal_mail-2359457-4.patch6.6 KBJeroenT
#3 remove_drupal_mail-2359457-3.patch5.6 KBJeroenT
#1 remove_drupal_mail-2359457-1.patch5.55 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_mail-2359457-1.patch5.55 KB
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Related issues: +#2358991: Remove usage of drupal_mail()
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
FileSize
remove_drupal_mail-2359457-3.patch5.6 KB
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Status: Postponed » Needs review
FileSize
remove_drupal_mail-2359457-4.patch6.6 KB

Removed full mail.inc file. There is a reference to mail.inc in core.api.php I removed it, but maybe we should reference to the mailer classes.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Novice, +@deprecated

Looks good for me.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Removal of unused and deprecated code is permitted by the beta evaluation. Committed 8de33db and pushed to 8.0.x. Thanks!

  • alexpott committed 8de33db on 8.0.x 
    Issue #2359457 by JeroenT: Remove drupal_mail()

Status: Fixed » Closed (fixed)

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