jenkins-bot has submitted this change and it was merged.
Change subject: Send IsCVNMEM with PT request
......................................................................
Send IsCVNMEM with PT request
Also minor cleanup.
Change-Id: Iba4af71054d08ed620226956af076f1975ee13ca
---
M worldpay_gateway/worldpay.adapter.php
1 file changed, 12 insertions(+), 7 deletions(-)
Approvals:
Awight: Looks good to me, approved
jenkins-bot: Verified
diff --git a/worldpay_gateway/worldpay.adapter.php
b/worldpay_gateway/worldpay.adapter.php
index fb9114b..0e7703c 100644
--- a/worldpay_gateway/worldpay.adapter.php
+++ b/worldpay_gateway/worldpay.adapter.php
@@ -844,13 +844,13 @@
case 'GenerateToken':
if ( $this->isESOP() ) {
// This parameter will cause WP to use
the iframe code path.
-
$this->transactions['GenerateToken']['request'][] = 'IsHosted';
-
$this->transactions['GenerateToken']['values']['IsHosted'] = 1;
+
$this->transactions[$transaction]['request'][] = 'IsHosted';
+
$this->transactions[$transaction]['values']['IsHosted'] = 1;
// Tell WP we want that CVV field.
-
$this->transactions['GenerateToken']['request'][] = 'IsCVNMEM';
-
$this->transactions['GenerateToken']['values']['IsCVNMEM'] = 1;
+
$this->transactions[$transaction]['request'][] = 'IsCVNMEM';
+
$this->transactions[$transaction]['values']['IsCVNMEM'] = 1;
// Translate the iframe to our language.
-
$this->transactions['GenerateToken']['request'][] = 'LAN';
+
$this->transactions[$transaction]['request'][] = 'LAN';
}
$result = parent::do_transaction( $transaction
);
@@ -869,9 +869,14 @@
break;
case 'AuthorizePaymentForFraud':
- if ( ! $this->isESOP() ) {
+ if ( $this->isESOP() ) {
+ // WP said we should send this here.
+
$this->transactions[$transaction]['request'][] = 'IsCVNMEM';
+
$this->transactions[$transaction]['values']['IsCVNMEM'] = 1;
+ } else {
// Perform CVV and AVS verification for
account (deposit not allowed)
-
$this->transactions['AuthorizePaymentForFraud']['values']['IsVerify'] = 1;
+
$this->transactions[$transaction]['request'][] = 'IsVerify';
+
$this->transactions[$transaction]['values']['IsVerify'] = 1;
}
$this->addRequestData( array( 'cvv' =>
$this->get_cvv() ) );
$this->store_cvv_in_session( null ); // Remove
the CVV from the session
--
To view, visit https://gerrit.wikimedia.org/r/245980
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iba4af71054d08ed620226956af076f1975ee13ca
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/DonationInterface
Gerrit-Branch: master
Gerrit-Owner: Cdentinger <[email protected]>
Gerrit-Reviewer: AndyRussG <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Cdentinger <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: Ssmith <[email protected]>
Gerrit-Reviewer: XenoRyet <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
