diff options
author | Jason Gerard DeRose <jderose@redhat.com> | 2009-01-20 09:38:39 -0700 |
---|---|---|
committer | Jason Gerard DeRose <jderose@redhat.com> | 2009-01-20 09:38:39 -0700 |
commit | f22f3eabb20e6c46f78ba5c40d75bb52b7872060 (patch) | |
tree | cffba110af8d5b3b13de2ede525ce6feea473231 /ipalib | |
parent | 55fba5420d8ea57931937728102094492ca73d86 (diff) | |
download | freeipa-f22f3eabb20e6c46f78ba5c40d75bb52b7872060.tar.gz freeipa-f22f3eabb20e6c46f78ba5c40d75bb52b7872060.tar.xz freeipa-f22f3eabb20e6c46f78ba5c40d75bb52b7872060.zip |
Fixed Param.validate() so that self.query is checked after self.required
Diffstat (limited to 'ipalib')
-rw-r--r-- | ipalib/parameters.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/ipalib/parameters.py b/ipalib/parameters.py index 7cc93e905..76d88347c 100644 --- a/ipalib/parameters.py +++ b/ipalib/parameters.py @@ -490,13 +490,12 @@ class Param(ReadOnly): :param value: A proposed value for this parameter. """ - # FIXME: this should be after 'if value is None:' - if self.query: - return if value is None: if self.required: raise RequirementError(name=self.name) return + if self.query: + return if self.multivalue: if type(value) is not tuple: raise TypeError( |