Don't trigger onChange unnecessarily

React-Admin CI · React-Admin CI · commit 3ff5554b1034 · 2024-10-25T10:00:44.000+02:00
diff --git a/packages/ra-ui-materialui/src/input/DateInput.tsx b/packages/ra-ui-materialui/src/input/DateInput.tsx
@@ -133,7 +133,7 @@ export const DateInput = ({
             (localInputRef.current.valueAsDate != null &&
                 !isNaN(new Date(localInputRef.current.valueAsDate).getTime()));
 
-        if (isNewValueValid) {
+        if (isNewValueValid && field.value !== newValue) {
             field.onChange(newValue ?? '');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ export const DateInput = ({`
`133`	`133`	`(localInputRef.current.valueAsDate != null &&`
`134`	`134`	`!isNaN(new Date(localInputRef.current.valueAsDate).getTime()));`
`135`	`135`
`136`		`- if (isNewValueValid) {`
	`136`	`+ if (isNewValueValid && field.value !== newValue) {`
`137`	`137`	`field.onChange(newValue ?? '');`
`138`	`138`	`}`
`139`	`139`