Ejegg has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/373101 )
Change subject: WX parser: set gateway_txn_id on refunds
......................................................................
WX parser: set gateway_txn_id on refunds
Note that it's the same as the parent id. Unhelpful, but that's how
we've been doing things.
Bug: T173457
Change-Id: I3d57121f02e47b68a3c4eb0e521ec3aa189c8ff6
---
M PaymentProviders/Ingenico/Audit/IngenicoAudit.php
M PaymentProviders/Ingenico/Tests/phpunit/AuditTest.php
2 files changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig
refs/changes/01/373101/1
diff --git a/PaymentProviders/Ingenico/Audit/IngenicoAudit.php
b/PaymentProviders/Ingenico/Audit/IngenicoAudit.php
index 2de3b84..5d03343 100644
--- a/PaymentProviders/Ingenico/Audit/IngenicoAudit.php
+++ b/PaymentProviders/Ingenico/Audit/IngenicoAudit.php
@@ -120,6 +120,9 @@
protected function parseRefund( DOMElement $recordNode, $type ) {
$record = $this->xmlToArray( $recordNode, $this->refundMap );
$record['type'] = $type;
+ // FIXME: Transaction ID is the same as the parent transaction
ID.
+ // That's not helpful...
+ $record['gateway_txn_id'] = $record['gateway_parent_id'];
if ( $record['installment'] < 0 ) {
// Refunds have negative EffortID. Weird.
// TODO: for refunds of recurring payments, determine
whether the
diff --git a/PaymentProviders/Ingenico/Tests/phpunit/AuditTest.php
b/PaymentProviders/Ingenico/Tests/phpunit/AuditTest.php
index 5b68013..a5d6d8e 100644
--- a/PaymentProviders/Ingenico/Tests/phpunit/AuditTest.php
+++ b/PaymentProviders/Ingenico/Tests/phpunit/AuditTest.php
@@ -85,6 +85,7 @@
'date' => 1500942220,
'gross' => 100,
'gateway_parent_id' => '123456789',
+ 'gateway_txn_id' => '123456789',
'installment' => 1,
'gross_currency' => 'USD',
'type' => 'refund',
--
To view, visit https://gerrit.wikimedia.org/r/373101
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3d57121f02e47b68a3c4eb0e521ec3aa189c8ff6
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
