diff options
author | Petr Viktorin <pviktori@redhat.com> | 2012-02-16 07:11:56 -0500 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2012-03-12 17:16:14 +0100 |
commit | 7cfc16ca58dfb22bc6e9cd519e6ecc7a10435fa1 (patch) | |
tree | a638850c0a1ba891376f49f5f3a2c8ced66ea696 /ipalib/frontend.py | |
parent | 1dc11a01d7e2a8e561b3a79aa97bf0939cd3fd25 (diff) | |
download | freeipa-7cfc16ca58dfb22bc6e9cd519e6ecc7a10435fa1.tar.gz freeipa-7cfc16ca58dfb22bc6e9cd519e6ecc7a10435fa1.tar.xz freeipa-7cfc16ca58dfb22bc6e9cd519e6ecc7a10435fa1.zip |
Enforce that required attributes can't be set to None in CRUD Update
The `required` parameter attribute didn't distinguish between cases
where the parameter is not given and all, and where the parameter is
given but empty. The case of updating a required attribute couldn't
be validated properly, because when it is given but empty, validators
don't run.
This patch introduces a new flag, 'nonempty', that specifies the
parameter can be missing (if not required), but it can't be None.
This flag gets added automatically to required parameters in CRUD
Update.
Diffstat (limited to 'ipalib/frontend.py')
-rw-r--r-- | ipalib/frontend.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipalib/frontend.py b/ipalib/frontend.py index 028e17e79..da25b4c1a 100644 --- a/ipalib/frontend.py +++ b/ipalib/frontend.py @@ -651,7 +651,7 @@ class Command(HasParam): """ for param in self.params(): value = kw.get(param.name, None) - param.validate(value, self.env.context) + param.validate(value, self.env.context, supplied=param.name in kw) def verify_client_version(self, client_version): """ |