Fix laravel-ide-helper generate:model #517
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is attempt to fix #516, as on post #516 (comment) pointed out, it already provide the arg to not execute save.
so we can leverage that, if user id is not present, we set the method
getWalletargument$saveas false.this will ensure, when authenticated user id will auto save,
when execute without any authentication for this instance, laravel-ide-helper would not execute save
➜ laravel-wallet git:(master) ✗ phpunit --stop-on-failure PHPUnit 9.5.20 #StandWithUkraine Runtime: PHP 8.0.8 Configuration: /Users/keatliang/Downloads/laravel-wallet/phpunit.xml Warning: No code coverage driver available ............................................................... 63 / 193 ( 32%) ............................................................... 126 / 193 ( 65%) ............................................................... 189 / 193 ( 97%) .... 193 / 193 (100%) Time: 00:06.076, Memory: 56.00 MB OK (193 tests, 1653 assertions)