From e4564ab33b73d6c1d8004ebe34c573919b443d6d Mon Sep 17 00:00:00 2001 From: Petr Voborník Date: Feb 29 2012 12:07:22 +0000 Subject: Moved is_empty method from field to IPA object is_empty method represents IPA UI standard of evaluating whether value is empty. Therefore is should be placed in IPA object instead of IPA.field to allow reuse in different locations. https://fedorahosted.org/freeipa/ticket/2351 --- diff --git a/install/ui/dns.js b/install/ui/dns.js index c849eec..8125f9d 100644 --- a/install/ui/dns.js +++ b/install/ui/dns.js @@ -1845,8 +1845,8 @@ IPA.dns.netaddr_field = function(spec) { var values = that.field_save(); //check for empty value: null, [''], '', [] - var orig_empty = that.is_empty(that.values); - var new_empty= that.is_empty(values); + var orig_empty = IPA.is_empty(that.values); + var new_empty= IPA.is_empty(values); if (orig_empty && new_empty) return false; if (orig_empty != new_empty) return true; diff --git a/install/ui/field.js b/install/ui/field.js index b821fd9..d5e3fe3 100644 --- a/install/ui/field.js +++ b/install/ui/field.js @@ -103,7 +103,7 @@ IPA.field = function(spec) { that.validate_required = function() { var values = that.save(); - if (that.is_empty(values) && that.is_required() && that.enabled) { + if (IPA.is_empty(values) && that.is_required() && that.enabled) { that.valid = false; that.show_error(IPA.messages.widget.validation.required); return false; @@ -124,7 +124,7 @@ IPA.field = function(spec) { var values = that.save(); - if (that.is_empty(values)) { + if (IPA.is_empty(values)) { return that.valid; } @@ -268,8 +268,8 @@ IPA.field = function(spec) { var values = that.save(); //check for empty value: null, [''], '', [] - var orig_empty = that.is_empty(that.values); - var new_empty= that.is_empty(values); + var orig_empty = IPA.is_empty(that.values); + var new_empty= IPA.is_empty(values); if (orig_empty && new_empty) return false; if (orig_empty != new_empty) return true; @@ -296,22 +296,6 @@ IPA.field = function(spec) { return true; }; - that.is_empty = function(value) { - - var empty = false; - - if (!value) empty = true; - - if (value instanceof Array) { - empty = empty || value.length === 0 || - (value.length === 1) && (value[0] === ''); - } - - if (value === '') empty = true; - - return empty; - }; - /** * This function compares the original values and the * values entered in the UI. If the values have changed @@ -561,7 +545,7 @@ IPA.multivalued_field = function(spec) { that.hide_error(); that.valid = true; - if (that.is_empty(values)) { + if (IPA.is_empty(values)) { return that.valid; } diff --git a/install/ui/ipa.js b/install/ui/ipa.js index 433d7fe..441e646 100644 --- a/install/ui/ipa.js +++ b/install/ui/ipa.js @@ -1207,6 +1207,22 @@ IPA.create_options = function(labels, values) { return options; }; +IPA.is_empty = function(value) { + + var empty = false; + + if (!value) empty = true; + + if (value instanceof Array) { + empty = empty || value.length === 0 || + (value.length === 1) && (value[0] === ''); + } + + if (value === '') empty = true; + + return empty; +}; + IPA.config = { default_priority: 500 };