From f22f3eabb20e6c46f78ba5c40d75bb52b7872060 Mon Sep 17 00:00:00 2001
From: Jason Gerard DeRose <jderose@redhat.com>
Date: Tue, 20 Jan 2009 09:38:39 -0700
Subject: Fixed Param.validate() so that self.query is checked after
 self.required

---
 ipalib/parameters.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

(limited to 'ipalib/parameters.py')

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