search

eea / eea.facetednavigation

Faceted Navigation for Plone Classic
http://eea.github.com/docs/eea.facetednavigation
20 stars 56 forks source link

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

Open laulaz opened 4 years ago

laulaz commented 4 years ago

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 ...

avoinea commented 4 years ago

@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 commented 4 years ago

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