From 6177b04ce5e1096e0faccb3812402c81d50b177e Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Wed, 13 Sep 2023 14:02:13 +0200 Subject: [PATCH] [FIX] core: remove unnecessary `ast.unparse` Only available in Python 3.9, and for now at least Odoo remains committed to supporting Python 3.8. And it's not actually necessary: `literal_eval` supports ast node input, because internally it just `ast.parse`s the input then evaluates it, giving it an AST node just skips the parse step. closes odoo/odoo#135302 Signed-off-by: Xavier Morel (xmo) --- odoo/tests/form.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/odoo/tests/form.py b/odoo/tests/form.py index 5168bfc10b5..c4336005824 100644 --- a/odoo/tests/form.py +++ b/odoo/tests/form.py @@ -973,8 +973,8 @@ def get_static_context(context_str): result = {} for key_ast, val_ast in zip(context_ast.keys, context_ast.values): try: - key = ast.literal_eval(ast.unparse(key_ast)) - val = ast.literal_eval(ast.unparse(val_ast)) + key = ast.literal_eval(key_ast) + val = ast.literal_eval(val_ast) result[key] = val except ValueError: pass