-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
[Bug]: Throw error in GET list have parameter include is categories or menus #66
Comments
@ryanmitchell @sampoyigi please help me! |
Please don't tag people like that. If it's urgent request support from the website, otherwise be patient as this is an open source mostly volunteer led project. |
i so sorry. |
I just ran a quick test and this works fine for me on 3.2 and the latest versions of all extensions. Can you make sure you're on the latest codebase and if so can you confirm what laravel and PHP versions you are using as there may be a clash there. |
Im sure it latest codebase and Im using php version 7.4.26. and tasty v3.2.1. Thanks you for test. |
Have you modified the codebase in any way? |
Im not modify any codebase. Im using laravelv8.0. Error happen when upgrade project to v 3.2.1 |
Yes. I can see from your screenshot you have made modifications.... you have tax classes there |
Tax Class is different extension. |
I know, have you tried disabling it to see if thats the issue? |
@sampoyigi any thoughts? Its working fine for me in all environments. Could it be a WAMP thing? |
I'm unable to reproduce the error, maybe try disabling all third party extensions? |
What laravel version you use? |
I am also experiencing the same issue. TastyIgniter 3.2.2 |
@shomarim After many days debug, i found error occur on laravel 8.80, you can downgrade version to 8.74 to fix |
Thank you @BaoNguyenSE. That worked! I just tried different versions and it seems that the issue occurred with Laravel 8.77. I suspect this PR to be the cause: laravel/framework#40022 |
Thanks for investigating - yes it seems the upstream changes in Laravel have caused this. |
@BaoNguyenSE thanks for investigating. @shomarim can you please confirm if this issue still exists on laravel 8.80+, i'm unable to reproduce on 8.82 |
Fixed here tastyigniter/flame@ad1dfc0 |
What happened?
version use: v1.3.2
any api index have param include relationship one to many
ex: categories, menus,...
What did you expect to happen?
this return list relationship
Version
3.1.x
What browser are you seeing the problem on?
No response
Relevant log output
The text was updated successfully, but these errors were encountered: