Jetspeed 2
  1. Jetspeed 2
  2. JS2-767

Portlet Action not working when using JetspeedPowerTool

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.2
    • Fix Version/s: 2.1.3
    • Component/s: None
    • Labels:
      None

      Description

      When a portlet is included in a decorator using $jetspeed.renderPortletEntity, the action urls generated by this portlet doesn't work.

      This bug is due to the following code in class org.apache.jetspeed.container.ContainerValve :

      public void invoke(RequestContext request, ValveContext context) throws PipelineException
      {
      ...
      window = state.getPortletWindowOfAction();
      if (window != null && page.getFragmentById(window.getId().toString()) == null)

      { // target window doesn't exists anymore or the target page is not accessible (anymore) // remove any navigational state for the window state.removeState(window); // as this is an action request which cannot be handled, perform a direct redirect after sync state (for the other windows) redirect = true; }

      ...
      }

      Indeed, in this case, the valve should not perform a redirect because even if the portlet is not in the page (but in the decorator), the action have to be performed.

        Activity

        Nicolas Dutertry created issue -
        David Sean Taylor made changes -
        Field Original Value New Value
        Assignee David Sean Taylor [ taylor ]
        David Sean Taylor made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        David Sean Taylor made changes -
        Status In Progress [ 3 ] Open [ 1 ]
        Hide
        David Sean Taylor added a comment -
        Show
        David Sean Taylor added a comment - see Woonsans commits: http://svn.apache.org/viewvc?view=rev&revision=589933
        David Sean Taylor made changes -
        Fix Version/s 2.1.3 [ 12312573 ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Ate Douma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        80d 2h 45m 1 David Sean Taylor 28/Oct/07 17:47
        In Progress In Progress Open Open
        1d 10h 17m 1 David Sean Taylor 30/Oct/07 04:05
        Open Open Resolved Resolved
        1m 8s 1 David Sean Taylor 30/Oct/07 04:06
        Resolved Resolved Closed Closed
        1435d 17h 31m 1 Ate Douma 04/Oct/11 22:38

          People

          • Assignee:
            David Sean Taylor
            Reporter:
            Nicolas Dutertry
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development