I do know that radio and check are directly attached to their imodel. But the imodel does not provide an alternate value for them.
I just dont see why not to provide an ichoicerender on these components when providing it for the radiochoice and dropdown choice. We have no way of controlling the value of the radios when using this component. And since we cant overide the getValue we are either stuck or need to use another component. But that implicates some redesign, and it's a little late for that now on our project at least. Hmm this maybe something for wicket-stuff or extensions then.. regards Nino -----Original Message----- From: [EMAIL PROTECTED] on behalf of Igor Vaynberg Sent: Fri 30-03-2007 17:15 To: [email protected] Subject: Re: [Wicket-user] Radio.getValue? radio and check do not take an ichoicerenderer because they do not need it. they do not traverse over a list of choices and try to look one up, they have the choice attached directly to them via imodel. -igor On 3/30/07, Nino Wael <[EMAIL PROTECTED]> wrote: > > That was whý I suggested that we could have another constructor that took > an Ichoicerender. There are tons of places in other wicket core components > that supply a constructor that takes the ichoicerender. > > SO I think it should be ok? I would really like that ichoicerenderer would > be supported by every component where it makes sense... > > > Should we move this do the dev list? > > regards Nino > > ________________________________ > > Fra: [EMAIL PROTECTED] på vegne af Igor Vaynberg > Sendt: to 29-03-2007 19:11 > Til: [email protected] > Emne: Re: [Wicket-user] Radio.getValue? > > > On 3/29/07, Eelco Hillenius <[EMAIL PROTECTED]> wrote: > > > Looking at that code, I don't understand why we even need: > private short uuid = -1; > > as there is no code other than in getValue that changes the value, > and > that value can just be recreated everytime it is needed (as it is > nothing more than a call to getPage().getAutoIndex()). Am I > missing > something here? > > > yes, you are missing the fact that getautoindex() increments the index on > every call that is why we cache it in the uuid var. > > > > WDYT? > > > personally i dont like making getvalue non-final. this will patch this > component for this kind of testing, but what about other components that do > not have a stable name? are we going to have to start opening implementation > details on all of them just because users want to test it with something > static like httpunit? just my 2c. > > -igor > > > > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share > your > opinions on IT & business topics through brief surveys-and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Wicket-user mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/wicket-user > > >
<<winmail.dat>>
------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________ Wicket-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/wicket-user
