diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2011-02-23 10:43:40 -0500 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2011-03-02 20:31:28 +0100 |
| commit | 7b6b3c2276285802edf89950c43ed40c45fb7b22 (patch) | |
| tree | aac0657c974e58260bb9ec98bd0674d9c22249e7 /ipaserver | |
| parent | 81fd790cda41644d21b9deecdc5c6e3c50155b3d (diff) | |
| download | freeipa-review.tar.gz freeipa-review.tar.xz freeipa-review.zip | |
Replace only if old and new have nothing in commonreview
Jakub did the initial diagnosis of this, I added a fix for removing
the last entry when removing members and a test case.
ticket 1000
Diffstat (limited to 'ipaserver')
| -rw-r--r-- | ipaserver/plugins/ldap2.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipaserver/plugins/ldap2.py b/ipaserver/plugins/ldap2.py index 568792d1..ebbca60e 100644 --- a/ipaserver/plugins/ldap2.py +++ b/ipaserver/plugins/ldap2.py @@ -798,7 +798,7 @@ class ldap2(CrudBackend, Encoder): force_replace = False if k in self._FORCE_REPLACE_ON_UPDATE_ATTRS or is_single_value: force_replace = True - elif len(adds) == 1 and len(rems) == 1: + elif len(v) > 0 and len(v.intersection(old_v)) == 0: force_replace = True if adds: |