Stripe merge requestshttps://lab.civicrm.org/extensions/stripe/-/merge_requests2022-09-22T20:42:00Zhttps://lab.civicrm.org/extensions/stripe/-/merge_requests/198handle null balance trxn id when calculating fees2022-09-22T20:42:00ZJonGoldhandle null balance trxn id when calculating feesI had an invoice that is a pro-rated refund of a recurring payment. When I run the `Stripe.retryall` API, this crashes in `CRM_Core_Payment_StripeIPN::setInfo()` because we try to set a fee when the `$balanceTransactionID` is `NULL`.
Si...I had an invoice that is a pro-rated refund of a recurring payment. When I run the `Stripe.retryall` API, this crashes in `CRM_Core_Payment_StripeIPN::setInfo()` because we try to set a fee when the `$balanceTransactionID` is `NULL`.
Since `CRM_Core_Payment_Stripe::getFeeFromBalanceTransaction()` is clearly set up to handle this scenario (based on the docblock and the `if` statement) I've changed the function signature to allow a `NULL` value.6.7https://lab.civicrm.org/extensions/stripe/-/merge_requests/197ensure webhook data is populated.2022-09-22T20:41:51Zjamieensure webhook data is populated.This seems to fix https://lab.civicrm.org/extensions/stripe/-/issues/388 in the least intrusive way.
But I'm not sure it's right:
1. I'm not sure why I have to strip off "Stripe\StripeObject JSON: " from the string stored in the databa...This seems to fix https://lab.civicrm.org/extensions/stripe/-/issues/388 in the least intrusive way.
But I'm not sure it's right:
1. I'm not sure why I have to strip off "Stripe\StripeObject JSON: " from the string stored in the database. Should that be saved differently?
2. I'm not sure we should trust data stored in the webhook table. Maybe it would be better to do away with that call and simply set `setVerifyData(TRUE)` to always verify the data.6.7https://lab.civicrm.org/extensions/stripe/-/merge_requests/184Convert webhook checks to non static class. Fix #3692022-05-08T15:29:07Zmattwiremjw@mjwconsult.co.ukConvert webhook checks to non static class. Fix #3696.7https://lab.civicrm.org/extensions/stripe/-/merge_requests/176Release 6.72022-01-28T12:26:03Zmattwiremjw@mjwconsult.co.ukRelease 6.76.7https://lab.civicrm.org/extensions/stripe/-/merge_requests/169WIP notes on custom integrations2022-01-28T12:19:19ZRichWIP notes on custom integrations@mattwire I've begun making notes about how a custom integration can use Stripe.@mattwire I've begun making notes about how a custom integration can use Stripe.6.7https://lab.civicrm.org/extensions/stripe/-/merge_requests/173Add unit tests for Stripe.Importsubscription/ImportCharge API. Support...2021-12-04T12:35:08Zmattwiremjw@mjwconsult.co.ukAdd unit tests for Stripe.Importsubscription/ImportCharge API. Support...Add unit tests for Stripe.Importsubscription/ImportCharge API. Support importing subscriptions with future startdate/trial period (no invoice so we create a template contribution)Add unit tests for Stripe.Importsubscription/ImportCharge API. Support importing subscriptions with future startdate/trial period (no invoice so we create a template contribution)6.7