Email recursion bug

Resolved jomo
(@jonathanmoorebcsorg)

8 years, 9 months ago

Is there any known recursion bug on the email sending?

I just tried publishing 1 directory entry from the back end, the first entry: many hundreds of emails generated.

Viewing 4 replies - 1 through 4 (of 4 total)

Thread Starter jomo
(@jonathanmoorebcsorg)

8 years, 9 months ago

I got 700+ emails in this format before managing to stop it:

Your listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “[title] Listing “Inkston” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” published” is now available at https://yoursite.com/community/postman_sent_mail/your-listing-your-listing-your-listing-your-listing-site-326/ and can be viewed by the public.
Thread Starter jomo
(@jonathanmoorebcsorg)

8 years, 9 months ago
ok, it’s clearly the interaction between the Business Directory and Postman plugin which is being used to save and send email, but how it happens is due to weakness in the Business Directory code for example here from class-admin-listings.
```
    public function save_post( $post_id, $post) {
        if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
            return;

        // Handle listings saved admin-side.
        if ( is_admin() && isset( $_REQUEST['post_type'] ) && $_REQUEST['post_type'] == WPBDP_POST_TYPE ) {
            $listing = WPBDP_Listing::get( $post_id );
```
Here the filter is catching the save post which is actually the Email (postman is saving the outgoing email as a post).
Then since the $_REQUEST is still set (both the directory entry and the email have been saved in the same request), then the Business Directory reprocesses again, triggering another email, saved by Postman, triggering reprocessing by Business Directory…:
Plugin Author Business Directory Plugin
(@businessdirectoryplugin)

8 years, 9 months ago

Hi @jonathanmoorebcsorg,

Sorry that you ran into issues here!

I reported this to my developer on Sunday and he already pushed a fix for me today in our dev version.

Contact us here and I’d be happy to provide it immediately: http://businessdirectoryplugin.com/contact/

Otherwise, it will be in our next public release.

Thread Starter jomo
(@jonathanmoorebcsorg)

8 years, 9 months ago

Great response thanks!
I have a temporary fix so I’ll await the next release.

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘Email recursion bug’ is closed to new replies.