From 9623054b9df23ade324368b833bcfa7683e71cd7 Mon Sep 17 00:00:00 2001 From: glondu Date: Sun, 26 Mar 2006 06:06:56 +0200 Subject: [PATCH] Il y avait un bug plutt important dans base_classes_crans.blacklist : dans certains cas, la blackliste ne se mettait pas jour dans la base. C'est corrig maintenant, il faut s'attendre un changement de comportement de certains scripts. darcs-hash:20060326040656-68412-4ce89637dd615da90adb424aaecc45c4dbc3344c.gz --- gestion/ldap_crans.py | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/gestion/ldap_crans.py b/gestion/ldap_crans.py index 5daf197d..bec84954 100755 --- a/gestion/ldap_crans.py +++ b/gestion/ldap_crans.py @@ -902,10 +902,7 @@ class base_classes_crans(crans_ldap): new = new[1] if new == '': liste.pop(index) - # La ligne suivante est inutile, elle est laissée dans un souci de clarté self._set('blacklist', liste) - # La liste n'a pas changé, mais son contenu, lui, a bien changé - self.modifs.setdefault('blacklist', None) return liste else: index = -1 @@ -939,16 +936,12 @@ class base_classes_crans(crans_ldap): new_c = '$'.join(map(str, new)) new_c = preattr(new_c)[1] - touched = True if index != -1: - if liste[index] == new_c: - touched = False - else: - liste[index] = new_c + liste[index] = new_c else: liste.append(new_c) - if touched: + if liste != self._data['blacklist']: # La ligne suivante est inutile, elle est laissée dans un souci de clarté self._data['blacklist'] = liste self.modifs.setdefault('blacklist_' + new[2], None)