mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-08-26 08:38:34 +00:00
oleibman
33eefe7c9d
* Resolve Phpstan Messages - FINALE - Shared/OLE Reduce number of Phpstan messages by addressing their issues. This is the last of a series of related tickets to achieve that end. Only a handful of messages will be left in Phpstan baseline after this change is merged. Shared/Ole.php and Shared/Ole/ChainedBlockStream.php were completely uncovered in the test suite. So I wrote some tests and found they didn't work at all. There were 2 problems. The first was that ChainedBlockStream checks for a mode of `r`, but Ole invokes it with mode `rb`. ChainedBlockStream will now just check the first character of the mode. The second was a more interesting problem. Ole was reading "an *unsigned* long". However, what it should have been reading was "a 32-bit *signed* long". The irony of this mismatch is that, once the mode problem was corrected, 32-bit Php handled Ole and ChainedBlockStream correctly, but 64-bit Php did not. The code is corrected to work properly for both 32- and 64-bit. * Scrutinizer 2 dead assignments - sufficient to just make calls which are expected to fail.
27 lines
1.1 KiB
Plaintext
27 lines
1.1 KiB
Plaintext
parameters:
|
|
ignoreErrors:
|
|
-
|
|
message: "#^Cannot call method getTokenSubType\\(\\) on PhpOffice\\\\PhpSpreadsheet\\\\Calculation\\\\FormulaToken\\|null\\.$#"
|
|
count: 4
|
|
path: src/PhpSpreadsheet/Calculation/FormulaParser.php
|
|
|
|
-
|
|
message: "#^Cannot call method getTokenType\\(\\) on PhpOffice\\\\PhpSpreadsheet\\\\Calculation\\\\FormulaToken\\|null\\.$#"
|
|
count: 9
|
|
path: src/PhpSpreadsheet/Calculation/FormulaParser.php
|
|
|
|
-
|
|
message: "#^Cannot call method setTokenSubType\\(\\) on PhpOffice\\\\PhpSpreadsheet\\\\Calculation\\\\FormulaToken\\|null\\.$#"
|
|
count: 5
|
|
path: src/PhpSpreadsheet/Calculation/FormulaParser.php
|
|
|
|
-
|
|
message: "#^Cannot call method setValue\\(\\) on PhpOffice\\\\PhpSpreadsheet\\\\Calculation\\\\FormulaToken\\|null\\.$#"
|
|
count: 5
|
|
path: src/PhpSpreadsheet/Calculation/FormulaParser.php
|
|
|
|
-
|
|
message: "#^Strict comparison using \\=\\=\\= between PhpOffice\\\\PhpSpreadsheet\\\\Calculation\\\\FormulaToken and null will always evaluate to false\\.$#"
|
|
count: 1
|
|
path: src/PhpSpreadsheet/Calculation/FormulaParser.php
|