diff options
| author | Aurélien Bompard <aurelien@bompard.org> | 2013-12-01 11:53:08 +0100 |
|---|---|---|
| committer | Aurélien Bompard <aurelien@bompard.org> | 2013-12-01 16:00:37 +0100 |
| commit | 0bd5582a75d5a610885241ba52186033e2272424 (patch) | |
| tree | a1af1842d9ddce37f1a9ec2815e0d289fbb405f4 /hyperkitty/lib/mailman.py | |
| parent | 85b8ca13fc2e164971f7cbb06c4a0c5b0a1a0770 (diff) | |
| download | hyperkitty-0bd5582a75d5a610885241ba52186033e2272424.tar.gz hyperkitty-0bd5582a75d5a610885241ba52186033e2272424.tar.xz hyperkitty-0bd5582a75d5a610885241ba52186033e2272424.zip | |
Better handling of the user votes cache
Diffstat (limited to 'hyperkitty/lib/mailman.py')
| -rw-r--r-- | hyperkitty/lib/mailman.py | 9 |
1 files changed, 5 insertions, 4 deletions
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" |