summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRob Crittenden <rcritten@redhat.com>2010-03-19 09:47:35 -0400
committerJason Gerard DeRose <jderose@redhat.com>2010-03-19 08:33:42 -0600
commit664ae51eb6e76ceb3630687d2ee423f69fb0ba19 (patch)
tree5ca57ada9437c2c876011865b11c8479e624815e
parent99da0d88f066826fc33562045d47f6cc760633b5 (diff)
downloadfreeipa-664ae51eb6e76ceb3630687d2ee423f69fb0ba19.tar.gz
freeipa-664ae51eb6e76ceb3630687d2ee423f69fb0ba19.tar.xz
freeipa-664ae51eb6e76ceb3630687d2ee423f69fb0ba19.zip
Raise an error if no modifications were performed in an update.
This will alert the user that nothing was done and is handy when used with --attr=''. This can be used to delete a non-required attribute but can be set to any valid attribute, present or not. We should alert the user if they attempt to delete a non-existant value.
-rw-r--r--ipalib/plugins/baseldap.py2
1 files changed, 0 insertions, 2 deletions
diff --git a/ipalib/plugins/baseldap.py b/ipalib/plugins/baseldap.py
index 6e601253..13529e69 100644
--- a/ipalib/plugins/baseldap.py
+++ b/ipalib/plugins/baseldap.py
@@ -311,8 +311,6 @@ class LDAPUpdate(LDAPQuery, crud.Update):
ldap.update_entry(dn, entry_attrs)
except errors.NotFound:
self.obj.handle_not_found(*keys)
- except errors.EmptyModlist:
- pass
try:
(dn, entry_attrs) = ldap.get_entry(dn, attrs_list)