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

Compatible with Laravel 7 #230

Closed
wants to merge 1 commit into from
Closed

Conversation

renalpha
Copy link

@renalpha renalpha commented May 14, 2020

Fixes:
PHP Fatal error: Declaration of MyParcelNL\Sdk\src\Support\Collection::when($value, callable $callback, ?callable $default = NULL) must be compatible with Illuminate\Support\Collection::when($value, ?callable $callback = NULL, ?callable $default = NULL) in /vendor/myparcelnl/sdk/src/Support/Collection.php on line 38

Fixes:
PHP Fatal error:  Declaration of MyParcelNL\Sdk\src\Support\Collection::when($value, callable $callback, ?callable $default = NULL) must be compatible with Illuminate\Support\Collection::when($value, ?callable $callback = NULL, ?callable $default = NULL) in /Users/jasonhoendervanger/PhpstormProjects/deraad/vendor/myparcelnl/sdk/src/Support/Collection.php on line 38
@Cannonb4ll
Copy link

Is it possible to get this merged? My application update is being held back because of this issue.

@RickJeroen
Copy link
Contributor

This is already fixed in #225?

@renalpha
Copy link
Author

This is already fixed in #225?

yes

@Cannonb4ll
Copy link

I still seem to have this issue, not sure if you guys tried this but it has not been solved for me. Using version 4.

@Cannonb4ll
Copy link

Cannonb4ll commented May 22, 2020

Just checked, if you use version 4 the collection is not updated properly:

public function when($value, callable $callback, callable $default = null)

It is still the old function.

@RichardPerdaan
Copy link
Member

On this moment we work on v4.1.0. I expect to set this version live today

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

Successfully merging this pull request may close these issues.

None yet

4 participants