Project

General

Profile

Bug #99414

combo-plugin-gnm: reporting, corriger la portée des objets zips

Added by Nicolas Roche 4 months ago. Updated 27 days ago.

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

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

On veut faire marcher le reporting toodego (développement spécifique #35844),
sur l'instance Sathonay (#99320).

En recette (et en local) j'ai cette trace :

$ sudo -u combo /usr/bin/combo-manage tenant_command gnm_reporting -d portail-agent-sathonaycamp.guichet-recette.grandlyon.com
  File "/usr/lib/python3/dist-packages/combo_plugin_gnm/management/commands/gnm_reporting.py", line 69, in handle
    new_ods.writestr(f, zipf.read(f))
                        ^^^^^^^^^^^^
ValueError: Attempt to use ZIP archive that was already closed

Qui pour moi signifie que les objets zipfile semblent à présent être correctement fermés,
quand on veut les accéder en dehors de la porté du with qui les déclarent.
(le code est rangé là : https://git.entrouvert.org/entrouvert/combo-plugin-gnm/src/branch/main/combo_plugin_gnm/management/commands/gnm_reporting.py)

History

#1

Updated by Robot Gitea 4 months ago

  • Status changed from Nouveau to Solution proposée

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

#2

Updated by Robot Gitea 4 months ago

  • Status changed from Solution proposée to En cours

Emmanuel Cazenave (ecazenave) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

#3

Updated by Robot Gitea 4 months ago

  • Status changed from En cours to Solution proposée

Nicolas Roche (nroche) a demandé une relecture de Emmanuel Cazenave (ecazenave) sur une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 4 months ago

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

Emmanuel Cazenave (ecazenave) a approuvé une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea 4 months 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 Frédéric Péters 27 days ago

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

Also available in: Atom PDF