permission_required decorator should take any iterable of permissions
As it came up in the discussion on ticket #24914 on GitHub (https://github.com/django/django/pull/4749#discussion_r31776720), there is no reason why permission_required only takes lists and tuples of permissions, while has_perms
itself can take any iterable. To be consistent with the new mixins and other parts of Django where both strings and iterables of strings are accepted (see e.g. model._meta.ordering), we should change this logic. I will prepare a pull request with the same logic that is used in other places.
Change History
(7)
Description: |
modified (diff)
|
Has patch: |
set
|
Owner: |
set to Raphael Michel
|
Status: |
new → assigned
|
Summary: |
permission_required decorator should take any iterable if permissions → permission_required decorator should take any iterable of permissions
|
Triage Stage: |
Unreviewed → Accepted
|
Needs tests: |
set
|
Patch needs improvement: |
set
|
Needs documentation: |
unset
|
Needs tests: |
unset
|
Patch needs improvement: |
unset
|
Triage Stage: |
Accepted → Ready for checkin
|
Resolution: |
→ fixed
|
Status: |
assigned → closed
|
I created the pull request: https://github.com/django/django/pull/4790