JWardee
Forum Replies Created
-
Hey @eddr
Version 2.1.12 has just been released which resolves this issue.
Let me know if you run into any other issues 🙂
James
Hi @eddr
Thanks for getting in touch, glad you like the plugin!
I’ve managed to reproduce the issue and have logged a bug report on our GitHub tracker. I’ll tackle this as part of the next patch. I’m going to mark this topic as resolved as we use GitHub issues to track feedback.
Let me know if you run into any other issues 🙂
James
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] Failure to Log Emails with Large ContentI think this is a good idea. I’ve added this to our GitHub tracker so I’ll close the issue here. Feel free to leave any additional comments.
Cheers, James
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] Does not catch WooCommerce 10.0.4 emailsHey @gutding,
No problem, thanks for updating your post.
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] WP 6.8.1 Compatibility InquiryHello @hemasusi,
2.1.10 works fine but haven’t tested 2.1.7, although it should also work with no issues. I don’t believe WP has released any breaking changes.
I hope that helps!
Hey @titsmaker, thanks for your patience in waiting for me to reply to this! Both of the issues mentioned are currently logged in our GitHub issues (here and here). These bits have slipped down the priority list (as you say, you inquired about this a while ago!) but I currently don’t have any time to look into these, and I’m not sure when I’ll be freed up.
If this is high priority for you I’m happy for you to implement the features yourself and open a PR. Otherwise I’ll let you know once I have an update 🙂
Cheers, James
Thank you for the suggestion, I’ve added it to our GitHub tracker and will look into it further when I get some time. I’m going to mark this topic as resolved because I track everything through GitHub 🙂
Thanks again,
JamesForum: Plugins
In reply to: [Mail logging - WP Mail Catcher] PHP Deprecated ErrorHi Qilin,
has_capisn’t used anywhere in the plugin. Could you share the stack trace in your logs so we can see what led up to the error rather than just the error itself. Also what version of WordPress and PHP are you using?Thanks,
JamesForum: Plugins
In reply to: [Mail logging - WP Mail Catcher] emails not to be savedHi @f14m07, I’ve just released version 2.1.9 which should solve your issue. Let me know, if it’s still a problem 🙂
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] emails not to be savedHey guys, @joy0114 your code snippet (pasted below) looks fine to me, I’ve tested this locally and it correctly doesn’t log emails that have “Database Backup” in their description. The only thing I could imagine that could be catching you out is that it’s case sensitive – so “database backup” would still be logged.
add_filter('wp_mail_catcher_before_success_log_save','my_mail_catch_filter');
function my_mail_catch_filter($transformedArgs){
if (str_contains($transformedArgs['subject'], "Database Backup")) {
return false;
}
return $transformedArgs;
}@f14m07, I’ve looked into your issue and I’ve found a bug. I’ll release a fix this weekend and will reply here once it’s out
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] emails not to be savedHi @f14m07, yes your example is correct. If a message is sent with a subject of either
XXXXXorYYYYYthe log will not be saved. Otherwise it’ll proceed as normal.If you’d like more info then an idea would be to take a look at the unit tests for the hook. An example of which can be seen here. You probably couldn’t find references to the hook inside the code as all mentions are prefixed with a namespace. If you search the code for
_before_success_log_saveor_before_error_log_saveyou should find them eg hereHope that helps!
Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] emails not to be savedHey @f14m07, just letting you know that this functionality has been released in v2.1.8 the hook you’ll need is
wp_mail_catcher_before_success_log_saveand the documentation can be found in the “Hooks and actions” section of the plugin page.Forum: Plugins
In reply to: [Mail logging - WP Mail Catcher] How to delete this pluginHi Nicola,
It’s likely been installed as a “must use plugin“. You’ll need to move the plugin from the mu-plugins folder to the plugins folder whereby it’ll appear in the plugins menu as normal allowing you to go through the normal uninstall process.
However I must stress that there’s a risk of you breaking your site if this is done incorrectly. If you’re unsure I’d recommend reaching out to whoever installed it as an mu-plugin.
Because this isn’t a bug with the plugin itself I’m going to close this. I hope this info helps!
Cheers,
JamesForum: Plugins
In reply to: [Mail logging - WP Mail Catcher] emails not to be savedHi @f14m07, currently this isn’t possible but adding a hook to cover this is a nice idea and would also handle some other edge-cases. I’ve added this as a feature request on our GitHub tracker, I’ll aim to roll this out in the next release. Not sure when that’ll be currently but I’ll message back here once it’s available.
I’m going to close this issue here as it’s much easier for me to organise this via GitHub 🙂
Cheers,
JamesForum: Plugins
In reply to: [Mail logging - WP Mail Catcher] Why am I seeing these Headers?Hey @gauchan, I’m not sure I understand, do you mean that your attached screenshot appears above the table of logs?
It’s likely there’s either another plugin or some custom code is causing this to appear, it would be worth right clicking on the page and going to “inspect element” and taking a look at the console to see if there are any JavsaScript errors that could point you to what’s going on.
I’m going to mark this as resolved as it doesn’t sound like an issue caused by Mail Catcher. But feel free to send more information through and I can log it in our GitHub tracker if it turns out to be a bug with Mail Catcher.
Hope that helps!