Re: Wicketstuff editable grid - 7.1.0

2016-02-26 Thread Mihir Chhaya
Done as below: https://github.com/wicketstuff/core/issues/471 -Mihir. On Wed, Feb 24, 2016 at 12:06 PM, Martin Grigorov wrote: > On Wed, Feb 24, 2016 at 5:59 PM, Mihir Chhaya > wrote: > > > Thanks, Martin for this. > > Would this be an JIRA entry? As a bug or as an enhancement? > > > > WicketS

Re: Wicketstuff editable grid - 7.1.0

2016-02-24 Thread Martin Grigorov
On Wed, Feb 24, 2016 at 5:59 PM, Mihir Chhaya wrote: > Thanks, Martin for this. > Would this be an JIRA entry? As a bug or as an enhancement? > WicketStuff GitHub issue. It a problem in EditableGrid, not in Wicket. > > For now, It seems separating the editable grids on two different panels > w

Re: Wicketstuff editable grid - 7.1.0

2016-02-24 Thread Mihir Chhaya
Thanks, Martin for this. Would this be an JIRA entry? As a bug or as an enhancement? For now, It seems separating the editable grids on two different panels would solve the problem. As the newActionsColumn method in the EditableGrid is private, I can't override. -Mihir. On Wed, Feb 24, 2016 at 1

Re: Wicketstuff editable grid - 7.1.0

2016-02-24 Thread Martin Grigorov
Hi, The problem is in EditableGrid. When you click on "Save" button it executes org.wicketstuff.egrid.column.EditableGridActionsPanel#newEditLink -> onClick(). Here it does: send(getPage(), Broadcast.BREADTH, rowItem); As you can see this broadcasts the event to the whole page instead of just the

Re: Wicketstuff editable grid - 7.1.0

2016-02-24 Thread Martin Grigorov
No sorry! I didn't had a chance yet. Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On Wed, Feb 24, 2016 at 3:39 PM, Mihir Chhaya wrote: > Martin, > > Just wanted to check if you got chance to look at the GitHub code and if it > is enough or any additional informa

Re: Wicketstuff editable grid - 7.1.0

2016-02-24 Thread Mihir Chhaya
Martin, Just wanted to check if you got chance to look at the GitHub code and if it is enough or any additional information is needed? Thanks, -Mihir. On Mon, Feb 22, 2016 at 10:26 AM, Mihir Chhaya wrote: > Sure thing; I have put the code online at > https://github.com/mihirchhaya/egrid71 > >

Re: Wicketstuff editable grid - 7.1.0

2016-02-22 Thread Mihir Chhaya
Sure thing; I have put the code online at https://github.com/mihirchhaya/egrid71 Thanks, -Mihir. On Sat, Feb 20, 2016 at 6:50 AM, Martin Grigorov wrote: > Hi, > > Please share the mini application somewhere, e.g. GitHub. > > Martin Grigorov > Wicket Training and Consulting > https://twitter.com

Re: Wicketstuff editable grid - 7.1.0

2016-02-20 Thread Martin Grigorov
Hi, Please share the mini application somewhere, e.g. GitHub. Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On Fri, Feb 19, 2016 at 7:30 PM, Mihir Chhaya wrote: > Ok, so creating a small project with two panels(Person and Customer) and a > page for similar use c

Re: Wicketstuff editable grid - 7.1.0

2016-02-19 Thread Mihir Chhaya
Ok, so creating a small project with two panels(Person and Customer) and a page for similar use case resulted in following stacktrace. The panels are added in the order of PersonPanel, CustomerPanel in the parent page. The line# 163 in onSave method of EditableGrid is calling EditableGrid.this.onS

Re: Wicketstuff editable grid - 7.1.0

2016-02-19 Thread Mihir Chhaya
Just realized that last two lines could be confusing: the decisionCode is the property of class PcaCodes in pcapanel, but showing up for exccodepanel row item. On Fri, Feb 19, 2016 at 12:06 PM, Mihir Chhaya wrote: > My mistake about the inmethod grid dependency. I don't know why I saw > compilat

Re: Wicketstuff editable grid - 7.1.0

2016-02-19 Thread Mihir Chhaya
My mistake about the inmethod grid dependency. I don't know why I saw compilation error in editable grid class when removed inmethod grid jar from classpath. For debugging the respond method as your suggestion, I do see correct class types for each rowItem (as attachment). But it fails with follow

Re: Wicketstuff editable grid - 7.1.0

2016-02-18 Thread Martin Grigorov
On Thu, Feb 18, 2016 at 10:53 PM, Mihir Chhaya wrote: > Thanks, Martin. > > By writing dependency on wicket in-method grid I meant editable grid > inherently referring inmethod grid. > https://github.com/wicketstuff/core/blob/master/editable-grid-parent/editable-grid/pom.xml https://github.com/w

Re: Wicketstuff editable grid - 7.1.0

2016-02-18 Thread Mihir Chhaya
Thanks, Martin. By writing dependency on wicket in-method grid I meant editable grid inherently referring inmethod grid. I did check the Ajax url earlier and following are the values in rendered html page for the panel. Wicket.Ajax.ajax({"u":"./appcontext?5-2.IBehaviorListener.0-admincontroltabs

Re: Wicketstuff editable grid - 7.1.0

2016-02-18 Thread Martin Grigorov
Hi, On Wed, Feb 17, 2016 at 5:09 PM, Mihir Chhaya wrote: > Hello, > > My apologies if this is not the right place to post wicketstuff related > issues. If so, then please advice me with right forum/link. > This is the right forum! > > I am using Wicketstuff-editable-grid-7.1.0 (with dependenc

Re: Wicketstuff editable grid - 7.1.0

2016-02-18 Thread Mihir Chhaya
Any suggestion, friends? On Wed, Feb 17, 2016 at 11:09 AM, Mihir Chhaya wrote: > Hello, > > My apologies if this is not the right place to post wicketstuff related > issues. If so, then please advice me with right forum/link. > > I am using Wicketstuff-editable-grid-7.1.0 (with dependency on > w