[stats_cableurs.py] small bugfixes

dreadful glimpse?

darcs-hash:20100420214952-bd074-78c5c60758fd88ca6c529588fe80715441ebd469.gz
This commit is contained in:
Antoine Durand-Gasselin 2010-04-20 23:49:52 +02:00
parent 790a358b96
commit 16bcafe5ad

View file

@ -1,5 +1,5 @@
#!/usr/bin/python #!/usr/bin/python
# -*- mode: python; coding: utf-8 -*- # -*- coding: utf-8 -*-
# #
# stats_cableurs.py # stats_cableurs.py
# ----------------- # -----------------
@ -30,7 +30,7 @@ sys.path.append('/usr/scripts/lib')
from optparse import OptionParser, OptionGroup from optparse import OptionParser, OptionGroup
sys.path.append("/usr/scripts/gestion/") sys.path.append("/usr/scripts/gestion/")
from ldap_crans import CransLdap from ldap_crans import CransLdap, Adherent
from config import ann_scol from config import ann_scol
@ -211,8 +211,8 @@ class StatsCableursMachines(StatsCableursBase):
# on ne va pas non plus compter les actions qu'on fait sur # on ne va pas non plus compter les actions qu'on fait sur
# ses propres machines. # ses propres machines.
if machine.proprietaire().compte() != champ[2]: proprio = machine.proprietaire()
if isinstance(proprio, Adherent) and proprio.compte() != champ[2]:
# Il se peut qu'une personne sans les droits câbleur ait # Il se peut qu'une personne sans les droits câbleur ait
# effectué une action. # effectué une action.
try: try:
@ -350,7 +350,7 @@ if __name__ == "__main__":
# options pour décider le nombre de câbleurs à afficher # options pour décider le nombre de câbleurs à afficher
group = OptionGroup(parser, "Options de filtrage", group = OptionGroup(parser, "Options de filtrage",
"Option pour filtrer les câbleurs à afficher") u"Option pour filtrer les câbleurs à afficher")
group.add_option('-a', '--all', help= u"Affiche tous les câbleurs", group.add_option('-a', '--all', help= u"Affiche tous les câbleurs",
action='store_const', const=0, dest='top') action='store_const', const=0, dest='top')
group.add_option('-t', '--top', metavar= "NB", type= 'int', dest='top', group.add_option('-t', '--top', metavar= "NB", type= 'int', dest='top',
@ -358,8 +358,8 @@ if __name__ == "__main__":
parser.add_option_group(group) parser.add_option_group(group)
# options pour le format d'affichage des câbleurs # options pour le format d'affichage des câbleurs
group = OptionGroup(parser, "Format d'affichage", group = OptionGroup(parser, u"Format d'affichage",
"Pour définir le format d'affichage des câbleurs") u"Pour définir le format d'affichage des câbleurs")
group.add_option('-d', '--droits', help= u"Affiche les droits du câbleur", group.add_option('-d', '--droits', help= u"Affiche les droits du câbleur",
action='callback', callback= update_fqn, action='callback', callback= update_fqn,
callback_kwargs= { 'fmt': ' (%(droits)s)'}) callback_kwargs= { 'fmt': ' (%(droits)s)'})
@ -378,8 +378,8 @@ if __name__ == "__main__":
parser.add_option_group(group) parser.add_option_group(group)
# options de verbosité # options de verbosité
group = OptionGroup(parser, "Commander la verbosité", group = OptionGroup(parser, u"Commander la verbosité",
"Pour définir ce que le script doit afficher en plus") u"Pour définir ce que le script doit afficher en plus")
group.add_option('-D', '--debug', group.add_option('-D', '--debug',
help= u"Affiche des informations de debuggage", help= u"Affiche des informations de debuggage",
action='store_const', const='d', dest= 'debug') action='store_const', const='d', dest= 'debug')
@ -390,8 +390,8 @@ if __name__ == "__main__":
parser.add_option_group(group) parser.add_option_group(group)
# options sur la durée étudiée # options sur la durée étudiée
group = OptionGroup(parser, "Durée étudiée", group = OptionGroup(parser, u"Durée étudiée",
"Pour définir la durée sur laquelle on travaille") u"Pour définir la durée sur laquelle on travaille")
group.add_option('-f', '--for', metavar= 'N', group.add_option('-f', '--for', metavar= 'N',
help= u"Affiche les statistiqes depuis N mois", help= u"Affiche les statistiqes depuis N mois",
type= 'int', dest='nbmois') type= 'int', dest='nbmois')