Cdentinger has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/321016

Change subject: Don't throw exception on IPN verify failure
......................................................................

Don't throw exception on IPN verify failure

It's being super annoying with the email

Change-Id: Icaf05cb569e05744d20ba78af71639059db787e3
---
M PaymentProviders/PayPal/PayPalPaymentsAPI.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig 
refs/changes/16/321016/1

diff --git a/PaymentProviders/PayPal/PayPalPaymentsAPI.php 
b/PaymentProviders/PayPal/PayPalPaymentsAPI.php
index 9eb9b49..c6c78eb 100644
--- a/PaymentProviders/PayPal/PayPalPaymentsAPI.php
+++ b/PaymentProviders/PayPal/PayPalPaymentsAPI.php
@@ -45,7 +45,7 @@
                } elseif ( $response === 'INVALID' ) {
                        return false;
                } else {
-                       throw new RuntimeException(
+                       Logger::debug(
                                "Unknown response from PayPal IPN PB: 
[{$response}].\n" .
                                "Verbose logging: $logged"
                        );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icaf05cb569e05744d20ba78af71639059db787e3
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Cdentinger <cdentin...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to