Issue 2822 - Added support for new subscription types

sitescripts/subscriptions/subscriptionParser.py

Index: sitescripts/subscriptions/subscriptionParser.py
===================================================================
--- a/sitescripts/subscriptions/subscriptionParser.py
+++ b/sitescripts/subscriptions/subscriptionParser.py
@@ -193,7 +193,7 @@
 
     if len(self.variants) == 0:
       warn('No list locations given in %s' % (path))
-    if self.type != 'ads' and self.type != 'other':
+    if self.type not in ['ads', 'anti-adblock', 'other', 'malware', 'social', 'privacy']:
       warn('Unknown type given in %s' % (path))
     if self.digest != 'daily' and self.digest != 'weekly':
       warn('Unknown digest frequency given in %s' % (path))
@@ -201,7 +201,7 @@
       warn('Unknown digest day given in %s' % (path))
       self.digestDay = 'wed'
     self.digestDay = weekdays[self.digestDay[0:3].lower()]
-    if self.recommendation != None and (self.languages == None or not re.search(r'\S', self.languages)):
+    if self.recommendation != None and self.type == 'ads' and (self.languages == None or not re.search(r'\S', self.languages)):

[Sebastian Noack, 2015/08/31 11:41:06]

  not re.search(r'\S', self.languages)

.. is equivalent to ..

  self.languages.strip()

[Sebastian Noack, 2015/08/31 11:41:06]

Also please use |is None| or |is not None| instead ==/!= when comparing to None.

[Thomas Greiner, 2015/08/31 15:27:47]

Done but I kept the "not" to achieve the same results because the regular
expression is checking for non-whitespaces.

[Thomas Greiner, 2015/08/31 15:27:47]

Done.

       warn('Recommendation without languages in %s' % (path))
     if len(self.supplements) == 0:
       for [name, url, complete] in self.variants:
@@ -218,7 +218,8 @@
   for filedata in lists.itervalues():
     for supplements in filedata.supplements:
       if supplements in lists:
-        lists[supplements].supplemented.append(filedata)
+        if lists[supplements].type == filedata.type:
+          lists[supplements].supplemented.append(filedata)
       else:
         warn('Subscription %s supplements an unknown subscription %s' % (filedata.name, supplements))