From 48ac049736c0702905b2896bf30a79894c939369 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Mon, 24 Aug 2015 03:29:13 -0400 Subject: [PATCH] Avoid using a loop when calculating recent document list. --- mayan/apps/documents/managers.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/mayan/apps/documents/managers.py b/mayan/apps/documents/managers.py index 29a064a3b5..2f7bd3434f 100644 --- a/mayan/apps/documents/managers.py +++ b/mayan/apps/documents/managers.py @@ -20,8 +20,9 @@ class RecentDocumentManager(models.Manager): # document already in the recent list, just save to force # accessed date and time update new_recent.save() - for recent_to_delete in self.model.objects.filter(user=user)[setting_recent_count.value:]: - recent_to_delete.delete() + + recent_to_delete = self.filter(user=user).values_list('pk', flat=True)[setting_recent_count.value:] + self.filter(pk__in=list(recent_to_delete)).delete() def get_for_user(self, user): document_model = apps.get_model('documents', 'document')