Re: [vote] release wicket 1.4.12

2010-09-15 Thread Jan Kriesten
+1 Please test the release and offer your vote: [ ] Yes, release [ ] No, don't release it (and please tell us why)

CompoundPropertyModel constructor

2010-09-15 Thread Ernesto Reinaldo Barreiro
My apologies if this has been discussed before... Right now the constructor of CompoundPropertyModel is as follows: /** * Constructor * * @param object *The model object, which may or may not implement IModel */ public

Re: Wicket Example on GAE (Google App Engine) - Form post back

2010-09-15 Thread Clint Checketts
Ha ha. You make them sound like a sort of candy. Sorry to pull the newbie card. But any tips on how to create a patch? Unfortunately the page on the Wicket site is blank ( http://wicket.apache.org/contribute/patch.html) Maybe its written in Whitespace and I need a decoder ring. I'm guessing I

Re: Wicket Example on GAE (Google App Engine) - Form post back

2010-09-15 Thread Martijn Dashorst
svn diff patch.txt in the root folder usually gives something useful Martijn On Wed, Sep 15, 2010 at 3:53 PM, Clint Checketts checke...@gmail.com wrote: Ha ha. You make them sound like a sort of candy. Sorry to pull the newbie card. But any tips on how to create a patch? Unfortunately the

Re: CompoundPropertyModel constructor

2010-09-15 Thread Martin Grigorov
Looks good to me. I've never seen that this ctor looks this way and I don't remember discussion related to that. On Wed, Sep 15, 2010 at 3:22 PM, Ernesto Reinaldo Barreiro reier...@gmail.com wrote: My apologies if this has been discussed before... Right now the constructor of

Re: Wicket Example on GAE (Google App Engine) - Form post back

2010-09-15 Thread Ernesto Reinaldo Barreiro
Or use your IDE to generate them:-) Ernesto On Wed, Sep 15, 2010 at 3:56 PM, Martijn Dashorst martijn.dasho...@gmail.com wrote: svn diff patch.txt in the root folder usually gives something useful Martijn On Wed, Sep 15, 2010 at 3:53 PM, Clint Checketts checke...@gmail.com wrote: Ha ha.

Re: CompoundPropertyModel constructor

2010-09-15 Thread Major Péter
Maybe IModel? extends T is better, or? Regards, Peter 2010-09-15 16:02 keltezéssel, Martin Grigorov írta: Looks good to me. I've never seen that this ctor looks this way and I don't remember discussion related to that. On Wed, Sep 15, 2010 at 3:22 PM, Ernesto Reinaldo Barreiro

Re: CompoundPropertyModel constructor

2010-09-15 Thread Ernesto Reinaldo Barreiro
Shall I provide a patch for 1.5 and attach it to a JIRA ticket? Probably this change cannot be applied to 1.4.x (as it might break existing apps) Ernesto On Wed, Sep 15, 2010 at 4:02 PM, Martin Grigorov mgrigo...@apache.org wrote: Looks good to me. I've never seen that this ctor looks this way

Re: CompoundPropertyModel constructor

2010-09-15 Thread Martin Grigorov
Don't hesitate to provide patches ;-) On Wed, Sep 15, 2010 at 4:06 PM, Ernesto Reinaldo Barreiro reier...@gmail.com wrote: Shall I provide a patch for 1.5 and attach it to a JIRA ticket? Probably this change cannot be applied to 1.4.x (as it might break existing apps) Ernesto On Wed, Sep

Re: Wicket Stuff commit access

2010-09-15 Thread Igor Vaynberg
done -igor On Wed, Sep 15, 2010 at 12:46 AM, Gabriel Bucher gabriel.buc...@gmail.com wrote: Could someone please grant me svn access to the wicketstuff? my sf account name is 'buchi' thanks gab

Re: CompoundPropertyModel constructor

2010-09-15 Thread Martin Grigorov
Looking at the code I see the problem is org.apache.wicket.model.CompoundPropertyModel.setChainedModel(IModel?) On Wed, Sep 15, 2010 at 4:27 PM, Martin Grigorov mgrigo...@apache.orgwrote: Don't hesitate to provide patches ;-) On Wed, Sep 15, 2010 at 4:06 PM, Ernesto Reinaldo Barreiro