CRM-12376
authorDonald A. Lobo <lobo@civicrm.org>
Thu, 18 Apr 2013 03:34:43 +0000 (20:34 -0700)
committerDonald A. Lobo <lobo@civicrm.org>
Thu, 18 Apr 2013 03:34:43 +0000 (20:34 -0700)
----------------------------------------
* CRM-12376: CiviMail job with no remaining deliveries gets stuck in Running
  http://issues.civicrm.org/jira/browse/CRM-12376

CRM/Mailing/BAO/Job.php

index 704484cedcc570885335613725b5ec565707606a..8c81602584ef9ae60c2ca6cf63d00e06aae954d4 100644 (file)
@@ -526,7 +526,10 @@ VALUES (%1, %2, %3, %4, %5, %6, %7)
       CRM_Core_Smarty::registerStringResource();
     }
 
-    $isDelivered = FALSE;
+    // CRM-12376
+    // This handles the edge case scenario where all the mails
+    // have been delivered in prior jobs
+    $isDelivered = TRUE;
 
     // make sure that there's no more than $config->mailerBatchLimit mails processed in a run
     while ($eq->fetch()) {
@@ -534,7 +537,8 @@ VALUES (%1, %2, %3, %4, %5, %6, %7)
       // CRM_Utils_System::xMemory( "$mailsProcessed: " );
       // }
 
-      if ($config->mailerBatchLimit > 0 &&
+      if (
+        $config->mailerBatchLimit > 0 &&
         $mailsProcessed >= $config->mailerBatchLimit
       ) {
         if (!empty($fields)) {