Project

General

Profile

Anomalie #16921

Erreur de conversion fichier au Format .PDF (Casse de l'extension)

Added by Antoine HAGUES about 1 year ago. Updated 11 months ago.

Status:
Développé / Analysé (S)
Priority:
2-Sérieux
Assignee:
Damien BUREL
Target version:
Start date:
04/22/2021
Due date:
Tags Courrier:
20.03.20, 20.10.12, 21.03.4

Description

Changement dans le code dans le fichier -> /var/www/html/MaarchCourrier/src/app/convert/controllers/ConvertPdfController.php ligne 164

Remplacer la ligne

~~~ php
$resource = file_get_contents("{$tmpPath}{$fileNameOnTmp}.pdf");
~~~

par
~~~ php
if (is_file("{$tmpPath}{$fileNameOnTmp}.PDF")) {
$resource = file_get_contents("{$tmpPath}{$fileNameOnTmp}.PDF");
} else {
$resource = file_get_contents("{$tmpPath}{$fileNameOnTmp}.pdf");
}


  

History

#1 Updated by Support Maarch about 1 year ago

  • Status changed from A qualifier to A traiter

#2 Updated by Emmanuel DILLARD about 1 year ago

  • Status changed from A traiter to Complément d'Informations
  • Assignee changed from Maarch Courrier DEV TEAM to Antoine HAGUES

Indiquer la version du code.

Pb de casse du .pdf ?

#3 Updated by Antoine HAGUES about 1 year ago

  • Assignee changed from Antoine HAGUES to EDI PO

Version du code 21.03 mais vu aussi en 20.03 et 20.10

Oui c'est ca, il ne convertis pas les fichiers au format .PDF

Voir avec Guillaume problème constaté avec lui.

#4 Updated by Emmanuel DILLARD about 1 year ago

  • Subject changed from Erreur de conversion fichier au Format .PDF to Erreur de conversion fichier au Format .PDF (Casse de l'extension)
  • Status changed from Complément d'Informations to A étudier
  • Assignee changed from EDI PO to Florian AZIZIAN
  • Target version changed from Inscription Backlog Courrier to 21.03 (stable)

#5 Updated by Florian AZIZIAN about 1 year ago

  • Status changed from A étudier to Prêt à développer
  • Assignee changed from Florian AZIZIAN to EDI PO

A la ligne 152 de src/app/convert/controllers/ConvertPdfController.php, il faudrait plutôt mettre ça :

copy($pathToDocument, $tmpPath.$fileNameOnTmp.'.'.strtolower($docInfo["extension"]));

Aussi, lorsqu'on enregistre le document, ce serait bien de mettre l'extension en minuscule, dans les docservers et dans la colonne "format" de res_letterbox.
Cela éviterait d'avoir d'autres cas à d'autres endroits.

#6 Updated by Florian AZIZIAN about 1 year ago

  • Project changed from Backlog to CURRENT SPRINT
  • Status changed from Prêt à développer to En cours de dev (S)
  • Assignee deleted (EDI PO)
  • Target version changed from 21.03 (stable) to Courrier Develop
  • Tags Courrier 20.03.20, 20.10.12, 21.03.4 added

#7 Updated by Damien BUREL about 1 year ago

  • Assignee set to Damien BUREL

#8 Updated by Damien BUREL about 1 year ago

Bug non reproduit que ça soit en develop, 21.03 ou 20.10. Guillaume ne reproduit pas non plus. Vu le code, je ne vois pas comment le problème peut se poser.
Je modifie juste l'enregistrement du format en minuscule (colonne format) mais cela n'a aucune incidence sur ce problème.

Effectivement si le courrier provient de Maarch AutoImport, le fichier peut avoir un format en majuscule. On peut donc reproduire ce problème comme ça.
Correctif effectué en develop, 21.03, 20.10 et 20.03.

#9 Updated by Damien BUREL about 1 year ago

  • Assignee changed from Damien BUREL to Antoine HAGUES

#10 Updated by Damien BUREL about 1 year ago

  • Status changed from En cours de dev (S) to Complément d'Informations

#11 Updated by Damien BUREL about 1 year ago

  • Assignee changed from Antoine HAGUES to Damien BUREL

#12 Updated by Damien BUREL about 1 year ago

  • Status changed from Complément d'Informations to En cours

#13 Updated by Florian AZIZIAN about 1 year ago

  • Status changed from En cours to En cours de dev (S)

#15 Updated by Damien BUREL about 1 year ago

  • Status changed from En cours de dev (S) to Développé / Analysé (S)

#17 Updated by Emmanuel DILLARD about 1 year ago

  • Project changed from CURRENT SPRINT to Backlog

#18 Updated by Emmanuel DILLARD 11 months ago

  • Project changed from Backlog to Backlog Courrier
  • Target version changed from Courrier Develop to Develop

Also available in: Atom PDF