Skip to content
Snippets Groups Projects
Commit 90e1c7f7 authored by mattwire's avatar mattwire
Browse files

Response to test webhook now includes payment processor name from CiviCRM...

Response to test webhook now includes payment processor name from CiviCRM (makes it easier to be sure which payment processor is responding)
parent fc72ff62
No related branches found
No related tags found
No related merge requests found
...@@ -119,8 +119,9 @@ class CRM_Core_Payment_StripeIPN { ...@@ -119,8 +119,9 @@ class CRM_Core_Payment_StripeIPN {
// Special case if this is the test webhook // Special case if this is the test webhook
if (substr($parameters->id, -15, 15) === '_00000000000000') { if (substr($parameters->id, -15, 15) === '_00000000000000') {
http_response_code(200); http_response_code(200);
$test = (boolean) $this->_paymentProcessor->getPaymentProcessor()['is_test'] ? '(Test processor)' : '(Live processor)'; $test = (boolean) $this->_paymentProcessor->getPaymentProcessor()['is_test'] ? '(Test)' : '(Live)';
echo "Test webhook from Stripe ({$parameters->id}) received successfully by CiviCRM {$test}."; $name = $this->_paymentProcessor->getPaymentProcessor()['name'];
echo "Test webhook from Stripe ({$parameters->id}) received successfully by CiviCRM: {$name} {$test}.";
exit(); exit();
} }
......
...@@ -9,11 +9,12 @@ Releases use the following numbering system: ...@@ -9,11 +9,12 @@ Releases use the following numbering system:
* **[BC]**: Items marked with [BC] indicate a breaking change that will require updates to your code if you are using that code in your extension. * **[BC]**: Items marked with [BC] indicate a breaking change that will require updates to your code if you are using that code in your extension.
## Release 6.5.4 (not yet released 2020-11-01) ## Release 6.5.4 (not yet released 2020-11-06)
**This release REQUIRES that you upgrade mjwshared to 0.9.6**. **This release REQUIRES that you upgrade mjwshared to 0.9.6**.
* Use `CRM.payment.isAJAXPaymentForm()` to check if we should load on backend forms. There is no change for the user, we are just switching to an identical shared function to reduce code duplication. * Use `CRM.payment.isAJAXPaymentForm()` to check if we should load on backend forms. There is no change for the user, we are just switching to an identical shared function to reduce code duplication.
* Fix [#272](https://lab.civicrm.org/extensions/stripe/-/issues/272) "undefined property" PHP notice. * Fix [#272](https://lab.civicrm.org/extensions/stripe/-/issues/272) "undefined property" PHP notice.
* Response to test webhook now includes payment processor name from CiviCRM (makes it easier to be sure which payment processor is responding).
## Release 6.5.3 ## Release 6.5.3
......
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
<author>Matthew Wire (MJW Consulting)</author> <author>Matthew Wire (MJW Consulting)</author>
<email>mjw@mjwconsult.co.uk</email> <email>mjw@mjwconsult.co.uk</email>
</maintainer> </maintainer>
<releaseDate>2020-11-01</releaseDate> <releaseDate>2020-11-06</releaseDate>
<version>6.5.4-dev</version> <version>6.5.4-dev</version>
<develStage>beta</develStage> <develStage>beta</develStage>
<compatibility> <compatibility>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment