plone / Products.CMFPlone

The core of the Plone content management system
https://plone.org
GNU General Public License v2.0
248 stars 188 forks source link

Refactor site creation in Products.CMFPlone to use plone.distribution #3961

Open ericof opened 4 months ago

ericof commented 4 months ago

Part of #3854

Tasks

mauritsvanrees commented 3 months ago

Without plone.distribution available, do we want to remove the Advanced options? I can imagine that we want to point users to either distributions or just let them install the extra profiles via the Add-ons control panel.

Screenshot 2024-06-19 at 19 31 45

Actually, you can't install "Zope Root Cookie Login" (Products.PlonePAS:root-cookie profile) in the control panel. You would need to do that via portal_setup. But that is really an advanced corner case. Since it works on the Zope root, you don't even want it available for install from within Plone. So I think that is fine. Anyway, you can still do that in a Distribution. :-)

mauritsvanrees commented 3 months ago

Progress during Buschenschankensprint (in the train back actually)

mauritsvanrees commented 3 months ago

The CMFPlone PR does not "Refactor site creation to be like the function from plone.distribution (supporting a distribution)". I don't really know what that would like like if plone.distribution is not available, which is the only way that the plone-addsite view is even visible.

Ah, or do you mean that the factory.addPloneSite function should be able to install a distribution if plone.distribution is available? Then this function would accept an extra distribution keyword argument, and call a function in plone.distribution. Something like that?