Mwalker has submitted this change and it was merged.

Change subject: send all the... required fields
......................................................................


send all the... required fields

Change-Id: I207245cc2a0ac876b058c4e447ecd242cac69aa8
---
M PaymentProviders/GlobalCollect/ExpatriatedMessages/GlobalCollectMessage.php
M PaymentProviders/GlobalCollect/ExpatriatedMessages/PaymentMessage.php
2 files changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Mwalker: Verified; Looks good to me, approved



diff --git 
a/PaymentProviders/GlobalCollect/ExpatriatedMessages/GlobalCollectMessage.php 
b/PaymentProviders/GlobalCollect/ExpatriatedMessages/GlobalCollectMessage.php
index 22bcf9b..4720087 100644
--- 
a/PaymentProviders/GlobalCollect/ExpatriatedMessages/GlobalCollectMessage.php
+++ 
b/PaymentProviders/GlobalCollect/ExpatriatedMessages/GlobalCollectMessage.php
@@ -39,6 +39,10 @@
                        $queueMsg->$destKey = $this->$key;
                }
 
+               if ( !property_exists( $queueMsg, 'email' ) ) {
+                       $queueMsg->email = '[email protected]';
+               }
+
                $queueMsg->gateway = 'globalcollect';
                $queueMsg->correlationId = 
"{$queueMsg->gateway}-{$queueMsg->order_id}";
 
diff --git 
a/PaymentProviders/GlobalCollect/ExpatriatedMessages/PaymentMessage.php 
b/PaymentProviders/GlobalCollect/ExpatriatedMessages/PaymentMessage.php
index 17befeb..ddc5020 100644
--- a/PaymentProviders/GlobalCollect/ExpatriatedMessages/PaymentMessage.php
+++ b/PaymentProviders/GlobalCollect/ExpatriatedMessages/PaymentMessage.php
@@ -22,7 +22,7 @@
                'currency_code' => array('map' => 'currency'),
                'effort_id',
                'merchant_id',
-               'order_id',
+               'order_id' => array('map' => 'gateway_txn_id'),
                'payment_method_id',
                'payment_product_id' => array('map' => 'payment_product'),
                'payment_reference',

-- 
To view, visit https://gerrit.wikimedia.org/r/108741
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I207245cc2a0ac876b058c4e447ecd242cac69aa8
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Adamw <[email protected]>
Gerrit-Reviewer: Katie Horn <[email protected]>
Gerrit-Reviewer: Mwalker <[email protected]>

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

Reply via email to