range field now correctly takes 0 into account (#4822)

apostrophecms · Dec 17, 2024 · 1038a65 · 1038a65
1 parent 72eeeae
commit 1038a65
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@
 * a11y improvements for context menus.
 * Fixes broken widget preview URL when the image is overridden (module improve) and external build module is registered.
 * Inject dynamic custom bundle CSS when using external build module with no CSS entry point.
+* Range field now correctly takes 0 into account.
 
 ### Adds
 

diff --git a/modules/@apostrophecms/schema/ui/apos/logic/AposInputRange.js b/modules/@apostrophecms/schema/ui/apos/logic/AposInputRange.js
@@ -32,7 +32,7 @@ export default {
     // The range spec defaults to a value of midway between the min and max
     // Example: a range with an unset value and a min of 0 and max of 100 will become 50
     // This does not allow ranges to go unset :(
-    if (!this.next) {
+    if (!this.next && this.next !== 0) {
       this.unset();
     }
   },
@@ -47,7 +47,7 @@ export default {
     },
     validate(value) {
       if (this.field.required) {
-        if (!value) {
+        if (!value && value !== 0) {
           return 'required';
         }
       }