diff options
author | Rob Crittenden <rcritten@redhat.com> | 2007-12-07 17:07:03 -0500 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2007-12-07 17:07:03 -0500 |
commit | 00489597d8bbd772a0a02562309b1e1acbf796fc (patch) | |
tree | 2ef8e31170ee47759bb19239f37963cabebfaf4a /ipa-python/ipavalidate.py | |
parent | 5e4a1629545a981c1183ec9d904f7072f6583284 (diff) | |
download | freeipa-00489597d8bbd772a0a02562309b1e1acbf796fc.tar.gz freeipa-00489597d8bbd772a0a02562309b1e1acbf796fc.tar.xz freeipa-00489597d8bbd772a0a02562309b1e1acbf796fc.zip |
Rework input validation to more closely match what we require in the UI
Diffstat (limited to 'ipa-python/ipavalidate.py')
-rw-r--r-- | ipa-python/ipavalidate.py | 28 |
1 files changed, 19 insertions, 9 deletions
diff --git a/ipa-python/ipavalidate.py b/ipa-python/ipavalidate.py index 918c34a62..1750b8397 100644 --- a/ipa-python/ipavalidate.py +++ b/ipa-python/ipavalidate.py @@ -19,7 +19,7 @@ import re -def email(mail, notEmpty=True): +def Email(mail, notEmpty=True): """Do some basic validation of an e-mail address. Return 0 if ok Return 1 if not @@ -49,7 +49,7 @@ def email(mail, notEmpty=True): return 0 -def plain(text, notEmpty=False): +def Plain(text, notEmpty=False, allowSpaces=True): """Do some basic validation of a plain text field Return 0 if ok Return 1 if not @@ -57,23 +57,33 @@ def plain(text, notEmpty=False): If notEmpty is True the this will return an error if the field is "" or None. """ - textRE = re.compile(r"^[a-zA-Z_\-0-9\'\ ]*$") - - if not text and notEmpty is True: - return 1 - - if text is None: + if (text is None) or (not text.strip()): if notEmpty is True: return 1 else: return 0 + if allowSpaces: + textRE = re.compile(r"^[a-zA-Z_\-0-9\'\ ]*$") + else: + textRE = re.compile(r"^[a-zA-Z_\-0-9\']*$") if not textRE.search(text): return 1 return 0 -def path(text, notEmpty=False): +def String(text, notEmpty=False): + """A string type. This is much looser in what it allows than plain""" + + if text is None or not text.strip(): + if notEmpty is True: + return 1 + else: + return 0 + + return 0 + +def Path(text, notEmpty=False): """Do some basic validation of a path Return 0 if ok Return 1 if not |