Bug #45565
export pdf : variable locale 'temp_dir' référencée avant assignation
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
29 juillet 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Vu dans #45562
Fichiers
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Fichier 0001-export_to_pdf-make-temp-variable-known-regardless-of.patch 0001-export_to_pdf-make-temp-variable-known-regardless-of.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Sujet changé de export pdf : variable locale 'temp_file' référencée avant assignation à export pdf : variable locale 'temp_dir' référencée avant assignation
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
Et le mieux serait en plus d'attraper les éventuelles IOError lors de la tentative de création du fichier temporaire. Je vais regarder ça.
Mis à jour par Thomas Noël il y a plus de 3 ans
On pourrait juste sortir le « temp_dir = tempfile.mkdtemp() » du try, et voilà, ça crashera comme quand libreoffice crashe. (ça ne doit jamais arriver et on doit être au courant le plus tôt possible quand ça arrive)
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Fichier 0001-wf-explicitly-crash-when-temporary-export-files-cann.patch 0001-wf-explicitly-crash-when-temporary-export-files-cann.patch ajouté
- Statut changé de En cours à Solution proposée
En fait je m'attendais à ce que cette erreur de référencement apparaisse sous la forme
IOError: <blabla could not create temporary file blabla> During handling of the above exception, another exception occurred: UnboundLocalError: local variable 'temp_dir' referenced before assignment
mais dans #45562, l'IOError, si elle a bien eu lieu, n'apparaît pas.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6f763c4045dfcb9adf8511ea0d3a76a134a85dbb Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Wed Jul 29 15:17:57 2020 +0200 wf: explicitly crash when temporary export files cannot be created (#45565)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
wf: explicitly crash when temporary export files cannot be created (#45565)