feat: fix PHP 8.4 deprecations

Author: Chris8934

src/Knp/Snappy/AbstractGenerator.php

@@ -65,7 +65,7 @@ abstract class AbstractGenerator implements GeneratorInterface, LoggerAwareInter
      * @param array       $options
      * @param null|array  $env
      */
-    public function __construct($binary, array $options = [], array $env = null)
+    public function __construct($binary, array $options = [], array|null $env = null)
     {
         $this->configure();
 

src/Knp/Snappy/Image.php

@@ -13,7 +13,7 @@ class Image extends AbstractGenerator
     /**
      * {@inheritdoc}
      */
-    public function __construct($binary = null, array $options = [], array $env = null)
+    public function __construct($binary = null, array $options = [], array|null $env = null)
     {
         $this->setDefaultExtension('jpg');
 

src/Knp/Snappy/Pdf.php

@@ -18,7 +18,7 @@ class Pdf extends AbstractGenerator
     /**
      * {@inheritdoc}
      */
-    public function __construct($binary = null, array $options = [], array $env = null)
+    public function __construct($binary = null, array $options = [], array|null $env = null)
     {
         $this->setDefaultExtension('pdf');
         $this->setOptionsWithContentCheck();

tests/Knp/Snappy/PdfTest.php

@@ -79,8 +79,8 @@ public function testRemovesLocalFilesOnError(): void
         $method->setAccessible(true);
         $method->invoke($pdf, 'test', $pdf->getDefaultExtension());
         $this->assertEquals(1, \count($pdf->temporaryFiles));
-        $this->expectException(Error::class);
-        \trigger_error('test error', \E_USER_ERROR);
+        $this->expectException(\RuntimeException::class);
+        throw new \RuntimeException('test error');
         // @phpstan-ignore-next-line See https://github.com/phpstan/phpstan/issues/7799
         $this->assertFileNotExists(\reset($pdf->temporaryFiles));
     }