- Removes check for signup confirmation when selecting subscribers
This commit is contained in:
@@ -65,15 +65,8 @@ class Scheduler {
|
||||
function processPostNotificationNewsletter($newsletter, $queue) {
|
||||
$next_run_date = $this->getQueueNextRunDate($newsletter->schedule);
|
||||
$segments = unserialize($newsletter->segments);
|
||||
if((boolean) Setting::getValue('signup_confirmation.enabled')) {
|
||||
$subscribers = Subscriber::getSubscribedInSegments($segments)
|
||||
->findArray();
|
||||
}
|
||||
else {
|
||||
$subscribers = SubscriberSegment::whereIn('segment_id', $segments)
|
||||
->where('status', 'subscribed')
|
||||
->findArray();
|
||||
}
|
||||
$subscribers = Subscriber::getSubscribedInSegments($segments)
|
||||
->findArray();
|
||||
$subscribers = Helpers::arrayColumn($subscribers, 'subscriber_id');
|
||||
$subscribers = array_unique($subscribers);
|
||||
if(!count($subscribers) || !$this->checkIfNewsletterChanged($newsletter)) {
|
||||
|
@@ -84,17 +84,9 @@ class SendingQueue {
|
||||
$queue = \MailPoet\Models\SendingQueue::create();
|
||||
$queue->newsletter_id = $newsletter->id;
|
||||
|
||||
if((boolean) Setting::getValue('signup_confirmation.enabled')) {
|
||||
$subscribers = Subscriber::getSubscribedInSegments($data['segments'])
|
||||
->findArray();
|
||||
}
|
||||
else {
|
||||
$subscribers = SubscriberSegment::whereIn('segment_id', $data['segments'])
|
||||
->where('status', 'subscribed')
|
||||
->findArray();
|
||||
}
|
||||
$subscribers = Subscriber::getSubscribedInSegments($data['segments'])
|
||||
->findArray();
|
||||
$subscribers = Helpers::arrayColumn($subscribers, 'subscriber_id');
|
||||
|
||||
$subscribers = array_unique($subscribers);
|
||||
if(!count($subscribers)) {
|
||||
return array(
|
||||
|
Reference in New Issue
Block a user