style: minor style fixes

Author: fstagni

src/DIRAC/ConfigurationSystem/Client/LocalConfiguration.py

@@ -412,8 +412,8 @@ def __parseCommandLine(self):
 
         groupArgs = []
         step = 0
-        for i in range(len(self.commandArgumentList)):
-            argMarking, description, mandatory, values, default = self.commandArgumentList[i]
+        for i, commandArgument in enumerate(self.commandArgumentList):
+            argMarking, description, mandatory, values, default = commandArgument
 
             # Check whether the required arguments are given in the command line
             if len(self.commandArgList) <= (i + step):
@@ -433,7 +433,7 @@ def __parseCommandLine(self):
                 if values and cArg not in values:
                     gLogger.fatal(
                         "Error when parsing command line arguments: "
-                        '"%s" does not match the allowed values for %s' % (cArg, argMarking)
+                        f'"{cArg}" does not match the allowed values for {argMarking}'
                     )
                     self.showHelp(exitCode=1)
 
@@ -444,30 +444,39 @@ def __parseCommandLine(self):
     def __loadCFGFiles(self):
         """
         Loads possibly several cfg files, in order:
-        1. ~/.dirac.cfg
-        2. cfg files pointed by DIRACSYSCONFIG env variable (comma-separated)
+        1. cfg files pointed by DIRACSYSCONFIG env variable (comma-separated)
+        2. ~/.dirac.cfg
         3. cfg files specified in addCFGFile calls
         4. cfg files that come from the command line
         """
         errorsList = []
+
+        # 1. $DIRACSYSCONFIG
         if "DIRACSYSCONFIG" in os.environ:
             diracSysConfigFiles = os.environ["DIRACSYSCONFIG"].replace(" ", "").split(",")
             for diracSysConfigFile in reversed(diracSysConfigFiles):
                 gLogger.debug(f"Loading file from DIRACSYSCONFIG {diracSysConfigFile}")
                 gConfigurationData.loadFile(diracSysConfigFile)
+
+        # 2. ~/.dirac.cfg
         gConfigurationData.loadFile(os.path.expanduser("~/.dirac.cfg"))
+
+        # 3. cfg files specified in addCFGFile calls
         for fileName in self.additionalCFGFiles:
             gLogger.debug(f"Loading file {fileName}")
             retVal = gConfigurationData.loadFile(fileName)
             if not retVal["OK"]:
                 gLogger.debug(f"Could not load file {fileName}: {retVal['Message']}")
                 errorsList.append(retVal["Message"])
+
+        # 4. cfg files that come from the command line
         for fileName in self.cliAdditionalCFGFiles:
             gLogger.debug(f"Loading file {fileName}")
             retVal = gConfigurationData.loadFile(fileName)
             if not retVal["OK"]:
                 gLogger.debug(f"Could not load file {fileName}: {retVal['Message']}")
                 errorsList.append(retVal["Message"])
+
         return errorsList
 
     def __addUserDataToConfiguration(self):

src/DIRAC/ConfigurationSystem/private/ConfigurationData.py

@@ -7,9 +7,10 @@
 import _thread
 import time
 import datetime
-import DIRAC
 
 from diraccfg import CFG
+
+import DIRAC
 from DIRAC.Core.Utilities.File import mkDir
 from DIRAC.Core.Utilities import List
 from DIRAC.Core.Utilities.ReturnValues import S_OK, S_ERROR
@@ -251,15 +252,13 @@ def getAutoPublish(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/AutoPublish", self.localCFG)
         if value and value.lower() in ("no", "false", "n"):
             return False
-        else:
-            return True
+        return True
 
     def getAutoSlaveSync(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/AutoSlaveSync", self.localCFG)
         if value and value.lower() in ("no", "false", "n"):
             return False
-        else:
-            return True
+        return True
 
     def getServers(self):
         return list(self.remoteServerList)
@@ -290,8 +289,7 @@ def isMaster(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/Master", self.localCFG)
         if value and value.lower() in ("yes", "true", "y"):
             return True
-        else:
-            return False
+        return False
 
     def getServicesPath(self):
         return "/Services"