search

mplodowski / formbuilder-plugin-public

https://octobercms.com/plugin/renatio-formbuilder
https://octobercms.com/plugin/renatio-formbuilder
2 stars 0 forks source link

A 2nd FormLog is created when SendEmailMessage::sendAutoresponderMessage() is called #26

Closed SamZimple closed 1 year ago

SamZimple commented 1 year ago

Describe the bug A 2nd FormLog is created when SendEmailMessage::sendAutoresponderMessage() is called

To Reproduce Steps to reproduce the behavior:

  1. Create a form in the CMS
  2. Configure Autoresponder for that form
  3. Submit the form on the frontend
  4. Check the FormLogs section of the CMS to confirm that 2 Logs were created for a single form submission.

Expected behavior Only 1 FormLog should be created per form submission

Screenshots Capture

October version 2.2.25

Plugin version 2.2.0

Additional context This problem can be fixed by removing line 59 in formbuilder/listeners/SendEmailMessage.php, in the function sendAutoresponderMessage() remove $this->setHeaders($message);

mplodowski commented 1 year ago

This is by design, because autoresponder send new mail message with separate recipient and mail template.

In new version I will add a flag so this can be disabled.

mplodowski commented 1 year ago

This is available in version 3.1.2.