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( |