You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-dev@portals.apache.org by "David Hay (JIRA)" <ji...@apache.org> on 2006/09/06 15:58:27 UTC

[jira] Commented: (PLUTO-247) PortalURLParser: Avoid putting '?' and '&' characters in query string if unnecessary

    [ http://issues.apache.org/jira/browse/PLUTO-247?page=comments#action_12432853 ] 
            
David Hay commented on PLUTO-247:
---------------------------------

I apparently didn't have enough nits to pick ;-)

I happened to find this because we have some javascript code that assumed that the base URL was free of query parameters and always appended a "?" character to the URL.  When we moved that code into a portlet, of course it stopped working becuase the "?" was already there.

I realize that the problem was really with our code (which has been fixed) so this is really just a cosmetic fix, rather than a functional one.

> PortalURLParser: Avoid putting '?' and '&' characters in query string if unnecessary
> ------------------------------------------------------------------------------------
>
>                 Key: PLUTO-247
>                 URL: http://issues.apache.org/jira/browse/PLUTO-247
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portal driver
>    Affects Versions: 1.1.0-beta1
>            Reporter: David Hay
>         Assigned To: David DeWolf
>            Priority: Trivial
>             Fix For: 1.1.0
>
>
> When building the query string in PortalURLParser, the URL ends up containing a '?' character, whether there's a query string or not (e.g. http://host/path/?)  While not incorrect, it doesn't look very nice.
> A similar problem exists for query parameters.  The query string always starts with an '&' character.  (e.g. http://host/path/?&param=value)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira