Done, see here:  https://issues.apache.org/jira/browse/WICKET-934 WICKET-934 


Matej Knopp-2 wrote:
> 
> I thought we were trimming textfield strings, but now i see it's no
> longer the case. Is it a regression or a feature?
> 
> Also the missing message doesn't make much sense, can you please
> report a JIRA issue and attach a quick start project?
> 
> -Matej
> 
> On 9/6/07, Alex Objelean <[EMAIL PROTECTED]> wrote:
>>
>> I've discovered the following issue:
>> When using a DateTextField in a form (with the format: dd/MM/yyyy), user
>> by
>> mistake add an extra space (for instance: "06/09/2007 ") and submit the
>> form. The are two problems:
>>
>> 1) I would expect the convertor to convert this value correct, but the
>> AbstractConverter.parse method throws a ConversionException because:
>> (position.getIndex() != stringValue.length())
>>
>> 2) The above exception is not reported anywhere, because validate method
>> does not check if the input is valid after converting its value:
>>
>>
>>
>> >       public final void validate()
>> >       {
>> >               validateRequired();
>> >               if (isValid())
>> >               {
>> >                       convertInput();
>> >
>> >                       if (isValid() && isRequired() &&
>> getConvertedInput() == null &&
>> > isInputNullable())
>> >                       {
>> >                               reportRequiredError();
>> >                       }
>> >                         /*isn't something like this missing here?*/ if
>> > (!isValid()) { /*report Conversion error*/ }
>> >                       if (isValid())
>> >                       {
>> >                               validateValidators();
>> >                       }
>> >               }
>> >       }
>> >
>>
>> Any thoughts?
>>
>> Thank you!
>> Alex.
>> --
>> View this message in context:
>> http://www.nabble.com/Wicket-validation-flaw--tf4390183.html#a12517215
>> Sent from the Wicket - User mailing list archive at Nabble.com.
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
> 
> 

-- 
View this message in context: 
http://www.nabble.com/Wicket-validation-flaw--tf4390183.html#a12519551
Sent from the Wicket - User mailing list archive at Nabble.com.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to