[2.0.0-beta1] Add alterInfo('component_info') hook

Is activating the component module enough?

Maybe we will remove _template all together.
Let's wait and discuss.

If we remove this property, don't forget to:

• Revert this change : 🐛 [2.0.0-alpha2] Wrong template name leads to Deprecated: basename(): Passing null to parameter #1 ($path) Fixed
• Update the specification
• Close the SDC issue: ✨ Allow declaration of template path in SDC Active

Finally, we keep it.

Christian.wiedemann → made their first commit to this issue’s fork.

I used component_definition_alter instead of component_info for the hook name. Thought it is more straight forward.

Indeed, you did:

$this->moduleHandler->alter('component_definition', $definition);

Instead of:

$this->alterInfo('component_info');

Doing that, we are diverging from the comment in the SDC codebase, I hope we will not have a concurrent implementation if SDC implements the alter hook later.

Jean, what is your opinion about this?

I think IMHO that we should keep 'component_info', so if the core sdc enable this alter, we'll know we have to remove it, and there will be break on existing code.

Or ui_patterns_component_definition. For me component_info is altering the plugin not the plugin definition.

Automatically closed - issue fixed for 2 weeks with no activity.