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.
chx, perhaps we need to make this a more general issue, with a title like 'Allow modules to be imported by the configuration system' ?
i've thought about this a bit, and i'm pretty sure we'll need to special case module changes during import. probably via a pass that handles modules first, then the rest of the changes.
i'm happy to open another issue if this is not what you had in mind for this issue.
@chx I've trying to figure a case: we have module node and comment (comment does not depends in node after #1790764: [meta] Convert comment module to field_api,
1) both disabled, 2) user calls to enable both (without dependency the order is unpredictable) 3) modules are trying to find each other state to install/update something... - seem rc posible
Comments
Comment #1
andypostthis could lead to race-conditions, also I cares only about install module moment
Comment #2
chx CreditAttribution: chx commentedI have no idea what do you mean. Importing config is likely a maintenance mode thing anyways, what race condition?
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedtagging.
chx, perhaps we need to make this a more general issue, with a title like 'Allow modules to be imported by the configuration system' ?
i've thought about this a bit, and i'm pretty sure we'll need to special case module changes during import. probably via a pass that handles modules first, then the rest of the changes.
i'm happy to open another issue if this is not what you had in mind for this issue.
Comment #4
andypost@chx I've trying to figure a case: we have module node and comment (comment does not depends in node after #1790764: [meta] Convert comment module to field_api,
1) both disabled, 2) user calls to enable both (without dependency the order is unpredictable) 3) modules are trying to find each other state to install/update something... - seem rc posible
Comment #5
chx CreditAttribution: chx commentedI still don't get it , the operation is the exact same as clicking on the Extend screen to enable modules. That's what this issue is about.
Comment #6
alexpottClosed in favour of a newer and more constructive issue #1808248: Add a separate module install/uninstall step to the config import process