Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Juergen Donnerstag
I tried to apply the patch provided yesterday but the code obviously has changed. May be you can create a new patch on the latest version? Juergen On 8/20/06, Jean-Baptiste Quenot [EMAIL PROTECTED] wrote: Hi, While we are [1]talking about the hacks required to make Wicket work, there's

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Jean-Baptiste Quenot
* Juergen Donnerstag: I tried to apply the patch provided yesterday but the code obviously has changed. May be you can create a new patch on the latest version? OK, I just updated the patch. But the previous one applied successfully for me: $ cd wicket/branches/WICKET_1_2/wicket $ patch

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Juergen Donnerstag
Ah, it is a 1.2 patch. I applied it on my laptop. Because it changes bookmarkable URL we'll definitely On 8/20/06, Jean-Baptiste Quenot [EMAIL PROTECTED] wrote: * Juergen Donnerstag: I tried to apply the patch provided yesterday but the code obviously has changed. May be you can create a

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Jean-Baptiste Quenot
* Juergen Donnerstag: Ah, it is a 1.2 patch. I applied it on my laptop. Because it changes bookmarkable URL we'll definitely Yes it is in branch 1.2. But tell me, we'll definitely... what? -- Jean-Baptiste Quenot aka John Banana Qwerty http://caraldi.com/jbq/

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Johan Compagner
abrupt abort of the sentence?What does that patch break? Our stable bookmarkable urls?johanOn 8/20/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:Ah, it is a 1.2 patch. I applied it on my laptop. Because it changes bookmarkable URL we'll definitelyOn 8/20/06, Jean-Baptiste Quenot [EMAIL

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Jean-Baptiste Quenot
* Johan Compagner: abrupt abort of the sentence? What does that patch break? Our stable bookmarkable urls? See attached unit tests output. I kept only the relevant parts. Basically « : » is converted to « %3A », and catching up would require a search and replace in the unit tests. But

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Juergen Donnerstag
Because URL querystrings will change, and because bookmarkable URLs are changed also, that means that some web browser bookmarks won't work anymore. Juergen On 8/20/06, Jean-Baptiste Quenot [EMAIL PROTECTED] wrote: * Johan Compagner: abrupt abort of the sentence? What does that patch

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Jean-Baptiste Quenot
* Juergen Donnerstag: Because URL querystrings will change, and because bookmarkable URLs are changed also, that means that some web browser bookmarks won't work anymore. That is not likely to be true. The bookmark will be eg:

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Juergen Donnerstag
Though I agree in theory, have you tested it? On which browsers? And the new would be /WicketTester/WicketTester?wicket%3AbookmarkablePage=%3Awicket.markup.html.basic.SimplePage_3 Juergen On 8/20/06, Jean-Baptiste Quenot [EMAIL

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Juergen Donnerstag
I asked myself that question as well but URLEncoder/URLDecoder are converting these. Juergen On 8/20/06, Johan Compagner [EMAIL PROTECTED] wrote: do we have to do such ugly urls? i don't like that. is : really a char that has to be encoded? johan On 8/20/06, Juergen Donnerstag [EMAIL

Re: [Wicket-user] Handle String array in PageParameters

2006-08-20 Thread Johan Compagner
do we have to do such ugly urls?i don't like that. is : really a char that has to be encoded?johanOn 8/20/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:Though I agree in theory, have you tested it? On which browsers? And the new would

[Wicket-user] Handle String array in PageParameters

2006-08-19 Thread Jean-Baptiste Quenot
Hi, While we are [1]talking about the hacks required to make Wicket work, there's an old one I submitted: [ 1493672 ] Handle String array in PageParameters http://sourceforge.net/tracker/index.php?func=detailaid=1493672group_id=119783atid=684977 Basically the following code: