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

Patch the Laravel framework #14

Open
pionl opened this issue Jul 6, 2022 · 2 comments
Open

Patch the Laravel framework #14

pionl opened this issue Jul 6, 2022 · 2 comments

Comments

@pionl
Copy link
Contributor

pionl commented Jul 6, 2022

For quality improvements would be great to patch Laravel framework using cweagans/composer-patches

The patches would be in separate composer package to make it reusable without using larastrict (and maybe optional?).

@pionl pionl changed the title Path the Laravel framework Patch the Laravel framework Jul 6, 2022
@pionl
Copy link
Contributor Author

pionl commented Jul 26, 2023

@pionl
Copy link
Contributor Author

pionl commented Aug 8, 2023

Expression support in various where conditions

  413    Parameter #4 $second of method                                                                   
         Illuminate\Database\Eloquent\Builder<Illuminate\Database\Eloquent\Model>::leftJoin()             
         expects string|null, Illuminate\Database\Query\Expression given.  
 403    Parameter #1 $columns of method                                                                                        
         Illuminate\Database\Eloquent\Builder<Illuminate\Database\Eloquent\Model>::whereNull()                                  
         expects array<string>|string, Illuminate\Database\Query\Expression                                                     
         given.    

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

1 participant