[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#9264: Merge backport patches from plone.app.dexterity into Plone +--- Reporter: optilude|Owner: optilude Type: PLIP| Status: closed Priority: major |Milestone: 4.0 Component: Infrastructure | Resolution: fixed Keywords: | +--- Changes (by esteele): * status: assigned = closed * resolution: = fixed Comment: (In [30473]) Removing PLIP branches. Closes #9264. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:32 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 esteele): This PLIP has been accepted for merging into Plone 4.0 The final vote was: Alec Mitchell +1 David Glick +1 Erik Rose +1 Laurence Rowe +1 Matthew Wilkes - Ross Patterson +1 Please merge your branches into the Plone 4.0 head by end-of-day Friday Oct 16. If you need assistance with merging, please contact me. We'll be assigning a documentation ticket to this PLIP shortly. Please assist the docs team in documenting the changes and new features that this PLIP introduces. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:30 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 esteele): Please assist the doc team in creating/updating documentation relating to this PLIP. See #9608. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:31 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 davisagli): (In [29877]) update review refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:28 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 esteele): Your PLIP has passed the Framework team's initial review. Feel free to discuss any suggested changes either here in the PLIP ticket or on the mailing lists. Final deadline for this PLIP is set for September 30. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:27 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [29666]) Improve performance by not re-calculating FTI availability. Refs #9264. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:26 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [29630]) Simplify by removing IAdding implicit view lookup. Refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:21 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [29631]) Simplify by removing IAdding implicit view lookup. Refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:22 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [29632]) Update notes and try to address performance issues. Refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:23 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 davisagli): (In [29500]) add FWT review, refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:19 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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 ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [28562]) Branch for PLIP refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:13 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [28563]) Branch for PLIP refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:14 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [28567]) Add build and documentation for PLIP refs #9264 -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:15 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [28568]) Implement PLIP refs #9264. Note: as with the existing code, the tests are in plone.app.contentmenu. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:16 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9264: Merge backport patches from plone.app.dexterity into Plone
#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): (In [28570]) Implement PLIP refs #9264. -- Ticket URL: http://dev.plone.org/plone/ticket/9264#comment:17 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories