fix(TDC-7476/form): check onFinish and onChange before invoking for t…

…he Text component (#5297)
Talend · Apr 30, 2024 · eb201ac · eb201ac
1 parent 54230de
commit eb201ac
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/.changeset/dirty-elephants-dream.md b/.changeset/dirty-elephants-dream.md
@@ -0,0 +1,5 @@
+---
+"@talend/react-forms": patch
+---
+
+fix: check onFinish and onChange before invoking for the Text component
diff --git a/packages/forms/src/UIForm/fields/Text/Text.component.js b/packages/forms/src/UIForm/fields/Text/Text.component.js
@@ -36,8 +36,16 @@ export default function Text(props) {
 		autoComplete,
 		autoFocus,
 		disabled: disabled || valueIsUpdating,
-		onBlur: event => onFinish(event, { schema }),
-		onChange: event => onChange(event, { schema, value: convertValue(type, event.target.value) }),
+		onBlur: event => {
+			if (onFinish) {
+				onFinish(event, { schema });
+			}
+		},
+		onChange: event => {
+			if (onChange) {
+				onChange(event, { schema, value: convertValue(type, event.target.value) });
+			}
+		},
 		placeholder,
 		readOnly,
 		type,