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

          Jeseung Choi created issue -
          Hide
          Jeseung Choi added a comment -

          I patched this issue.

          Show
          Jeseung Choi added a comment - I patched this issue.
          Jeseung Choi made changes -
          Field Original Value New Value
          Attachment layout_portlets_remove_patch.patch [ 12391974 ]
          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 .
          Ate Douma made changes -
          Assignee Woonsan Ko [ woon_san ]
          Ate Douma made changes -
          Link This issue is related to JS2-832 [ 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!
          Woonsan Ko made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Woonsan Ko made changes -
          Link This issue is blocked by JS2-832 [ JS2-832 ]
          Ate Douma made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          1d 5h 49m 1 Woonsan Ko 14/Oct/08 08:06
          Resolved Resolved Closed Closed
          1085d 13h 1m 1 Ate Douma 04/Oct/11 21:08

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development