Broadcast sends email to all all subscribers of other newsletters

[speedy200man]

Hello! Thank you so much for this amazing plugin. My tests show that when a send a broadcast to 1 newsletter subscribers, all subscribers of all lists get the email. Can you please fix this ?

Thank you !

[speedy200man]

Hi guys! I made a fix for this issue.

Step 1: In the source code of newmail.php please replace the line 24 with $recipients = (!empty($_POST['recipients']))?$_POST['recipients']:$nid;

The default behavior is to populate the column in the database with the value from this field. However, the javascript library / function is doing nothing, hence the first issue related to the code.

Step 2:

In the source of processes/broadcast_processor.php, the query used to filter the recipients is not correct. so, on line 34 you should replace the old query with the new one

$getSubscribersForBroadcastQuery = sprintf( "SELECT subscribers.* FROM {$wpdb->prefix}wpr_subscribers as subscribers LEFT JOIN {$wpdb->prefix}wpr_newsletters as newsletters on (newsletters.id=subscribers.fid) WHERE 1 AND subscribers.active=1 AND subscribers.confirmed=1 AND subscribers.fid='".$nid."' " );

This will take in account only the subscribers that enrolled to the specified newsletter in the broadcast record.

Hope this helps and thank you again for this plugin! Cezar

[lukeheights]

Cezar, How do I get the latest version of the plugin? Do I need to make all the code changes you have been posting individually? Thanks

[speedy200man]

Hey!

I see that the work on this plugin has been discontinued. Making the changes posted by me will make the plugin work correctly.

Have fun ! Cezar

---

From: lukeheights [email protected] To: nodesman/javelin [email protected] Cc: speedy200man [email protected] Sent: Thursday, January 2, 2014 11:38 PM Subject: Re: [javelin] Broadcast sends email to all all subscribers of other newsletters (#282)

Cezar, How do I get the latest version of the plugin? Do I need to make all the code changes you have been posting individually? Thanks — Reply to this email directly or view it on GitHub.

[GordonFreeman]

It's actually not discontinued.

[speedy200man]

It was written on the plugin page, sorry if I misunderstood...

---

From: Gordon [email protected] To: nodesman/javelin [email protected] Cc: speedy200man [email protected] Sent: Friday, January 3, 2014 11:27 AM Subject: Re: [javelin] Broadcast sends email to all all subscribers of other newsletters (#282)

It's actually not discontinued. — Reply to this email directly or view it on GitHub.