Project

General

Profile

Bug #45565

export pdf : variable locale 'temp_dir' référencée avant assignation

Added by Paul Marillonnet 7 days ago. Updated 2 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
29 Jul 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Vu dans #45562

0001-export_to_pdf-make-temp-variable-known-regardless-of.patch View (1.11 KB) Paul Marillonnet, 29 Jul 2020 03:30 PM

0001-wf-explicitly-crash-when-temporary-export-files-cann.patch View (907 Bytes) Paul Marillonnet, 30 Jul 2020 11:13 AM

Associated revisions

Revision 6f763c40 (diff)
Added by Paul Marillonnet 5 days ago

wf: explicitly crash when temporary export files cannot be created (#45565)

History

#2 Updated by Paul Marillonnet 7 days ago

#3 Updated by Paul Marillonnet 7 days ago

  • Subject changed from export pdf : variable locale 'temp_file' référencée avant assignation to export pdf : variable locale 'temp_dir' référencée avant assignation

#4 Updated by Paul Marillonnet 7 days ago

  • Status changed from Solution proposée to 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.

#5 Updated by Thomas Noël 6 days ago

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)

#6 Updated by Paul Marillonnet 6 days ago

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.

#7 Updated by Thomas Noël 6 days ago

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

#8 Updated by Frédéric Péters 5 days ago

  • Status changed from Solution validée to 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)

#9 Updated by Frédéric Péters 2 days ago

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

Also available in: Atom PDF