fix: default values

ForestAdmin · Sep 12, 2024 · 7ccb8f6 · 7ccb8f6
1 parent b099557
commit 7ccb8f6
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/packages/agent/src/utils/forest-schema/generator-actions.ts b/packages/agent/src/utils/forest-schema/generator-actions.ts
@@ -145,6 +145,8 @@ export default class SchemaGeneratorActions {
     const fields: ActionField[] = [];
     let layout: ActionLayoutElement[] = [];
 
+    if (!formElements) return { fields: [], layout: [] };
+
     formElements.forEach(element => {
       if (element.type === 'Layout') {
         hasLayout = true;

diff --git a/packages/agent/test/routes/modification/action/action.test.ts b/packages/agent/test/routes/modification/action/action.test.ts
@@ -573,7 +573,10 @@ describe('ActionRoute', () => {
         },
       );
 
-      expect(context.response.body).toEqual({ fields: [{ field: 'firstname', type: 'String' }] });
+      expect(context.response.body).toEqual({
+        fields: [{ field: 'firstname', type: 'String' }],
+        layout: [],
+      });
     });
 
     test('handleHook should generate the form if called with changehook params', async () => {
@@ -620,7 +623,10 @@ describe('ActionRoute', () => {
         },
       );
 
-      expect(context.response.body).toEqual({ fields: [{ field: 'firstname', type: 'String' }] });
+      expect(context.response.body).toEqual({
+        fields: [{ field: 'firstname', type: 'String' }],
+        layout: [],
+      });
     });
   });
 

diff --git a/packages/agent/test/utils/forest-schema/generator-action-field-widget.test.ts b/packages/agent/test/utils/forest-schema/generator-action-field-widget.test.ts
@@ -13,7 +13,6 @@ describe('GeneratorActionFieldWidget', () => {
     });
 
     it('should return null when the field type is Collection', () => {
-      // @ts-expect-error Collection type does not support widget
       const result = GeneratorActionFieldWidget.buildWidgetOptions({
         type: 'Collection',
         label: 'Label',
@@ -25,7 +24,6 @@ describe('GeneratorActionFieldWidget', () => {
     });
 
     it('should return null when the field type is Enum', () => {
-      // @ts-expect-error Collection type does not support widget
       const result = GeneratorActionFieldWidget.buildWidgetOptions({
         type: 'Enum',
         label: 'Label',
@@ -38,7 +36,6 @@ describe('GeneratorActionFieldWidget', () => {
     });
 
     it('should return null when the field type is EnumList', () => {
-      // @ts-expect-error Collection type does not support widget
       const result = GeneratorActionFieldWidget.buildWidgetOptions({
         type: 'EnumList',
         label: 'Label',

diff --git a/packages/datasource-customizer/src/decorators/actions/collection.ts b/packages/datasource-customizer/src/decorators/actions/collection.ts
@@ -162,7 +162,7 @@ export default class ActionCollectionDecorator extends CollectionDecorator {
     field: DynamicField,
     data: Record<string, unknown>,
   ): Promise<DynamicField> {
-    if (field.label === undefined) {
+    if (data[field.label] === undefined) {
       const defaultValue = await this.evaluate(context, null, field.defaultValue);
       data[field.label] = defaultValue;
     }