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

If newsletter has filterSegmentId, pause if it doesn't exist

Browse Source 
MAILPOET-5509
This commit is contained in:
John Oleksowicz
2023-08-11 14:30:52 -05:00
committed by Aschepikov
parent 2a97189d97
commit 9b8c4910c1
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
mailpoet/lib/Cron/Workers/SendingQueue/SendingQueue.php
View File

@@ -197,8 +197,15 @@ class SendingQueue {
$this->mailerTask->configureMailer($newsletter); $this->mailerTask->configureMailer($newsletter);
// get newsletter segments // get newsletter segments
$newsletterSegmentsIds = $newsletterEntity->getSegmentIds(); $newsletterSegmentsIds = $newsletterEntity->getSegmentIds();
$segmentIdsToCheck = $newsletterSegmentsIds;
$filterSegmentId = $newsletterEntity->getFilterSegmentId();
if (is_int($filterSegmentId)) {
$segmentIdsToCheck[] = $filterSegmentId;
}
// Pause task in case some of related segments was deleted or trashed // Pause task in case some of related segments was deleted or trashed
if ($newsletterSegmentsIds && !$this->checkDeletedSegments($newsletterSegmentsIds)) { if ($newsletterSegmentsIds && !$this->checkDeletedSegments($segmentIdsToCheck)) {
$this->loggerFactory->getLogger(LoggerFactory::TOPIC_NEWSLETTERS)->info( $this->loggerFactory->getLogger(LoggerFactory::TOPIC_NEWSLETTERS)->info(
'pause task in sending queue due deleted or trashed segment', 'pause task in sending queue due deleted or trashed segment',
['task_id' => $queue->taskId] ['task_id' => $queue->taskId]