- πΊπΈUnited States laura.gates
Just to note on 9.5.x that this patch introduces this fatal error. Porting this patch will need to be done in a way that also doesn't break 9.4.x or 9.5.x
I'm working on my D10 upgrade but haven't flipped the switch yet.
- πΊπ¦Ukraine voleger Ukraine, Rivne
This is a duplicate issue of π [Drupal 10, Drush 11] PHP Fatal error on install Closed: duplicate
- π«π·France GuillaumeDuveau Toulouse
https://git.drupalcode.org/project/csv_serialization/-/merge_requests/12 in this issue is more complete than https://git.drupalcode.org/project/csv_serialization/-/merge_requests/10... on the issue https://www.drupal.org/project/csv_serialization/issues/3317133 π [Drupal 10, Drush 11] PHP Fatal error on install Closed: duplicate , because it also fixes decode.
- π¦πΊAustralia fenstrat Australia
Confirming this is RTBC.
Also marked π [Drupal 10, Drush 11] PHP Fatal error on install Closed: duplicate as a duplicate of this.
Attached is a patch version of the MR for use in composer.patches
- πΊπΈUnited States jennypanighetti
I'm on 9.5.3 and cannot get this patch to apply. Not sure why though...
I have tested and applied the patch for Drupal 10 setup. This #11 patch works on Drupal 10. Thank you.
- π«π·France benjbmc
Patch in #11 is working for me on D10.0.2, and my export tests are fine in CSV and XLSX (with contrib modules Views Data Export and Excel Serialization). Thanks
- π¨π΄Colombia jidrone
I tested on Drupal 10.0.2, and it is working as expected.
- πΊπΈUnited States scotwith1t Birmingham, AL
+1 for RTBC (Hey @jidrone!! Long time no see!)
- πΊπΈUnited States markdorison
I have offered to co-maintain this module in order to get this and any other D10-related issues committed and packaged for a D10 release. I have reached out to the maintainer directly as well.
π¬ Offering to co-maintain CSV Serialization Fixed
- Status changed to Closed: duplicate
over 1 year ago 6:34pm 23 February 2023 - πΊπΈUnited States markdorison
These changes were already committed to the 3.x branch here β . Look out for a D10 release for that branch soon.
I still have the same error when I updated this module for a D10 release
PHP Fatal error: Declaration of Drupal\csv_serialization\Encoder\CsvEncoder::encode($data, string $format, array $context = []): string must be compatible with Symfony\Component\Serializer\Encoder\EncoderInterface::encode($data, $format, array $context = []) in C:\work\drupal10\web\modules\contrib\csv_serialization\src\Encoder\CsvEncoder.php on line 125 [warning] Drush command terminated abnormally.
- πΊπΈUnited States markdorison
These changes seem to actually be causing this issue on D9. Detailed in π CsvEncoder::encode must be compatible with Symfony\Component\Serializer\Encoder\EncoderInterface::encode Fixed