autopep8 action fixes

Author: glk0

ganga/GangaCore/GPIDev/Lib/File/MassStorageFile.py

@@ -387,23 +387,36 @@ def validate(self):
                     missingKeywords.append(item)
 
             if len(missingKeywords):
-                return (False, 'Error in %s.outputfilenameformat field : missing keywords %s ' % (_getName(self), ','.join(missingKeywords)))
+                return (
+                    False, 'Error in %s.outputfilenameformat field : missing keywords %s ' %
+                    (_getName(self), ','.join(missingKeywords)))
 
             if isSplitJob is False and self.outputfilenameformat.find('{sjid}') > -1:
-                return (False, 'Error in %s.outputfilenameformat field :  job is non-split, but {\'sjid\'} keyword found' % _getName(self))
+                return (
+                    False,
+                    'Error in %s.outputfilenameformat field :  job is non-split, but {\'sjid\'} keyword found' %
+                    _getName(self))
 
             if isJob is False and self.outputfilenameformat.find('{sjid}') > -1:
-                return (False, 'Error in %s.outputfilenameformat field :  no parent job, but {\'sjid\'} keyword found' % _getName(self))
+                return (
+                    False,
+                    'Error in %s.outputfilenameformat field :  no parent job, but {\'sjid\'} keyword found' %
+                    _getName(self))
 
             if isJob is False and self.outputfilenameformat.find('{jid}') > -1:
-                return (False, 'Error in %s.outputfilenameformat field :  no parent job, but {\'jid\'} keyword found' % _getName(self))
+                return (
+                    False,
+                    'Error in %s.outputfilenameformat field :  no parent job, but {\'jid\'} keyword found' %
+                    _getName(self))
 
             invalidUnixChars = ['"', ' ']
             test = self.outputfilenameformat.replace('{jid}', 'a').replace('{sjid}', 'b').replace('{fname}', 'c')
 
             for invalidUnixChar in invalidUnixChars:
                 if test.find(invalidUnixChar) > -1:
-                    return (False, 'Error in %s.outputfilenameformat field :  invalid char %s found' % (_getName(self), invalidUnixChar))
+                    return (
+                        False, 'Error in %s.outputfilenameformat field :  invalid char %s found' %
+                        (_getName(self), invalidUnixChar))
 
         return (True, '')
 
@@ -438,9 +451,13 @@ def getWNInjectedScript(self, outputFiles, indent, patternsToZip, postProcessLoc
             if outputFile.outputfilenameformat is not None and outputFile.outputfilenameformat != '':
                 outputfilenameformat = outputFile.outputfilenameformat
 
-            massStorageCommands.append(['massstorage', outputFile.namePattern, outputfilenameformat,
-                                        massStorageConfig['mkdir_cmd'], massStorageConfig['cp_cmd'],
-                                        massStorageConfig['ls_cmd'], os.path.expanduser(os.path.expandvars(massStorageConfig['path']))])
+            massStorageCommands.append(['massstorage',
+                                        outputFile.namePattern,
+                                        outputfilenameformat,
+                                        massStorageConfig['mkdir_cmd'],
+                                        massStorageConfig['cp_cmd'],
+                                        massStorageConfig['ls_cmd'],
+                                        os.path.expanduser(os.path.expandvars(massStorageConfig['path']))])
 
         script_location = os.path.join(os.path.dirname(os.path.abspath(inspect.getfile(inspect.currentframe()))),
                                        'scripts/MassStorageFileWNScript.py.template')

ganga/GangaCore/Runtime/Repository_runtime.py

@@ -179,7 +179,7 @@ def shutdown():
     from GangaCore.Utility.logging import getLogger
     logger = getLogger()
     logger.info('Registry Shutdown')
-    #import traceback
+    # import traceback
     # traceback.print_stack()
 
     # Flush all repos before we shut them down
@@ -205,7 +205,7 @@ def shutdown():
     for itr, registry in enumerate(all_registries):
         thisName = registry.name
         try:
-            if not thisName in started_registries:
+            if thisName not in started_registries:
                 continue
             # in case this is called repeatedly, only call shutdown once
             started_registries.remove(thisName)