Skip to content

Commit 078846c

Browse files
oleibmanoleibman
committed
Phpstan Level 9 - Part 8 of Many - TextData
1 parent 729c4dc commit 078846c

File tree

7 files changed

+29
-137
lines changed

7 files changed

+29
-137
lines changed

‎phpstan-baseline.neon

-114
Original file line numberDiff line numberDiff line change
@@ -419,117 +419,3 @@ parameters:
419419
identifier: argument.type
420420
count: 9
421421
path: src/PhpSpreadsheet/Calculation/Statistical/Trends.php
422-
423-
-
424-
message: '#^Method PhpOffice\\PhpSpreadsheet\\Calculation\\TextData\\CharacterConvert\:\:unicodeToOrd\(\) should return int but returns mixed\.$#'
425-
identifier: return.type
426-
count: 1
427-
path: src/PhpSpreadsheet/Calculation/TextData/CharacterConvert.php
428-
429-
-
430-
message: '#^Binary operation "\." between mixed and mixed results in an error\.$#'
431-
identifier: binaryOp.invalid
432-
count: 1
433-
path: src/PhpSpreadsheet/Calculation/TextData/Concatenate.php
434-
435-
-
436-
message: '#^Cannot cast mixed to string\.$#'
437-
identifier: cast.string
438-
count: 1
439-
path: src/PhpSpreadsheet/Calculation/TextData/Concatenate.php
440-
441-
-
442-
message: '#^Parameter \#1 \$ignoreEmpty of static method PhpOffice\\PhpSpreadsheet\\Calculation\\TextData\\Concatenate\:\:evaluateTextJoinArray\(\) expects bool, mixed given\.$#'
443-
identifier: argument.type
444-
count: 1
445-
path: src/PhpSpreadsheet/Calculation/TextData/Concatenate.php
446-
447-
-
448-
message: '#^Parameter \#1 \$separator of function implode expects array\|string, mixed given\.$#'
449-
identifier: argument.type
450-
count: 1
451-
path: src/PhpSpreadsheet/Calculation/TextData/Concatenate.php
452-
453-
-
454-
message: '#^Cannot cast mixed to int\.$#'
455-
identifier: cast.int
456-
count: 6
457-
path: src/PhpSpreadsheet/Calculation/TextData/Extract.php
458-
459-
-
460-
message: '#^Method PhpOffice\\PhpSpreadsheet\\Calculation\\TextData\\Extract\:\:validateTextBeforeAfter\(\) should return array\|string but returns mixed\.$#'
461-
identifier: return.type
462-
count: 1
463-
path: src/PhpSpreadsheet/Calculation/TextData/Extract.php
464-
465-
-
466-
message: '#^Cannot cast mixed to string\.$#'
467-
identifier: cast.string
468-
count: 3
469-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
470-
471-
-
472-
message: '#^Method PhpOffice\\PhpSpreadsheet\\Calculation\\TextData\\Format\:\:VALUE\(\) should return array\|DateTimeInterface\|float\|int\|string but returns mixed\.$#'
473-
identifier: return.type
474-
count: 2
475-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
476-
477-
-
478-
message: '#^Parameter \#1 \$dateValue of static method PhpOffice\\PhpSpreadsheet\\Calculation\\DateTimeExcel\\DateValue\:\:fromString\(\) expects array\|bool\|float\|int\|string\|null, mixed given\.$#'
479-
identifier: argument.type
480-
count: 1
481-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
482-
483-
-
484-
message: '#^Parameter \#1 \$haystack of function str_contains expects string, mixed given\.$#'
485-
identifier: argument.type
486-
count: 1
487-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
488-
489-
-
490-
message: '#^Parameter \#1 \$haystack of function strpos expects string, mixed given\.$#'
491-
identifier: argument.type
492-
count: 1
493-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
494-
495-
-
496-
message: '#^Parameter \#1 \$string of function trim expects string, mixed given\.$#'
497-
identifier: argument.type
498-
count: 1
499-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
500-
501-
-
502-
message: '#^Parameter \#1 \$timeValue of static method PhpOffice\\PhpSpreadsheet\\Calculation\\DateTimeExcel\\TimeValue\:\:fromString\(\) expects array\|bool\|float\|int\|string\|null, mixed given\.$#'
503-
identifier: argument.type
504-
count: 1
505-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
506-
507-
-
508-
message: '#^Parameter \#2 \$subject of static method Composer\\Pcre\\Preg\:\:matchAllWithOffsets\(\) expects string, mixed given\.$#'
509-
identifier: argument.type
510-
count: 1
511-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
512-
513-
-
514-
message: '#^Parameter \#3 \$subject of function str_replace expects array\<string\>\|string, mixed given\.$#'
515-
identifier: argument.type
516-
count: 2
517-
path: src/PhpSpreadsheet/Calculation/TextData/Format.php
518-
519-
-
520-
message: '#^Cannot cast mixed to int\.$#'
521-
identifier: cast.int
522-
count: 1
523-
path: src/PhpSpreadsheet/Calculation/TextData/Helpers.php
524-
525-
-
526-
message: '#^Cannot cast mixed to string\.$#'
527-
identifier: cast.string
528-
count: 1
529-
path: src/PhpSpreadsheet/Calculation/TextData/Helpers.php
530-
531-
-
532-
message: '#^Method PhpOffice\\PhpSpreadsheet\\Calculation\\TextData\\Replace\:\:executeSubstitution\(\) should return string but returns mixed\.$#'
533-
identifier: return.type
534-
count: 1
535-
path: src/PhpSpreadsheet/Calculation/TextData/Replace.php

‎src/PhpSpreadsheet/Calculation/TextData/CharacterConvert.php

+1
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ private static function unicodeToOrd(string $character): int
8181
$retVal = 0;
8282
$iconv = iconv('UTF-8', 'UCS-4LE', $character);
8383
if ($iconv !== false) {
84+
/** @var false|int[] */
8485
$result = unpack('V', $iconv);
8586
if (is_array($result) && isset($result[1])) {
8687
$retVal = $result[1];

‎src/PhpSpreadsheet/Calculation/TextData/Concatenate.php

+7-6
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@ private static function concatenate2Args(array|string $operand1, null|array|bool
7878
return $operand2[$row][$column];
7979
}
8080
$operand1[$row][$column]
81-
= Calculation::boolToString($operand1[$row][$column])
82-
. Calculation::boolToString($operand2[$row][$column]);
81+
= StringHelper::convertToString($operand1[$row][$column], convertBool: true)
82+
. StringHelper::convertToString($operand2[$row][$column], convertBool: true);
8383
if (mb_strlen($operand1[$row][$column]) > DataType::MAX_STRING_LENGTH) {
8484
$operand1 = ExcelError::CALC();
8585
$errorFound = true;
@@ -91,7 +91,7 @@ private static function concatenate2Args(array|string $operand1, null|array|bool
9191
} elseif (ErrorValue::isError($operand2, true) === true) {
9292
$operand1 = (string) $operand2;
9393
} else {
94-
$operand1 .= (string) Calculation::boolToString($operand2);
94+
$operand1 .= StringHelper::convertToString($operand2, convertBool: true);
9595
if (mb_strlen($operand1) > DataType::MAX_STRING_LENGTH) {
9696
$operand1 = ExcelError::CALC();
9797
}
@@ -103,17 +103,17 @@ private static function concatenate2Args(array|string $operand1, null|array|bool
103103
/**
104104
* TEXTJOIN.
105105
*
106-
* @param mixed $delimiter The delimter to use between the joined arguments
106+
* @param null|string|string[] $delimiter The delimiter to use between the joined arguments
107107
* Or can be an array of values
108-
* @param mixed $ignoreEmpty true/false Flag indicating whether empty arguments should be skipped
108+
* @param null|bool|bool[] $ignoreEmpty true/false Flag indicating whether empty arguments should be skipped
109109
* Or can be an array of values
110110
* @param mixed $args The values to join
111111
*
112112
* @return array|string The joined string
113113
* If an array of values is passed for the $delimiter or $ignoreEmpty arguments, then the returned result
114114
* will also be an array with matching dimensions
115115
*/
116-
public static function TEXTJOIN(mixed $delimiter = '', mixed $ignoreEmpty = true, mixed ...$args): array|string
116+
public static function TEXTJOIN($delimiter = '', $ignoreEmpty = true, mixed ...$args): array|string
117117
{
118118
if (is_array($delimiter) || is_array($ignoreEmpty)) {
119119
return self::evaluateArrayArgumentsSubset(
@@ -127,6 +127,7 @@ public static function TEXTJOIN(mixed $delimiter = '', mixed $ignoreEmpty = true
127127

128128
$delimiter ??= '';
129129
$ignoreEmpty ??= true;
130+
/** @var array */
130131
$aArgs = Functions::flattenArray($args);
131132
$returnValue = self::evaluateTextJoinArray($ignoreEmpty, $aArgs);
132133

‎src/PhpSpreadsheet/Calculation/TextData/Extract.php

+7-7
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,9 @@ public static function before(mixed $text, $delimiter, mixed $instance = 1, mixe
139139
return $e->getMessage();
140140
}
141141

142-
$instance = (int) $instance;
143-
$matchMode = (int) $matchMode;
144-
$matchEnd = (int) $matchEnd;
142+
$instance = (int) StringHelper::convertToString($instance);
143+
$matchMode = (int) StringHelper::convertToString($matchMode);
144+
$matchEnd = (int) StringHelper::convertToString($matchEnd);
145145

146146
$split = self::validateTextBeforeAfter($text, $delimiter, $instance, $matchMode, $matchEnd, $ifNotFound);
147147
if (is_string($split)) {
@@ -203,9 +203,9 @@ public static function after(mixed $text, $delimiter, mixed $instance = 1, mixed
203203
return $e->getMessage();
204204
}
205205

206-
$instance = (int) $instance;
207-
$matchMode = (int) $matchMode;
208-
$matchEnd = (int) $matchEnd;
206+
$instance = (int) StringHelper::convertToString($instance);
207+
$matchMode = (int) StringHelper::convertToString($matchMode);
208+
$matchEnd = (int) StringHelper::convertToString($matchEnd);
209209

210210
$split = self::validateTextBeforeAfter($text, $delimiter, $instance, $matchMode, $matchEnd, $ifNotFound);
211211
if (is_string($split)) {
@@ -234,7 +234,7 @@ private static function validateTextBeforeAfter(string $text, null|array|string
234234
$delimiter = self::buildDelimiter($delimiter);
235235

236236
if (preg_match('/' . $delimiter . "/{$flags}", $text) === 0 && $matchEnd === 0) {
237-
return $ifNotFound;
237+
return is_array($ifNotFound) ? $ifNotFound : StringHelper::convertToString($ifNotFound);
238238
}
239239

240240
$split = preg_split('/' . $delimiter . "/{$flags}", $text, 0, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);

‎src/PhpSpreadsheet/Calculation/TextData/Format.php

+10-7
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,7 @@ public static function VALUE(mixed $value = '')
192192
return $e->getMessage();
193193
}
194194
if (!is_numeric($value)) {
195+
$value = StringHelper::convertToString($value);
195196
$numberValue = str_replace(
196197
StringHelper::getThousandsSeparator(),
197198
'',
@@ -212,14 +213,14 @@ public static function VALUE(mixed $value = '')
212213
if ($timeValue !== ExcelError::VALUE()) {
213214
Functions::setReturnDateType($dateSetting);
214215

215-
return $timeValue;
216+
return $timeValue; //* @phpstan-ignore-line
216217
}
217218
}
218219
$dateValue = Functions::scalar(DateTimeExcel\DateValue::fromString($value));
219220
if ($dateValue !== ExcelError::VALUE()) {
220221
Functions::setReturnDateType($dateSetting);
221222

222-
return $dateValue;
223+
return $dateValue; //* @phpstan-ignore-line
223224
}
224225
Functions::setReturnDateType($dateSetting);
225226

@@ -250,23 +251,23 @@ public static function valueToText(mixed $value, mixed $format = false): array|s
250251
$value = $value->getPlainText();
251252
}
252253
if (is_string($value)) {
253-
$value = ($format === true) ? Calculation::wrapResult($value) : $value;
254+
$value = ($format === true) ? StringHelper::convertToString(Calculation::wrapResult($value)) : $value;
254255
$value = str_replace("\n", '', $value);
255256
} elseif (is_bool($value)) {
256257
$value = Calculation::getLocaleBoolean($value ? 'TRUE' : 'FALSE');
257258
}
258259

259-
return (string) $value;
260+
return StringHelper::convertToString($value);
260261
}
261262

262263
private static function getDecimalSeparator(mixed $decimalSeparator): string
263264
{
264-
return empty($decimalSeparator) ? StringHelper::getDecimalSeparator() : (string) $decimalSeparator;
265+
return empty($decimalSeparator) ? StringHelper::getDecimalSeparator() : StringHelper::convertToString($decimalSeparator);
265266
}
266267

267268
private static function getGroupSeparator(mixed $groupSeparator): string
268269
{
269-
return empty($groupSeparator) ? StringHelper::getThousandsSeparator() : (string) $groupSeparator;
270+
return empty($groupSeparator) ? StringHelper::getThousandsSeparator() : StringHelper::convertToString($groupSeparator);
270271
}
271272

272273
/**
@@ -293,7 +294,9 @@ public static function NUMBERVALUE(mixed $value = '', mixed $decimalSeparator =
293294
return $e->getMessage();
294295
}
295296

296-
if (!is_numeric($value)) {
297+
/** @var null|array|scalar $value */
298+
if (!is_array($value) && !is_numeric($value)) {
299+
$value = StringHelper::convertToString($value);
297300
$decimalPositions = Preg::matchAllWithOffsets('/' . preg_quote($decimalSeparator, '/') . '/', $value, $matches);
298301
if ($decimalPositions > 1) {
299302
return ExcelError::VALUE();

‎src/PhpSpreadsheet/Calculation/TextData/Helpers.php

+3-2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
use PhpOffice\PhpSpreadsheet\Calculation\Functions;
88
use PhpOffice\PhpSpreadsheet\Calculation\Information\ErrorValue;
99
use PhpOffice\PhpSpreadsheet\Calculation\Information\ExcelError;
10+
use PhpOffice\PhpSpreadsheet\Shared\StringHelper;
1011

1112
class Helpers
1213
{
@@ -31,7 +32,7 @@ public static function extractString(mixed $value, bool $throwIfError = false):
3132
throw new CalcExp($value);
3233
}
3334

34-
return (string) $value;
35+
return StringHelper::convertToString($value);
3536
}
3637

3738
public static function extractInt(mixed $value, int $minValue, int $gnumericNull = 0, bool $ooBoolOk = false): int
@@ -87,6 +88,6 @@ public static function validateInt(mixed $value, bool $throwIfError = false): in
8788
throw new CalcExp($value);
8889
}
8990

90-
return (int) $value;
91+
return (int) StringHelper::convertToString($value);
9192
}
9293
}

‎src/PhpSpreadsheet/Calculation/TextData/Replace.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,6 @@ private static function executeSubstitution(string $text, string $fromText, stri
111111
--$instance;
112112
}
113113

114-
return Functions::scalar(self::REPLACE($text, ++$pos, StringHelper::countCharacters($fromText), $toText));
114+
return StringHelper::convertToString(Functions::scalar(self::REPLACE($text, ++$pos, StringHelper::countCharacters($fromText), $toText)));
115115
}
116116
}

0 commit comments

Comments
 (0)