You're right, it'd be good if the Renderer did the check in getConvertedValue(), but not in getSubmittedValue().
-- Adam On 9/12/07, Simon Lessard <[EMAIL PROTECTED]> wrote: > > Oups it seems that I stripped some important information when I reread > myself (one of the few times I did). > > The size check is done in getSubmittedValue so it's called mainly during > conversion which is correct, but I don't think getSubmittedValue should do > any validation, just return the String[]. > > > ~ Simon > > On 9/12/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > > > > > > > On 9/12/07, Simon Lessard < [EMAIL PROTECTED]> wrote: > > > > > > Hi all, > > > > > > Shuttle behavior is not the same as other selectMany components for > > > purpose of retrieving the submitted values in two way. Firstly, shuttles > > > submit a single parameter with values separated by ';'. That difference > > > cannot really be changed. However, shuttles also do a first validation in > > > that phase > > > > > > ... which phase, what method? > > > > -- Adam > > > > > > by comparing the submitted value list size with the LoV size (imply > > > SelectItem resolution at that point). However, I think that latter > > > validation shouldn't exist there and should be delayed until > > > getSelectedIndices is called inside SimpleSelectManyRenderer as with all > > > other selectMany components. > > > > > > > > > Thoughts, objections? > > > > > > ~ Simon > > > > > > > >
