plone / plone.app.widgets

Integrating plone.widgets into plone
Other
7 stars 38 forks source link

Simplify usage of navigation root #142

Closed thet closed 8 years ago

thet commented 8 years ago

/cc @jensens

jensens commented 8 years ago

I dont think this works.

An INavigationRoot is less than a Site. https://github.com/plone/plone.app.layout/blob/master/plone/app/layout/navigation/interfaces.py#L13

Only in lineage a Childsite is both, INavigationRoot and ISite: https://github.com/collective/collective.lineage/blob/master/src/collective/lineage/utils.py#L20

But in the other common use case of plone.app.multilingual its just a INavigationRoot: https://github.com/plone/plone.app.multilingual/blob/master/src/plone/app/multilingual/content/lrf.py#L35 https://github.com/plone/plone.app.multilingual/blob/master/src/plone/app/multilingual/interfaces.py#L21