Jetspeed 2
  1. Jetspeed 2
  2. JS2-909

Layout Portlets removed Page logic is incorrectly worked.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.3
    • Fix Version/s: 2.2.0
    • Component/s: Layout
    • Labels:
      None
    • Environment:
      Window XP, Tomcat 5.5, Java 1.5.x

      Description

      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);

        Issue Links

          Activity

          Hide
          Jeseung Choi added a comment -

          I patched this issue.

          Show
          Jeseung Choi added a comment - I patched this issue.
          Hide
          Ate Douma added a comment -

          Woonsan, I've assigned this issue to you as I think it might be related to JS2-832.

          Show
          Ate Douma added a comment - Woonsan, I've assigned this issue to you as I think it might be related to JS2-832 .
          Hide
          Woonsan Ko added a comment -

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

          Show
          Woonsan Ko added a comment - A folder should be updated after removing a page. Thank you very much for the patch, Jeseung Choi!

            People

            • Assignee:
              Woonsan Ko
              Reporter:
              Jeseung Choi
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development