Mandrill is changing the names of args in the filter (3 posts)

  1. Sudar
    Posted 11 months ago #

    I am the author of the email-log WordPress plugin, which allows you to log emails sent thorugh WordPress. In order to do that, I listen to the wp_mail filter in WordPress.

    A user of my plugin recently complained that if an email is sent through wpmandrill plugin then it is not getting logged. I looked into it and found that the wpmandrill plugin is changing the names of the array elements that are passed to the wp_mail filter.

    In https://core.trac.wordpress.org/browser/tags/3.9/src/wp-includes/pluggable.php#L236 you can see that the original array key is "message". But in https://plugins.trac.wordpress.org/browser/wpmandrill/trunk/wpmandrill.php#L1646 it is changed to "html".

    The wpmandrill plugin should not change the array keys of args passed through filters. It will nice if it can be changed back to "message" in wpmandrill plugin, so that it doesn't break other plugins.


  2. MC_Will
    Plugin Author

    Posted 10 months ago #

    Hi Sudar,

    Thanks a lot for pointing out this bug. We'll fix it in the next release.

    Our apologies to your users.

  3. Sudar
    Posted 10 months ago #

    Thanks for agreeing to fix this.

    I have one request though. Kindly update this thread once you have made the release.


You must log in to post.

About this Plugin

About this Topic