Closed (outdated)
Project:
Drupal core
Version:
8.4.0-rc1
Component:
migration system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Sep 2017 at 11:56 UTC
Updated:
7 Apr 2019 at 07:48 UTC
Jump to comment: Most recent
Comments
Comment #2
heddnI'm pretty sure we fixed this in the past few years. Can someone confirm?
Comment #3
quietone commentedYes, this has been fixed, sort of.
The migration now has this, which will limit the length of the id. However, is uses a length of 32 and should be less than 32, to allow characters for the string that makes the field unique. Thats because te MakeUniqueBase process plugin gets the substr using length before adding the string that makes it unque.
This length was added in #2247903: Add a substr options to DedupeBase process plugin to several migrations.
So, I think this can be close and we open an issue to check all the migrations that use length 32. Of course make_unique_entity_field should also be a bit smarter. Made a new issue to look into that #3046397: Change length in make_unique_entity_field to less than 32
Comment #4
quietone commentedClosing