Noissue - Adapt best practices for trailing commas (cms)

cms/sources.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-present 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 97 matching lines @@
     #
     # Localizable files helpers
     #
 
     @staticmethod
     def localizable_file_filename(locale, filename):
         return 'locales/%s/%s' % (locale, filename)
 
     def list_localizable_files(self):
         default_locale = self.read_config().get('general', 'defaultlocale')
-        return filter(
-            lambda f: os.path.splitext(f)[1].lower() != '.json',
-            self.list_files('locales/%s' % default_locale)
-        )
+        return filter(lambda f: os.path.splitext(f)[1].lower() != '.json',
+                      self.list_files('locales/%s' % default_locale))
 
     def has_localizable_file(self, locale, filename):
         return self.has_file(self.localizable_file_filename(locale, filename))
 
     def read_localizable_file(self, locale, filename):
         return self.read_file(self.localizable_file_filename(locale, filename), binary=True)[0]
 
     #
     # Static file helpers
     #
@@ skipping 43 matching lines @@
         if self.has_locale(locale, page):
             filename = self.locale_filename(locale, page)
             filedata, filepath = self.read_file(filename)
             try:
                 localedata = json.loads(filedata)
                 for key, value in localedata.iteritems():
                     result[key] = value['message']
             except KeyError as ke:
                 if ke.message != 'message':
                     raise
-                raise ValueError(
-                    'The content of translations file for page "{}", '
-                    'language "{}" ({}) is not a valid translations file: '
-                    '"message" key is missing for string: "{}"'
-                    .format(page, locale, filepath, key)
-                )
+                raise ValueError('The content of translations file for page '
+                                 '"{}", language "{}" ({}) is not a valid '
+                                 'translations file: "message" key is missing '
+                                 'for string: "{}"'.format(page, locale,
+                                                           filepath, key))
             except (AttributeError, ValueError):
-                raise ValueError(
-                    'The content of translations file for page "{}", '
-                    'language "{}" ({}) is not a valid JSON dictionary'
-                    .format(page, locale, filepath)
-                )
+                raise ValueError('The content of translations file for page '
+                                 '"{}", language "{}" ({}) is not a valid '
+                                 'JSON object'.format(page, locale, filepath))
 
         return result
 
     #
     # Template helpers
     #
 
     @staticmethod
     def template_filename(template):
         return 'templates/%s.tmpl' % template
@@ skipping 243 matching lines @@
             'resolve_link',
             'read_config',
             'read_template',
             'read_locale',
             'read_include',
             'exec_file',
         ]:
             setattr(source, fname, _memoize(getattr(source, fname)))
 
     return source