Simple changes for phpstan.

Author: CraicOverflow89

src/AbstractPDF.php

@@ -462,23 +462,23 @@ public function getLandscape(): ?bool
      * Renders a string of HTML to a PDF
      *
      * @param  string $content Content to render
-     * @return resource
+     * @return resource|null
      */
     abstract public function renderContent(string $content);
 
     /**
     * Renders a URL to a PDF
     *
     * @param  string $url URL to render
-    * @return resource
+    * @return resource|null
     */
     abstract public function renderURL(string $url);
 
     /**
      * Renders a local file to a PDF
      *
      * @param  string $path Local file to render
-     * @return resource
+     * @return resource|null
      */
     abstract public function renderFile(string $path);
 }

src/Browserless.php

@@ -22,7 +22,7 @@ class Browserless extends AbstractPDF
     */
     private $pdfEndpoint = '/pdf';
     /**
-     * @var object
+     * @var \GuzzleHttp\Client
      */
     private $client;
     /**
@@ -40,7 +40,7 @@ class Browserless extends AbstractPDF
 
     /**
      * @param string $apiKey api key from browserless.io
-     * @param object $client custom Guzzle client
+     * @param \GuzzleHttp\Client $client custom Guzzle client
      */
     public function __construct(string $apiKey = null, $client = null)
     {
@@ -149,7 +149,7 @@ public function getRotation(): ?int
     /**
      * Gets the payload of JSON options to be sent to browserless, minus the `url` or `html` property
      *
-     * @return array
+     * @return array<string, mixed>
      */
     public function getFormattedOptions(): array
     {
@@ -227,7 +227,7 @@ public function getFormattedOptions(): array
     }
 
     /**
-     * @param  array  $options
+     * @param  array<string, mixed>  $options
      * @return resource
      */
     private function render(array $options)

src/Chrome.php

@@ -16,7 +16,7 @@ class Chrome extends AbstractPDF
      * an array of temporary file handles
      * they are unlinked after rendering
      *
-     * @var array
+     * @var array<string>
      */
     private $handles = [];
 
@@ -43,7 +43,7 @@ public function __construct($binary = 'chrome-pdf', $processClass = null)
     /**
      * A factory for creating process classes, to execute the binary
      *
-     * @param  array $cmdline The command to run
+     * @param  array<string> $cmdline The command to run
      * @return object
      */
     public function createProcess($cmdline)
@@ -100,7 +100,7 @@ public function getMarginString(): ?string
     /**
      * Gets an array of options to pass to the command line
      *
-     * @return array
+     * @return array<mixed>
      */
     private function getCommandFlags(): array
     {
@@ -139,12 +139,12 @@ private function getCommandFlags(): array
             $opts[] = "--no-displayHeaderFooter";
         }
         if ($this->getHeader()) {
-            $headerFile = $this->getFileForString($this->getHeader());
+            $headerFile = $this->getFileForString($this->getHeader() ?: '');
             $opts[] = "--headerTemplate";
             $opts[] = $headerFile;
         }
         if ($this->getFooter()) {
-            $footerFile = $this->getFileForString($this->getFooter());
+            $footerFile = $this->getFileForString($this->getFooter() ?: '');
             $opts[] = "--footerTemplate";
             $opts[] = $footerFile;
         }
@@ -188,7 +188,7 @@ private function getCommandFlags(): array
      */
     private function getFileForString(string $content): string
     {
-        $tmpfile = tempnam(sys_get_temp_dir(), 'chromepdf');
+        $tmpfile = tempnam(sys_get_temp_dir(), 'chromepdf') ?: '';
         // Chrome must see the correct extension to load it as html
         rename($tmpfile, $tmpfile .= '.html');
         $this->handles[] = $tmpfile;
@@ -200,12 +200,12 @@ private function getFileForString(string $content): string
     /**
      * Execute the local renderer with the given options
      *
-     * @param  array $options
+     * @param  array<string> $options
      * @return string A path to a temporary file containing the rendered PDF
      */
     private function executeBinary(array $options): string
     {
-        $output = tempnam(sys_get_temp_dir(), 'chromepdf');
+        $output = tempnam(sys_get_temp_dir(), 'chromepdf') ?: '';
         $options[] = "--path";
         $options[] = $output;
         array_unshift($options, $this->binary, 'pdf');
@@ -229,7 +229,7 @@ public function renderContent(string $content)
         $opts[] = $file;
 
         $outputFile = $this->executeBinary($opts);
-        return fopen($outputFile, 'r');
+        return fopen($outputFile, 'r') ?: null;
     }
 
     /**
@@ -242,7 +242,7 @@ public function renderURL(string $url)
         $opts[] = $url;
 
         $outputFile = $this->executeBinary($opts);
-        return fopen($outputFile, 'r');
+        return fopen($outputFile, 'r') ?: null;
     }
 
     /**
@@ -255,6 +255,6 @@ public function renderFile(string $path)
         $opts[] = $path;
 
         $outputFile = $this->executeBinary($opts);
-        return fopen($outputFile, 'r');
+        return fopen($outputFile, 'r') ?: null;
     }
 }