From 191386038c9165c862c902e0b075be4938ef0986 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Sun, 8 Jan 2012 06:50:35 -0400 Subject: [PATCH] Don't update recent documents for anonymous users --- apps/documents/managers.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/apps/documents/managers.py b/apps/documents/managers.py index 189535a67d..b564d44885 100644 --- a/apps/documents/managers.py +++ b/apps/documents/managers.py @@ -11,15 +11,16 @@ from .conf.settings import RECENT_COUNT class RecentDocumentManager(models.Manager): def add_document_for_user(self, user, document): - self.model.objects.filter(user=user, document=document).delete() - new_recent = self.model(user=user, document=document, datetime_accessed=datetime.now()) - new_recent.save() - to_delete = self.model.objects.filter(user=user)[RECENT_COUNT:] - for recent_to_delete in to_delete: - recent_to_delete.delete() + if user.is_authenticated(): + self.model.objects.filter(user=user, document=document).delete() + new_recent = self.model(user=user, document=document, datetime_accessed=datetime.now()) + new_recent.save() + to_delete = self.model.objects.filter(user=user)[RECENT_COUNT:] + for recent_to_delete in to_delete: + recent_to_delete.delete() def get_for_user(self, user): - if not user.is_anonymous(): + if user.is_authenticated(): return [recent_document.document for recent_document in self.model.objects.filter(user=user)] else: return []