fix all formatting issue

Author: sebastianMindee

.github/workflows/_static-analysis.yml

@@ -34,4 +34,7 @@ jobs:
     - name: Run lint
       run: |
         composer lint
-    
+
+    - name: Run PHPStan
+      run: |
+        composer phpstan

composer.json

@@ -34,7 +34,7 @@
   ],
   "scripts": {
     "lint": "php-cs-fixer fix --dry-run --diff",
-    "phpstan": "phpstan analyse src --level max",
+    "phpstan": "phpstan analyse src --level 6",
     "format": "php-cs-fixer fix"
   }
 }

src/Extraction/ExtractedImage.php

@@ -85,7 +85,7 @@ public function writeToFile(string $outputPath, ?string $format = null, int $qua
         $quality = min(100, max(0, $quality));
         if ('png' === $format) {
             $finalQuality = round($quality * 0.09);
-            $this->image->setOption('png:compression-level', (string)$finalQuality);
+            $this->image->setOption('png:compression-level', (string) $finalQuality);
         } elseif (in_array($format, ['jpg', 'jpeg'], true)) {
             $this->image->setImageCompression(Imagick::COMPRESSION_JPEG);
         }

src/Geometry/BBoxUtils.php

@@ -30,32 +30,21 @@ public static function generateBBoxFromPolygon(Polygon $polygon): ?BBox
     /**
      * Generates a BBox from an array of polygons. Returns null if no polygons are provided.
      *
-     * @param array<Polygon> $polygons Series of polygons to get the BBox of.
+     * @param array<Polygon|null> $polygons Series of polygons to get the BBox of.
      */
     public static function generateBBoxFromPolygons(array $polygons): ?BBox
     {
-        if (!$polygons) {
-            return null;
-        }
-        $merged = null;
+        $bboxes = [];
+
         foreach ($polygons as $polygon) {
-            if (empty($polygon)) {
+            if (null === $polygon || !$polygon->getCoordinates()) {
                 continue;
-            } else {
-                if (empty($merged)) {
-                    $merged = $polygon;
-                }
-            }
-            if ($merged !== $polygon) {
-                $merged = PolygonUtils::merge($merged, $polygon);
             }
+
+            $bboxes[] = self::generateBBoxFromPolygon($polygon);
         }
-        return new BBox(
-            $merged->getMinX(),
-            $merged->getMaxX(),
-            $merged->getMinY(),
-            $merged->getMaxY(),
-        );
+
+        return self::mergeBBoxes($bboxes);
     }
 
     /**

src/Geometry/PolygonUtils.php

@@ -18,7 +18,6 @@ abstract class PolygonUtils
      * Gets the centroid (Point) of a set of points.
      *
      * @param array<Point>|Polygon $vertices Array of points.
-     * @return Point
      */
     public static function getCentroid(mixed $vertices): Point
     {
@@ -49,7 +48,7 @@ public static function getCentroid(mixed $vertices): Point
     public static function compareOnY(Polygon $polygon1, Polygon $polygon2): int
     {
         $sort = ($polygon1->getMinY() - $polygon2->getMinY());
-        if ($sort == 0) {
+        if ($sort === 0.0) {
             return 0;
         }
         return $sort < 0 ? -1 : 1;

src/PDF/CustomFPDI.php

@@ -30,14 +30,14 @@ class CustomFPDI extends Fpdi
      */
     public function rotate(float $angle, float $x = -1, float $y = -1): void
     {
-        if ($x == -1) {
+        if ((int) $x === -1) {
             $x = $this->x;
         }
-        if ($y == -1) {
+        if ((int) $y === -1) {
             $y = $this->y;
         }
 
-        if ((int) $angle != 0) {
+        if ((int) $angle !== 0) {
             $angle = -$angle;
         }
         $angle *= M_PI / 180;
@@ -72,6 +72,7 @@ protected function _endpage(): void
         }
         parent::_endpage();
     }
+
     /**
      * Starts a new transformation.
      *
@@ -80,6 +81,7 @@ public function startTransform(): void
     {
         $this->_out('q');
     }
+
     /**
      * Stops the current transformation.
      *

src/PDF/PDFCompressor.php

@@ -18,7 +18,6 @@
 use Imagick;
 use SplFileObject;
 
-
 /**
  * PDF compression class.
  */

src/PDF/PDFUtils.php

@@ -86,7 +86,7 @@ public static function hasSourceText(string $pdfPath): bool
      *
      * @param string $pdfPath Path to the PDF file.
      * @return array<integer, string> A page-indexed array of text elements.
-     *               Each text element includes text content, position, font, size, and color.
+     *                                Each text element includes text content, position, font, size, and color.
      * @throws MindeePDFException Throws if the PDF can't be parsed or text elements can't be extracted.
      */
     public static function extractPagesTextElements(string $pdfPath): array
@@ -153,7 +153,7 @@ public static function downgradePDFVersion(string $inputPath): string
      *
      * @param Page $page Page object.
      * @return array<array<string, string|float|Font|null>> An array of text elements, each containing text content,
-     * position, font, size, and color.
+     *                                                      position, font, size, and color.
      * @throws MindeePDFException Throws if the text elements can't be extracted.
      */
     public static function extractTextElements(Page $page): array
@@ -169,11 +169,11 @@ public static function extractTextElements(Page $page): array
                 if (isset($text[1])) {
                     $textElements[] = [
                         'text' => $text[1],
-                        'rotation' => rad2deg((float)($text[0][2])),
-                        'x' => (float)($text[0][4]),
-                        'y' => (float)($text[0][5]),
+                        'rotation' => rad2deg((float) ($text[0][2])),
+                        'x' => (float) ($text[0][4]),
+                        'y' => (float) ($text[0][5]),
                         'font' => $page->getFont($text[2]),
-                        'size' => (float)($text[3]),
+                        'size' => (float) ($text[3]),
                     ];
                 }
             }

src/Parsing/SummaryHelper.php

@@ -20,7 +20,7 @@ class SummaryHelper
      */
     public static function formatFloat(?float $number): string
     {
-        if ($number === null) {
+        if (null === $number) {
             return '';
         }
         $formatted = number_format($number, 5, '.', '');
@@ -54,7 +54,7 @@ public static function cleanOutString(string $inputString): string
      */
     protected static function escapeSpecialChars(?string $string): ?string
     {
-        if ($string == null) {
+        if (null === $string) {
             return null;
         }
         $find = ["\n", "\t", "\r"];

src/V1/Client.php

@@ -407,7 +407,7 @@ public function parse(
         ?PredictMethodOptions $options = null,
         ?PageOptions $pageOptions = null
     ): PredictResponse {
-        if ($options == null) {
+        if (null === $options) {
             $options = new PredictMethodOptions();
         }
         if ($pageOptions !== null && $inputDoc instanceof LocalInputSource && $inputDoc->isPDF()) {
@@ -437,10 +437,10 @@ public function enqueueAndParse(
         ?PollingOptions $asyncOptions = null,
         ?PageOptions $pageOptions = null
     ): AsyncPredictResponse {
-        if ($options == null) {
+        if (null === $options) {
             $options = new PredictMethodOptions();
         }
-        if ($asyncOptions == null) {
+        if (null === $asyncOptions) {
             $asyncOptions = new PollingOptions();
         }
 
@@ -492,7 +492,7 @@ public function enqueue(
         ?PredictMethodOptions $options = null,
         ?PageOptions $pageOptions = null
     ): AsyncPredictResponse {
-        if ($options == null) {
+        if (null === $options) {
             $options = new PredictMethodOptions();
         }
         if ($pageOptions !== null && $inputDoc instanceof LocalInputSource && $inputDoc->isPDF()) {
@@ -561,7 +561,7 @@ public function executeWorkflow(
         ?WorkflowOptions $options = null,
         ?PageOptions $pageOptions = null
     ): WorkflowResponse {
-        if ($options == null) {
+        if (null === $options) {
             $options = new WorkflowOptions();
         }
         if ($pageOptions !== null && $inputDoc instanceof LocalInputSource && $inputDoc->isPDF()) {