Merge branch 'main' into avoid-full-lodash-import

12kb · web-flow · commit 343efbad5f3b · 2025-10-29T14:39:47.000+03:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,12 @@
 # Changelog
 
+## [5.9.1](https://github.com/gravity-ui/dynamic-forms/compare/v5.9.0...v5.9.1) (2025-10-23)
+
+
+### Bug Fixes
+
+* **utils:** imporve nullish-checking for number inputs ([#327](https://github.com/gravity-ui/dynamic-forms/issues/327)) ([537d5f6](https://github.com/gravity-ui/dynamic-forms/commit/537d5f68e6db46eb4cba165d58ad8d50c2d8ba9f))
+
 ## [5.9.0](https://github.com/gravity-ui/dynamic-forms/compare/v5.8.1...v5.9.0) (2025-09-10)
 
 
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@gravity-ui/dynamic-forms",
-  "version": "5.9.0",
+  "version": "5.9.1",
   "description": "",
   "license": "MIT",
   "main": "build/cjs/index.js",
diff --git a/src/lib/core/components/Form/Controller/utils.tsx b/src/lib/core/components/Form/Controller/utils.tsx
@@ -239,7 +239,7 @@ export const getFieldInitials = <
     }
 
     if (isNil(value)) {
-        if (spec.defaultValue) {
+        if (!isNil(spec.defaultValue)) {
             value = transformArrIn(spec.defaultValue) as DirtyValue;
         }
         // if the spec with type array or object, and this spec has "required === true",
@@ -298,7 +298,7 @@ export const getFieldMethods = <
         let value = transformArrIn(_value);
 
         if (isNumberSpec(spec) && !error) {
-            value = (value ? Number(value) : undefined) as DirtyValue;
+            value = (isNil(value) || value === '' ? undefined : Number(value)) as DirtyValue;
         }
 
         let newChildErrors: Record<string, ValidateError> = {...state.childErrors};

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@gravity-ui/dynamic-forms",`
`3`		`- "version": "5.9.0",`
	`3`	`+ "version": "5.9.1",`
`4`	`4`	`"description": "",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"main": "build/cjs/index.js",`
Original file line number	Diff line number	Diff line change
`@@ -239,7 +239,7 @@ export const getFieldInitials = <`
`239`	`239`	`}`
`240`	`240`
`241`	`241`	`if (isNil(value)) {`
`242`		`- if (spec.defaultValue) {`
	`242`	`+ if (!isNil(spec.defaultValue)) {`
`243`	`243`	`value = transformArrIn(spec.defaultValue) as DirtyValue;`
`244`	`244`	`}`
`245`	`245`	`// if the spec with type array or object, and this spec has "required === true",`
`@@ -298,7 +298,7 @@ export const getFieldMethods = <`
`298`	`298`	`let value = transformArrIn(_value);`
`299`	`299`
`300`	`300`	`if (isNumberSpec(spec) && !error) {`
`301`		`- value = (value ? Number(value) : undefined) as DirtyValue;`
	`301`	`+ value = (isNil(value) \|\| value === '' ? undefined : Number(value)) as DirtyValue;`
`302`	`302`	`}`
`303`	`303`
`304`	`304`	`let newChildErrors: Record<string, ValidateError> = {...state.childErrors};`