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

[objets/machine] Pour blacklist_actif on appel plutôt super().blacklist_actif au lieu de la réimplémenté

Browse source
This commit is contained in:
Valentin Samir 2014-02-19 19:02:41 +01:00
parent 838c4e9a73
commit 840cef04c7
1 changed files with 4 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

12
objets.py
View file

@ -705,14 +705,10 @@ class machine(CransLdapObject):
self._proprio = new_cransldapobject(self.conn, parent_dn, self.mode) self._proprio = new_cransldapobject(self.conn, parent_dn, self.mode)
return self._proprio return self._proprio
def blacklist_actif(self): def blacklist_actif(self, excepts=[]):
u"""Renvoie la liste des blacklistes actives sur la machine et le proprio u"""Renvoie la liste des blacklistes actives sur la machine et le proprio"""
Améliorations possibles: black=self.proprio().blacklist_actif(excepts)
- Proposer de filtrer les blacklistes avec un arg supplémentaire ? black.extend(super(machine, self).blacklist_actif(excepts))
- Vérifier les blacklistes des machines pour les adhérents ?"""
black=self.proprio().blacklist_actif()
attrs = (self.attrs if self.mode not in ["w", "rw"] else self._modifs)
black.extend(filter((lambda bl: bl.is_actif()), attrs.get("blacklist",[])))
return black return black
def _post_creation(self): def _post_creation(self):