You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Ingram Chen (JIRA)" <ji...@apache.org> on 2007/05/06 05:01:15 UTC

[jira] Created: (WICKET-539) Mutiple window support with QueryStringUrlCodingStrategy cause bad url

Mutiple window support with QueryStringUrlCodingStrategy cause bad url 
-----------------------------------------------------------------------

                 Key: WICKET-539
                 URL: https://issues.apache.org/jira/browse/WICKET-539
             Project: Wicket
          Issue Type: Bug
          Components: wicket
    Affects Versions: 1.2.6
         Environment:  linux 2.6.x
 tomcat5, 6
 jdk6
            Reporter: Ingram Chen
         Attachments: quickstart.tar.gz

When using QueryStringUrlCodingStrategy with multiple window support, for example: 

mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));

and use BookmarkablePageLink:

public Index() {
	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
}

ideally the url should be:
/context/app/mountPath?foo=123

if open the link in new window, in 1.2.5, the url is
/context/app/mountPath?foo=123&wicket:pageMapName=wicket-0

however, in 1.2.6. the url become:
/context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Ingram Chen (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ingram Chen closed WICKET-539.
------------------------------


We apply related patches and do custom build for 1.2.x for ourself, so this is not urgent for us. 
and we will migrate to 1.3 soon.

i think this issue can be closed.

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot updated WICKET-539:
----------------------------------------

    Attachment: 20070601-wicket-WICKET-539.txt

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Ingram Chen (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12501110 ] 

Ingram Chen commented on WICKET-539:
------------------------------------

hmm.. if we can't fix for 1.2.x branch, at least provide some workaround to avoid it. 
This bug is not just a corner case, it already hurts our production system.

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12500659 ] 

Jean-Baptiste Quenot commented on WICKET-539:
---------------------------------------------

Applying this fix to the 1.2.x branch means we also fix WICKET-65 for consistency.  That's too much of a change for the maintenance branch in my opinion.

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot updated WICKET-539:
----------------------------------------

    Fix Version/s:     (was: trunk)
                   1.3.0-beta2

This issue is not fixed in the 1.2.x branch indeed. The fix is only available in Wicket trunk == Wicket 1.3.  Based on the fix I made (look at the Subversion commits tab in this issue), it should be possible to come up with a patch for the 1.2.x branch however.  Any chance you can prepare one?

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot updated WICKET-539:
----------------------------------------

    Summary: QueryStringUrlCodingStrategy does not handle multi-valued parameters  (was: Mutiple window support with QueryStringUrlCodingStrategy cause bad url )

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>         Assigned To: Jean-Baptiste Quenot
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot resolved WICKET-539.
-----------------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 1.3.0-beta3)
                   1.3.0-beta2

This issue is actually fixed in beta2 already.  If you want this issue to be backported on the 1.2.x branch, please poll the users list.

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Eelco Hillenius (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12501247 ] 

Eelco Hillenius commented on WICKET-539:
----------------------------------------

See discussion here http://www.nabble.com/Re%3A-%28WICKET-539%29-QueryStringUrlCodingStrategy-does-not-handle-multi-valued-parameters-tf3863672.html

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Ingram Chen (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ingram Chen updated WICKET-539:
-------------------------------

    Attachment: patch.tar.gz

I back port your changes to 1.2.x (just copy and paste), see attachment

for assertion in QueryStringUrlCodingStrategyTest, I change

assertEquals("mount/point?a=1&a=2&b=1", path);
to 
assertEquals("/mount/point?a=1&a=2&b=1", path);

The quick start works. But I don't know why path contains "/" in 1.2.x but not in 1.3. 


> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta2
>
>         Attachments: patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (WICKET-539) Mutiple window support with QueryStringUrlCodingStrategy cause bad url

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot reassigned WICKET-539:
-------------------------------------------

    Assignee: Jean-Baptiste Quenot

> Mutiple window support with QueryStringUrlCodingStrategy cause bad url 
> -----------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>         Assigned To: Jean-Baptiste Quenot
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Martijn Dashorst (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Martijn Dashorst updated WICKET-539:
------------------------------------

    Fix Version/s:     (was: 1.3.0-beta2)
                   1.3.0-beta3

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: 1.3.0-beta3
>
>         Attachments: 20070601-wicket-WICKET-539.txt, patch.tar.gz, quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Jean-Baptiste Quenot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Quenot resolved WICKET-539.
-----------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.3

Fixed in trunk, thanks for reporting.  Note that there is a similar problem described in WICKET-65

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>         Assigned To: Jean-Baptiste Quenot
>             Fix For: 1.3
>
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WICKET-539) Mutiple window support with QueryStringUrlCodingStrategy cause bad url

Posted by "Ingram Chen (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ingram Chen updated WICKET-539:
-------------------------------

    Attachment: quickstart.tar.gz

quick start to reproduce the bug

> Mutiple window support with QueryStringUrlCodingStrategy cause bad url 
> -----------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (WICKET-539) QueryStringUrlCodingStrategy does not handle multi-valued parameters

Posted by "Ingram Chen (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WICKET-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ingram Chen reopened WICKET-539:
--------------------------------


I tried current 1.2.x trunk (2007/05/31) with quick start
but it seems that it's still broken.

Could you run quick-test which I provided again to verify ?
 

> QueryStringUrlCodingStrategy does not handle multi-valued parameters
> --------------------------------------------------------------------
>
>                 Key: WICKET-539
>                 URL: https://issues.apache.org/jira/browse/WICKET-539
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.2.6
>         Environment:  linux 2.6.x
>  tomcat5, 6
>  jdk6
>            Reporter: Ingram Chen
>            Assignee: Jean-Baptiste Quenot
>             Fix For: trunk
>
>         Attachments: quickstart.tar.gz
>
>
> When using QueryStringUrlCodingStrategy with multiple window support, for example: 
> mount("/mountPath", new QueryStringUrlCodingStrategy("/mountPath", Other.class));
> and use BookmarkablePageLink:
> public Index() {
> 	add(new BookmarkablePageLink("link", Other.class).setParameter("foo", 123L));
> }
> ideally the url should be:
> /context/app/mountPath?foo=123
> if open the link in new window, in 1.2.5, the url is
> /context/app/mountPath?foo=123&wicket:pageMapName=wicket-0
> however, in 1.2.6. the url become:
> /context/app/mountPath?foo=%5BLjava.lang.String%3B%406f0241&wicket:pageMapName=wicket-0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.