Support PHP 8.2 league/csv

Patch file provided for anyone that may need it.

The last submitted patch, 4: migrate_source_csv-3406342.patch, failed testing. View results →

pookmish → changed the visibility of the branch 3406342-support-php-8.2 to hidden.

This fix not working for me.

Patch #7 fixes the problem for me, but for a different situation: migration source is a CSV file with no headers, and PHP 8.1.26.

Confirmed.
Upgraded to PHP 8.2 and Drupal 10.1.7

Suddenly CSV could not be read throwing this error:

[error] Could not retrieve source count from migrate_ct_open_rec: The header mapper indexes should only contain positive integer or 0.

While it should display the migration definition using drush ms
It would error every time and not count records in the source file nor updates or new records.

Applied this patch and now it reads the file.

Can issue commands again:
drush mr test_migration_name
drush ms test_migration_name --update --sync

etc.

Had the same issue, and can confirm that changing array_flip to array_keys solved it.

I can also confirm the patch at #7 fixes the issue. (same issue as described in #10 but using PHP 8.1 and Drupal 10.1.7)

I had the same issue

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).

using PHP 8.1.22, Drupal 10.1.7, migrate_source_csv 8.x-3.5, league/csv 9.12.0

The patch at #7 fixed the issue for me as well. Thanks!

Works great in Drupal 10.2 with PHP 8.2, thanks @pookmish. Raising to Major, since the module is currently broken. Can it be committed?

+1 We are having the same issue.

+1

Thanks for the fixes here.

Thanks @heddn, the new 8.x-3.6 release works perfectly.

Automatically closed - issue fixed for 2 weeks with no activity.