Re: [Django] #34284: Permission to access adminsites

3 views
Skip to first unread message

Django

unread,
Jan 23, 2023, 7:04:07 AM1/23/23
to django-...@googlegroups.com
#34284: Permission to access adminsites
-------------------------------+--------------------------------------
Reporter: Al Mahdi | Owner: nobody
Type: New feature | Status: closed
Component: contrib.admin | Version: dev
Severity: Normal | Resolution: wontfix
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+--------------------------------------
Changes (by Mariusz Felisiak):

* status: new => closed
* resolution: => wontfix


Comment:

Thanks for the ticket, however Django Admin is not a
[https://docs.djangoproject.com/en/stable/ref/contrib/admin/#module-
django.contriab.admin universal tool for building an app] and adding a new
layer of access control (like any new customization option) is always
controversial. Moreover, I don't think it's needed, you should be able to
override
[https://docs.djangoproject.com/en/stable/ref/contrib/admin/#django.contrib.admin.AdminSite.has_permission
AdminSite.has_permission()] and control permissions for each `AdminSite`.

--
Ticket URL: <https://code.djangoproject.com/ticket/34284#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Jan 23, 2023, 1:51:47 PM1/23/23
to django-...@googlegroups.com
#34284: Permission to access adminsites
-------------------------------+--------------------------------------
Reporter: Al Mahdi | Owner: nobody
Type: New feature | Status: closed
Component: contrib.admin | Version: dev
Severity: Normal | Resolution: wontfix
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+--------------------------------------

Comment (by Al Mahdi):

Thanks. Overriding the {{{has_permission}}} method, works for me.

--
Ticket URL: <https://code.djangoproject.com/ticket/34284#comment:4>

Reply all
Reply to author
Forward
0 new messages