From f0de8bd515b30514d402673c1bd4a2a8c94b02e6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pierre-Elliott=20B=C3=A9cue?= Date: Fri, 17 Jul 2015 09:52:05 -0600 Subject: [PATCH] =?UTF-8?q?Supprime=20ce=20qui=20concerne=20la=20carte=20d?= =?UTF-8?q?'=C3=A9tudiant?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- objets.py | 36 +++++++-------------------------- printing/templates/adherent | 2 -- printing/templates/templates.py | 28 +++++++------------------ services.py | 2 +- 4 files changed, 15 insertions(+), 53 deletions(-) diff --git a/objets.py b/objets.py index a64ce15..3494958 100644 --- a/objets.py +++ b/objets.py @@ -782,9 +782,6 @@ class CransLdapObject(object): blacklist_liste=[] # blacklistes virtuelle si on est un adhérent pour carte étudiant et chambre invalides if isinstance(self, adherent): - if not self.carte_ok(): - bl = attributs.blacklist(u'%s$%s$%s$%s' % ('-', '-', 'carte_etudiant', ''), {}, self.conn) - blacklist_liste.append(bl) if self['chbre'][0] == '????': bl = attributs.blacklist(u'%s$%s$%s$%s' % ('-', '-', 'chambre_invalide', ''), {}, self.conn) blacklist_liste.append(bl) @@ -1006,17 +1003,9 @@ class proprio(CransLdapObject): return super(proprio, self).may_be(what, liste) - - def sursis_carte(self): - for h in self['historique'][::-1]: - x = re.match("(.*),.* : .*(paiement\+%s|inscription).*" % (config.ann_scol,), h.value) - if x != None: - return ((time.time()-time.mktime(time.strptime(x.group(1),'%d/%m/%Y %H:%M'))) <= config.sursis_carte) - return False - def access_ok(self): u"""Renvoie si le propriétaire a payé et donné sa carte pour l'année en cours""" - return self.paiement_ok() and self.carte_ok() + return self.paiement_ok() def fin_adhesion(self): """Retourne la date de fin d'adhésion""" @@ -1037,31 +1026,20 @@ class proprio(CransLdapObject): for bl in self.blacklist_actif(): if bl['type'] == 'paiement': return False - old_style_paiement = config.ann_scol in self['paiement'] or (config.periode_transitoire and (config.ann_scol - 1) in self['paiement']) if isinstance(self, adherent): fin_paiement = min(self.fin_adhesion(), self.fin_connexion()) else: fin_paiement = self.fin_adhesion() - new_style_paiement = time.time() < fin_paiement or (config.periode_transitoire and config.debut_periode_transitoire <= fin_paiement <= config.fin_periode_transitoire) - return (old_style_paiement or new_style_paiement) + paiement = time.time() < fin_paiement or (config.periode_transitoire and config.debut_periode_transitoire <= fin_paiement <= config.fin_periode_transitoire) + return paiement def carte_ok(self): - u"""Renvoie si le propriétaire a donné sa carte pour l'année en cours, en prenant en compte les periode transitoires et le sursis carte""" - if self.dn == variables.base_dn: - return True - elif 'club' in self["objectClass"]: - return True - elif config.periode_transitoire or not config.bl_carte_et_actif: - return True - else: - return bool(self.get('carteEtudiant', [])) or self.sursis_carte() + """Dummy""" + return True def carte_controle(self): - u"""Renvoie si la carte a été controlé pour l'année en cours par le trésorier""" - if self["controle"]: - return "c" in str(self["controle"][0]) - else: - return False + """Dummy""" + return True # TODO: gérer cela en modifiant un attribut ? def solde(self, diff, comment=u"", login=None): diff --git a/printing/templates/adherent b/printing/templates/adherent index 2c2f90c..74c7e36 100644 --- a/printing/templates/adherent +++ b/printing/templates/adherent @@ -15,6 +15,4 @@ {{conn}} {% if o.get('controle', []) and 'p' in o.controle.0.value %}{{"(OK)"|coul('vert')}}{% endif %} -{{cetud}} {% if o.get('controle', []) and 'c' in o.controle.0.value %}{{"(OK)"|coul('vert')}}{% endif %} - {% endblock%} diff --git a/printing/templates/templates.py b/printing/templates/templates.py index 59143aa..404a381 100644 --- a/printing/templates/templates.py +++ b/printing/templates/templates.py @@ -227,12 +227,11 @@ def list_adherents(adherents, width=None): [a['aid'][0], u' '.join(unicode(i) for i in a['prenom'] + a['nom']), a['chbre'][0], style('o', 'vert') if a.paiement_ok() else style('n', 'rouge'), - style('o', 'vert') if a.carte_ok() else style('n', 'rouge'), u', '.join(unicode(m['host'][0]).split('.',1)[0] for m in a.machines()) ] for a in adherents ], - titre = [u'aid', u'Prénom Nom', u'Chbre', u'P', u'C', u'Machines'], - largeur = [5, 35, 5, 1, 1, '*'], - alignement = ['d', 'c', 'c', 'c', 'c', 'g'], + titre = [u'aid', u'Prénom Nom', u'Chbre', u'P', u'Machines'], + largeur = [5, 35, 5, 1, '*'], + alignement = ['d', 'c', 'c', 'c', 'g'], width=width) def list_clubs(clubs, width=None): @@ -250,16 +249,12 @@ def list_clubs(clubs, width=None): def proprio(proprio, params): params['o']=proprio etat_administratif=[] - if proprio.paiement_ok() and proprio.carte_ok(): + if proprio.paiement_ok(): etat_administratif.append(style(u"à jour", "vert")) - if not proprio.carte_ok(): - etat_administratif.append(style(u"manque carte d'étudiant", "violet")) if not proprio.paiement_ok(): etat_administratif.append(style(u"cotisation non réglée", "violet")) if proprio.fin_adhesion() >= time.time(): adh = style(u"Adhésion jusqu'au %s" % (time.strftime("%d/%m/%Y %H:%M:%S", time.localtime(proprio.fin_adhesion())),), "vert") - elif config.ann_scol in proprio['paiement']: - adh = style(u"Adhésion pour la période %s-%s ok." % (config.ann_scol, config.ann_scol+1), "vert") elif proprio.paiement_ok(): adh = style(u"Adhésion terminée, mais il y a un sursis.", 'orange') else: @@ -267,27 +262,18 @@ def proprio(proprio, params): params["adh"] = adh if proprio.fin_connexion() >= time.time(): conn = style(u"Connexion jusqu'au %s" % (time.strftime("%d/%m/%Y %H:%M:%S", time.localtime(proprio.fin_connexion())),), "vert") - elif config.ann_scol in proprio['paiement']: - conn = style(u"Connexion pour la période %s-%s ok." % (config.ann_scol, config.ann_scol+1), "vert") elif proprio.paiement_ok(): conn = style(u"Connexion terminée, mais il y a un sursis.", 'orange') else: conn = style(u"Pas connecté actuellement.", 'rouge') params["conn"] = conn - if proprio.get('carteEtudiant', []): - cetud = style(u"Carte d'étudiant fournie.", 'vert') - elif proprio.sursis_carte() or proprio.carte_ok(): - cetud = style(u"Carte d'étudiant non fournie, mais en sursis.", 'orange') - else: - cetud = style(u"Pas de carte d'étudiant") - params["cetud"] = cetud - params['etat_administratif']=etat_administratif + params['etat_administratif'] = etat_administratif try: if proprio.machines(): - params['machines']=list_machines(proprio.machines()) + params['machines'] = list_machines(proprio.machines()) if proprio.factures(): - params['factures']=list_factures(proprio.factures()) + params['factures'] = list_factures(proprio.factures()) # On essaye d'afficher un adhérent qui n'existe pas encore # et donc, on ne peut pas récupérer ses objets enfant except ldap.NO_SUCH_OBJECT: diff --git a/services.py b/services.py index 3b8097f..1627277 100644 --- a/services.py +++ b/services.py @@ -19,7 +19,7 @@ services_to_attrs['filtrage_machines'] = [ attributs.ipHostNumber, attributs.ip6 services_to_attrs['surveillance_machines'] = services_to_attrs['filtrage_machines'] services_to_attrs['filtrage_exemptions'] = [ attributs.exempt ] services_to_attrs['surveillance_exemptions'] = services_to_attrs['filtrage_exemptions'] -services_to_attrs['macip'] = [ attributs.ipHostNumber, attributs.ip6HostNumber, attributs.macAddress, attributs.paiement, attributs.carteEtudiant, attributs.finConnexion ] +services_to_attrs['macip'] = [ attributs.ipHostNumber, attributs.ip6HostNumber, attributs.macAddress, attributs.paiement, attributs.finConnexion ] services_to_attrs['dns'] = [ attributs.ipHostNumber, attributs.ip6HostNumber, attributs.sshFingerprint, attributs.host, attributs.hostAlias, attributs.dnsIpv6 , attributs.hostCert, attributs.portTCPin, attributs.portUDPin ] services_to_attrs['blacklist'] = [ attributs.blacklist, attributs.chbre, attributs.mailInvalide ] + services_to_attrs['macip'] services_to_attrs['ports'] = [ attributs.portUDPout, attributs.portUDPin, attributs.portTCPout, attributs.portTCPin ]