Fix(CanvasForm): Fix commit

KaotoIO · Jan 14, 2025 · 314ea7f · 314ea7f
1 parent a8a104a
commit 314ea7f
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 12 deletions.
diff --git a/packages/ui/src/components/Form/CustomAutoFields.tsx b/packages/ui/src/components/Form/CustomAutoFields.tsx
@@ -42,7 +42,7 @@ export function CustomAutoFields({
     cleanQueryTerm,
     omitFields,
   );
-  const propertiesArray = getFieldGroups(filteredProperties!);
+  const propertiesArray = getFieldGroups(filteredProperties);
 
   if (
     canvasFormTabsContext?.selectedTab !== 'All' &&

diff --git a/packages/ui/src/components/Form/customField/CustomNestField.tsx b/packages/ui/src/components/Form/customField/CustomNestField.tsx
@@ -56,7 +56,7 @@ export const CustomNestField = connectField(
       props.properties as KaotoSchemaDefinition['schema']['properties'],
       cleanQueryTerm,
     );
-    const propertiesArray = getFieldGroups(filteredProperties!);
+    const propertiesArray = getFieldGroups(filteredProperties);
     if (propertiesArray.common.length === 0 && Object.keys(propertiesArray.groups).length === 0) return null;
 
     return (

diff --git a/packages/ui/src/utils/get-field-groups.ts b/packages/ui/src/utils/get-field-groups.ts
@@ -1,6 +1,9 @@
 import { getValue } from './get-value';
+import { isDefined } from './is-defined';
+
+export const getFieldGroups = (fields?: { [name: string]: unknown }) => {
+  if (!isDefined(fields)) return { common: [], groups: {} };
 
-export const getFieldGroups = (fields: { [name: string]: unknown }) => {
   const propertiesArray = Object.entries(fields).reduce(
     (acc, [name, definition]) => {
       const group: string = getValue(definition, 'group', '');

diff --git a/packages/ui/src/utils/get-filtered-properties.ts b/packages/ui/src/utils/get-filtered-properties.ts
@@ -13,13 +13,13 @@ export function getFilteredProperties(
 
   const filteredFormSchema = Object.entries(properties).reduce(
     (acc, [property, definition]) => {
-      if (definition['type'] === 'object' && 'properties' in definition) {
-        const subFilteredSchema = getFilteredProperties(definition['properties'], filter);
-        if (Object.keys(subFilteredSchema!).length > 0) {
-          acc![property] = { ...definition, properties: subFilteredSchema };
-        }
-      } else {
-        if ((!omitFields || !omitFields.includes(property)) && property.toLowerCase().includes(filter)) {
+      if (!omitFields?.includes(property)) {
+        if (definition['type'] === 'object' && 'properties' in definition) {
+          const subFilteredSchema = getFilteredProperties(definition['properties'], filter);
+          if (Object.keys(subFilteredSchema!).length > 0) {
+            acc![property] = { ...definition, properties: subFilteredSchema };
+          }
+        } else if (property.toLowerCase().includes(filter)) {
           acc![property] = definition;
         }
       }

diff --git a/packages/ui/src/utils/get-required-properties-schema.ts b/packages/ui/src/utils/get-required-properties-schema.ts
@@ -32,8 +32,8 @@ export function getRequiredPropertiesSchema(
         if (Object.keys(subSchema.properties as object).length > 0) {
           acc[property] = subSchema;
         }
-      } else {
-        if (isDefined(requiredProperties) && requiredProperties.indexOf(property) > -1) acc[property] = definition;
+      } else if (isDefined(requiredProperties) && requiredProperties.indexOf(property) > -1) {
+        acc[property] = definition;
       }
 
       return acc;