magepal / magento2-gmail-smtp-app

Configure Magento 2 to send email using Google App, Gmail, Amazon Simple Email Service (SES), Microsoft Office365 and many other SMTP (Simple Mail Transfer Protocol) servers
https://www.magepal.com
319 stars 148 forks source link

Magento 2.2.9 "We can't save the customer" caused by this module #146

Closed ChrHartwig closed 5 years ago

ChrHartwig commented 5 years ago

Magento version #:

2.2.9

Edition (EE, CE, OS, etc):

CE

Expected behavior:

When new customer registers, the user is redirected to the user account.

Actual behavior:

User is redirected to registration page with error message "We can't save the customer"

Steps to reproduce:

Preconditions

Solution

Change file Model/ZendMailTwo/Smtp.php ` if ($message->getReplyTo() === null && $dataHelper->getConfigSetReplyTo()) { foreach ($returnPathEmail as $address) { $message->setReplyTo($address); } }

    if ($returnPathEmail !== null && $dataHelper->getConfigSetFrom()) {
        foreach ($returnPathEmail as $address) {
            $message->setFrom($address);
        }
    }

`

to

` if ($message->getReplyTo() === null && $dataHelper->getConfigSetReplyTo()) { if (is_string($returnPathEmail)) { $message->setReplyTo(trim($returnPathEmail)); } elseif ($returnPathEmail instanceof AddressList) { foreach ($returnPathEmail as $address) { $message->setReplyTo($address); } } }

    if ($returnPathEmail !== null && $dataHelper->getConfigSetFrom()) {
        if (is_string($returnPathEmail)) {
            $message->setFrom(trim($returnPathEmail));
        } elseif ($returnPathEmail instanceof AddressList) {
            foreach ($returnPathEmail as $address) {
                $message->setFrom($address);
            }
        }
    }`
srenon commented 5 years ago

I will release a new version shortly... Do you have any Mirasvit extensions installed?

ChrHartwig commented 5 years ago

I will release a new version shortly thx :)

Do you have any Mirasvit extensions installed? No, i don't have any Miravisit extension installed.

srenon commented 5 years ago

@Neusta-Hartwig ... Fixed in 2.6.4