Opened 9 years ago
Last modified 9 years ago
#24929 closed Cleanup/optimization
permission_required decorator should take any iterable if permissions — at Initial Version
| Reported by: | Raphael Michel | Owned by: | |
|---|---|---|---|
| Component: | contrib.auth | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
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_perm itself can take any iterable. To be consistent with the new mixins and other parts of Django where boths 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.
Note:
See TracTickets
for help on using tickets.