No acquistion during publish traverse

Introduction

The problem with “acquisition” and publishTraverse is that the current method returns too many different URLs for the same content. For instance here is some potential url for the “kb” page of the plone.org website

and here is a generic “Plone” site with two content items “a” and “b” (folderish or not)

All the urls above returns 200 with the same content, while I would like the “canonical url” to return 200 and the other to return 404.

The behaviour described above constitute a problem because:

• multiple url for the same content is a problem for SEO and is confusing to people. For SEO, in the latest versions Plone introduced the canonical META, but IMHO it’s just a workaround. People are confused. For example: sometimes some of my editors ask me: “I can’t remove the http://example.com/Plone/a/b/a/page. Can you do it for me?”

• the page doesn’t seem really the same on all urls: if you open https://plone.org/documentation/kb and https://plone.org/documentation/manual/kb the second has a portlet that the first is missing

• removing page from external cache (varnish or squid), for example after a content modification, will be a pain. This is because for the same content there could be multiple urls without any control or rules (collective.purgebyid solves this)

• when using subsite (or multiple plone site on the same zope app) the problem is even more annoying: suppose that “a” is a subsite (marked with INavigationRoot) for http://a.example.org and “b” for http://b.example.org. Opening the url http://a.example.org/b will probably show the homepage of site “a” inside the “b” site. collective.siteisolation and probably collective.lineage do something to isolate subsite, but IMHO again are only workarounds.

Usage

This is a monkey patch for publishTraverse method of Zope2’s ZPublisher.BaseRequest.DefaultPublishTraverse and a monkey patch for Products.Archetypes.BaseObject.BaseObject.__bobo_traverse__

By default invalid traverse is only logged as warning.

For enable raising exceptions, you need to manually modify config.py changing DRYRUN to False.

Or using plone.recipe.zope2instance >= 4.2.14, e.g.:

[instance]
recipe = plone.recipe.zope2instance
eggs =
experimental.noacquisition
...
initialization =
from experimental.noacquisition import config
config.DRYRUN = False

Warning

Don’t use it, if you don’t know exactly what are you doing… at least use leaving DRYRUN = True.

Tests

This add-on is tested using Travis CI. The current status of the add-on is :

Other solutions

There is a more elegant solution in a branch of Products.CMFPlone, that makes use of IPubAfterTraversal event instead of a monkey patch. But seems that currently it doesn’t works for all cases, at least when there is a custom traversal at the end of the request (take a look at the tests inside this package). https://github.com/plone/Products.CMFPlone/tree/publication-through-explicit-acquisition

There is also another package with same approach as CMFPlone’s branch: collective.explicitacquisition

Changelog

1.0.0b4 (2018-05-14)

• Zope2 2.13.27 (no changes) [mamico]

1.0.0b3 (2017-05-09)

• Zope2 2.13.26 (no changes) [mamico]

1.0.0b2 (2016-06-10)

• Zope2 2.13.24 [mamico]

1.0.0b1 (2015-10-23)

• Zope2 2.13.23 (Plone 4.3.7/5.0) [mamico]

1.0.0a5 (2014-10-31)

• Nothing changed yet.

1.0.0a4 (2014-10-31)

• Initial release

Project details

Uploaded source