Mwalker has submitted this change and it was merged.

Change subject: when response is REFUSED, operations=null
......................................................................


when response is REFUSED, operations=null

Change-Id: I90054bd914ab53b5376b84874a24f031bec3aa22
---
M SmashPig/PaymentProviders/Adyen/ExpatriatedMessages/Authorisation.php
1 file changed, 3 insertions(+), 3 deletions(-)

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



diff --git 
a/SmashPig/PaymentProviders/Adyen/ExpatriatedMessages/Authorisation.php 
b/SmashPig/PaymentProviders/Adyen/ExpatriatedMessages/Authorisation.php
index 4af9606..edeac2b 100644
--- a/SmashPig/PaymentProviders/Adyen/ExpatriatedMessages/Authorisation.php
+++ b/SmashPig/PaymentProviders/Adyen/ExpatriatedMessages/Authorisation.php
@@ -28,10 +28,10 @@
 
                $this->paymentMethod = $msgObj->paymentMethod;
 
-               if ( is_array( $msgObj->operations->string ) ) {
-                       $this->operations = $msgObj->operations->string;
+               if ( $msgObj->operations ) {
+                       $this->operations = (array)$msgObj->operations->string;
                } else {
-                       $this->operations = array( $msgObj->operations->string 
);
+                       $this->operations = array();
                }
 
                $this->reason = $msgObj->reason;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I90054bd914ab53b5376b84874a24f031bec3aa22
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/PaymentsListeners
Gerrit-Branch: master
Gerrit-Owner: Adamw <[email protected]>
Gerrit-Reviewer: Mwalker <[email protected]>
Gerrit-Reviewer: Siebrand <[email protected]>

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

Reply via email to