Mwalker has submitted this change and it was merged.
Change subject: Explicit about which date we log
......................................................................
Explicit about which date we log
The constructed message could have been normalized to use recurring keys by
this point, so we cannot depend on out['date'].
Change-Id: Ie63820c092331d077fcf66cfbf70aec7eeedeb78
---
M audit/paypal/TrrFile.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Mwalker: Looks good to me, approved
diff --git a/audit/paypal/TrrFile.py b/audit/paypal/TrrFile.py
index 0551944..c2ff81f 100644
--- a/audit/paypal/TrrFile.py
+++ b/audit/paypal/TrrFile.py
@@ -106,7 +106,7 @@
return
if self.crm.transaction_exists(gateway_txn_id=out['gateway_txn_id'],
gateway='paypal'):
-
log.info("-Duplicate\t{id}\t{date}\t{type}".format(id=out['gateway_txn_id'],
date=out['date'], type=queue))
+
log.info("-Duplicate\t{id}\t{date}\t{type}".format(id=out['gateway_txn_id'],
date=row['Transaction Initiation Date'], type=queue))
return
if 'last_name' not in out and queue != 'refund':
--
To view, visit https://gerrit.wikimedia.org/r/104690
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie63820c092331d077fcf66cfbf70aec7eeedeb78
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/tools
Gerrit-Branch: master
Gerrit-Owner: Adamw <awi...@wikimedia.org>
Gerrit-Reviewer: Katie Horn <kh...@wikimedia.org>
Gerrit-Reviewer: Mwalker <mwal...@wikimedia.org>
Gerrit-Reviewer: Ssmith <ssm...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
