Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix post notifications to work with dynamic segments

Browse Source 
[PREMIUM-38]
This commit is contained in:
Pavel Dohnal
2017-10-17 15:30:38 +01:00
parent f1b373924f
commit 16beda530a
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Cron/Workers/Scheduler.php
View File

@@ -7,6 +7,7 @@ use MailPoet\Models\Newsletter;
use MailPoet\Models\SendingQueue;
use MailPoet\Models\Subscriber;
use MailPoet\Models\SubscriberSegment;
use MailPoet\Segments\SubscribersFinder;
use MailPoet\Util\Helpers;
use MailPoet\Newsletter\Scheduler\Scheduler as NewsletterScheduler;
@@ -73,12 +74,11 @@ class Scheduler {
if(empty($segments)) {
return $this->deleteQueueOrUpdateNextRunDate($queue, $newsletter);
}
$segment_ids = array_map(function($segment) {
return (int)$segment['id'];
}, $segments);
// ensure that subscribers are in segments
$subscribers = Subscriber::getSubscribedInSegments($segment_ids)->findArray();
$finder = new SubscribersFinder();
$subscribers = $finder->getSubscribersByList($segments);
$subscribers = Helpers::flattenArray($subscribers);
if(empty($subscribers)) {