Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP8 deprecating warning #10

Open
theblindfrog opened this issue Aug 22, 2021 · 3 comments
Open

PHP8 deprecating warning #10

theblindfrog opened this issue Aug 22, 2021 · 3 comments

Comments

@theblindfrog
Copy link

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

@ExpDev07
Copy link
Owner

Could u fix this in ur most resent PR? Thanks :)

@theblindfrog
Copy link
Author

theblindfrog commented Aug 23, 2021

Can do! But this is a breaking change to any folks using v1.1.0 of the package

@ExpDev07
Copy link
Owner

Yes, let's just keep this issue open then, and not do anything about it right now :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants