From 73a170a388f06f7db54199fdcf65ab9a92d29248 Mon Sep 17 00:00:00 2001 From: Benjamin Tissoires Date: Thu, 22 Feb 2024 16:30:01 +0100 Subject: [PATCH] openapi: fix breakage introduced with blank return Introduced by commit f8ef2e33de2d ("cards.js Added a control to check error if card is not updated"), a blank return was added at line 3914. The generate_openapi script assumed a return statement always has an argument, and this was crashing. Fixes #5320 --- openapi/generate_openapi.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/openapi/generate_openapi.py b/openapi/generate_openapi.py index 684aebc63..2398f1d57 100644 --- a/openapi/generate_openapi.py +++ b/openapi/generate_openapi.py @@ -66,7 +66,8 @@ def get_req_body_elems(obj, elems): get_req_body_elems(obj.left, elems) get_req_body_elems(obj.right, elems) elif obj.type in ('ReturnStatement', 'UnaryExpression'): - get_req_body_elems(obj.argument, elems) + if obj.argument is not None: + get_req_body_elems(obj.argument, elems) elif obj.type == 'Identifier': return obj.name elif obj.type in ['Literal', 'FunctionDeclaration', 'ThrowStatement']: