Issue 5456 - Add support to specify urlFilters in the notification repository

sitescripts/notifications/parser.py

Index: sitescripts/notifications/parser.py
diff --git a/sitescripts/notifications/parser.py b/sitescripts/notifications/parser.py
index 3593ee80d3107d1d34fb4dfcc2861892129b049a..4fbc92a7031399f8b141f13d8675f0b3fee1d075 100644
--- a/sitescripts/notifications/parser.py
+++ b/sitescripts/notifications/parser.py
@@ -92,6 +92,9 @@ 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'] = ["{}^$document".format(v.upper())

[Vasily Kuznetsov, 2017/08/03 17:08:52]

Our style guide recommends single quotes and using "+" for concatenating just
two strings. I believe flake8-eyeo should complain about those. Also the
indentation of the second line should align with the opening brace in this case
(not sure if flake8 would complain, but we've basically agreed to use one of the
two styles:

    foo = [bar
               for bar in baz]
    foo = [
        bar
        for bar in baz
    ]

when multiline literals, comprehension expressions or long parameter lists need
to be split over several lines). So probably something like this would be more
"kosher":

            current['urlFilters'] = [v.upper() + '$document'
	    			     for v in value.split()]

[wspee, 2017/08/04 08:41:23]

Done.

+                for v in value.split()]
         else:
             raise Exception("Unknown parameter '%s' in file '%s'" % (key, name))