Viewing 3 posts - 1 through 3 (of 3 total)
  • Author
    Posts
  • #1135298

    Hello,

    There is a flaw in the file class-form-generator.php in that it has the same filter used twice (avf_form_mail_header), once for the message and once for the auto-responder which means it gets called twice, overwriting the header for both when adding custom Reply-to or other header items (undesirable!). Each section should have its own filter to avoid overwrites.

    Feature Request: It looks to me like the class should be re-written so that there is a header variable for each type of email to be sent, and these header variables should be stored as key/value paired arrays so that CC’s, Reply-tos and other header items can be added to a “stack” via filters, and then the Enfold class can “implode” them with “\r\n” when ready to send.

    Thanks for reading,
    Andy

    • This topic was modified 5 years, 3 months ago by betaphase.
    #1135308

    I also wanted to add that the auto-responder is fired twice. When I var_dumped the $new_post content to the error log I saw 4 passes through the avf_form_mail_header filter; not 2 which is expected since avf_form_mail_header is included twice.

    #1135960

    Hi betaphase,

    Thank you for bringing this up. Our devs will look into this.

    Best regards,
    Victoria

Viewing 3 posts - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.