Django

Code

Ticket #796 (closed: fixed)

Opened 3 years ago

Last modified 2 years ago

Return no permission rather than a traceback for anonymous users

Reported by: garthk Assigned to: adrian
Milestone: Component: Core framework
Version: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: 0 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

Description

Quick and easy patch: 

Index: django/core/extensions.py
===================================================================
--- django/core/extensions.py   (revision 1234)
+++ django/core/extensions.py   (working copy)
@@ -55,7 +55,8 @@
     def __getitem__(self, perm_name):
         return self.user.has_perm("%s.%s" % (self.module_name, perm_name))
     def __nonzero__(self):
-        return self.user.has_module_perms(self.module_name)
+        return hasattr(self.user, 'has_module_perms') \
+                and self.user.has_module_perms(self.module_name)

 class PermWrapper:
     def __init__(self, user):

Attachments

Change History

11/14/05 17:58:12 changed by garthk

Or, fix #797. :)

11/15/05 08:43:56 changed by adrian

  • status changed from new to closed.
  • resolution set to fixed.

(In [1243]) Fixed #796 -- Gave AnonymousUser? a has_module_perms method.

Add/Change #796 (Return no permission rather than a traceback for anonymous users)




Change Properties
Action