Python interface: various PR bugs caught by Copilot

Author: PetrilloAtWork

sbnalg/gallery/python/LArSoftUtils.py

@@ -226,7 +226,7 @@ def serviceClassToConfigKeys(serviceClassName: str) -> list[str]:
   candidates = []
 
   # remove namespaces (namespaces are not part of any candidate)
-  try: serviceClassName = serviceClassName[:serviceClassName.index('::')]
+  try: serviceClassName = serviceClassName[:serviceClassName.rindex('::')]
   except ValueError: pass # no namespace
 
   if not serviceClassName: return [] # ?!
@@ -284,10 +284,10 @@ def readServiceConfig(getConfig, configKey, returnConfigKey = True):
     = tuple(base + suffix for base in serviceClassToConfigKeys(configKey))
 
   Logger.debug("Configuration from candidates: '%s'", "', '".join(configKeys))
-  for configKey in configKeys:
-    try: config = getConfig(configKey)
+  for candidateKey in configKeys:
+    try: config = getConfig(candidateKey)
     except Exception: continue
-    return (config, configKey) if returnConfigKey else config
+    return (config, candidateKey) if returnConfigKey else config
   raise RuntimeError(f"No configuration for service key '{configKey}'")
 # readServiceConfig()
 
@@ -701,6 +701,8 @@ def fullConfig(self): return self.serviceTable is None
     def isValid(self): return self.configPath is not None
     def hasExtraConfig(self): return bool(self.extraConfig)
     def needsCustom(self): return not self.fullConfig() or self.hasExtraConfig()
+    def serviceTableName(self):
+      return 'services' if self.fullConfig() else self.serviceTable
 
     def addExtraConfig(self, extra): self.extraConfig += "\n" + extra
 
@@ -811,7 +813,7 @@ def loadConfiguration(self):
         '\n# ==============================='
         .format(
           configPath=configurationInfo.configPath,
-          serviceTable=configurationInfo.serviceTable,
+          serviceTable=configurationInfo.serviceTableName(),
           extraConfig=configurationInfo.extraConfig,
           )
         )

sbnalg/gallery/python/cppUtils.py

@@ -148,9 +148,10 @@ def loadMany(self, *pathSpecs, extraPaths = [], force = False, loadAll = False):
     `{ relPath: 'larcorealg_Geometry', extraPaths: [] }`.
     If a path specification is not a dictionary, it is assumed to be a `relPath`
     and it is equivalent to having `{ relPath: pathSpec }`.
-    If an argument `extraPath` or `force` is specified in the path
-    specification, a value is used from the `extraPath` and `force` arguments
-    of this function.
+    If `extraPaths` or `force` are not specified in a path specification,
+    they default to the `extraPaths` and `force` arguments of this function;
+    if they are specified in the path specification, their values override
+    the defaults from this function.
     
     If `loadAll` is set, all loads are attempted; the return value is a pair:
     whether an exception was thrown, and a list of all return values or