Remove all mixed in param type where reasonable (except Calculcation/)

PowerKiKi · PowerKiKi · commit ad9fe0a13bba · 2024-01-22T22:59:24.000+08:00
diff --git a/src/PhpSpreadsheet/Cell/AdvancedValueBinder.php b/src/PhpSpreadsheet/Cell/AdvancedValueBinder.php
@@ -17,7 +17,7 @@ class AdvancedValueBinder extends DefaultValueBinder implements IValueBinder
      * @param Cell $cell Cell to bind value to
      * @param mixed $value Value to bind in cell
      */
-    public function bindValue(Cell $cell, $value = null): bool
+    public function bindValue(Cell $cell, mixed $value = null): bool
     {
         if ($value === null) {
             return parent::bindValue($cell, $value);
diff --git a/src/PhpSpreadsheet/Cell/CellAddress.php b/src/PhpSpreadsheet/Cell/CellAddress.php
@@ -34,18 +34,18 @@ public function __destruct()
      * @phpstan-assert int|numeric-string $columnId
      * @phpstan-assert int|numeric-string $rowId
      */
-    private static function validateColumnAndRow(mixed $columnId, mixed $rowId): void
+    private static function validateColumnAndRow(int|string $columnId, int|string $rowId): void
     {
         if (!is_numeric($columnId) || $columnId <= 0 || !is_numeric($rowId) || $rowId <= 0) {
             throw new Exception('Row and Column Ids must be positive integer values');
         }
     }
 
-    public static function fromColumnAndRow(mixed $columnId, mixed $rowId, ?Worksheet $worksheet = null): self
+    public static function fromColumnAndRow(int|string $columnId, int|string $rowId, ?Worksheet $worksheet = null): self
     {
         self::validateColumnAndRow($columnId, $rowId);
 
-        return new self(Coordinate::stringFromColumnIndex($columnId) . ((string) $rowId), $worksheet);
+        return new self(Coordinate::stringFromColumnIndex($columnId) . $rowId, $worksheet);
     }
 
     public static function fromColumnRowArray(array $array, ?Worksheet $worksheet = null): self
@@ -55,7 +55,7 @@ public static function fromColumnRowArray(array $array, ?Worksheet $worksheet =
         return self::fromColumnAndRow($columnId, $rowId, $worksheet);
     }
 
-    public static function fromCellAddress(mixed $cellAddress, ?Worksheet $worksheet = null): self
+    public static function fromCellAddress(string $cellAddress, ?Worksheet $worksheet = null): self
     {
         return new self($cellAddress, $worksheet);
     }
diff --git a/src/PhpSpreadsheet/Cell/DefaultValueBinder.php b/src/PhpSpreadsheet/Cell/DefaultValueBinder.php
@@ -16,7 +16,7 @@ class DefaultValueBinder implements IValueBinder
      * @param Cell $cell Cell to bind value to
      * @param mixed $value Value to bind in cell
      */
-    public function bindValue(Cell $cell, $value): bool
+    public function bindValue(Cell $cell, mixed $value): bool
     {
         // sanitize UTF-8 strings
         if (is_string($value)) {
diff --git a/src/PhpSpreadsheet/Cell/StringValueBinder.php b/src/PhpSpreadsheet/Cell/StringValueBinder.php
@@ -67,7 +67,7 @@ public function setConversionForAllValueTypes(bool $suppressConversion = false):
      * @param Cell $cell Cell to bind value to
      * @param mixed $value Value to bind in cell
      */
-    public function bindValue(Cell $cell, $value): bool
+    public function bindValue(Cell $cell, mixed $value): bool
     {
         if (is_object($value)) {
             return $this->bindObjectValue($cell, $value);
diff --git a/src/PhpSpreadsheet/Chart/Axis.php b/src/PhpSpreadsheet/Chart/Axis.php
@@ -112,9 +112,9 @@ public function __construct()
     /**
      * Get Series Data Type.
      */
-    public function setAxisNumberProperties(mixed $format_code, ?bool $numeric = null, int $sourceLinked = 0): void
+    public function setAxisNumberProperties(string $format_code, ?bool $numeric = null, int $sourceLinked = 0): void
     {
-        $format = (string) $format_code;
+        $format = $format_code;
         $this->axisNumber['format'] = $format;
         $this->axisNumber['source_linked'] = $sourceLinked;
         if (is_bool($numeric)) {
diff --git a/src/PhpSpreadsheet/Chart/Chart.php b/src/PhpSpreadsheet/Chart/Chart.php
@@ -126,7 +126,7 @@ class Chart
      * Create a new Chart.
      * majorGridlines and minorGridlines are deprecated, moved to Axis.
      */
-    public function __construct(mixed $name, ?Title $title = null, ?Legend $legend = null, ?PlotArea $plotArea = null, mixed $plotVisibleOnly = true, string $displayBlanksAs = DataSeries::EMPTY_AS_GAP, ?Title $xAxisLabel = null, ?Title $yAxisLabel = null, ?Axis $xAxis = null, ?Axis $yAxis = null, ?GridLines $majorGridlines = null, ?GridLines $minorGridlines = null)
+    public function __construct(string $name, ?Title $title = null, ?Legend $legend = null, ?PlotArea $plotArea = null, bool $plotVisibleOnly = true, string $displayBlanksAs = DataSeries::EMPTY_AS_GAP, ?Title $xAxisLabel = null, ?Title $yAxisLabel = null, ?Axis $xAxis = null, ?Axis $yAxis = null, ?GridLines $majorGridlines = null, ?GridLines $minorGridlines = null)
     {
         $this->name = $name;
         $this->title = $title;
diff --git a/src/PhpSpreadsheet/Chart/DataSeries.php b/src/PhpSpreadsheet/Chart/DataSeries.php
@@ -232,7 +232,7 @@ public function getPlotLabels(): array
      *
      * @return DataSeriesValues|false
      */
-    public function getPlotLabelByIndex(mixed $index): bool|DataSeriesValues
+    public function getPlotLabelByIndex(int $index): bool|DataSeriesValues
     {
         $keys = array_keys($this->plotLabel);
         if (in_array($index, $keys)) {
@@ -257,7 +257,7 @@ public function getPlotCategories(): array
      *
      * @return DataSeriesValues|false
      */
-    public function getPlotCategoryByIndex(mixed $index): bool|DataSeriesValues
+    public function getPlotCategoryByIndex(int $index): bool|DataSeriesValues
     {
         $keys = array_keys($this->plotCategory);
         if (in_array($index, $keys)) {
@@ -304,7 +304,7 @@ public function getPlotValues(): array
      *
      * @return DataSeriesValues|false
      */
-    public function getPlotValuesByIndex(mixed $index): bool|DataSeriesValues
+    public function getPlotValuesByIndex(int $index): bool|DataSeriesValues
     {
         $keys = array_keys($this->plotValues);
         if (in_array($index, $keys)) {
diff --git a/src/PhpSpreadsheet/Chart/PlotArea.php b/src/PhpSpreadsheet/Chart/PlotArea.php
@@ -88,7 +88,7 @@ public function getPlotGroup(): array
     /**
      * Get Plot Series by Index.
      */
-    public function getPlotGroupByIndex(mixed $index): DataSeries
+    public function getPlotGroupByIndex(int $index): DataSeries
     {
         return $this->plotSeries[$index];
     }
diff --git a/src/PhpSpreadsheet/Chart/Properties.php b/src/PhpSpreadsheet/Chart/Properties.php
@@ -403,7 +403,7 @@ protected function getShadowPresetsMap(int $presetsOption): array
     /**
      * Get value of array element.
      */
-    protected function getArrayElementsValue(mixed $properties, mixed $elements): mixed
+    protected function getArrayElementsValue(array $properties, array|int|string $elements): mixed
     {
         $reference = &$properties;
         if (!is_array($elements)) {
diff --git a/src/PhpSpreadsheet/Helper/Dimension.php b/src/PhpSpreadsheet/Helper/Dimension.php
@@ -56,7 +56,7 @@ class Dimension
      * Phpstan bug has been fixed; this function allows us to
      * pass Phpstan whether fixed or not.
      */
-    private static function stanBugFixed(mixed $value): array
+    private static function stanBugFixed(array|int|null $value): array
     {
         return is_array($value) ? $value : [null, null];
     }
diff --git a/src/PhpSpreadsheet/Reader/Html.php b/src/PhpSpreadsheet/Reader/Html.php
@@ -998,7 +998,7 @@ private function applyInlineStyle(Worksheet &$sheet, int $row, string $column, a
     /**
      * Check if has #, so we can get clean hex.
      */
-    public function getStyleColor(mixed $value): string
+    public function getStyleColor(?string $value): string
     {
         $value = (string) $value;
         if (str_starts_with($value, '#')) {
diff --git a/src/PhpSpreadsheet/Reader/Xls/Escher.php b/src/PhpSpreadsheet/Reader/Xls/Escher.php
@@ -50,15 +50,13 @@ class Escher
 
     /**
      * The object to be returned by the reader. Modified during load.
-     *
-     * @var BSE|BstoreContainer|DgContainer|DggContainer|\PhpOffice\PhpSpreadsheet\Shared\Escher|SpContainer|SpgrContainer
      */
-    private $object;
+    private BSE|BstoreContainer|DgContainer|DggContainer|\PhpOffice\PhpSpreadsheet\Shared\Escher|SpContainer|SpgrContainer $object;
 
     /**
      * Create a new Escher instance.
      */
-    public function __construct(mixed $object)
+    public function __construct(BSE|BstoreContainer|DgContainer|DggContainer|\PhpOffice\PhpSpreadsheet\Shared\Escher|SpContainer|SpgrContainer $object)
     {
         $this->object = $object;
     }
diff --git a/src/PhpSpreadsheet/Reader/Xlsx/Chart.php b/src/PhpSpreadsheet/Reader/Xlsx/Chart.php
@@ -79,7 +79,8 @@ public function readChart(SimpleXMLElement $chartElements, string $chartName): \
         $chartElementsC = $chartElements->children($this->cNamespace);
 
         $XaxisLabel = $YaxisLabel = $legend = $title = null;
-        $dispBlanksAs = $plotVisOnly = null;
+        $dispBlanksAs = null;
+        $plotVisOnly = false;
         $plotArea = null;
         $rotX = $rotY = $rAngAx = $perspective = null;
         $xAxis = new Axis();
@@ -381,7 +382,7 @@ public function readChart(SimpleXMLElement $chartElements, string $chartName): \
 
                                 break;
                             case 'plotVisOnly':
-                                $plotVisOnly = self::getAttributeString($chartDetails, 'val');
+                                $plotVisOnly = (bool) self::getAttributeString($chartDetails, 'val');
 
                                 break;
                             case 'dispBlanksAs':
diff --git a/src/PhpSpreadsheet/Reader/Xlsx/Styles.php b/src/PhpSpreadsheet/Reader/Xlsx/Styles.php
@@ -429,8 +429,8 @@ public function styles(): array
      *
      * @param mixed $array (usually array, in theory can be false)
      */
-    private static function getArrayItem(mixed $array, int $key = 0): ?SimpleXMLElement
+    private static function getArrayItem(mixed $array): ?SimpleXMLElement
     {
-        return is_array($array) ? ($array[$key] ?? null) : null;
+        return is_array($array) ? ($array[0] ?? null) : null;
     }
 }
diff --git a/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php b/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php
@@ -14,7 +14,7 @@ public function __construct(Spreadsheet $spreadsheet)
         $this->spreadsheet = $spreadsheet;
     }
 
-    public function viewSettings(SimpleXMLElement $xmlWorkbook, mixed $mainNS, array $mapSheetId, bool $readDataOnly): void
+    public function viewSettings(SimpleXMLElement $xmlWorkbook, string $mainNS, array $mapSheetId, bool $readDataOnly): void
     {
         // Default active sheet index to the first loaded worksheet from the file
         $this->spreadsheet->setActiveSheetIndex(0);
diff --git a/src/PhpSpreadsheet/Shared/Date.php b/src/PhpSpreadsheet/Shared/Date.php
@@ -195,7 +195,7 @@ public static function convertIsoDate(mixed $value): float|int
      *
      * @return DateTime PHP date/time object
      */
-    public static function excelToDateTimeObject($excelTimestamp, $timeZone = null): DateTime
+    public static function excelToDateTimeObject(float|int $excelTimestamp, null|DateTimeZone|string $timeZone = null): DateTime
     {
         $timeZone = ($timeZone === null) ? self::getDefaultTimezone() : self::validateTimeZone($timeZone);
         if (Functions::getCompatibilityMode() == Functions::COMPATIBILITY_EXCEL) {
diff --git a/src/PhpSpreadsheet/Shared/OLE/PPS.php b/src/PhpSpreadsheet/Shared/OLE/PPS.php
@@ -175,7 +175,7 @@ public function getPpsWk(): string
      *
      * @return int The index for this PPS
      */
-    public static function savePpsSetPnt(array &$raList, mixed $to_save, mixed $depth = 0): int
+    public static function savePpsSetPnt(array &$raList, mixed $to_save, int $depth = 0): int
     {
         if (!is_array($to_save) || (empty($to_save))) {
             return 0xFFFFFFFF;
diff --git a/src/PhpSpreadsheet/Spreadsheet.php b/src/PhpSpreadsheet/Spreadsheet.php
@@ -287,10 +287,10 @@ public function getRibbonXMLData(string $what = 'all'): null|array|string //we n
     /**
      * store binaries ribbon objects (pictures).
      */
-    public function setRibbonBinObjects(mixed $BinObjectsNames, mixed $BinObjectsData): void
+    public function setRibbonBinObjects(mixed $binObjectsNames, mixed $binObjectsData): void
     {
-        if ($BinObjectsNames !== null && $BinObjectsData !== null) {
-            $this->ribbonBinObjects = ['names' => $BinObjectsNames, 'data' => $BinObjectsData];
+        if ($binObjectsNames !== null && $binObjectsData !== null) {
+            $this->ribbonBinObjects = ['names' => $binObjectsNames, 'data' => $binObjectsData];
         } else {
             $this->ribbonBinObjects = null;
         }
@@ -318,16 +318,6 @@ public function setUnparsedLoadedData(array $unparsedLoadedData): void
         $this->unparsedLoadedData = $unparsedLoadedData;
     }
 
-    /**
-     * return the extension of a filename. Internal use for a array_map callback (php<5.3 don't like lambda function).
-     */
-    private function getExtensionOnly(mixed $path): string
-    {
-        $extension = pathinfo($path, PATHINFO_EXTENSION);
-
-        return substr($extension, 0);
-    }
-
     /**
      * retrieve Binaries Ribbon Objects.
      */
@@ -351,7 +341,7 @@ public function getRibbonBinObjects(string $what = 'all'): ?array
                     && isset($this->ribbonBinObjects['data']) && is_array($this->ribbonBinObjects['data'])
                 ) {
                     $tmpTypes = array_keys($this->ribbonBinObjects['data']);
-                    $ReturnData = array_unique(array_map([$this, 'getExtensionOnly'], $tmpTypes));
+                    $ReturnData = array_unique(array_map(fn (string $path): string => pathinfo($path, PATHINFO_EXTENSION), $tmpTypes));
                 } else {
                     $ReturnData = []; // the caller want an array... not null if empty
                 }
diff --git a/src/PhpSpreadsheet/Writer/Ods/Settings.php b/src/PhpSpreadsheet/Writer/Ods/Settings.php
@@ -124,7 +124,7 @@ private function writeSplitValue(XMLWriter $objWriter, string $splitMode, string
 
     private function writeFreezePane(XMLWriter $objWriter, Worksheet $worksheet): void
     {
-        $freezePane = CellAddress::fromCellAddress($worksheet->getFreezePane());
+        $freezePane = CellAddress::fromCellAddress($worksheet->getFreezePane() ?? '');
         if ($freezePane->cellAddress() === 'A1') {
             return;
         }
diff --git a/src/PhpSpreadsheet/Writer/Xls/Escher.php b/src/PhpSpreadsheet/Writer/Xls/Escher.php
@@ -16,10 +16,8 @@ class Escher
 {
     /**
      * The object we are writing.
-     *
-     * @var Blip|BSE|BstoreContainer|DgContainer|DggContainer|Escher|SpContainer|SpgrContainer
      */
-    private $object;
+    private Blip|BSE|BstoreContainer|DgContainer|DggContainer|Escher|SpContainer|SpgrContainer|SharedEscher $object;
 
     /**
      * The written binary data.
@@ -39,7 +37,7 @@ class Escher
     /**
      * Constructor.
      */
-    public function __construct(mixed $object)
+    public function __construct(Blip|BSE|BstoreContainer|DgContainer|DggContainer|self|SpContainer|SpgrContainer|SharedEscher $object)
     {
         $this->object = $object;
     }
diff --git a/src/PhpSpreadsheet/Writer/Xlsx/Chart.php b/src/PhpSpreadsheet/Writer/Xlsx/Chart.php
@@ -26,7 +26,7 @@ class Chart extends WriterPart
      *
      * @return string XML Output
      */
-    public function writeChart(\PhpOffice\PhpSpreadsheet\Chart\Chart $chart, mixed $calculateCellValues = true): string
+    public function writeChart(\PhpOffice\PhpSpreadsheet\Chart\Chart $chart, bool $calculateCellValues = true): string
     {
         // Create XML writer
         $objWriter = null;
diff --git a/tests/PhpSpreadsheetTests/Cell/CellAddressTest.php b/tests/PhpSpreadsheetTests/Cell/CellAddressTest.php
@@ -41,7 +41,7 @@ public static function providerCreateFromCellAddress(): array
     /**
      * @dataProvider providerCreateFromCellAddressException
      */
-    public function testCreateFromCellAddressException(int|string $cellAddress): void
+    public function testCreateFromCellAddressException(string $cellAddress): void
     {
         $this->expectException(Exception::class);
         $this->expectExceptionMessage(
@@ -60,7 +60,6 @@ public static function providerCreateFromCellAddressException(): array
             [''],
             ['IV'],
             ['12'],
-            [123],
         ];
     }
 

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ class AdvancedValueBinder extends DefaultValueBinder implements IValueBinder`
`17`	`17`	`* @param Cell $cell Cell to bind value to`
`18`	`18`	`* @param mixed $value Value to bind in cell`
`19`	`19`	`*/`
`20`		`- public function bindValue(Cell $cell, $value = null): bool`
	`20`	`+ public function bindValue(Cell $cell, mixed $value = null): bool`
`21`	`21`	`{`
`22`	`22`	`if ($value === null) {`
`23`	`23`	`return parent::bindValue($cell, $value);`
Original file line number	Diff line number	Diff line change
`@@ -34,18 +34,18 @@ public function __destruct()`
`34`	`34`	`* @phpstan-assert int\|numeric-string $columnId`
`35`	`35`	`* @phpstan-assert int\|numeric-string $rowId`
`36`	`36`	`*/`
`37`		`- private static function validateColumnAndRow(mixed $columnId, mixed $rowId): void`
	`37`	`+ private static function validateColumnAndRow(int\|string $columnId, int\|string $rowId): void`
`38`	`38`	`{`
`39`	`39`	`if (!is_numeric($columnId) \|\| $columnId <= 0 \|\| !is_numeric($rowId) \|\| $rowId <= 0) {`
`40`	`40`	`throw new Exception('Row and Column Ids must be positive integer values');`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`
`44`		`- public static function fromColumnAndRow(mixed $columnId, mixed $rowId, ?Worksheet $worksheet = null): self`
	`44`	`+ public static function fromColumnAndRow(int\|string $columnId, int\|string $rowId, ?Worksheet $worksheet = null): self`
`45`	`45`	`{`
`46`	`46`	`self::validateColumnAndRow($columnId, $rowId);`
`47`	`47`
`48`		`- return new self(Coordinate::stringFromColumnIndex($columnId) . ((string) $rowId), $worksheet);`
	`48`	`+ return new self(Coordinate::stringFromColumnIndex($columnId) . $rowId, $worksheet);`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`public static function fromColumnRowArray(array $array, ?Worksheet $worksheet = null): self`
`@@ -55,7 +55,7 @@ public static function fromColumnRowArray(array $array, ?Worksheet $worksheet =`
`55`	`55`	`return self::fromColumnAndRow($columnId, $rowId, $worksheet);`
`56`	`56`	`}`
`57`	`57`
`58`		`- public static function fromCellAddress(mixed $cellAddress, ?Worksheet $worksheet = null): self`
	`58`	`+ public static function fromCellAddress(string $cellAddress, ?Worksheet $worksheet = null): self`
`59`	`59`	`{`
`60`	`60`	`return new self($cellAddress, $worksheet);`
`61`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ class DefaultValueBinder implements IValueBinder`
`16`	`16`	`* @param Cell $cell Cell to bind value to`
`17`	`17`	`* @param mixed $value Value to bind in cell`
`18`	`18`	`*/`
`19`		`- public function bindValue(Cell $cell, $value): bool`
	`19`	`+ public function bindValue(Cell $cell, mixed $value): bool`
`20`	`20`	`{`
`21`	`21`	`// sanitize UTF-8 strings`
`22`	`22`	`if (is_string($value)) {`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ public function setConversionForAllValueTypes(bool $suppressConversion = false):`
`67`	`67`	`* @param Cell $cell Cell to bind value to`
`68`	`68`	`* @param mixed $value Value to bind in cell`
`69`	`69`	`*/`
`70`		`- public function bindValue(Cell $cell, $value): bool`
	`70`	`+ public function bindValue(Cell $cell, mixed $value): bool`
`71`	`71`	`{`
`72`	`72`	`if (is_object($value)) {`
`73`	`73`	`return $this->bindObjectValue($cell, $value);`
Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ class Chart`
`126`	`126`	`* Create a new Chart.`
`127`	`127`	`* majorGridlines and minorGridlines are deprecated, moved to Axis.`
`128`	`128`	`*/`
`129`		`- public function __construct(mixed $name, ?Title $title = null, ?Legend $legend = null, ?PlotArea $plotArea = null, mixed $plotVisibleOnly = true, string $displayBlanksAs = DataSeries::EMPTY_AS_GAP, ?Title $xAxisLabel = null, ?Title $yAxisLabel = null, ?Axis $xAxis = null, ?Axis $yAxis = null, ?GridLines $majorGridlines = null, ?GridLines $minorGridlines = null)`
	`129`	`+ public function __construct(string $name, ?Title $title = null, ?Legend $legend = null, ?PlotArea $plotArea = null, bool $plotVisibleOnly = true, string $displayBlanksAs = DataSeries::EMPTY_AS_GAP, ?Title $xAxisLabel = null, ?Title $yAxisLabel = null, ?Axis $xAxis = null, ?Axis $yAxis = null, ?GridLines $majorGridlines = null, ?GridLines $minorGridlines = null)`
`130`	`130`	`{`
`131`	`131`	`$this->name = $name;`
`132`	`132`	`$this->title = $title;`
Original file line number	Diff line number	Diff line change
`@@ -232,7 +232,7 @@ public function getPlotLabels(): array`
`232`	`232`	`*`
`233`	`233`	`* @return DataSeriesValues\|false`
`234`	`234`	`*/`
`235`		`- public function getPlotLabelByIndex(mixed $index): bool\|DataSeriesValues`
	`235`	`+ public function getPlotLabelByIndex(int $index): bool\|DataSeriesValues`
`236`	`236`	`{`
`237`	`237`	`$keys = array_keys($this->plotLabel);`
`238`	`238`	`if (in_array($index, $keys)) {`
`@@ -257,7 +257,7 @@ public function getPlotCategories(): array`
`257`	`257`	`*`
`258`	`258`	`* @return DataSeriesValues\|false`
`259`	`259`	`*/`
`260`		`- public function getPlotCategoryByIndex(mixed $index): bool\|DataSeriesValues`
	`260`	`+ public function getPlotCategoryByIndex(int $index): bool\|DataSeriesValues`
`261`	`261`	`{`
`262`	`262`	`$keys = array_keys($this->plotCategory);`
`263`	`263`	`if (in_array($index, $keys)) {`
`@@ -304,7 +304,7 @@ public function getPlotValues(): array`
`304`	`304`	`*`
`305`	`305`	`* @return DataSeriesValues\|false`
`306`	`306`	`*/`
`307`		`- public function getPlotValuesByIndex(mixed $index): bool\|DataSeriesValues`
	`307`	`+ public function getPlotValuesByIndex(int $index): bool\|DataSeriesValues`
`308`	`308`	`{`
`309`	`309`	`$keys = array_keys($this->plotValues);`
`310`	`310`	`if (in_array($index, $keys)) {`
Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ public function getPlotGroup(): array`
`88`	`88`	`/**`
`89`	`89`	`* Get Plot Series by Index.`
`90`	`90`	`*/`
`91`		`- public function getPlotGroupByIndex(mixed $index): DataSeries`
	`91`	`+ public function getPlotGroupByIndex(int $index): DataSeries`
`92`	`92`	`{`
`93`	`93`	`return $this->plotSeries[$index];`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -403,7 +403,7 @@ protected function getShadowPresetsMap(int $presetsOption): array`
`403`	`403`	`/**`
`404`	`404`	`* Get value of array element.`
`405`	`405`	`*/`
`406`		`- protected function getArrayElementsValue(mixed $properties, mixed $elements): mixed`
	`406`	`+ protected function getArrayElementsValue(array $properties, array\|int\|string $elements): mixed`
`407`	`407`	`{`
`408`	`408`	`$reference = &$properties;`
`409`	`409`	`if (!is_array($elements)) {`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ class Dimension`
`56`	`56`	`* Phpstan bug has been fixed; this function allows us to`
`57`	`57`	`* pass Phpstan whether fixed or not.`
`58`	`58`	`*/`
`59`		`- private static function stanBugFixed(mixed $value): array`
	`59`	`+ private static function stanBugFixed(array\|int\|null $value): array`
`60`	`60`	`{`
`61`	`61`	`return is_array($value) ? $value : [null, null];`
`62`	`62`	`}`