Throw exceptions on pdf size and orientation parsing

Author: firstred

src/Util/Util.php

@@ -32,7 +32,9 @@
 use Exception;
 use Firstred\PostNL\Exception\InvalidArgumentException;
 use setasign\Fpdi\Fpdi;
+use setasign\Fpdi\PdfParser\PdfParserException;
 use setasign\Fpdi\PdfParser\StreamReader;
+use setasign\Fpdi\PdfReader\PdfReaderException;
 
 /**
  * Class Util.
@@ -85,38 +87,36 @@ public static function urlEncode($arr, $prefix = null)
     /**
      * @param string $pdf Raw PDF string
      *
-     * @return array|false|string Returns an array with the dimensions or ISO size and orientation
-     *                            The orientation is in FPDF format, so L for Landscape and P for Portrait
-     *                            Sizes are in mm
+     * @return array Returns an array with the dimensions or ISO size and orientation
+     *               The orientation is in FPDF format, so L for Landscape and P for Portrait
+     *               Sizes are in mm
+     *
+     * @throws PdfParserException|PdfReaderException
      */
     public static function getPdfSizeAndOrientation($pdf)
     {
-        try {
-            $fpdi = new Fpdi('P', 'mm');
-            $fpdi->setSourceFile(StreamReader::createByString($pdf));
-            // import page 1
-            $tplIdx1 = $fpdi->importPage(1);
-            $size = $fpdi->getTemplateSize($tplIdx1);
-            $width = $size['width'];
-            $height = $size['height'];
-            $orientation = $size['orientation'];
-
-            $length = 'P' === $orientation ? $height : $width;
-            if ($length >= (148 - static::ERROR_MARGIN) && $length <= (148 + static::ERROR_MARGIN)) {
-                $iso = 'A6';
-            } elseif ($length >= (210 - static::ERROR_MARGIN) && $length <= (210 + static::ERROR_MARGIN)) {
-                $iso = 'A5';
-            } elseif ($length >= (420 - static::ERROR_MARGIN) && $length <= (420 + static::ERROR_MARGIN)) {
-                $iso = 'A3';
-            } elseif ($length >= (594 - static::ERROR_MARGIN) && $length <= (594 + static::ERROR_MARGIN)) {
-                $iso = 'A2';
-            } elseif ($length >= (841 - static::ERROR_MARGIN) && $length <= (841 + static::ERROR_MARGIN)) {
-                $iso = 'A1';
-            } else {
-                $iso = 'A4';
-            }
-        } catch (Exception $e) {
-            return false;
+        $fpdi = new Fpdi('P', 'mm');
+        $fpdi->setSourceFile(StreamReader::createByString($pdf));
+        // import page 1
+        $tplIdx1 = $fpdi->importPage(1);
+        $size = $fpdi->getTemplateSize($tplIdx1);
+        $width = $size['width'];
+        $height = $size['height'];
+        $orientation = $size['orientation'];
+
+        $length = 'P' === $orientation ? $height : $width;
+        if ($length >= (148 - static::ERROR_MARGIN) && $length <= (148 + static::ERROR_MARGIN)) {
+            $iso = 'A6';
+        } elseif ($length >= (210 - static::ERROR_MARGIN) && $length <= (210 + static::ERROR_MARGIN)) {
+            $iso = 'A5';
+        } elseif ($length >= (420 - static::ERROR_MARGIN) && $length <= (420 + static::ERROR_MARGIN)) {
+            $iso = 'A3';
+        } elseif ($length >= (594 - static::ERROR_MARGIN) && $length <= (594 + static::ERROR_MARGIN)) {
+            $iso = 'A2';
+        } elseif ($length >= (841 - static::ERROR_MARGIN) && $length <= (841 + static::ERROR_MARGIN)) {
+            $iso = 'A1';
+        } else {
+            $iso = 'A4';
         }
 
         return [