Removing 'utf8_bom' setting value requirement

Created on 10 May 2019, over 5 years ago
Updated 17 November 2023, about 1 year ago

When exporting CSV from views via the `views_data_export`, the generated files are receiving no BOM. This is happening because the `csv_serialization` module is requiring that a 'utf8_bom' setting value be provided. However, all the `csv_serialization` module does with this value is confirm that it was passed; i.e. at no point does `csv_serialization` actually use the value, and is instead using BOM_UTF8 constant from vendor/league/csv/src/AbstractCsv.php. This means that even though `views_data_export` is correctly specifying 'utf8' for the 'encoding' setting, the resulting file is not properly encoded. Withal, this patch removes the 'utf8_bom' setting requirement.

Note that the 'utf8_bom' setting being a requirement appears to have come from issue 2859030 β†’ .

πŸ› Bug report
Status

Closed: duplicate

Version

1.4

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States emanaton

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024