I'm using datetextfield from the wicket-extensions.
Alex.
igor.vaynberg wrote:
>
> actually we do trim them.
>
> convertinput() calls convertvalue() which does trim. so if you override
> convertvalue() itself you need to trim yourself. however i do not see it
> being overwritten anywhere. is this datetextfield from extensions or
> wicket-datetime?
>
> -igor
>
>
> On 9/6/07, Matej Knopp <[EMAIL PROTECTED]> 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#a12536802
Sent from the Wicket - User mailing list archive at Nabble.com.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
