Add allow-plugins

[fritzmg]

The current composer.json is still missing the allow-plugins section of the contao/managed-edition causing prompts when installing via composer create-project.

[aschempp]

hmmm… having this in the composer.json will prevent the Contao Manager von automatically configuring it though 😞

[fritzmg]

We include it for contao/managed-edition as well though, so you don't have to confirm anything when running composer create-project. So I think we should include it here too?

[aschempp]

contao/managed-edition is not used by the Contao Manager though. I agree this is a problem, I don't know how to fix it though 🤔

[fritzmg]

But why is it a problem for the Contao Manager in the first place? If contao/managed-edition and contao/contao-demo (and isotope/isotope-demo for that matter) come with appropriately allowed plugins, no issues should occur during installation?

[fritzmg]

And the Contao Manager can always alter the composer.json afterwards.

[aschempp]

We can never know what plugins a user will install, e.g. php-http/discovery was added in a patch version, and that would now prevent any installation of packages in an existing installation.

And the Contao Manager can always alter the composer.json afterwards.

The Contao Manager does not alter the composer.json user configuration. It is explicitly meant so you can allow or deny plugins if you want to, but the default should be everything is allowed (which is done automatically if the setting is not present).

[fritzmg]

The Contao Manager does not alter the composer.json user configuration.

The Contao Manager for example updates the post-update-cmd and post-install-cmd scripts to use vendor/bin/contao-setup. So why not also remove the allow-plugins section after initial installation?

but the default should be everything is allowed (which is done automatically if the setting is not present).

But then we should add this default everywhere.

Anyway, what other solution do you propose?

[aschempp]

Anyway, what other solution do you propose?

I don't have any other solution 🤷
However, I would also argue that anyone installing the demo through Composer should be aware of Composer features like allowing plugins, and should be able to decide whether a plugin is wanted or not. Not saying that is best user experience but… 😬

[aschempp]

So why not also remove the allow-plugins section after initial installation?

That could be a viable option. But then we can only update this repo once the feature is implemented in the Contao Manager.

[fritzmg]

However, I would also argue that anyone installing the demo through Composer should be aware of Composer features like allowing plugins, and should be able to decide whether a plugin is wanted or not. Not saying that is best user experience but… 😬

Regardless I would argue that

• composer create-project contao/managed-edition
• composer create-project contao/contao-demo
• composer create-project isotope/isotope-demo
• …

should all work the same and not require different arguments from one another.