Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In user_user_role_insert() we create system actions to remove and add the role. This means that we will get unnecessary config conflicts on UUIDs - we should provide defaults.
This change depends on #2127573: Stop creating node, comment and custom block fields automatically and provide defaults in CMI to fully function as this sets the sync flag properly on module install - but this should not affect our tests atm.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2127583.1.patch | 2.39 KB | alexpott |
Comments
Comment #1
alexpottComment #2
alexpottComment #3
vijaycs85Looks good to me. +1 to RTBC.
Comment #4
gddI think this is great but I expressed some concerns with isSyncing() over in #2127573: Stop creating node, comment and custom block fields automatically and provide defaults in CMI
Comment #5
alexpott1: 2127583.1.patch queued for re-testing.
Comment #6
xjmComment #7
xjmComment #8
alexpottDiscussed with yched, heyrocker, xjm, and mtift. We need to be able to support programmatic creation of config entities on install time due to the following scenario:
Agreed to close this issue in favour of exploring other solutions which will be documented on #2121751: [META] Making configuration synchronisation work
Comment #9
xjm