#2358993: Remove usage of drupal_mail_system() removed the usages of drupal_mail_system. Now let's remove it!

CommentFileSizeAuthor
#4 remove-2359453-4.patch1.36 KBjeroent
#1 remove-2359453-1.patch1.37 KBjeroent

Comments

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
StatusFileSize
new remove-2359453-1.patch1.37 KB
jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Postponed » Needs review
Related issues: +#2358993: Remove usage of drupal_mail_system()

Status: Needs review » Needs work

The last submitted patch, 1: remove-2359453-1.patch, failed testing.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs work » Needs review
StatusFileSize
new remove-2359453-4.patch1.36 KB
ianthomas_uk’s picture

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

Confirmed there are no references to this function, and the patch just removes it.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

I updated https://www.drupal.org/node/2191201.

Committed/pushed to 8.0.x, thanks!

  • catch committed d3f58a5 on 8.0.x 
    Issue #2359453 by JeroenT: Remove drupal_mail_system()

Status: Fixed » Closed (fixed)

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