Fix #1161

apache · Mar 5, 2024 · 3ce8124 · 3ce8124
1 parent f46c7e9
commit 3ce8124
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 6 deletions.
diff --git a/karavan-app/src/main/webui/src/designer/property/property/DslPropertyField.tsx b/karavan-app/src/main/webui/src/designer/property/property/DslPropertyField.tsx
@@ -116,7 +116,7 @@ export function DslPropertyField(props: Props) {
         if (checkChanges) {
             const interval = setInterval(() => {
                 if (props.value !== textValue) {
-                    if (isVariable) {
+                    if (isVariable && textValue !== undefined) {
                         propertyChanged(property.name, variableType.concat(textValue));
                     } else {
                         propertyChanged(property.name, textValue);
@@ -312,7 +312,11 @@ export function DslPropertyField(props: Props) {
                            value={textValue?.toString()}
                            customIcon={property.type !== 'string' ?
                                <Text component={TextVariants.p}>{property.type}</Text> : undefined}
-                           onBlur={_ => propertyChanged(property.name, variableType.concat(textValue))}
+                           onBlur={_ => {
+                               if (textValue !== undefined) {
+                                   propertyChanged(property.name, variableType.concat(textValue))
+                               }
+                           }}
                            onFocus={_ => setCheckChanges(true)}
                            onChange={(_, v) => {
                                setTextValue(v);

diff --git a/karavan-designer/src/designer/property/property/DslPropertyField.tsx b/karavan-designer/src/designer/property/property/DslPropertyField.tsx
@@ -116,7 +116,7 @@ export function DslPropertyField(props: Props) {
         if (checkChanges) {
             const interval = setInterval(() => {
                 if (props.value !== textValue) {
-                    if (isVariable) {
+                    if (isVariable && textValue !== undefined) {
                         propertyChanged(property.name, variableType.concat(textValue));
                     } else {
                         propertyChanged(property.name, textValue);
@@ -312,7 +312,11 @@ export function DslPropertyField(props: Props) {
                            value={textValue?.toString()}
                            customIcon={property.type !== 'string' ?
                                <Text component={TextVariants.p}>{property.type}</Text> : undefined}
-                           onBlur={_ => propertyChanged(property.name, variableType.concat(textValue))}
+                           onBlur={_ => {
+                               if (textValue !== undefined) {
+                                   propertyChanged(property.name, variableType.concat(textValue))
+                               }
+                           }}
                            onFocus={_ => setCheckChanges(true)}
                            onChange={(_, v) => {
                                setTextValue(v);

diff --git a/karavan-space/src/designer/property/property/DslPropertyField.tsx b/karavan-space/src/designer/property/property/DslPropertyField.tsx
@@ -116,7 +116,7 @@ export function DslPropertyField(props: Props) {
         if (checkChanges) {
             const interval = setInterval(() => {
                 if (props.value !== textValue) {
-                    if (isVariable) {
+                    if (isVariable && textValue !== undefined) {
                         propertyChanged(property.name, variableType.concat(textValue));
                     } else {
                         propertyChanged(property.name, textValue);
@@ -312,7 +312,11 @@ export function DslPropertyField(props: Props) {
                            value={textValue?.toString()}
                            customIcon={property.type !== 'string' ?
                                <Text component={TextVariants.p}>{property.type}</Text> : undefined}
-                           onBlur={_ => propertyChanged(property.name, variableType.concat(textValue))}
+                           onBlur={_ => {
+                               if (textValue !== undefined) {
+                                   propertyChanged(property.name, variableType.concat(textValue))
+                               }
+                           }}
                            onFocus={_ => setCheckChanges(true)}
                            onChange={(_, v) => {
                                setTextValue(v);