- Issue created by @lolgm
- 🇫🇷France flocondetoile Lyon
MR11 is not mergeable.
Also you changed the entity type definition for the Site entity.
So you need to update this definition too for existing sites, with a hook_update_N. See https://git.drupalcode.org/project/commerce/-/blob/3.0.x/modules/order/c... (commerce_order_update_8210()
) as an example. - 🇫🇷France flocondetoile Lyon
What impact of such change if content_moderation is not enabled on a project with micro_site ?
- 🇵🇹Portugal lolgm
@flocondetoile Thanks for the feedback.
I added the
hook_update_N
implementation (micro_site_update_8107
) in MR !11, following the example provided in #4.From my testing, this change doesn't have any impact when the content_moderation module is disabled. It appears that this handler is loaded only when content_moderation is enabled. You can confirm this behavior by reviewing the methods
entityTypeAlter()
andaddModerationToEntityType()
in the classDrupal\content_moderation\EntityTypeInfo
:
https://git.drupalcode.org/project/drupal/-/blob/10.4.x/core/modules/con...