[7.x] Add option to specify a custom guard for the policy #33210
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
When using
artisan make:policy, policies can only be generated with the user model found in the default guard's provider. This PR allows an optional--guardoption to be specified to override this default behaviour with a non-default guard.It also adds an
--any-guardoption, which overrides--guard, which will instead type hintIlluminate\Contracts\Auth\Access\Authorizablein the generated policy, allowing any guard to be used with the policy.This PR should be fully backwards compatible as omitting both new options results in the existing behaviour (the default guard's provider model is used).
I wanted this because I have two guards, one for
Administrators and one forUsers.