Projet

Général

Profil

Bug #19550

agoraplus: erreur lors de la récuperation des reservations periscolaires d'un enfant

Ajouté par Serghei Mihai il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
-
Début:
19 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:


Fichiers

Révisions associées

Révision ea1959ae (diff)
Ajouté par Serghei Mihai il y a plus de 6 ans

agoraplus: fix child's family membreship (#19550)

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 6 ans

pfff

#2

Mis à jour par Frédéric Péters il y a plus de 6 ans

pfff.

L'occasion de jeter un œil à https://jenkins.entrouvert.org/job/passerelle/violations/file/passerelle/contrib/agoraplus/models.py/ et voir ce que pylint avait détecté.

Noter aussi la ligne 957, arrivée par le même patch :

+    def is_child_in_name_id_family(self, child_id, name_id):
+        family = self.get_family(name_id=name_id)
+        for link in AgoraPlusLink.filter(name_id=name_id):
+            family = get_agoraplus_family(link.login)
+            for child in family['children']:

get_agoraplus_family pas déclaré.

Noter aussi la ligne 75, d'un autre commit mais erreur aussi (not enough arguments for format string)  :

                'failure %s: we expected %s and got: %r' % (expected_type.__name__, response))

Oui, pfff.

#3

Mis à jour par Serghei Mihai il y a plus de 6 ans

Déjà, correction pour is_child_in_name_id_family.
Pour les arguments de l'exception cas dans #19559.

#4

Mis à jour par Thomas Noël il y a plus de 6 ans

Ack

#5

Mis à jour par Serghei Mihai il y a plus de 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit ea1959ae5c67a22ed2b3b09637e733da10792c24 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Oct 19 12:30:23 2017 +0200

    agoraplus: fix child's family membreship (#19550)
#6

Mis à jour par Serghei Mihai il y a plus de 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF