Dev/Core 25 Wrap the mailing split_job function in a transaction to e…

…nsure that there is no possibility of duplicate maiing jobs created
civicrm · Mar 20, 2018 · e286514 · e286514
1 parent 52fab18
commit e286514
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/CRM/Mailing/BAO/MailingJob.php b/CRM/Mailing/BAO/MailingJob.php
@@ -402,7 +402,7 @@ public function split_job($offset = 200) {
     $recipient_count = CRM_Mailing_BAO_Recipients::mailingSize($this->mailing_id);
 
     $jobTable = CRM_Mailing_DAO_MailingJob::getTableName();
-
+    $transaction = new CRM_Core_Transaction();
     $dao = new CRM_Core_DAO();
 
     $sql = "
@@ -437,6 +437,7 @@ public function split_job($offset = 200) {
         CRM_Core_DAO::executeQuery($sql, $params);
       }
     }
+    $transaction->commit();
   }
 
   /**