portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From j...@apache.org
Subject [jira] Closed: (JS2-4) ActionURL parameters not accessible: wronly encoded
Date Wed, 05 May 2004 21:24:56 GMT

   The following issue has been closed.

   Resolver: David Sean Taylor
       Date: Wed, 5 May 2004 2:24 PM

i've deprecated the getAllParameters method.
Im keeping it around for reference when working on JS2-9
View the issue:

Here is an overview of the issue:
        Key: JS2-4
    Summary: ActionURL parameters not accessible: wronly encoded
       Type: Bug

     Status: Closed
   Priority: Blocker
 Resolution: FIXED

    Project: Jetspeed 2

   Assignee: David Sean Taylor
   Reporter: Ate Douma

    Created: Thu, 1 Apr 2004 6:38 AM
    Updated: Wed, 5 May 2004 2:24 PM
Environment: WindowsXP, J2SE 1.4.2_03, Tomcat 4.1.29

o.a.j.services.information.PortletURLProviderImpl.toString() encodes ActionURL parameter using
pluto's NamespaceMapper.encode(...).

This seems to be copied from pluto's o.a.p.portalImpl.core.PortletURLProviderImpl.toString()
where the exact same handling can be found BUT there its commented out! (since the first version
in cvs even).

The result of the current implementation is that ActionURL parameters are prefixed by 'Pluto_[PortletWindowId]_'.
These parameters are thus not accessible by their expected name in processAction.

In Pluto, ActionURL parameters are simply non-encoded put into the url which to me seems perfectly
fine as only one portlet can be target with an ActionURL and its id is already encoded otherwise
in the url. ActionURL parameters thus can be treated just like normal form parameters (non
of these will be propagated as render parameters anyway).

So, my suggestion is to remove the NamespaceMapper.encode(...) handling in o.a.j.services.information.PortletURLProviderImpl.toString()
just as in Pluto.

Note: the current cvs state of ActionURL and RenderURL seems to be broken (no parameters are
rendered anymore). I tested this on a cvs checkout from 2004-03-26)

This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:

If you want more information on JIRA, or have a bug to report see:

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

View raw message