From e62ecc50369a92f811e3d44ce6fd0838cdec0a06 Mon Sep 17 00:00:00 2001 From: Vlad Date: Mon, 4 Apr 2016 21:09:15 -0400 Subject: [PATCH] - Fixes duplicate detection check --- lib/Cron/Workers/Scheduler.php | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/Cron/Workers/Scheduler.php b/lib/Cron/Workers/Scheduler.php index 162b4c083e..ac81f872fe 100644 --- a/lib/Cron/Workers/Scheduler.php +++ b/lib/Cron/Workers/Scheduler.php @@ -69,11 +69,7 @@ class Scheduler { ->findArray(); $subscribers = Helpers::arrayColumn($subscribers, 'subscriber_id'); $subscribers = array_unique($subscribers); - if(!count($subscribers)) { - $queue->delete(); - return; - } - if(!$this->checkIfNewsletterChanged($newsletter)) { + if(!count($subscribers) || !$this->checkIfNewsletterChanged($newsletter)) { $queue->scheduled_at = $next_run_date; $queue->save(); return; @@ -141,7 +137,7 @@ class Scheduler { $rendered_newsletter = $renderer->render(); $new_hash = md5($rendered_newsletter['html']); $old_hash = $last_run_queue->newsletter_rendered_body_hash; - return $new_hash !== $old_hash; + return $new_hash === $old_hash; } private function getQueueNextRunDate($schedule) {