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: wicket-user@lists.sourceforge.net 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
<<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 Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user