[ZCM] [ZC] 2240/ 5 Comment "change aquisition handling"
"Collector: Zope Bugs, Features, and Patches ..." <[email protected]>
| Newsgroups | gmane.comp.web.zope.devel.collector-monitor |
|---|---|
| Message-ID | <[email protected]> |
Issue #2240 Update (Comment) "change aquisition handling"
Status Rejected, Zope/feature medium
To followup, visit:
http://www.zope.org/Collectors/Zope/2240
==============================================================
= Comment - Entry #5 by tseaver on Nov 21, 2006 1:05 pm
I would say that such a patch should be structured as a subclass
of Acquisition.Implicit, rather than adding the behavior directly
to Implicit. Application code which wanted this behavior would
then do something like::
>>> from Acquisition import Dynamic
>>> class MyClass(Dynamic):
... def aq_dynamic(self, name):
""" Majyk """
instead of
>>> from Acquisition import Implicit
>>> class MyClass(Implicit):
... def aq_dynamic(self, name):
""" Majyk """
________________________________________
= Reject - Entry #4 by ajung on Nov 21, 2006 11:08 am
Status: Pending => Rejected
________________________________________
= Comment - Entry #3 by ajung on Nov 21, 2006 11:05 am
Rejecting this patch because it is too odd.
________________________________________
= Comment - Entry #2 by ajung on Nov 21, 2006 10:46 am
The documentation of the patch appears scary to me. And it is unlikely that we want to change the acquisition behavior of Zope.
There is already enough too much magic involved.
________________________________________
= Request - Entry #1 by tonim on Nov 21, 2006 10:38 am
Uploaded: "zope-aq-dynamic.patch"
- http://www.zope.org/Collectors/Zope/2240/zope-aq-dynamic.patch/view
There's a patch that changes the aquisition behaviour of Zope. This patch is required for ERP5. Read a complete description here: http://www.erp5.org/WhyAqDynamicIsRequired
Attached you find an updated patch against Zope 2.10.0. Please note that the test script ends up in a weird location and probably needs to be moved.
It would be nice if the patch could be integrated and thus make it possible to run ERP5 with a standard Zope.
==============================================================