>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3413894. --> Reported by: [caesius](https://www.drupal.org/user/1807592) Related to !8 >>> <p>Per #8 Drupal core starting with 10.2 does not require that migrate IDs not include spaces. Since this has never effected migrations except when needing to roll back, we probably don't need to update the minimum Drupal version in the composer file, especially since 10.1 is already EOL.</p> <p>Revert <span class="drupalorg-gitlab-issue-link drupalorg-gitlab-link-wrapper"><a href="https://git.drupalcode.org/project/migrate_source_csv/-/work_items/3379184" class="drupalorg-gitlab-link">https://git.drupalcode.org/project/migrate_source_csv/-/work_items/3379184</a></span></p> <p><em>Original description:</em></p> <p>In the parent issue, a change was implemented to disallow spaces in source IDs. However, this change was not documented in an explicit change record, and existing migrations which have source IDs containing spaces will break.</p> <p>There is no documentation regarding exactly how to fix migrations affected by this issue. Can the CSVs continue to use column names with spaces while the migration configuration needs the spaces deleted? Or do both the original CSV and the migrate configuration need to remove spaces?</p> <p>E.g., would a CSV that has a "Some Column" column name properly map to a "SomeColumn" ID? Or would that not work?</p> <p>Since this change leads to a minor version upgrade breaking previously-working migrations (a change which maybe should've waited for a major version increment), it's pretty important to get this properly documented.</p> <p>Additionally, if this update completely prohibits the use of CSVs that includes spaces, it would be preferable to have a code solution to allow spaces in CSV column names and simply remove spaces before mapping columns to source IDs, since updating how CSV headers are exported from a particular source may not be trivial or possible. In these cases manual edits to CSV headers would be needed.</p> > Related issue: [Issue #3379184](https://www.drupal.org/node/3379184)