-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restore Default Options doesn't restore default options #181
Comments
This probably also affects Comment Mail Lite, but I haven't confirmed yet. |
Try changing this line. Instead of $args = array(__NAMESPACE__ => array('restore_default_options' => time())); I think this is a browser caching quirk. The redirection is getting cached. |
I think you're right. I realized I was testing on my personal site, where there are quite a few plugins. I just tested on my clean install and the Restore Defaults works fine. |
Confirmed. This fixes the issue. |
@renzms Could you submit a PR for me? For both Pro and Lite. |
@raamdev PRs submitted! |
Next Lite Release Changelog:
|
Next Pro Release Changelog:
|
Comment Mail v151224 has been released and includes changes from this GitHub Issue: See the v151224 announcement for further details. This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#181). |
Testing the dev branch (as of 2015-12-15) of Comment Mail Pro, clicking "Restore Default Options" doesn't actually restore any default options. You can test this by enabling the plugin or disabling data safeguards, saving options, and then clicking "Restore Default Options". The defaults (Comment Mail disabled, Data Safeguards enabled) should be restored, but they are not.
The text was updated successfully, but these errors were encountered: