Project

General

Profile

Bug #74184

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

Added by Frédéric Péters about 1 month ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
04 February 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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.

Associated revisions

Revision 2c6168cc (diff)
Added by Nicolas Roche about 1 month ago

toulouse-maelis: accept empty referential (#74184)

History

#1

Updated by Nicolas Roche about 1 month ago

  • Assignee set to 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

Updated by Frédéric Péters about 1 month ago

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

Updated by Gitea (Bot) Gitea about 1 month ago

  • Status changed from Nouveau to Solution proposée

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

#4

Updated by Gitea (Bot) Gitea about 1 month ago

  • Status changed from Solution proposée to Solution validée

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

#5

Updated by Gitea (Bot) Gitea about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)

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

#6

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF