nstaeger / wp-post-email-notification Goto Github PK

• category

See https://wordpress.org/support/topic/additional-placeholders/

Accidentally an email was sent when new menu-items where created for a menu.

Is there a simple way to make the plugin work with a custom post type?

• Place holder text
• Submit button label

• Initial timeout when scheduling the job (https://github.com/nstaeger/wp-post-email-notification/blob/master/src/Model/JobModel.php#L39)
• Timeout between batches (https://github.com/nstaeger/wp-post-email-notification/blob/master/src/Model/JobModel.php#L126)

My site title contains an ampersand (it is a "private" site, so I will not link it here) and in the email subject which is templated with (in the admin): Nouveau post sur @@blog.name!

When rendered on my email, it reads as Nouveau post sur toto & tata!

Hello,

I realise that this plugin is (probably) no longer supported, but can you update it, to allow deletion of the plugin, in later versions of Wordpress?

If I try to delete it through the Wordpress plugin page, I get an 'Deletion failed: There has been a critical error on your website' message and the following in the PHP log:

[23-May-2020 11:15:35 UTC] PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function Nstaeger\WpPostEmailNotification\Model\JobModel::__construct(), 1 passed in /home/user/public_html/travel/wp-content/plugins/wp-post-email-notification/uninstall.php on line 24 and exactly 2 expected in /home/user/public_html/travel/wp-content/plugins/wp-post-email-notification/src/Model/JobModel.php:23
Stack trace:
#0 /home/user/public_html/travel/wp-content/plugins/wp-post-email-notification/uninstall.php(24): Nstaeger\WpPostEmailNotification\Model\JobModel->__construct(Object(Nstaeger\CmsPluginFramework\Broker\Wordpress\WordpressDatabaseBroker))
#1 /home/user/public_html/travel/wp-admin/includes/plugin.php(1222): include('/home/user/...')
#2 /home/user/public_html/travel/wp-admin/includes/plugin.php(956): uninstall_plugin('wp-post-email-n...')
#3 /home/user/public_html/travel/wp-admin/includes/ajax-actions.php(4545): delete_plugins(Array)
#4 /home/user/public_html/travel/wp-includes/class-wp-hook.php(287): wp_ajax_de in /home/user/public_html/travel/wp-content/plugins/wp-post-email-notification/src/Model/JobModel.php on line 23

Thanks.

• total number of people mail was sent to
• no of email reads
• no of email link clicks
• open rate & click rates
• if possible who clicked

one of your vendor scripts is calling wp_localize_script wrong the last param should be an array here

plugins/wp-post-email-notification/vendor/nstaeger/cms-plugin-framework/src/Broker/Wordpress/WordpressAssetBroker.php: wp_localize_script($asset->getName(), 'ajaxurl', admin_url('admin-ajax.php'));

should be

./assets/plugins/wp-post-email-notification/vendor/nstaeger/cms-plugin-framework/src/Broker/Wordpress/WordpressAssetBroker.php: wp_localize_script($asset->getName(), 'ajaxurl', array ( admin_url('admin-ajax.php') ) );

Could e.g. be done by setting a value within the database (wp-options or jobs-table) before fetching the next batch of users and updating the job.