Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: sitescripts/notifications/web/notification.py

Issue 4883267715072000: Issue 2276 - Handle groups in notification.json requests (Closed)
Patch Set: Extract group parsing Created June 12, 2015, 12:11 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « sitescripts/notifications/web/__init__.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 # coding: utf-8
2
3 # This file is part of the Adblock Plus web scripts,
4 # Copyright (C) 2006-2015 Eyeo GmbH
5 #
6 # Adblock Plus is free software: you can redistribute it and/or modify
7 # it under the terms of the GNU General Public License version 3 as
8 # published by the Free Software Foundation.
9 #
10 # Adblock Plus is distributed in the hope that it will be useful,
11 # but WITHOUT ANY WARRANTY; without even the implied warranty of
12 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 # GNU General Public License for more details.
14 #
15 # You should have received a copy of the GNU General Public License
16 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
17
18 import copy
19 import json
20 import random
21 import time
22 import urlparse
23
24 from sitescripts.notifications.parser import load_notifications
25 from sitescripts.web import url_handler
26
27 def _determine_groups(version, notifications):
28 version_groups = dict(x.split("/") for x in version.split("-")[1:]
29 if x.count("/") == 1)
30 groups = []
31 for notification in notifications:
32 if "variants" not in notification:
33 continue
34 group_id = notification["id"]
35 if group_id in version_groups:
36 groups.append({"id": group_id, "variant": int(version_groups[group_id])})
37 return groups
38
39 def _assign_groups(notifications):
40 groups = []
41 selection = random.random()
42 start = 0
43 for notification in notifications:
44 if "variants" not in notification:
45 continue
46 group = {"id": notification["id"], "variant": 0}
47 groups.append(group)
48 for i, variant in enumerate(notification["variants"]):
49 sample_size = variant["sample"]
50 end = start + sample_size
51 selected = sample_size > 0 and start <= selection <= end
52 start = end
53 if selected:
54 group["variant"] = i + 1
55 break
56 return groups
57
58 def _get_active_variant(notifications, groups):
59 for group in groups:
60 group_id = group["id"]
61 variant = group["variant"]
62 if variant == 0:
63 continue
64 notification = next(x for x in notifications if x["id"] == group_id)
65 notification = copy.deepcopy(notification)
66 notification.update(notification["variants"][variant - 1])
67 for key_to_remove in ("sample", "variants"):
68 notification.pop(key_to_remove, None)
69 return notification
70
71 def _generate_version(groups):
72 version = time.strftime("%Y%m%d%H%M", time.gmtime())
73 for group in groups:
74 version += "-%s/%s" % (group["id"], group["variant"])
75 return version
76
77 def _create_response(notifications, groups):
78 active_variant = _get_active_variant(notifications, groups)
79 if active_variant:
80 notifications = [active_variant]
81 else:
82 notifications = [x for x in notifications if "variants" not in x]
83 response = {
84 "version": _generate_version(groups),
85 "notifications": notifications
86 }
87 return json.dumps(response, ensure_ascii=False, indent=2,
88 separators=(",", ": "), sort_keys=True)
89
90 @url_handler("/notification.json")
91 def notification(environ, start_response):
92 params = urlparse.parse_qs(environ.get("QUERY_STRING", ""))
93 version = params.get("lastVersion", [""])[0]
94 notifications = load_notifications()
95 groups = _determine_groups(version, notifications)
96 if not groups:
97 groups = _assign_groups(notifications)
98 response = _create_response(notifications, groups)
99 start_response("200 OK",
100 [("Content-Type", "application/json; charset=utf-8")])
101 return response.encode("utf-8")
OLDNEW
« no previous file with comments | « sitescripts/notifications/web/__init__.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld