[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
plip-advisories at lists.plone.org
plip-advisories at lists.plone.org
Tue Sep 1 14:17:19 UTC 2009
#9264: Merge backport patches from plone.app.dexterity into Plone
----------------------------+-----------------------------------------------
Reporter: optilude | Owner: optilude
Type: PLIP | Status: assigned
Priority: major | Milestone: 4.0
Component: Infrastructure | Resolution:
Keywords: |
----------------------------+-----------------------------------------------
Comment(by optilude):
Some thoughts on the review notes:
- The slow-down is obviously disappointing. I'd like to do some more
benchmarking, both to confirm the results and to figure out where the
problem really lies.
- We could maybe make support for this optional somehow? Dexterity
certainly needs it. AT doesn't presently.
- I think we should just rip out the IAdding guess-the-view support. It
really complicates the code, and it's very easy to use the folder/add
action to get a /+/foo URL to work. Since IAdding is not used by any core
code, I think it can just go into the upgrade notes. It may also gain us
some performance.
- It may worth raising the performance issue with the CMF guys. This is a
CMF feature after all.
Martin
--
Ticket URL: <http://dev.plone.org/plone/ticket/9264#comment:20>
Plone <http://plone.org>
Plone Content Management System
More information about the PLIP-Advisories
mailing list