Hi Laurence!

Laurence Rowe wrote:
On 5 September 2012 19:21, Laurence Rowe <l...@lrowe.co.uk> wrote:
Instead of removing the RequestContainer, it could be replaced with a
zope.globalrequest aware RequestContainer. That might be cleaner than
rewrapping in individual utilities, and would work with Zope 2.13.

I gave this a go in

It seems to work fine with the CMF trunk tests even when I remove all
RequestContainer wrapping from both CMFCore and CMFDefault (the
CMFDefault tests then fail with five.localsitemanage trunk.)



Unfortunately there's a trade-off:

Modernizing the RequestContainer concept makes it possible to move forward in some areas without breaking existing code. That's a good thing.

But on the other hand it makes it easy to write bad code. New code should not rely on this. People should write views if their code depends on the request, not utilities.

I think this discussion is closely related to your plans for Zope 4: If Zope 4 will (re-)enable the get-request-by-acquisition pattern everywhere, it doesn't make much sense to be more restrictive in CMF 2.3 on Zope 2.13.

Please consider providing tools for people who want to write clean code. Documentation, warnings, maybe even a switch for disabling the legacy behavior.



Zope-CMF maillist  -  Zope-CMF@zope.org

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests

Reply via email to