Awight has uploaded a new change for review.

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

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(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig 
refs/changes/65/295665/1

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: newchange
Gerrit-Change-Id: Ib63671f7c50937a4261489abd48cb6fb6935e18c
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: deployment
Gerrit-Owner: Awight <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to