Pylint says "no".

This commit is contained in:
Pierre-Elliott Bécue 2013-01-28 22:53:38 +01:00
parent d12ad982cd
commit b578b4a6a7

View file

@ -204,15 +204,15 @@ class lc_ldap(ldap.ldapobject.LDAPObject):
m = new_cransldapobject(self, dn, ldif = attrs)
parent_dn = dn.split(',', 1)[1]
if not machines.has_key(parent_dn):
machines[parent_dn]=[]
machines[parent_dn] = []
machines[parent_dn].append(m)
elif (dn.startswith('aid=') or dn.startswith('cid=') or dn==base_dn) and not parent.has_key(dn):
parent[dn]=new_cransldapobject(self, dn, ldif = attrs)
allmachines=[]
elif (dn.startswith('aid=') or dn.startswith('cid=') or dn == base_dn) and not parent.has_key(dn):
parent[dn] = new_cransldapobject(self, dn, ldif = attrs)
allmachines = []
for dn,mlist in machines.items(): # on associe propriétaires et machines
parent[dn]._machines=mlist
parent[dn]._machines = mlist
for m in mlist:
m._proprio=parent[dn]
m._proprio = parent[dn]
allmachines.append(m)
return allmachines,parent.values() # on renvoie la liste des machines et des adherents (dont club et crans)
@ -221,7 +221,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject):
s'éxécuter le plus rapidement possible. On dumpe malgré tout
toute la base, c'est pour pouvoir aussi rajouter à moindre coût
les propriétaires."""
machines,_=self.allMachinesAdherents()
machines,_ = self.allMachinesAdherents()
return machines
def allAdherents(self):
@ -229,7 +229,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject):
s'éxécuter le plus rapidement possible. On dumpe malgré tout
toute la base, c'est pour pouvoir aussi rajouter à moindre coût
les machines."""
_,adherents=self.allMachinesAdherents()
_,adherents = self.allMachinesAdherents()
return adherents
def newMachine(self, parent, realm, uldif):