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/crud.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/crud.py')
-rw-r--r-- | ipalib/crud.py | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/ipalib/crud.py b/ipalib/crud.py index b9dfb025e..12edbf58a 100644 --- a/ipalib/crud.py +++ b/ipalib/crud.py @@ -186,20 +186,29 @@ class Update(PKQuery): for option in super(Update, self).get_options(): yield option for option in self.obj.params_minus_pk(): + new_flags = option.flags attribute = 'virtual_attribute' not in option.flags + if option.required: + # Required options turn into non-required, since not specifying + # them means that they are not changed. + # However, they cannot be empty (i.e. explicitly set to None). + new_flags = new_flags.union(['nonempty']) if 'no_update' in option.flags: continue if 'ask_update' in option.flags: yield option.clone( attribute=attribute, query=False, required=False, - autofill=False, alwaysask=True + autofill=False, alwaysask=True, flags=new_flags, ) elif 'req_update' in option.flags: yield option.clone( attribute=attribute, required=True, alwaysask=False, + flags=new_flags, ) else: - yield option.clone(attribute=attribute, required=False, autofill=False) + yield option.clone(attribute=attribute, required=False, + autofill=False, flags=new_flags, + ) if not self.extra_options_first: for option in super(Update, self).get_options(): yield option |