[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