Noissue - Adapt quotes for compliance with our coding style in sitescripts

sitescripts/subscriptions/subscriptionParser.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2016 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 21 matching lines @@
 class Subscription(object):
     def define_property(propName, readonly=False):
         if readonly:
             return property(lambda self: self._data[propName])
         else:
             def set_property(self, value):
                 self._data[propName] = value
 
             return property(lambda self: self._data[propName], set_property)
 
-    name = define_property("name")
-    type = define_property("type")
-    maintainer = define_property("maintainer")
-    email = define_property("email")
-    specialization = define_property("specialization")
-    languages = define_property("languages")
-    recommendation = define_property("recommendation", readonly=True)
-    deprecated = define_property("deprecated", readonly=True)
-    unavailable = define_property("unavailable", readonly=True)
-    catchall = define_property("catchall", readonly=True)
-    supplements = define_property("supplements", readonly=True)
-    supplementsType = define_property("supplementsType", readonly=True)
-    supplemented = define_property("supplemented", readonly=True)
-    variants = define_property("variants", readonly=True)
-    homepage = define_property("homepage")
-    contact = define_property("contact")
-    forum = define_property("forum")
-    faq = define_property("faq")
-    blog = define_property("blog")
-    changelog = define_property("changelog")
-    policy = define_property("policy")
-    digest = define_property("digest")
-    digestDay = define_property("digestDay")
+    name = define_property('name')
+    type = define_property('type')
+    maintainer = define_property('maintainer')
+    email = define_property('email')
+    specialization = define_property('specialization')
+    languages = define_property('languages')
+    recommendation = define_property('recommendation', readonly=True)
+    deprecated = define_property('deprecated', readonly=True)
+    unavailable = define_property('unavailable', readonly=True)
+    catchall = define_property('catchall', readonly=True)
+    supplements = define_property('supplements', readonly=True)
+    supplementsType = define_property('supplementsType', readonly=True)
+    supplemented = define_property('supplemented', readonly=True)
+    variants = define_property('variants', readonly=True)
+    homepage = define_property('homepage')
+    contact = define_property('contact')
+    forum = define_property('forum')
+    faq = define_property('faq')
+    blog = define_property('blog')
+    changelog = define_property('changelog')
+    policy = define_property('policy')
+    digest = define_property('digest')
+    digestDay = define_property('digestDay')
 
     def __init__(self, path, data):
         self._data = {
             'name': None,
             'type': 'ads',
             'maintainer': None,
             'email': None,
             'specialization': None,
             'languages': None,
             'deprecated': False,
@@ skipping 102 matching lines @@
                 languagenames = []
                 for language in value.split(','):
                     if settings.has_option('languages', language):
                         languagenames.append(settings.get('languages', language))
                     else:
                         warn('Unknown language code %s in %s' % (language, path))
                 self._data['languageSpecialization'] = ', '.join(languagenames)
 
         if 'languageSpecialization' in self._data:
             if self.specialization != None:
-                self.specialization += ", " + self._data['languageSpecialization']
+                self.specialization += ', ' + self._data['languageSpecialization']
             else:
                 self.specialization = self._data['languageSpecialization']
             del self._data['languageSpecialization']
 
         for group in mandatory:
             found = False
             for key in group:
                 if self._data[key] != None:
                     found = True
             if not found:
-                str = ", ".join(group)
+                str = ', '.join(group)
                 warn('None of the attributes %s present in %s' % (str, path))
 
         if len(self.variants) == 0:
             warn('No list locations given in %s' % (path))
         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))
         if not self.digestDay[0:3].lower() in weekdays:
             warn('Unknown digest day given in %s' % (path))
@@ skipping 56 matching lines @@
 def getFallbackData():
     repo = os.path.abspath(get_config().get('subscriptions', 'repository'))
     redirectdata = subprocess.check_output(['hg', '-R', repo, 'cat', '-r', 'default', os.path.join(repo, 'redirects')])
     gonedata = subprocess.check_output(['hg', '-R', repo, 'cat', '-r', 'default', os.path.join(repo, 'gone')])
     return (redirectdata, gonedata)
 
 
 def _validate_URL(url):
     parse_result = urlparse(url)
     return parse_result.scheme in ('http', 'https') and parse_result.netloc != ''