From 395cf90fbc0afffde82ac35a87349a098014aee8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pierre-Elliott=20B=C3=A9cue?= Date: Wed, 29 Jan 2014 00:55:48 +0100 Subject: [PATCH] Killing trailing spaces --- lc_ldap.py | 8 ++++---- objets.py | 28 ++++++++++++++-------------- 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/lc_ldap.py b/lc_ldap.py index 8e76168..0378c7f 100644 --- a/lc_ldap.py +++ b/lc_ldap.py @@ -209,7 +209,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject, object): if not machines.has_key(parent_dn): machines[parent_dn] = [] machines[parent_dn].append(m) - elif (dn.startswith('aid=') or dn.startswith('cid=') or dn == variables.base_dn) and not parent.has_key(dn): + elif (dn.startswith('aid=') or dn.startswith('cid=') or dn == variables.base_dn) and not parent.has_key(dn): parent[dn] = objets.new_cransldapobject(self, dn, mode, uldif=ldif_to_uldif(attrs)) allmachines = [] for dn, mlist in machines.iteritems(): # on associe propriétaires et machines @@ -269,7 +269,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject, object): # On récupère le premier id libre dans la plages s'il n'est pas # déjà précisé dans le ldif rid = uldif.setdefault('rid', [unicode(self._find_id('rid', realm))]) - + # La machine peut-elle avoir une ipv4 ? if 'v6' not in realm: uldif['ipHostNumber'] = [ unicode(crans_utils.ip4_of_rid(int(rid[0]))) ] @@ -346,7 +346,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject, object): res = self.search_s(variables.base_dn, ldap.SCOPE_SUBTREE, '%s=*' % attr, attrlist = [attr]) nonfree = [ int(r[1].get(attr)[0]) for r in res if r[1].get(attr) ] nonfree.sort() - + plage = None # On récupère la plage des mids if realm != None: @@ -389,7 +389,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject, object): Teste le rapport entre le dn fourni et self Retourne une liste qui s'ajoutera à la liste des droits """ - + if objdn.endswith(self.dn) and objdn != self.dn: return [attributs.parent] diff --git a/objets.py b/objets.py index 87a4f71..fd0844b 100644 --- a/objets.py +++ b/objets.py @@ -131,13 +131,13 @@ class CransLdapObject(object): if not res: raise ValueError ('objet inexistant: %s' % dn) self.dn, ldif = res[0] - + # L'objet sortant de la base ldap, on ne fait pas de vérifications sur # l'état des données. uldif = lc_ldap.ldif_to_uldif(ldif) self.attrs = attributs.AttrsDict(self.conn, uldif, Parent=self) self._modifs = attributs.AttrsDict(self.conn, uldif, Parent=self) - + if mode in ['w', 'rw']: if not self.may_be(variables.modified, self.conn.droits + self.conn._check_parent(dn) + self.conn._check_self(dn)): raise EnvironmentError("Vous n'avez pas le droit de modifier cet objet.") @@ -168,7 +168,7 @@ class CransLdapObject(object): ### mais on devrait tout recoder pour utiliser l'historique LDAP""" assert isinstance(login, unicode) assert isinstance(chain, unicode) - + new_line = u"%s, %s : %s" % (time.strftime(attributs.historique.FORMAT), login, chain) # Attention, le __setitem__ est surchargé, mais pas .append sur l'historique self["historique"] = self.get("historique", []) + [new_line] @@ -244,7 +244,7 @@ class CransLdapObject(object): # Services à relancer services.services_to_restart(self.conn, {}, self._modifs, created_object=[self]) self._post_creation() - + # Vérifications après insertion. self.check_modifs() @@ -495,9 +495,9 @@ class CransLdapObject(object): class ObjectFactory(object): """Utilisée pour enregistrer toutes les classes servant à instancier un objet LDAP. Elle sert à les récupérer à partir de leur nom LDAP. - + Cette classe n'est jamais instanciée. - + """ _classes = {} @@ -539,7 +539,7 @@ class proprio(CransLdapObject): super(proprio, self).__init__(conn, dn, mode, ldif) self._machines = None self._factures = None - + def sursis_carte(self): for h in self['historique'][::-1]: x = re.match("(.*),.* : .*(paiement\+%s|inscription).*" % (config.ann_scol,), h.value) @@ -584,7 +584,7 @@ class proprio(CransLdapObject): bool_carte = True return bool_carte return True - + # XXX - To Delete def update_solde(self, diff, comment=u"", login=None): """Modifie le solde du proprio. diff peut être négatif ou positif.""" @@ -592,14 +592,14 @@ class proprio(CransLdapObject): login = self.conn.current_login assert isinstance(diff, int) or isinstance(diff, float) assert isinstance(comment, unicode) - + solde = float(self["solde"][0].value) new_solde = solde + diff - + # On vérifie qu'on ne dépasse par le découvert autorisé if new_solde < config.impression.decouvert: raise ValueError(u"Solde minimal atteint, opération non effectuée.") - + transaction = u"credit" if diff >=0 else u"debit" new_solde = u"%.2f" % new_solde self.history_add(login, u"%s %.2f Euros [%s]" % (transaction, abs(diff), comment)) @@ -662,7 +662,7 @@ class machine(CransLdapObject): if not self._proprio: self._proprio = new_cransldapobject(self.conn, parent_dn, self.mode) return self._proprio - + def blacklist_actif(self): u"""Renvoie la liste des blacklistes actives sur la machine et le proprio Améliorations possibles: @@ -835,7 +835,7 @@ class adherent(proprio): if u'posixAccount' in [str(o) for o in self.attrs['objectClass']]: return self.attrs['uid'][0] - + elif login: fn = crans_utils.strip_accents(unicode(self.attrs['prenom'][0]).capitalize()) ln = crans_utils.strip_accents(unicode(self.attrs['nom'][0]).capitalize()) @@ -851,7 +851,7 @@ class adherent(proprio): if os.path.exists("/var/mail/" + login): raise ValueError('Création du compte impossible : /var/mail/%s existant' % login) - + if not self.full: self.attribs = self.attribs + crans_account_attribs self.full = True