use None as default for optional string config param. Fix #5309 (#5312)

dmwm · May 13, 2024 · 26e1b3e · 26e1b3e
1 parent 8383629
commit 26e1b3e
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/python/CRABClient/ClientMapping.py b/src/python/CRABClient/ClientMapping.py
@@ -21,8 +21,8 @@
                   'saveoutput'     : {'default': True,       'config': ['General.transferOutputs'],         'type': 'BooleanType', 'required': False},
                   'savelogsflag'   : {'default': False,      'config': ['General.transferLogs'],            'type': 'BooleanType', 'required': False},
                   'faillimit'      : {'default': None,       'config': ['General.failureLimit'],            'type': 'IntType',     'required': False},
-                  'inputdata'      : {'default': '',         'config': ['Data.inputDataset'],               'type': 'StringType',  'required': False},
-                  'primarydataset' : {'default': '',         'config': ['Data.outputPrimaryDataset'],       'type': 'StringType',  'required': False},
+                  'inputdata'      : {'default': None,       'config': ['Data.inputDataset'],               'type': 'StringType',  'required': False},
+                  'primarydataset' : {'default': None,       'config': ['Data.outputPrimaryDataset'],       'type': 'StringType',  'required': False},
                   'nonvaliddata'   : {'default': False,      'config': ['Data.allowNonValidInputDataset'],  'type': 'BooleanType', 'required': False},
                   'userfiles'      : {'default': [],         'config': ['Data.userInputFiles'],             'type': 'ListType',    'required': False},
                   'dbsurl'         : {'default': 'global',   'config': ['Data.inputDBS'],                   'type': 'StringType',  'required': False},
@@ -36,7 +36,7 @@
                   'publication'    : {'default': True,       'config': ['Data.publication'],                'type': 'BooleanType', 'required': False},
                   'publishdbsurl'  : {'default': 'phys03',   'config': ['Data.publishDBS'],                 'type': 'StringType',  'required': False},
                   #the 2 here to the publishname was added because of https://github.com/dmwm/CRABServer/issues/4947
-                  'publishname2'   : {'default': '',         'config': ['Data.outputDatasetTag'],           'type': 'StringType',  'required': False},
+                  'publishname2'   : {'default': None,       'config': ['Data.outputDatasetTag'],           'type': 'StringType',  'required': False},
                   'jobtype'        : {'default': 'Analysis', 'config': ['JobType.pluginName',
                                                                         'JobType.externalPluginFile'],      'type': 'StringType',  'required': False},
                   'generator'      : {'default': 'pythia',   'config': ['JobType.generator'],               'type': 'StringType',  'required': False},