fix(simple form iterator): defaulted values to null instead of empty string

Author: Krzysztof Marek Witkowski

packages/ra-core/src/form/FormDataConsumer.spec.tsx

@@ -1,15 +1,15 @@
 import * as React from 'react';
-import { render, waitFor, screen, fireEvent } from '@testing-library/react';
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 
 import FormDataConsumer, { FormDataConsumerView } from './FormDataConsumer';
 import { testDataProvider } from '../dataProvider';
 import {
     AdminContext,
+    ArrayInput,
     BooleanInput,
     SimpleForm,
-    TextInput,
     SimpleFormIterator,
-    ArrayInput,
+    TextInput,
 } from 'ra-ui-materialui';
 import expect from 'expect';
 
@@ -146,7 +146,7 @@ describe('FormDataConsumerView', () => {
 
         fireEvent.click(screen.getByLabelText('ra.action.add'));
 
-        expect(globalScopedFormData).toEqual({ name: '' });
+        expect(globalScopedFormData).toEqual({ name: null });
 
         fireEvent.change(
             screen.getByLabelText('resources.undefined.fields.authors.name'),

packages/ra-ui-materialui/src/input/ArrayInput/SimpleFormIterator.spec.tsx

@@ -1,10 +1,10 @@
 import * as React from 'react';
 import {
-    screen,
-    render,
     fireEvent,
-    waitFor,
     getByLabelText,
+    render,
+    screen,
+    waitFor,
 } from '@testing-library/react';
 import expect from 'expect';
 import { FormDataConsumer, testDataProvider } from 'ra-core';
@@ -770,7 +770,7 @@ describe('<SimpleFormIterator />', () => {
         await waitFor(() => {
             expect(save).toHaveBeenCalledWith(
                 {
-                    emails: [{ email: '' }],
+                    emails: [{ email: null }],
                 },
                 expect.anything()
             );
@@ -853,7 +853,7 @@ describe('<SimpleFormIterator />', () => {
             expect(save).toHaveBeenCalledWith(
                 {
                     id: 1,
-                    emails: [{ email: '', role: '' }],
+                    emails: [{ email: null, role: null }],
                 },
                 expect.anything()
             );

packages/ra-ui-materialui/src/input/ArrayInput/SimpleFormIterator.tsx

@@ -75,7 +75,7 @@ export const SimpleFormIterator = (props: SimpleFormIteratorProps) => {
         const { id, ...rest } = fields[0];
         initialDefaultValue.current = rest;
         for (const k in initialDefaultValue.current)
-            initialDefaultValue.current[k] = '';
+            initialDefaultValue.current[k] = null;
     }
 
     const addField = useCallback(
@@ -104,7 +104,7 @@ export const SimpleFormIterator = (props: SimpleFormIteratorProps) => {
                             input.props.source
                         ) {
                             defaultValue[input.props.source] =
-                                input.props.defaultValue ?? '';
+                                input.props.defaultValue ?? null;
                         }
                     });
                 }