[jira] Resolved: (JS2-909) Layout Portlets removed Page logic is incorrectly worked.

["Woonsan Ko (JIRA)" <je...@portals.apache.org>]

     [ https://issues.apache.org/jira/browse/JS2-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Woonsan Ko resolved JS2-909.
----------------------------

    Resolution: Fixed

A folder should be updated after removing a page.
Thank you very much for the patch, Jeseung Choi!

> Layout Portlets removed Page logic is incorrectly worked.
> ---------------------------------------------------------
>
>                 Key: JS2-909
>                 URL: https://issues.apache.org/jira/browse/JS2-909
>             Project: Jetspeed 2
>          Issue Type: Bug
>          Components: Layout
>    Affects Versions: 2.1.3
>         Environment: Window XP, Tomcat 5.5, Java 1.5.x
>            Reporter: Jeseung Choi
>            Assignee: Woonsan Ko
>             Fix For: 2.2
>
>         Attachments: layout_portlets_remove_patch.patch
>
>
> Layout Portlets removed Page logic is incorrectly worked.
> 1) login jetspeed-portal.
> 2) click editPage button.
> 3) remove current page in page edit pane.
> 4) browser go to removed page.
> 5) portal engine does not work correctly.
> After inspecting this issue, I came to the following conclusion:
> Removed page after parent folder update is problem.
> // line 372-387 of MultiColumnPortlet.java 
> Folder parent = (Folder)requestPage.getParent();
> if (parent != null)
> {
>    List orderList = parent.getDocumentOrder();
>    if (orderList != null)
>    {
>        String name = requestPage.getName();
>        if (orderList.indexOf(name) > -1)
>        {
>            orderList.remove(name);
>            parent.setDocumentOrder(orderList);
>            pageManager.updateFolder(parent);
>        }
>    }
> } 
> pageManager.removePage(requestPage);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org