From 32639c8ef3593d6d20fdd150ee73a6a9c4e759bc Mon Sep 17 00:00:00 2001 From: spez Date: Wed, 3 Jun 2009 14:12:36 -0700 Subject: [PATCH] rename the recentclick cookie for the millionth time --- r2/r2/controllers/api.py | 2 +- r2/r2/controllers/reddit_base.py | 7 ++++--- r2/r2/public/static/js/reddit.js | 6 +++--- r2/r2/templates/frametoolbar.html | 2 +- 4 files changed, 9 insertions(+), 8 deletions(-) diff --git a/r2/r2/controllers/api.py b/r2/r2/controllers/api.py index 03ab5acf8..59e543d92 100644 --- a/r2/r2/controllers/api.py +++ b/r2/r2/controllers/api.py @@ -1365,7 +1365,7 @@ class ApiController(RedditController): def GET_gadget(self, form, jquery, type, links): if not links and type == 'click': # malformed cookie, clear it out - set_user_cookie('recentclicks', '') + set_user_cookie('recentclicks2', '') if not links: return diff --git a/r2/r2/controllers/reddit_base.py b/r2/r2/controllers/reddit_base.py index 0327f14f4..fb5a4e0ef 100644 --- a/r2/r2/controllers/reddit_base.py +++ b/r2/r2/controllers/reddit_base.py @@ -139,6 +139,7 @@ def read_user_cookie(name): def set_user_cookie(name, val): uname = c.user.name if c.user_is_loggedin else "" c.cookies[uname + '_' + name] = Cookie(value = val) + valid_click_cookie = fullname_regex(Link, True).match def set_recent_clicks(): @@ -146,21 +147,21 @@ def set_recent_clicks(): if not c.user_is_loggedin: return - click_cookie = read_user_cookie('recentclicks') + click_cookie = read_user_cookie('recentclicks2') if click_cookie: if valid_click_cookie(click_cookie): names = [ x for x in UniqueIterator(click_cookie.split(',')) if x ] if len(click_cookie) > 1000: names = names[:20] - set_user_cookie('recentclicks', ','.join(names)) + set_user_cookie('recentclicks2', ','.join(names)) #eventually this will look at the user preference names = names[:5] c.recent_clicks = Link._by_fullname(names, data = True, return_dict = False) else: #if the cookie wasn't valid, clear it - set_user_cookie('recentclicks', '') + set_user_cookie('recentclicks2', '') def read_mod_cookie(): cook = [s.split('=')[0:2] for s in read_user_cookie('mod').split(':') if s] diff --git a/r2/r2/public/static/js/reddit.js b/r2/r2/public/static/js/reddit.js index 400814bb8..ad700416c 100644 --- a/r2/r2/public/static/js/reddit.js +++ b/r2/r2/public/static/js/reddit.js @@ -485,7 +485,7 @@ function add_thing_id_to_cookie(id, cookie_name) { }; function clicked_items() { - var cookie = $.cookie_read('recentclicks'); + var cookie = $.cookie_read('recentclicks2'); if(cookie && cookie.data) { var fullnames = cookie.data.split(","); /* don't return empty ones */ @@ -501,7 +501,7 @@ function clicked_items() { } function clear_clicked_items() { - var cookie = $.cookie_read('recentclicks'); + var cookie = $.cookie_read('recentclicks2'); cookie.data = ''; $.cookie_write(cookie); $('.gadget').remove(); @@ -542,7 +542,7 @@ function updateEventHandlers(thing) { /* mark as clicked */ $(this).addClass("click"); /* set the click cookie. */ - add_thing_to_cookie(this, "recentclicks"); + add_thing_to_cookie(this, "recentclicks2"); /* remember this as the last thing clicked */ var wasorganic = $(this).parents('.organic-listing').length > 0; last_click(thing, wasorganic); diff --git a/r2/r2/templates/frametoolbar.html b/r2/r2/templates/frametoolbar.html index dc680c9ff..771213f2f 100644 --- a/r2/r2/templates/frametoolbar.html +++ b/r2/r2/templates/frametoolbar.html @@ -149,7 +149,7 @@ ## add us to the click cookie