- Issue created by @RoSk0
- First commit to issue fork.
- last update
over 1 year ago Composer require failure - @ranjit1032002 opened merge request.
- last update
over 1 year ago Composer require failure - Status changed to Needs review
over 1 year ago 5:47am 18 May 2023 - 🇮🇳India Ranjit1032002
Created MR!9 for the issue mentioned, please review.
Thank you. -
theemstra →
committed df5e4ffd on 8.x-1.x authored by
Ranjit1032002 →
Issue #3358781: Replace setCellValueByColumnAndRow by setCellValue()...
-
theemstra →
committed df5e4ffd on 8.x-1.x authored by
Ranjit1032002 →
- Status changed to Fixed
over 1 year ago 11:38am 29 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
10 months ago 6:39pm 3 March 2024 - 🇩🇪Germany gngn
The commited patch did not work for me.
I got the error:Uncaught PHP Exception TypeError: "PhpOffice\\PhpSpreadsheet\\Worksheet\\Worksheet::setCellValue(): Argument #3 ($binder) must be of type ?PhpOffice\\PhpSpreadsheet\\Cell\\IValueBinder, string given, called in modules/contrib/xls_serialization/src/Encoder/Xls.php on line 171" at /vendor/phpoffice/phpspreadsheet/src/PhpSpreadsheet/Worksheet/Worksheet.php line 1176
That is because
Worksheet::setCellValue($coordinate, $value, ?IValueBinder $binder = null)
expects the first parameter $coordinate to be:
array|CellAddress|string $coordinate Coordinate of the cell as a string, eg: 'C5'; or as an array of [$columnIndex, $row] (e.g. [3, 5]), or a CellAddress object.So we need to change:
- $sheet->setCellValue($column, $i + 2, $this->formatValue($value)); + $sheet->setCellValue([$column, $i + 2], $this->formatValue($value));
This is done in 🐛 Tests are failing on 10.1 Fixed . So you need to also apply the patch from this issue (after the patch from here).