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

Unified Diff: sitescripts/notifications/parser.py

Issue 29500676: Issue 5456 - Add support to specify urlFilters in the notification repository (Closed)
Patch Set: Use assert instead of assertEqual Created Sept. 21, 2017, 10:09 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | sitescripts/notifications/test/parser.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/notifications/parser.py
diff --git a/sitescripts/notifications/parser.py b/sitescripts/notifications/parser.py
index f7a14008cb701dd21b184cafd79fc8e85a0bff4b..d8679b89b625495b593b2484b4b0cc6a27cc6329 100644
--- a/sitescripts/notifications/parser.py
+++ b/sitescripts/notifications/parser.py
@@ -92,6 +92,11 @@ def _parse_notification(data, name):
current[key] = datetime.datetime.strptime(value, '%Y-%m-%dT%H:%M')
elif key == 'interval':
current[key] = int(value)
+ elif key == 'urls':
+ current['urlFilters'] = [
+ v.upper() + '^$document'
+ for v in value.split()
+ ]
else:
raise Exception("Unknown parameter '%s' in file '%s'" % (key, name))
« no previous file with comments | « no previous file | sitescripts/notifications/test/parser.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld