PDF Writer : Add config for defining the default font

docs/changes/1.x/1.2.0.md

@@ -10,6 +10,7 @@
 - Word2007 Reader : Added option to disable loading images by [@aelliott1485](https://github.com/aelliott1485) in GH-2450
 - HTML Writer : Added border-spacing to default styles for table by [@kernusr](https://github.com/kernusr) in GH-2451
 - Word2007 Reader : Support for table cell borders and margins by [@kernusr](https://github.com/kernusr) in GH-2454
+- PDF Writer : Add config for defining the default font by [@MikeMaldini](https://github.com/MikeMaldini) in [#2262](https://github.com/PHPOffice/PHPWord/pull/2262) & [#2468](https://github.com/PHPOffice/PHPWord/pull/2468)
 
 ### Bug fixes
 

docs/usage/writers.md

@@ -30,6 +30,24 @@ $writer = IOFactory::createWriter($oPhpWord, 'PDF');
 $writer->save(__DIR__ . '/sample.pdf');
 ```
 
+### Options
+
+You can define options like :
+* `font`: default font
+
+Options must be defined before creating the writer.
+
+``` php
+use PhpOffice\PhpWord\Settings;
+
+Settings::setPdfRendererOptions([
+    'font' => 'Arial'
+]);
+
+$writer = IOFactory::createWriter($oPhpWord, 'PDF');
+$writer->save(__DIR__ . '/sample.pdf');
+```
+
 ## RTF
 The name of the writer is `RTF`.
 

src/PhpWord/IOFactory.php

@@ -28,19 +28,18 @@ abstract class IOFactory
      * Create new writer.
      *
      * @param string $name
-     * @param array $config
      *
      * @return WriterInterface
      */
-    public static function createWriter(PhpWord $phpWord, $name = 'Word2007', $config = array())
+    public static function createWriter(PhpWord $phpWord, $name = 'Word2007')
     {
         if ($name !== 'WriterInterface' && !in_array($name, ['ODText', 'RTF', 'Word2007', 'HTML', 'PDF'], true)) {
             throw new Exception("\"{$name}\" is not a valid writer.");
         }
 
         $fqName = "PhpOffice\\PhpWord\\Writer\\{$name}";
 
-        return ($name == 'PDF' && !empty($config)) ? new $fqName($phpWord, $config) : new $fqName($phpWord);
+        return new $fqName($phpWord);
     }
 
     /**

src/PhpWord/Settings.php

@@ -89,6 +89,13 @@ class Settings
      */
     private static $pdfRendererName;
 
+    /**
+     * Options used for rendering PDF files.
+     *
+     * @var array
+     */
+    private static $pdfRendererOptions = [];
+
     /**
      * Directory Path to the external Library used for rendering PDF files.
      *
@@ -226,6 +233,22 @@ public static function getPdfRendererPath(): ?string
         return self::$pdfRendererPath;
     }
 
+    /**
+     * Set options of the external library for rendering PDF files.
+     */
+    public static function setPdfRendererOptions(array $options): void
+    {
+        self::$pdfRendererOptions = $options;
+    }
+
+    /**
+     * Return the PDF Rendering Options.
+     */
+    public static function getPdfRendererOptions(): array
+    {
+        return self::$pdfRendererOptions;
+    }
+
     /**
      * Location of external library to use for rendering PDF files.
      *

src/PhpWord/Writer/PDF/AbstractRenderer.php

@@ -77,13 +77,11 @@ abstract class AbstractRenderer extends HTML
      * Create new instance.
      *
      * @param PhpWord $phpWord PhpWord object
-     * @param array $config
-     *
-     * @throws \PhpOffice\PhpWord\Exception\Exception
      */
-    public function __construct(PhpWord $phpWord, $config = array())
+    public function __construct(PhpWord $phpWord)
     {
         parent::__construct($phpWord);
+
         if ($this->includeFile != null) {
             $includeFile = Settings::getPdfRendererPath() . '/' . $this->includeFile;
             if (file_exists($includeFile)) {
@@ -96,7 +94,12 @@ public function __construct(PhpWord $phpWord, $config = array())
                 // @codeCoverageIgnoreEnd
             }
         }
-        if(isset($config['font']) && empty($config['font'])) $this->font = $config['font'];
+
+        // Configuration
+        $options = Settings::getPdfRendererOptions();
+        if (!empty($options['font'])) {
+            $this->setFont($options['font']);
+        }
     }
 
     /**

src/PhpWord/Writer/PDF/DomPDF.php

@@ -43,15 +43,12 @@ class DomPDF extends AbstractRenderer implements WriterInterface
      */
     protected function createExternalWriterInstance()
     {
-        if(!empty($this->font)){
-            $options = new Options();
-            $options->set('defaultFont', $this->font);
-            $instance = new DompdfLib($options);
-        }else{
-            $instance = new DompdfLib();
+        $options = new Options();
+        if ($this->getFont()) {
+            $options->set('defaultFont', $this->getFont());
         }
 
-        return $instance;
+        return new DompdfLib($options);
     }
 
     /**

src/PhpWord/Writer/PDF/MPDF.php

@@ -52,7 +52,12 @@ protected function createExternalWriterInstance()
     {
         $mPdfClass = $this->getMPdfClassName();
 
-        return new $mPdfClass();
+        $options = [];
+        if ($this->getFont()) {
+            $options['default_font'] = $this->getFont();
+        }
+
+        return new $mPdfClass($options);
     }
 
     /**

src/PhpWord/Writer/PDF/TCPDF.php

@@ -46,7 +46,13 @@ class TCPDF extends AbstractRenderer implements WriterInterface
      */
     protected function createExternalWriterInstance($orientation, $unit, $paperSize)
     {
-        return new \TCPDF($orientation, $unit, $paperSize);
+        $instance = new \TCPDF($orientation, $unit, $paperSize);
+
+        if ($this->getFont()) {
+            $instance->setFont($this->getFont(), $instance->getFontStyle(), $instance->getFontSizePt());
+        }
+
+        return $instance;
     }
 
     /**

tests/PhpWordTests/SettingsTest.php

@@ -41,6 +41,8 @@ class SettingsTest extends TestCase
 
     private $pdfRendererName;
 
+    private $pdfRendererOptions;
+
     private $pdfRendererPath;
 
     private $tempDir;
@@ -56,6 +58,7 @@ protected function setUp(): void
         $this->measurementUnit = Settings::getMeasurementUnit();
         $this->outputEscapingEnabled = Settings::isOutputEscapingEnabled();
         $this->pdfRendererName = Settings::getPdfRendererName();
+        $this->pdfRendererOptions = Settings::getPdfRendererOptions();
         $this->pdfRendererPath = Settings::getPdfRendererPath();
         $this->tempDir = Settings::getTempDir();
         $this->zipClass = Settings::getZipClass();
@@ -70,6 +73,7 @@ protected function tearDown(): void
         Settings::setMeasurementUnit($this->measurementUnit);
         Settings::setOutputEscapingEnabled($this->outputEscapingEnabled);
         Settings::setPdfRendererName($this->pdfRendererName);
+        Settings::setPdfRendererOptions($this->pdfRendererOptions);
         Settings::setPdfRendererPath($this->pdfRendererPath);
         Settings::setTempDir($this->tempDir);
         Settings::setZipClass($this->zipClass);
@@ -124,6 +128,22 @@ public function testSetGetPdfRenderer(): void
         self::assertEquals($domPdfPath, Settings::getPdfRendererPath());
     }
 
+    /**
+     * Test set/get PDF renderer.
+     */
+    public function testSetGetPdfOptions(): void
+    {
+        $domPdfPath = realpath(PHPWORD_TESTS_BASE_DIR . '/../vendor/dompdf/dompdf');
+
+        self::assertEquals([], Settings::getPdfRendererOptions());
+        self::assertNull(Settings::setPdfRendererOptions([
+            'font' => 'Arial',
+        ]));
+        self::assertEquals([
+            'font' => 'Arial',
+        ], Settings::getPdfRendererOptions());
+    }
+
     /**
      * Test set/get measurement unit.
      */

tests/PhpWordTests/Writer/PDF/DomPDFTest.php

@@ -75,4 +75,21 @@ public function testSetGetAbstractRendererProperties(): void
         $writer->setTempDir(Settings::getTempDir());
         self::assertEquals(Settings::getTempDir(), $writer->getTempDir());
     }
+
+    /**
+     * Test set/get abstract renderer options.
+     */
+    public function testSetGetAbstractRendererOptions(): void
+    {
+        define('DOMPDF_ENABLE_AUTOLOAD', false);
+
+        $rendererName = Settings::PDF_RENDERER_DOMPDF;
+        $rendererLibraryPath = realpath(PHPWORD_TESTS_BASE_DIR . '/../vendor/dompdf/dompdf');
+        Settings::setPdfRenderer($rendererName, $rendererLibraryPath);
+        Settings::setPdfRendererOptions([
+            'font' => 'Arial'
+        ]);
+        $writer = new PDF(new PhpWord());
+        self::assertEquals('Arial', $writer->getFont());
+    }
 }

tests/PhpWordTests/Writer/PDF/MPDFTest.php

@@ -50,4 +50,19 @@ public function testConstruct(): void
 
         unlink($file);
     }
+
+    /**
+     * Test set/get abstract renderer options.
+     */
+    public function testSetGetAbstractRendererOptions(): void
+    {
+        $rendererName = Settings::PDF_RENDERER_MPDF;
+        $rendererLibraryPath = realpath(PHPWORD_TESTS_BASE_DIR . '/../vendor/mpdf/mpdf');
+        Settings::setPdfRenderer($rendererName, $rendererLibraryPath);
+        Settings::setPdfRendererOptions([
+            'font' => 'Arial'
+        ]);
+        $writer = new PDF(new PhpWord());
+        self::assertEquals('Arial', $writer->getFont());
+    }
 }

tests/PhpWordTests/Writer/PDF/TCPDFTest.php

@@ -49,4 +49,19 @@ public function testConstruct(): void
 
         unlink($file);
     }
+
+    /**
+     * Test set/get abstract renderer options.
+     */
+    public function testSetGetAbstractRendererOptions(): void
+    {
+        $rendererName = Settings::PDF_RENDERER_TCPDF;
+        $rendererLibraryPath = realpath(PHPWORD_TESTS_BASE_DIR . '/../vendor/tecnickcom/tcpdf');
+        Settings::setPdfRenderer($rendererName, $rendererLibraryPath);
+        Settings::setPdfRendererOptions([
+            'font' => 'Arial'
+        ]);
+        $writer = new PDF(new PhpWord());
+        self::assertEquals('Arial', $writer->getFont());
+    }
 }