Merge pull request #783 from tentwofour/feature_slugified_filenames

[MediaBundle] Proposed fix/feature for web safe filenames
Kunstmaan · Oct 26, 2015 · fe159df · fe159df
2 parents b0fe094 + d0cbf39
commit fe159df
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/Kunstmaan/MediaBundle/Helper/File/FileHandler.php b/src/Kunstmaan/MediaBundle/Helper/File/FileHandler.php
@@ -11,6 +11,7 @@
 use Symfony\Component\HttpFoundation\File\File;
 use Symfony\Component\HttpFoundation\File\MimeType\MimeTypeGuesserInterface;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
+use Kunstmaan\UtilitiesBundle\Helper\Slugifier;
 
 /**
  * FileHandler
@@ -213,13 +214,18 @@ public function getOriginalFile(Media $media)
      */
     private function getFilePath(Media $media)
     {
-        $filename = $media->getOriginalFilename();
-        $filename = str_replace(array('/', '\\'), '', $filename);
+        $filename  = $media->getOriginalFilename();
+        $filename  = str_replace(array('/', '\\', '%'), '', $filename);
+        $slugifier = new Slugifier();
 
         if (!empty($this->blacklistedExtensions)) {
             $filename = preg_replace('/\.('.join('|', $this->blacklistedExtensions).')$/', '.txt', $filename);
         }
 
+        $parts    = pathinfo($filename);
+        $filename = $slugifier->slugify($parts['filename']);
+        $filename .= '.'.$parts['extension'];
+
         return sprintf(
             '%s/%s',
             $media->getUuid(),