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

Path widget doesn't work if Plone site id is "plone" #182

Open
laulaz opened this issue Nov 14, 2019 · 2 comments
Open

Path widget doesn't work if Plone site id is "plone" #182

laulaz opened this issue Nov 14, 2019 · 2 comments
Labels

Comments

@laulaz
Copy link
Contributor

laulaz commented Nov 14, 2019

If you want to use the path widget with a plone site named plone, you get a traceback :

Traceback (innermost last):
  Module ZPublisher.WSGIPublisher, line 155, in transaction_pubevents
  Module ZPublisher.WSGIPublisher, line 337, in publish_module
  Module ZPublisher.WSGIPublisher, line 255, in publish
  Module ZPublisher.mapply, line 85, in mapply
  Module ZPublisher.WSGIPublisher, line 61, in call_object
  Module eea.facetednavigation.caching.nocache, line 12, in replacement
  Module eea.facetednavigation.widgets.path.tree, line 168, in json_tree
  Module eea.facetednavigation.widgets.path.tree, line 90, in tree
  Module plone.app.layout.navigation.navtree, line 133, in buildFolderTree
AttributeError: 'Plone' object has no attribute 'getPhysicalPath'

Indeed, self.context.unrestrictedTraverse(url.strip('/'), None) on /plone returns a Products.Five.browser.metaconfigure.Plone object ...

@laulaz laulaz changed the title Path widget doesn't work in Plone 5 Path widget doesn't work if Plone site id is "plone" Nov 14, 2019
@avoinea
Copy link
Member

avoinea commented Nov 14, 2019

@laulaz Can't reproduce it:

docker run -it --rm -e ADDONS=eea.facetednavigation -e SITE=Plone -p 8080:8080 plone fg

Screenshot 2019-11-14 at 15 11 47

Screenshot 2019-11-14 at 15 13 31

Screenshot 2019-11-14 at 15 14 04

@avoinea
Copy link
Member

avoinea commented Nov 14, 2019

Ok, I think I missed the part with the Plone site ID = 'plone'. Indeed, issue confirmed.

@avoinea avoinea added the Bug label Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants