Support PHP 8.2 league/csv

Created on 23 January 2025, 2 months ago

Problem/Motivation

When using PHP 8.2 and the latest version of league/csv, an error is thrown due to the array structure of the header.

Steps to reproduce

Enable php 8.2
create a simple csv with a header row and a single record
Attempt to import
Observe the error in the database logs like:

League\Csv\SyntaxError: The header mapper indexes should only contain positive integer or 0. in League\Csv\Reader->computeHeader() (line 448 of vendor/league/csv/src/Reader.php).

Proposed resolution

Check for the header first. Same fix as in the CSV source: https://www.drupal.org/project/migrate_source_csv/issues/3406342 🐛 Support PHP 8.2 league/csv Fixed

🐛 Bug report

Status

Active

Version

1.0

Component

Code

Created by

🇧🇪Belgium dave.mentens

Live updates comments and jobs are added and updated live.

Merge Requests

!1Support PHP 8.2 league/csv
Open
🇧🇪Belgium dave.mentens
updated 2 months ago

Comments & Activities

Issue created by @dave.mentens
Merge request !1Issue #3501715: Correct header array for league/csv and PHP 8.2. → (Open) created by dave.mentens
Comment 2 months ago →
🇧🇪Belgium dave.mentens
Comment 2 months ago →
🇧🇪Belgium dave.mentens

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024