diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2012-09-11 13:52:10 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2012-09-19 11:09:09 +0200 |
| commit | 26166deb0c3a9938eeb94725705d49d259aa6e70 (patch) | |
| tree | d4bf0e63a61224de35b6e74b67f471bfb4ea4901 /install/ui/field.js | |
| parent | e39a109060ce8b92919c00aebd83a682a039d9b5 (diff) | |
| download | freeipa.git-26166deb0c3a9938eeb94725705d49d259aa6e70.tar.gz freeipa.git-26166deb0c3a9938eeb94725705d49d259aa6e70.tar.xz freeipa.git-26166deb0c3a9938eeb94725705d49d259aa6e70.zip | |
Fix integer validation when boundary value is empty string
There was an error in number validation check. If boundary value was an empty string, validation of a number always failed. This patch fixes the problem by not performing the check in these cases.
https://fedorahosted.org/freeipa/ticket/3066
Diffstat (limited to 'install/ui/field.js')
| -rw-r--r-- | install/ui/field.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/install/ui/field.js b/install/ui/field.js index 42da6f92..c5c999e6 100644 --- a/install/ui/field.js +++ b/install/ui/field.js @@ -448,13 +448,13 @@ IPA.metadata_validator = function(spec) { if (number) { - if (metadata.minvalue !== undefined && Number(value) < Number(metadata.minvalue)) { + if (IPA.defined(metadata.minvalue, true) && Number(value) < Number(metadata.minvalue)) { message = IPA.messages.widget.validation.min_value; message = message.replace('${value}', metadata.minvalue); return that.false_result(message); } - if (metadata.maxvalue !== undefined && Number(value) > Number(metadata.maxvalue)) { + if (IPA.defined(metadata.maxvalue, true) && Number(value) > Number(metadata.maxvalue)) { message = IPA.messages.widget.validation.max_value; message = message.replace('${value}', metadata.maxvalue); return that.false_result(message); |