Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#283 closed defect (invalid)

templatetags/log.py breaks on AnonymousUser

Reported by: garthk@… Owned by: Adrian Holovaty
Component: Core (Other) Version:
Severity: normal Keywords: templatetags log anonymous
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Adrian Holovaty)

Try logging in if you're already the anonymous user. Ouch! Trivial fix:

C:\dev\django-svn\trunk>svn diff
Index: django/templatetags/log.py
===================================================================
--- django/templatetags/log.py  (revision 416)
+++ django/templatetags/log.py  (working copy)
@@ -10,7 +10,10 @@

     def render(self, context):
         if self.user is not None and not self.user.isdigit():
-            self.user = context[self.user].id
+            try:
+                self.user = context[self.user].id
+            except AttributeError: # AnonymousUser doesn't have it
+                pass
         context[self.varname] = log.get_list(user_id__exact=self.user, limit=self.limit, select_related=True)
         return ''

Change History (2)

comment:1 by Adrian Holovaty, 19 years ago

Description: modified (diff)

(Fixed formatting in description.)

comment:2 by Adrian Holovaty, 19 years ago

Resolution: invalid
Status: newclosed

The admin-log templatetag isn't meant to be used out of the admin context. It's assumed that the user is logged in when the tag is used.

Note: See TracTickets for help on using tickets.
Back to Top