search

skydiver / wn-forms-plugin

Create easy (and almost magic) forms
https://magicforms.vercel.app
MIT License
21 stars 9 forks source link

Using `mail_replyto` component property crash sending mail #10

Open damsfx opened 2 years ago

damsfx commented 2 years ago

If mail_replyto component property is used to refer a form field, then the mail notification result in an error : Swift_RfcComplianceException: Address in mailbox given [email] does not comply with RFC 2822, 3.6.2.

In this case I want to use the email field of my form.

When I was looking for the cause of the problem, I realised that the line https://github.com/skydiver/wn-forms-plugin/blob/805f3aff2ba4ae768d61b5fedbae913e0cf1dcf6/classes/Mails/Notification.php#L74

May be changed to $message->replyTo($this->post[$this->properties['mail_replyto']]); if we want the reply field of mail to be set.

The only workaround currently found is to remove the reply_to property from the component.

willouch commented 2 years ago
damsfx commented 2 years ago

Well still waiting for a fix.
Did #15 will be merged to close this issue?

damsfx commented 2 years ago

@skydiver ... Is this plugin still maintained?