make valid filenames

Application/Model/AbstractClasses/pdfdocumentsGeneric.php

@@ -199,10 +199,18 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
     {
         // forced filename from setFilename()
         if ($this->filename) {
-            return $this->addFilenameExtension($this->filename);
+            return $this->addFilenameExtension(
+                $this->makeValidFileName(
+                    $this->filename
+                )
+            );
         }
 
-        return $this->addFilenameExtension($this->getTypeForFilename());
+        return $this->addFilenameExtension(
+            $this->makeValidFileName(
+                $this->getTypeForFilename()
+            )
+        );
     }
 
     /**

Application/Model/AbstractClasses/pdfdocumentsOrder.php

@@ -69,19 +69,25 @@ abstract class pdfdocumentsOrder extends pdfdocumentsGeneric implements orderInt
     {
         // forced filename from setFilename()
         if ($this->filename) {
-            return $this->addFilenameExtension($this->filename);
+            return $this->addFilenameExtension(
+                $this->makeValidFileName(
+                    $this->filename
+                )
+            );
         }
 
         $sTrimmedBillName = trim($this->getOrder()->getFieldData('oxbilllname'));
 
         return $this->addFilenameExtension(
-            implode(
+            $this->makeValidFileName(
-                '_',
+                implode(
-                [
+                    '_',
-                    $this->getTypeForFilename(),
+                    [
-                    $this->getOrder()->getFieldData('oxordernr'),
+                        $this->getTypeForFilename(),
-                    $sTrimmedBillName
+                        $this->getOrder()->getFieldData('oxordernr'),
-                ]
+                        $sTrimmedBillName
+                    ]
+                )
             )
         );
     }

Application/Model/Documents/invoicePdf.php

@@ -111,10 +111,12 @@ class invoicePdf extends pdfdocumentsOrder implements pdfdocumentsOrderinvoiceIn
     {
         $filename = parent::getFilename();
 
-        return str_replace(
+        $filename = str_replace(
             $this->getOrder()->getFieldData('oxordernr'),
             $this->getOrder()->getFieldData('oxbillnr'),
             $filename
         );
+
+        return $this->makeValidFileName($filename);
     }
 }