[MediaWiki-commits] [Gerrit] wikimedia...crm[master]: Fix AntifraudQueueConsumer constructor arguments

2016-08-30 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged.

Change subject: Fix AntifraudQueueConsumer constructor arguments
..


Fix AntifraudQueueConsumer constructor arguments

Time limit comes first, take the default 0 message limit.

Will fix payments-init args in another patch.

Change-Id: Ic68f832f2cbbba63ced561ded2010eda98effe07
---
M sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  AndyRussG: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module 
b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
index 9ae179b..b65b94f 100644
--- a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
+++ b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
@@ -82,7 +82,6 @@
 
   $fraudQueueConsumer = new AntifraudQueueConsumer(
 variable_get('fredge_payments_antifraud_queue', 'payments-antifraud'),
-false,
 $cycle_time
   );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/307680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic68f832f2cbbba63ced561ded2010eda98effe07
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Ejegg 
Gerrit-Reviewer: AndyRussG 
Gerrit-Reviewer: Awight 
Gerrit-Reviewer: Cdentinger 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] wikimedia...crm[master]: Fix AntifraudQueueConsumer constructor arguments

2016-08-30 Thread Ejegg (Code Review) 
Ejegg has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/307680

Change subject: Fix AntifraudQueueConsumer constructor arguments
..

Fix AntifraudQueueConsumer constructor arguments

Time limit comes first, message limit should be numeric.

Will fix payments-init args in another patch.

Change-Id: Ic68f832f2cbbba63ced561ded2010eda98effe07
---
M sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm 
refs/changes/80/307680/1

diff --git a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module 
b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
index 9ae179b..2e82b47 100644
--- a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
+++ b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
@@ -82,8 +82,8 @@
 
   $fraudQueueConsumer = new AntifraudQueueConsumer(
 variable_get('fredge_payments_antifraud_queue', 'payments-antifraud'),
-false,
-$cycle_time
+$cycle_time,
+0
   );
 
   $processed += $fraudQueueConsumer->dequeueMessages();

-- 
To view, visit https://gerrit.wikimedia.org/r/307680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic68f832f2cbbba63ced561ded2010eda98effe07
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Ejegg 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits