Minor code tweaks

Author: javiereguiluz

controller/upload_file.rst

@@ -127,8 +127,8 @@ Finally, you need to update the code of the controller that handles the form::
                 $brochureFile = $form['brochure']->getData();
                 $originalFilename = pathinfo($brochureFile->getClientOriginalName(), PATHINFO_FILENAME);
                 // this is needed to safely include the file name as part of the URL
-                $originalFilename = transliterator_transliterate('Any-Latin; Latin-ASCII; Lower()', $originalFilename);
-                $newFilename = $originalFilename.'-'.uniqid().'.'.$brochureFile->guessExtension();
+                $safeFilename = transliterator_transliterate('Any-Latin; Latin-ASCII; [^A-Za-z0-9_] remove; Lower()', $originalFilename);
+                $newFilename = $safeFilename.'-'.uniqid().'.'.$brochureFile->guessExtension();
 
                 // Move the file to the directory where brochures are stored
                 try {
@@ -227,8 +227,8 @@ logic to a separate service::
         public function upload(UploadedFile $file)
         {
             $originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
-            $originalFilename = transliterator_transliterate('Any-Latin; Latin-ASCII; Lower()', $originalFilename);
-            $fileName = $originalFilename.'-'.uniqid().'.'.$file->guessExtension();
+            $safeFilename = transliterator_transliterate('Any-Latin; Latin-ASCII; [^A-Za-z0-9_] remove; Lower()', $originalFilename);
+            $fileName = $safeFilename.'-'.uniqid().'.'.$file->guessExtension();
 
             try {
                 $file->move($this->getTargetDirectory(), $fileName);