I get it! you are talking about AjaxRequestTarget.add(form) by this call you tell Wicket to send new version of component to the page. So it is OK to re-add it
On Thu, Sep 6, 2012 at 8:54 PM, Maxim Solodovnik <[email protected]>wrote: > I would not add/remove Form > I would leave form on the page, but change the underlying object. > > I can modify your code a little bit to give an example > > > On Thu, Sep 6, 2012 at 8:42 PM, [email protected] < > [email protected]> wrote: > >> One question on my tiny wicket script attempts :) >> If you add something, [in my example target.add(form) ] >> shouldn't I remove the form before adding it? >> Or does wicket care about it automatically? >> >> Thanks! >> Sebastian >> >> 2012/9/6 [email protected] <[email protected]>: >> > I would not rename Java Objects now. >> > They have influence on the client side. >> > I would prefer renaming things when we have refactored to Ajax, as we >> > can more easily fix the possible changes in the UI. >> > >> > I have committed a small form component (UserForm.java). >> > Could you review those script lines if that is the way you would do it >> > in Wicket too? >> > >> > Sebastian >> > >> > 2012/9/6 Maxim Solodovnik <[email protected]>: >> >> What about replacing persistence.beans with data.entity or maybe >> dao.entity >> >> and move entities to one folder with same naming >> >> like: >> >> PollType -> RoomPollType >> >> Rooms_Organisation -> RoomsOrganisation etc. >> >> >> >> What do you think? >> >> >> >> On Thu, Sep 6, 2012 at 7:32 PM, [email protected] < >> [email protected] >> >>> wrote: >> >> >> >>> Sure, >> >>> >> >>> maybe at some point it would make sense to group some of the packages. >> >>> >> >>> Sebastian >> >>> >> >>> 2012/9/6 Maxim Solodovnik <[email protected]>: >> >>> > Maybe we can get rid of app package? >> >>> > like have >> >>> > org.apache.openmeetings.data package instead of >> >>> > org.apache.openmeetings.app.data? >> >>> > >> >>> > On Thu, Sep 6, 2012 at 7:15 PM, Maxim Solodovnik < >> [email protected] >> >>> >wrote: >> >>> > >> >>> >> The only places need also be updated are: >> >>> >> 1) *persistence.xml files >> >>> >> 2) openmeetings-applicationContext.xml >> >>> >> >> >>> >> I can handle it >> >>> >> >> >>> >> >> >>> >> On Thu, Sep 6, 2012 at 6:40 PM, [email protected] < >> >>> >> [email protected]> wrote: >> >>> >> >> >>> >>> Hi, >> >>> >>> >> >>> >>> one thing I constantly notice but did not resolve yet is our >> package >> >>> >>> structure. >> >>> >>> We should fix that before we try to apply to graduate. >> >>> >>> >> >>> >>> Our package structure is: >> >>> >>> org.openmeetings.* >> >>> >>> it should be: >> >>> >>> org.apache.openmeetings.* >> >>> >>> >> >>> >>> Maybe it is a good moment to do that refactoring right now. >> >>> >>> What do you think, are there critical areas where there could be >> >>> >>> potenial issues when doing this change? >> >>> >>> configs? What about the CLI installer, does it rely on some of the >> >>> >>> package path's in the scripts? >> >>> >>> >> >>> >>> Sebastian >> >>> >>> -- >> >>> >>> Sebastian Wagner >> >>> >>> https://twitter.com/#!/dead_lock >> >>> >>> http://www.webbase-design.de >> >>> >>> http://www.wagner-sebastian.com >> >>> >>> [email protected] >> >>> >>> >> >>> >> >> >>> >> >> >>> >> >> >>> >> -- >> >>> >> WBR >> >>> >> Maxim aka solomax >> >>> >> >> >>> > >> >>> > >> >>> > >> >>> > -- >> >>> > WBR >> >>> > Maxim aka solomax >> >>> >> >>> >> >>> >> >>> -- >> >>> Sebastian Wagner >> >>> https://twitter.com/#!/dead_lock >> >>> http://www.webbase-design.de >> >>> http://www.wagner-sebastian.com >> >>> [email protected] >> >>> >> >> >> >> >> >> >> >> -- >> >> WBR >> >> Maxim aka solomax >> > >> > >> > >> > -- >> > Sebastian Wagner >> > https://twitter.com/#!/dead_lock >> > http://www.webbase-design.de >> > http://www.wagner-sebastian.com >> > [email protected] >> >> >> >> -- >> Sebastian Wagner >> https://twitter.com/#!/dead_lock >> http://www.webbase-design.de >> http://www.wagner-sebastian.com >> [email protected] >> > > > > -- > WBR > Maxim aka solomax > -- WBR Maxim aka solomax
