laravel-cashier-stripe-connect
laravel-cashier-stripe-connect copied to clipboard
PHP8 deprecating warning
Hi there,
I'm getting this warning when running on PHP8
Deprecated: Required parameter $amount follows optional parameter $refundFee in /Users/Code/project/vendor/expdev07/laravel-cashier-stripe-connect/src/Concerns/ManagesTransfer.php on line 53
As far as I can tell, this is due to PHP 8 requiring required arg first, before optional items per this article
public function reverseTransferFromStripeAccount(Transfer $transfer, $refundFee = false, ?int $amount, array $options = []): TransferReverse
could be altered to
public function reverseTransferFromStripeAccount(Transfer $transfer, ?int $amount, $refundFee = false, array $options = []): TransferReverse
but this is a breaking change, I'd imagine
Could u fix this in ur most resent PR? Thanks :)
Can do! But this is a breaking change to any folks using v1.1.0
of the package
Yes, let's just keep this issue open then, and not do anything about it right now :).