Awight has submitted this change and it was merged.
Change subject: Don't try to serialize Capture job logger
......................................................................
Don't try to serialize Capture job logger
Oops, this caused a failmail and prevented sending the job to the
damaged queue
Change-Id: Ib63671f7c50937a4261489abd48cb6fb6935e18c
---
M PaymentProviders/Adyen/Jobs/ProcessCaptureRequestJob.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Awight: Verified; Looks good to me, approved
diff --git a/PaymentProviders/Adyen/Jobs/ProcessCaptureRequestJob.php
b/PaymentProviders/Adyen/Jobs/ProcessCaptureRequestJob.php
index c4f556c..68092c7 100644
--- a/PaymentProviders/Adyen/Jobs/ProcessCaptureRequestJob.php
+++ b/PaymentProviders/Adyen/Jobs/ProcessCaptureRequestJob.php
@@ -30,7 +30,7 @@
* @var TaggedLogger
*/
protected $logger;
-
+ protected $propertiesExcludedFromExport = array( 'logger' );
// Actions to take after examining capture request and queue message
const ACTION_PROCESS = 'process'; // all clear to capture payment
const ACTION_REJECT = 'reject'; // very likely fraud - cancel the
authorization
--
To view, visit https://gerrit.wikimedia.org/r/295665
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib63671f7c50937a4261489abd48cb6fb6935e18c
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: deployment
Gerrit-Owner: Awight <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Cdentinger <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
