Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
2 of 3 tasks
ericof opened this issue May 17, 2024 · 3 comments
Open
2 of 3 tasks

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

ericof opened this issue May 17, 2024 · 3 comments

Comments

@ericof
Copy link
Member

ericof commented May 17, 2024

Part of #3854

Tasks

  • Products.CMFPlone.browser.admin.AddPloneSite only registered if plone.distribution is not available
  • Refactor site creation to be like the function from plone.distribution (supporting a distribution)
  • Remove creation of example content (make sure those are still created in the test layer)
@mauritsvanrees
Copy link
Member

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
Copy link
Member

Progress during Buschenschankensprint (in the train back actually)

@mauritsvanrees
Copy link
Member

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?

mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Jul 26, 2024
Branch: refs/heads/master
Date: 2024-06-19T21:48:44+02:00
Author: Maurits van Rees (mauritsvanrees) <[email protected]>
Commit: plone/plone.app.testing@89dfa1d

PloneFixture: explicitly install plone.app.contenttypes:default.

The `addPloneSite` factory in Plone 6.1 no longer installs this by default.
See plone/Products.CMFPlone#3961

Files changed:
A news/3961.feature
M src/plone/app/testing/layers.py
Repository: plone.app.testing

Branch: refs/heads/master
Date: 2024-07-26T22:02:51+02:00
Author: Maurits van Rees (mauritsvanrees) <[email protected]>
Commit: plone/plone.app.testing@72970c7

Merge pull request #97 from plone/distributions

PloneFixture: explicitly install plone.app.contenttypes:default.

Files changed:
A news/3961.feature
M src/plone/app/testing/layers.py
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Jul 26, 2024
Branch: refs/heads/master
Date: 2024-06-19T21:48:44+02:00
Author: Maurits van Rees (mauritsvanrees) <[email protected]>
Commit: plone/plone.app.testing@89dfa1d

PloneFixture: explicitly install plone.app.contenttypes:default.

The `addPloneSite` factory in Plone 6.1 no longer installs this by default.
See plone/Products.CMFPlone#3961

Files changed:
A news/3961.feature
M src/plone/app/testing/layers.py
Repository: plone.app.testing

Branch: refs/heads/master
Date: 2024-07-26T22:02:51+02:00
Author: Maurits van Rees (mauritsvanrees) <[email protected]>
Commit: plone/plone.app.testing@72970c7

Merge pull request #97 from plone/distributions

PloneFixture: explicitly install plone.app.contenttypes:default.

Files changed:
A news/3961.feature
M src/plone/app/testing/layers.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

2 participants