Refactor logic for replacing variables

tests/Feature/PdfMaker/PdfMaker.php

@@ -2,8 +2,6 @@
 
 namespace Tests\Feature\PdfMaker;
 
-use DOMDocument;
-
 class PdfMaker
 {
     use PdfMakerUtilities;
@@ -42,9 +40,7 @@ class PdfMaker
             }
         }
 
-        foreach ($this->data['variables'] as $entry) {
+        $this->updateVariables($this->data['variables']);
-            $this->updateVariable($entry['id'], $entry['variable'], $entry['value']);
-        }
 
         return $this;
     }

tests/Feature/PdfMaker/PdfMakerTest.php

@@ -95,7 +95,7 @@ class PdfMakerTest extends TestCase
                 ],
             ],
             'variables' => [
-                ['id' => 'header', 'variable' => '$title', 'value' => 'Invoice Ninja'],
+                '$title' => 'Invoice Ninja',
             ],
         ];
 

tests/Feature/PdfMaker/PdfMakerUtilities.php

@@ -49,6 +49,15 @@ trait PdfMakerUtilities
         return $element;
     }
 
+    public function updateVariables(array $variables)
+    {
+        $html = strtr($this->getCompiledHTML(), $variables);
+
+        $this->document->loadHTML($html);
+
+        $this->document->saveHTML();
+    }
+
     public function updateVariable(string $element, string $variable, string $value)
     {
         $element = $this->document->getElementById($element);