Projet

Général

Profil

Bug #74184

toulouse-maelis : code si données de référentiel absentes

Ajouté par Frédéric Péters il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Sur le node2.test alerte comme quoi du code local a été modifié,

--- /usr/lib/python3/dist-packages/passerelle/contrib/toulouse_maelis/models.py    2023-02-02 14:35:19.000000000 +0100
+++ /tmp/m    2023-02-04 10:14:20.629634793 +0100
@@ -89,7 +89,7 @@

     def update_referential(self, referential_name, data, id_key, text_key):
         last_update = now()
-        for item in data:
+        for item in data or []:
             text = item[text_key] or ''
             if isinstance(text, int):
                 text = str(text)

Je n'ai pas vu de mention à ce sujet dans redmine.

Révisions associées

Révision 2c6168cc (diff)
Ajouté par Nicolas Roche il y a environ un an

toulouse-maelis: accept empty referential (#74184)

Historique

#1

Mis à jour par Nicolas Roche il y a environ un an

  • Assigné à mis à Nicolas Roche

C'est moi, j'ai averti sur le salon, mais un peu trop tard pour qu'on me donne la marche à suivre.

pour signaler que je vais toucher au code de maelis sur la recette : juste ajoute "or []" pour que le connecteur puisse attraper les référentiels suite à la mise à jour de Sigec ce vendredi soir.

On a vérifié avec Stéphane que l'import de la base de Toulouse par Sigec n'a rien cassé, parce qu'il y aura une la formation mardi.

#2

Mis à jour par Frédéric Péters il y a environ un an

C'est moi, j'ai averti sur le salon, mais un peu trop tard pour qu'on me donne la marche à suivre.

???

quand il y a un bug on fait un ticket, on propose la correction, elle est relue et intégrée, etc.

#3

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée

Nicolas Roche (nroche) a ouvert une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution proposée à Solution validée

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Nicolas Roche (nroche) a mergé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF