fix all lvl 6 errors (untested)

Author: sebastianMindee

src/Extraction/ImageExtractor.php

@@ -131,7 +131,7 @@ public function getPageCount(): int
     /**
      * Extract multiple images on a given page from a list of fields having position data.
      *
-     * @param array<BaseField> $fields List of Fields to extract.
+     * @param array<BaseField<mixed>> $fields List of Fields to extract.
      * @param integer $pageIndex The page index to extract, begins at 0.
      * @param null|string $outputName The base output filename, must have an image extension.
      *
@@ -215,7 +215,7 @@ public function extractPolygonFromPage(
     /**
      * Extracts a single image from a Position field.
      *
-     * @param BaseField $field The field to extract.
+     * @param BaseField<mixed> $field The field to extract.
      * @param integer $pageIndex The page index to extract, begins at 0.
      * @param integer $index The index to use for naming the extracted image.
      * @param string $filename The output filename.
@@ -265,7 +265,7 @@ public function getInputSource(): LocalInputSource
     /**
      * Extracts images from a page.
      *
-     * @param array<BaseField> $fields List of Fields to extract.
+     * @param array<BaseField<mixed>> $fields List of Fields to extract.
      * @param integer $pageIndex The page index to extract, begins at 0.
      * @param string $outputName Name of the created file.
      * @param string $format The output format.
@@ -316,7 +316,7 @@ protected function extractImageFromBbox(BBox $bbox, int|float $pageIndex): Imagi
      * Splits the filename into name and extension.
      *
      * @param string $filename Name of the file.
-     * @return array An array containing the name and extension of the file.
+     * @return array{0: string, 1: string} An array containing the name and extension of the file.
      */
     protected static function splitNameStrict(string $filename): array
     {

src/Extraction/PDFExtractor.php

@@ -86,7 +86,7 @@ public function getPageCount(): int
     /**
      * Extracts sub-documents from the source document using list of page indexes.
      *
-     * @param array|InvoiceSplitterV1InvoicePageGroups $pageIndexes List of sub-lists of pages to keep.
+     * @param array<array<integer>>|InvoiceSplitterV1InvoicePageGroups $pageIndexes List of sub-lists of pages to keep.
      *
      * @return ExtractedPDF[] list of extracted documents
      *
@@ -141,7 +141,7 @@ public function extractSubDocuments(mixed $pageIndexes): array
     /**
      * Extracts invoices as complete PDFs from the document.
      *
-     * @param array|InvoiceSplitterV1InvoicePageGroups $pageIndexes List of sub-lists of pages to keep.
+     * @param array<array<integer>>|InvoiceSplitterV1InvoicePageGroups $pageIndexes List of sub-lists of pages to keep.
      * @param boolean $strict Whether to trust confidence scores or not.
      *
      * @return ExtractedPDF[] a list of extracted invoices

src/Geometry/BBox.php

@@ -83,9 +83,9 @@ public function getMaxY(): float
     /**
      * Extends the BBox with the provided points.
      *
-     * @param array|Polygon $points Series of points to add to the BBox.
+     * @param array<Point>|Polygon $points Series of points to add to the BBox.
      */
-    public function extendWith(Polygon|array $points): void
+    public function extendWith(mixed $points): void
     {
         if ($points instanceof Polygon) {
             $sequence = $points->getCoordinates();

src/Geometry/MinMaxUtils.php

@@ -22,7 +22,7 @@ class MinMaxUtils
      */
     public static function getMinMaxY(mixed $points): MinMax
     {
-        if (is_a(Polygon::class, $points)) {
+        if ($points instanceof Polygon) {
             $points = $points->getCoordinates();
         }
         if (count($points) < 1) {
@@ -46,7 +46,7 @@ public static function getMinMaxY(mixed $points): MinMax
      */
     public static function getMinMaxX(mixed $points): MinMax
     {
-        if (is_a(Polygon::class, $points)) {
+        if ($points instanceof  Polygon) {
             $points = $points->getCoordinates();
         }
         if (count($points) < 1) {

src/Geometry/Polygon.php

@@ -27,7 +27,7 @@ class Polygon
     private MinMax $minMaxX;
 
     /**
-     * @param array<array<float|integer>>|null $coordinates Coordinates of the polygon as a set of Points.
+     * @param array<array<float|integer>>|array<Point>|null $coordinates Coordinates of the polygon as a set of Points.
      */
     public function __construct(?array $coordinates = null)
     {

src/Geometry/PolygonUtils.php

@@ -22,7 +22,7 @@ abstract class PolygonUtils
      */
     public static function getCentroid(mixed $vertices): Point
     {
-        if (is_a(Polygon::class, $vertices)) {
+        if ($vertices instanceof Polygon) {
             $vertices = $vertices->getCoordinates();
         }
         $verticesSum = count($vertices);

src/Input/LocalResponse.php

@@ -23,49 +23,36 @@ class LocalResponse
     private $file;
 
     /**
-     * @param resource|string|array $inputFile A string, path or file-like object to load as a local response.
+     * @param resource|string|array<string> $inputFile A string, path or file-like object to load as a local response.
      * @throws MindeeException Throws if the input file isn't acceptable.
      */
     public function __construct(mixed $inputFile)
     {
-        if (is_resource($inputFile) && get_resource_type($inputFile) === 'file') {
-            $content = fread($inputFile, fstat($inputFile)['size']);
-            $strStripped = str_replace(["\r", "\n"], '', $content);
-            $this->file = fopen('php://memory', 'r+');
-            fwrite($this->file, $strStripped);
-            rewind($this->file);
-        } elseif (is_resource($inputFile) && get_resource_type($inputFile) === 'stream') {
-            $content = stream_get_contents($inputFile);
-            $strStripped = str_replace(["\r", "\n"], '', $content);
-            $this->file = fopen('php://memory', 'r+');
-            fwrite($this->file, $strStripped);
-            rewind($this->file);
-        } elseif (is_string($inputFile) && file_exists($inputFile)) {
-            $content = file_get_contents($inputFile);
-            $strStripped = str_replace(["\r", "\n"], '', $content);
-            $this->file = fopen('php://memory', 'r+');
-            fwrite($this->file, $strStripped);
-            rewind($this->file);
+        if (is_resource($inputFile)) {
+            $resourceType = get_resource_type($inputFile);
+            if ($resourceType === 'file') {
+                $content = fread($inputFile, fstat($inputFile)['size']);
+            } elseif ($resourceType === 'stream') {
+                $content = stream_get_contents($inputFile);
+            } else {
+                throw new MindeeException("Unsupported resource type.", ErrorCode::USER_INPUT_ERROR);
+            }
         } elseif (is_string($inputFile)) {
-            $strStripped = str_replace(["\r", "\n"], '', $inputFile);
-            $this->file = fopen('php://memory', 'r+');
-            fwrite($this->file, $strStripped);
-            rewind($this->file);
-        } elseif (is_string($inputFile) || is_array($inputFile)) {
-            if (is_array($inputFile))
-            {
-                $inputFile = implode($inputFile);
+            if (file_exists($inputFile) && is_file($inputFile)) {
+                $content = file_get_contents($inputFile);
+            } else {
+                $content = $inputFile;
             }
-            $strStripped = str_replace(["\r", "\n"], '', $inputFile);
-            $this->file = fopen('php://memory', 'r+');
-            fwrite($this->file, $strStripped);
-            rewind($this->file);
+        } elseif (is_array($inputFile)) {
+            $content = implode('', $inputFile);
         } else {
-            throw new MindeeException(
-                "Incompatible type for input.",
-                ErrorCode::USER_INPUT_ERROR
-            );
+            throw new MindeeException("Incompatible type for input.", ErrorCode::USER_INPUT_ERROR);
         }
+
+        $strStripped = str_replace(["\r", "\n"], '', (string) $content);
+        $this->file = fopen('php://memory', 'r+');
+        fwrite($this->file, $strStripped);
+        rewind($this->file);
     }
 
     /**

src/PDF/PDFUtils.php

@@ -85,7 +85,7 @@ public static function hasSourceText(string $pdfPath): bool
      * Extracts text elements with their properties from all pages in a PDF.
      *
      * @param string $pdfPath Path to the PDF file.
-     * @return array<<integer, string>> A page-indexed array of text elements.
+     * @return array<integer, string> A page-indexed array of text elements.
      *               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.
      */
@@ -192,7 +192,7 @@ public static function extractTextElements(Page $page): array
      * @param string $fontName Name of the font/subfont.
      * @return array{family: string, style: string} The standard font & possible style.
      */
-    private static function standardizeFontName(string $fontName): array
+    protected static function standardizeFontName(string $fontName): array
     {
         $cleanName = preg_replace('/^.*?\+/', '', $fontName);
         $parts = explode('-', $cleanName, 2);

src/Parsing/SummaryHelper.php

@@ -77,7 +77,7 @@ public static function formatForDisplay($inputString = null, ?int $maxColSize =
             return 'False';
         }
         $inputString = self::escapeSpecialChars($inputString);
-        if (!$inputString || mb_strlen($inputString, "UTF-8") === 0) {
+        if (!$inputString) {
             return "";
         }
         if (!isset($maxColSize)) {

src/V1/Client.php

@@ -155,7 +155,7 @@ private function constructEndpoint(
         string $endpointOwner,
         string $endpointVersion
     ): Endpoint {
-        $endpointVersion = $endpointVersion != null && strlen($endpointVersion) > 0 ? $endpointVersion : '1';
+        $endpointVersion = $endpointVersion !== '' ? $endpointVersion : '1';
 
         $endpointSettings = new MindeeAPI($this->apiKey, $endpointName, $endpointOwner, $endpointVersion);
 
@@ -225,7 +225,7 @@ public function createEndpoint(string $endpointName, string $accountName, ?strin
             );
         }
         $accountName = $this->cleanAccountName($accountName);
-        if (!$version || $version == '') {
+        if (empty($version)) {
             error_log("Notice: no version provided for a custom build, will attempt to poll version 1 by default.");
             $version = "1";
         }