From 0bd5582a75d5a610885241ba52186033e2272424 Mon Sep 17 00:00:00 2001 From: Aurélien Bompard Date: Sun, 1 Dec 2013 11:53:08 +0100 Subject: Better handling of the user votes cache --- hyperkitty/lib/mailman.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'hyperkitty/lib/mailman.py') diff --git a/hyperkitty/lib/mailman.py b/hyperkitty/lib/mailman.py index 0e940ee..45f4289 100644 --- a/hyperkitty/lib/mailman.py +++ b/hyperkitty/lib/mailman.py @@ -61,16 +61,17 @@ def get_subscriptions(store, client, mm_user): # de-duplicate subscriptions if mlist in [ s["list_name"] for s in subscriptions ]: continue + posts_count = store.get_message_count_by_user_id( + mm_user.user_id, mlist) cache_key = "user:%s:list:%s:votes" % (mm_user.user_id, mlist) - likes, dislikes, posts_count = cache.get(cache_key, (None, None, None)) + likes, dislikes = cache.get(cache_key, (None, None)) if likes is None or dislikes is None or posts_count is None: email_hashes = store.get_message_hashes_by_user_id( mm_user.user_id, mlist) likes, dislikes, _myvote = get_votes(mlist, email_hashes) - posts_count = len(email_hashes) - cache.set(cache_key, (likes, dislikes, posts_count)) + cache.set(cache_key, (likes, dislikes)) all_posts_url = "%s?list=%s" % \ - (reverse("user_posts", mm_user.user_id), mlist) + (reverse("user_posts", args=[mm_user.user_id]), mlist) likestatus = "neutral" if likes - dislikes >= 10: likestatus = "likealot" -- cgit