Ejegg has uploaded a new change for review.
https://gerrit.wikimedia.org/r/319373
Change subject: Downgrade missing details to warning for RecordCaptureJob
......................................................................
Downgrade missing details to warning for RecordCaptureJob
We'll eventually get them from the logs when we parse the audit.
Change-Id: Id82c4ec731c4e9652db6571bd9fcaa30e469327e
---
M PaymentProviders/Adyen/Jobs/RecordCaptureJob.php
1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig
refs/changes/73/319373/1
diff --git a/PaymentProviders/Adyen/Jobs/RecordCaptureJob.php
b/PaymentProviders/Adyen/Jobs/RecordCaptureJob.php
index 2db5924..8e23090 100644
--- a/PaymentProviders/Adyen/Jobs/RecordCaptureJob.php
+++ b/PaymentProviders/Adyen/Jobs/RecordCaptureJob.php
@@ -65,7 +65,12 @@
$db->deleteMessage( $dbMessage );
} else {
- $logger->error(
+ // Sometimes we don't have a pending db row because the
donor made
+ // multiple attempts with the same order ID. It would
be nice if
+ // Adyen could prevent that, but let's not send a
failmail since
+ // we'll eventually get the donor details from the
payments log
+ // when we parse the audit.
+ $logger->warning(
"Could not find a processable message for
authorization Reference '{$this->originalReference}' " .
"and order ID
'{$this->merchantReference}'.",
$dbMessage
--
To view, visit https://gerrit.wikimedia.org/r/319373
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id82c4ec731c4e9652db6571bd9fcaa30e469327e
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
