Corrige l'upload des documents : conserve l'extension de fichier (#132)

CHANGELOG.md

@@ -14,6 +14,7 @@ Ce projet adhère au principe du [Semantic Versioning](https://semver.org/spec/v
 - Améliore les données affichées dans les PDF (fiches de sortie, devis et factures), en y ajoutant la référence
   (numéro client) du bénéficiaire (voir #122) et les numéros de téléphone des techniciens et des bénéficiaires.
 - Corrige l'affichage des numéros légaux de l'entreprise (SIRET, etc.) sur les devis et factures.
+- Corrige le nom des fichiers uploadés comme documents du matériel pour qu'il conserve son extension (#132).
 
 ## 0.12.0 (2021-03-29)
 

server/src/App/Config/functions.php

@@ -254,7 +254,10 @@ function moveUploadedFile($directory, UploadedFileInterface $uploadedFile)
 {
     $name = $uploadedFile->getClientFilename();
 
-    $slugify = new Cocur\Slugify\Slugify(['lowercase' => false]);
+    $slugify = new Cocur\Slugify\Slugify([
+        'regexp' => '/([^A-Za-z0-9\.]|-)+/',
+        'lowercase' => false,
+    ]);
     $nameSecure = $slugify->slugify($name);
 
     if (!is_dir($directory)) {

server/tests/other/FunctionsTest.php

@@ -115,10 +115,10 @@ public function testMoveUploadedFile(): void
         $destinationFolder = DATA_FOLDER . DS . 'materials' . DS . 'tests';
 
         // - Déplace le fichier de test d'upload dans
-        $file = new UploadedFile($sourceFile, 'Uploaded File for Tests', 'application/pdf', 13269);
+        $file = new UploadedFile($sourceFile, 'Uploaded File for Tests.pdf', 'application/pdf', 13269);
         // - Déplace le fichier de test d'upload dans le dossier d'un matériel
         $filename = moveUploadedFile($destinationFolder, $file);
-        $this->assertEquals('Uploaded-File-for-Tests', $filename);
+        $this->assertEquals('Uploaded-File-for-Tests.pdf', $filename);
         $destinationFile = $destinationFolder . DS . $filename;
         $this->assertTrue(file_exists($destinationFile));
         // - Remet le fichier dans son dossier d'origine