mirror of
https://git.mirrors.martin98.com/https://github.com/langgenius/dify.git
synced 2025-07-28 05:51:58 +08:00
fix: Refines None checks in result transformation
Simplifies the code by replacing type checks for None with direct comparisons, improving readability and consistency in handling None values during output validation. Signed-off-by: -LAN- <laipz8200@outlook.com>
This commit is contained in:
parent
559ab46ee1
commit
5f7f851b17
@ -195,7 +195,7 @@ class CodeNode(BaseNode[CodeNodeData]):
|
|||||||
if output_config.type == "object":
|
if output_config.type == "object":
|
||||||
# check if output is object
|
# check if output is object
|
||||||
if not isinstance(result.get(output_name), dict):
|
if not isinstance(result.get(output_name), dict):
|
||||||
if isinstance(result.get(output_name), type(None)):
|
if result.get(output_name) is None:
|
||||||
transformed_result[output_name] = None
|
transformed_result[output_name] = None
|
||||||
else:
|
else:
|
||||||
raise OutputValidationError(
|
raise OutputValidationError(
|
||||||
@ -223,7 +223,7 @@ class CodeNode(BaseNode[CodeNodeData]):
|
|||||||
elif output_config.type == "array[number]":
|
elif output_config.type == "array[number]":
|
||||||
# check if array of number available
|
# check if array of number available
|
||||||
if not isinstance(result[output_name], list):
|
if not isinstance(result[output_name], list):
|
||||||
if isinstance(result[output_name], type(None)):
|
if result[output_name] is None:
|
||||||
transformed_result[output_name] = None
|
transformed_result[output_name] = None
|
||||||
else:
|
else:
|
||||||
raise OutputValidationError(
|
raise OutputValidationError(
|
||||||
@ -244,7 +244,7 @@ class CodeNode(BaseNode[CodeNodeData]):
|
|||||||
elif output_config.type == "array[string]":
|
elif output_config.type == "array[string]":
|
||||||
# check if array of string available
|
# check if array of string available
|
||||||
if not isinstance(result[output_name], list):
|
if not isinstance(result[output_name], list):
|
||||||
if isinstance(result[output_name], type(None)):
|
if result[output_name] is None:
|
||||||
transformed_result[output_name] = None
|
transformed_result[output_name] = None
|
||||||
else:
|
else:
|
||||||
raise OutputValidationError(
|
raise OutputValidationError(
|
||||||
@ -265,7 +265,7 @@ class CodeNode(BaseNode[CodeNodeData]):
|
|||||||
elif output_config.type == "array[object]":
|
elif output_config.type == "array[object]":
|
||||||
# check if array of object available
|
# check if array of object available
|
||||||
if not isinstance(result[output_name], list):
|
if not isinstance(result[output_name], list):
|
||||||
if isinstance(result[output_name], type(None)):
|
if result[output_name] is None:
|
||||||
transformed_result[output_name] = None
|
transformed_result[output_name] = None
|
||||||
else:
|
else:
|
||||||
raise OutputValidationError(
|
raise OutputValidationError(
|
||||||
|
Loading…
x
Reference in New Issue
Block a user