From e27a3472924c29bd36ff3917a65114e50774a9dd Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <eileen@fuzion.co.nz>
Date: Thu, 13 Mar 2014 17:49:13 +1300
Subject: [PATCH] CRM-14319 - alter job api to not limit scheduled reminders

----------------------------------------
* CRM-14319: not all scheduled reminders are being sent
  http://issues.civicrm.org/jira/browse/CRM-14319
---
 api/v3/Job.php | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/api/v3/Job.php b/api/v3/Job.php
index 1c07ac6515..0cecad4303 100644
--- a/api/v3/Job.php
+++ b/api/v3/Job.php
@@ -185,6 +185,11 @@ function _civicrm_api3_job_geocode_spec(&$params) {
  *
  */
 function civicrm_api3_job_send_reminder($params) {
+  //note that $params['rowCount' can be overridden by one of the preferred syntaxes ($options['limit'] = x
+  //It's not clear whether than syntax can be passed in via the UI config - but this keeps the pre 4.4.4 behaviour
+  // in that case (ie. makes it unconfigurable via the UI). Another approach would be to set a default of 0
+  // in the _spec function - but since that is a deprecated value it seems more contentious than this approach
+  $params['rowCount'] = 0;
   $lock = new CRM_Core_Lock('civimail.job.EmailProcessor');
   if (!$lock->isAcquired()) {
     return civicrm_api3_create_error('Could not acquire lock, another EmailProcessor process is running');
-- 
2.25.1