crans/lc_ldap
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[lc_ldap] Quitte à faire du lazy attrify, autant le faire partout,

Browse source 
pour avoir une certaine homogénéité dans le code.
This commit is contained in:
Pierre-Elliott Bécue 2013-04-01 23:13:32 +02:00
parent b957a41cd7
commit 90a0b565cc
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
lc_ldap.py
View file

@ -465,26 +465,26 @@ class CransLdapObject(object):
orig = {} orig = {}
if ldif: if ldif:
if dn != base_dn: # new_cransldapobject ne donne pas de ldif formaté et utilise un ldif non formaté, donc on formate if dn != base_dn: # new_cransldapobject ne donne pas de ldif formaté et utilise un ldif non formaté, donc on formate
self.attrs = attributs.AttrsDict(conn, ldif, check_ctxt = False) self.attrs = attributs.AttrsDict(self.conn, ldif, check_ctxt = False)
else: else:
self.attrs = ldif self.attrs = ldif
self._modifs = attributs.AttrsDict(conn, ldif, check_ctxt = False) self._modifs = attributs.AttrsDict(self.conn, ldif, check_ctxt = False)
orig = ldif orig = ldif
elif dn != base_dn: elif dn != base_dn:
res = conn.search_s(dn, 0) res = self.conn.search_s(dn, 0)
if not res: if not res:
raise ValueError ('objet inexistant: %s' % dn) raise ValueError ('objet inexistant: %s' % dn)
self.dn, res_attrs = res[0] self.dn, res_attrs = res[0]
# L'objet sortant de la base ldap, on ne fait pas de vérifications sur # L'objet sortant de la base ldap, on ne fait pas de vérifications sur
# l'état des données. # l'état des données.
self.attrs = attributs.AttrsDict(conn, res_attrs, check_ctxt = False) self.attrs = attributs.AttrsDict(self.conn, res_attrs, check_ctxt = False)
# Pour test en cas de mode w ou rw # Pour test en cas de mode w ou rw
orig = res[0][1] orig = res[0][1]
self._modifs = attributs.AttrsDict(conn, res[0][1], check_ctxt = False) self._modifs = attributs.AttrsDict(self.conn, res[0][1], check_ctxt = False)
# Je m'interroge sur la pertinence de cette partie, je pense qu'elle n'est # Je m'interroge sur la pertinence de cette partie, je pense qu'elle n'est
# pas utile. -- PEB 27/01/2013 # pas utile. -- PEB 27/01/2013
@ -544,7 +544,7 @@ class CransLdapObject(object):
raise EnvironmentError("Impossible de modifier l'objet, peut-être n'existe-t-il pas ?") raise EnvironmentError("Impossible de modifier l'objet, peut-être n'existe-t-il pas ?")
# Vérification des modifications # Vérification des modifications
self.attrs = attributs.AttrDict(conn, self.conn.search_s(self.dn, 0)[0][1], check_ctxt=False) self.attrs = attributs.AttrsDict(self.conn, self.conn.search_s(self.dn, 0)[0][1], check_ctxt=False)
differences = [] differences = []
# On fait les différences entre les deux dicos # On fait les différences entre les deux dicos
for attr in set(self.attrs.keys()).union(set(self._modifs.keys())): for attr in set(self.attrs.keys()).union(set(self._modifs.keys())):