search

kmmbvnr / django-guardian-ng

Per object permissions for Django
https://django-guardian.readthedocs.io/
Other
1 stars 0 forks source link

Add get_login_url to the PermissionRequiredMixin #27

Open kmmbvnr opened 5 months ago

kmmbvnr commented 5 months ago

Original issue: https://github.com/django-guardian/django-guardian/issues/775

kmmbvnr commented 5 months ago

Actually Django's LoginRequiredMixin inherited from AccessMixin that provides get_login_url method

https://github.com/django/django/blob/main/django/contrib/auth/mixins.py#L21

So it worth to have the same in django-guardian

kmmbvnr commented 5 months ago

PermissionRequiredMixin should be inherited from standard Django PermissionRequiredMixin mixin