Fix primefaces#2465: InputNumber default to min on allowEmpty=false

melloware · Nov 25, 2021 · 4f24303 · 4f24303
1 parent 8461882
commit 4f24303
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/components/inputnumber/InputNumber.js b/src/components/inputnumber/InputNumber.js
@@ -783,13 +783,18 @@ export class InputNumber extends Component {
 
         if (valueStr != null) {
             newValue = this.parseValue(valueStr);
-            newValue = !newValue && !this.props.allowEmpty ? 0 : newValue;
+            newValue = this.evaluateEmpty(newValue);
             this.updateInput(newValue, insertedValueStr, operation, valueStr);
 
             this.handleOnChange(event, currentValue, newValue);
         }
     }
 
+    evaluateEmpty(newValue) {
+        const minimum = this.props.min || 0;
+        return !newValue && !this.props.allowEmpty ? minimum : newValue;
+    }
+
     handleOnChange(event, currentValue, newValue) {
         if (this.props.onChange && this.isValueChanged(currentValue, newValue)) {
             this.props.onChange({
@@ -907,7 +912,7 @@ export class InputNumber extends Component {
     }
 
     updateInputValue(newValue) {
-        newValue = !newValue && !this.props.allowEmpty ? 0 : newValue;
+        newValue = this.evaluateEmpty(newValue);
 
         const inputEl = this.inputRef.current;
         const value = inputEl.value;
@@ -920,8 +925,8 @@ export class InputNumber extends Component {
     }
 
     formattedValue(val) {
-        const newVal = !val && !this.props.allowEmpty ? 0 : val;
-        return this.formatValue(newVal);
+        const newValue = this.evaluateEmpty(val);
+        return this.formatValue(newValue);
     }
 
     concatValues(val1, val2) {