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
