Problem/Motivation

After installing the module I got:
Error sending email (from *** to *** with reply-to 1).
After debugging I found:

      $this->mailManager->mail('admin_content_notification', $key, $admin_email, 'en', $params, TRUE);

however mail exepts string|null, not TRUE.

Proposed resolution

Will provide the patch.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

Comments

RuslanP created an issue. See original summary.

ruslan piskarov’s picture

Status: Active » Needs review
StatusFileSize
new735 bytes

Adding the patch. Please review.

ruslan piskarov’s picture

StatusFileSize
new784 bytes

Sorry.
In the previous patch, I was wrong. After testing I got: "Error sending email (from *** to *** with reply-to not set)".
Attaching a new one.

  • 17d3fbb committed on 8.x-3.x
    fix issues #3007555, #3016410, #3016454
    
  • ab783b4 committed on 8.x-3.x
    fix issues #3007555, #3016410, #3016454
    

  • 1877c0f committed on 8.x-3.x
    fix issues #3007555, #3016410, #3016454
    
manish.upadhyay’s picture

Status: Needs review » Fixed
manish.upadhyay’s picture

manish.upadhyay’s picture

Status: Fixed » Closed (fixed)