Merge pull request #12 from tjodalv/configure-driver

Added ability to configure Browsershot driver instance

README.md

@@ -36,7 +36,15 @@ Currently two drivers are supported:
 
 ### Browsershot Driver
 
-This is the  driver and requires [spatie/browsershot](https://github.com/spatie/browsershot). Please follow the installation instructions for that package.
+This is the default driver and requires [spatie/browsershot](https://github.com/spatie/browsershot). Please follow the installation instructions for that package.
+
+You can configure the Browsershot driver via `BrowsershotDriver::configureUsing()` in your `AppServiceProvider`:
+
+```php
+BrowsershotDriver::configureUsing(
+    fn (Browsershot $browser) => $browser->setCustomTempPath(storage_path('tmp'))
+);
+```
 
 ### Wkhtmltopdf Driver
 

src/Drivers/BrowsershotDriver.php

@@ -2,21 +2,33 @@
 
 namespace pxlrbt\LaravelPdfable\Drivers;
 
+use Closure;
 use pxlrbt\LaravelPdfable\Pdfable;
 use Spatie\Browsershot\Browsershot;
 
 class BrowsershotDriver implements Driver
 {
+    protected static ?Closure $configureUsing = null;
+
+    public static function configureUsing(Closure $callback): void {
+        static::$configureUsing = $callback;
+    }
+
     public function getData(Pdfable $pdf): ?string
     {
         $html = $pdf->getView()->render();
         $page = $pdf->page();
 
+        $browser = Browsershot::html($html)
+            ->paperSize($page->getWidth(), $page->getHeight())
+            ->margins(...$page->getMargins());
+
+        if (self::$configureUsing !== null) {
+            $browser = call_user_func(static::$configureUsing, $browser);
+        }
+
         return base64_decode(
-            Browsershot::html($html)
-                ->paperSize($page->getWidth(), $page->getHeight())
-                ->margins(...$page->getMargins())
-                ->base64pdf()
+            $browser->base64pdf()
         );
     }
 }