PhpSpreadsheet/tests/PhpSpreadsheetTests/Calculation
oleibman faf6d819c6
Keep Calculated String Results Below 32K (#2921)
* Keep Calculated String Results Below 32K

This is the result of an investigation into issue #2884 (see also PR #2913). It is, unfortunately, not a fix for the original problem; see the discussion in that PR for why I don't think there is a practical fix for that specific problem at this time.

Excel limits strings to 32,767 characters. We already truncate strings to that length when added to the spreadsheet. However, we have been able to exceed that length as a result of the concatenation operator (Excel truncates); as a result of the CONCATENATE or TEXTJOIN functions (Excel returns #CALC!); or as a result of the REPLACE, REPT, SUBSTITUTE functions (Excel returns #VALUE!). This PR changes PhpSpreadsheet to return the same value as Excel in these cases. Note that Excel2003 truncates in all those cases; I don't think there is a way to differentiate that behavior in PhpSpreadsheet.

However, LibreOffice and Gnumeric do not have that limit; if they have a limit at all, it is much higher. It would be fairly easy to use existing settings to differentiate between Excel and LibreOffice/Gnumeric in this respect. I have not done so in this PR because I am not sure how useful that is, and I can easily see it leading to problems (read in a LibreOffice spreadsheet with a 33K cell and then output to an Excel spreadsheet). Perhaps it should be handled with an additional opt-in setting.

I changed the maximum size from a literal to a constant in the one place where it was already being enforced (Cell/DataType). I am not sure that is the best place for it to be defined; I am open to suggestions.

* Implement Some Suggestions

... from @MarkBaker.
2022-07-04 08:30:46 -07:00
..
Engine Handle defined names with the range operator. 2022-04-13 17:55:51 +02:00
Functions Handling of #REF! Errors in Subtotal, and More (#2902) 2022-06-25 22:08:32 -07:00
ArrayFormulaTest.php Convert all relevant Logical functions to support array arguments (#2600) 2022-02-18 02:56:23 +01:00
BinaryComparisonTest.php Extract some methods from the Calculation Engine into dedicated classes (#2537) 2022-02-04 16:02:29 +01:00
CalculationErrorTest.php New Error Reported with Phpstan 1.3 (#2481) 2022-01-03 17:32:52 -08:00
CalculationFunctionListTest.php Eliminate Most Scrutinizer Problems in Test Suite (#2699) 2022-03-21 13:58:42 -07:00
CalculationLoggingTest.php Unit test for the Calculation Engine debug log 2022-03-06 09:47:17 +01:00
CalculationSettingsTest.php Adjust Cell Reference regexp in Calculation Engine to handle worksheet names containing quotes (#2617) 2022-02-22 15:55:59 +01:00
CalculationTest.php Adjust Cell Reference regexp in Calculation Engine to handle worksheet names containing quotes (#2617) 2022-02-22 15:55:59 +01:00
DefinedNameConfusedForCellTest.php Reduce PHPStan error in tests 2021-04-12 11:10:23 +09:00
DefinedNamesCalculationTest.php Named formula implementation, and improved handling of Defined Names generally (#1535) 2020-07-26 12:00:06 +02:00
FormulaAsStringTest.php Reduce PHPStan error in tests 2021-04-12 11:10:23 +09:00
FunctionsTest.php Eliminate Most Scrutinizer Problems in Test Suite (#2699) 2022-03-21 13:58:42 -07:00
MergedCellTest.php Relax validation on merge cells to allow input of a single cell 2022-06-10 01:15:57 +02:00
NullEqualsZeroTest.php Add Test For NULL=0 (#2607) 2022-02-19 13:12:44 -08:00
ParseFormulaTest.php Escape double quotes in worksheet names for column range and row range references 2022-06-17 13:34:20 +02:00
RefErrorTest.php Handling of #REF! Errors in Subtotal, and More (#2902) 2022-06-25 22:08:32 -07:00
RowColumnReferenceTest.php Set precision for Database function unit tests, for RowColumnReference and Dimension Helper unit tests (#2604) 2022-02-18 21:37:26 +01:00
StringLengthTest.php Keep Calculated String Results Below 32K (#2921) 2022-07-04 08:30:46 -07:00
TranslationTest.php Reduce PHPStan error in tests 2021-04-12 11:10:23 +09:00
XlfnFunctionsTest.php Reduce PHPStan error in tests 2021-04-12 11:10:23 +09:00