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 []