diff --git a/JsonPreprocessor/CJsonPreprocessor.py b/JsonPreprocessor/CJsonPreprocessor.py index ca536060..0b2e924e 100644 --- a/JsonPreprocessor/CJsonPreprocessor.py +++ b/JsonPreprocessor/CJsonPreprocessor.py @@ -584,9 +584,6 @@ def __jsonUpdated(k, v, oJson, bNested, keyNested = ''): v = ldict['value'] except: raise Exception(f"The variable '{valueProcessed}' is not available!") - - if isinstance(v, str) and re.match('^\s*none|true|false\s*$', v.lower()): - v = '\"' + v + '\"' __jsonUpdated(k, v, oJson, bNested, keyNested) if keyNested != '': @@ -662,18 +659,6 @@ def jsonLoad(self, jFile : str, masterFile : bool = True): Preprocessed json file(s) as dictionary data structure ''' - def __handleStrNoneTrueFalse(objJson): - oJson = {} - for k, v in objJson.items(): - if isinstance(v, dict): - v = __handleStrNoneTrueFalse(v) - oJson[k] = v - elif isinstance(v, str) and re.match('^\s*none|true|false\s*$', v.lower()): - v = '\"' + v + '\"' - oJson[k] = v - else: - oJson[k] = v - return oJson def __handleListElements(sInput : str) -> str: items = re.split("\s*,\s*", sInput) @@ -777,7 +762,6 @@ def __handleListElements(sInput : str) -> str: except Exception as error: raise Exception(f"json file '{jFile}': '{error}'") - oJson = __handleStrNoneTrueFalse(oJson) os.chdir(currentDir) self.__checkDotInParamName(oJson) diff --git a/atest/jsonpreprocessor/test_jsonpreprocessor.py b/atest/jsonpreprocessor/test_jsonpreprocessor.py index 47b942ed..cf7cb6be 100644 --- a/atest/jsonpreprocessor/test_jsonpreprocessor.py +++ b/atest/jsonpreprocessor/test_jsonpreprocessor.py @@ -348,15 +348,15 @@ def test_none_true_false_datatype(self): sJsonfile = os.path.abspath("../testdata/config/06_override_params/project_config_04.json") oJsonPreprocessor = CJsonPreprocessor(syntax="python") oJsonData = oJsonPreprocessor.jsonLoad(sJsonfile) - assert oJsonData['string_null'] == "null" - assert oJsonData['string_none'] == '"None"' - assert oJsonData['string_true'] == '"True"' - assert oJsonData['string_false'] == '"False"' - assert oJsonData['convert_null_to_string'] == '"None"' - assert oJsonData['convert_none_to_string'] == '"None"' + assert oJsonData['string_null'] == 'null' + assert oJsonData['string_none'] == 'None' + assert oJsonData['string_true'] == 'True' + assert oJsonData['string_false'] == 'False' + assert oJsonData['convert_null_to_string'] == 'None' + assert oJsonData['convert_none_to_string'] == 'None' assert oJsonData['convert_float_to_string'] == "1.332" - assert oJsonData['convert_true_to_string'] == '"True"' - assert oJsonData['convert_false_to_string'] == '"False"' + assert oJsonData['convert_true_to_string'] == 'True' + assert oJsonData['convert_false_to_string'] == 'False' assert oJsonData['params']['global'] == JSONFORMAT_NONE_TRUE_FALSE['params']['global'] assert oJsonData['preprocessor']['definitions'] == JSONFORMAT_NONE_TRUE_FALSE['preprocessor']['definitions']