If you believe that the usecase is obscure, shouldn't this approach to be
reconsidered? Or at least discussed a little bit more, before the final
release is done?
Thank you!
igor.vaynberg wrote:
>
> i believe it was jonathan who had an obscure usecase that null input was
> supposed to be converted to a nonnull object.
>
> -igor
>
>
> On 8/14/07, Alex Objelean <[EMAIL PROTECTED]> wrote:
>>
>>
>>
>> I found a quick fix for my issue: instead of disabling the Textfield, I
>> make
>> it readonly... still wondering why the change has been made in the
>> convert()
>> method.
>>
>>
>> Matej Knopp-2 wrote:
>> >
>> > At this point I don't know why the check was removed, but i suppose
>> > there was a reason for it. I'm not sure whether we should support the
>> > state when client and server are out of sync.
>> >
>> > -Matej
>> >
>> > On 8/14/07, Alex Objelean <[EMAIL PROTECTED]> wrote:
>> >>
>> >> This means that the component enable state must be always in sync with
>> >> the
>> >> client side state? Shouldn't it be set automatically as not enabled
>> after
>> >> the submit occurs? On the other hand, the same code worked great on
>> the
>> >> wicket-1.2.x branch..
>> >>
>> >> Please help!
>> >>
>> >>
>> >>
>> >> Matej Knopp-2 wrote:
>> >> >
>> >> > I'm not sure if it's bug in wicket. So your <input> is disabled, but
>> >> > the TextField component is not? That's not good, you need to disable
>> >> > the TextField too in that case.
>> >> >
>> >> > -Matej
>> >> >
>> >> > On 8/14/07, Alex Objelean <[EMAIL PROTECTED]> wrote:
>> >> >>
>> >> >> After migrating from wicket-1.2.6 to wicket-1.3.0-beta2 I had the
>> >> >> following
>> >> >> problem:
>> >> >>
>> >> >> application throws ConversionException when trying to convert a
>> null
>> >> >> value
>> >> >> of the Textfield wich is disabled on the clientside.
>> >> >>
>> >> >> I've take a look on the convert() method of the FormComponent class
>> >> and
>> >> >> noticed the difference from the 1.2.x branch which may cause the
>> >> issue:
>> >> >>
>> >> >> This snippet of code is from 1.3.0-beta2
>> >> >> [CODE]
>> >> >> if (typeName == null) {
>> >> >> //string conversion code
>> >> >> } else {
>> >> >> //type conversion code
>> >> >> }
>> >> >> [/CODE]
>> >> >>
>> >> >> This snippet of code is from 1.2.6
>> >> >> [CODE]
>> >> >> if (type == null) {
>> >> >> //string conversion code
>> >> >> } else if (!Strings.isEmpty(getInput())) {
>> >> >> //type conversion code
>> >> >> }
>> >> >> [/CODE]
>> >> >>
>> >> >> As you can see, in the 1.3.0-beta2 version, the conversion does not
>> >> check
>> >> >> if
>> >> >> the getInput() is an empty string before performing type
>> conversion.
>> I
>> >> >> wonder if it is a bug, or it is something that I missed?
>> >> >>
>> >> >> Thank you!
>> >> >> Alex
>> >> >> --
>> >> >> View this message in context:
>> >> >>
>> >>
>> http://www.nabble.com/Wicket-1.3-beta2-validation-%28conversion%29-bug--tf4265733.html#a12140062
>> >> >> 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-1.3-beta2-validation-%28conversion%29-bug--tf4265733.html#a12141596
>> >> 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-1.3-beta2-validation-%28conversion%29-bug--tf4265733.html#a12157292
>> Sent from the Wicket - User mailing list archive at Nabble.com.
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>
>
--
View this message in context:
http://www.nabble.com/Wicket-1.3-beta2-validation-%28conversion%29-bug--tf4265733.html#a12175844
Sent from the Wicket - User mailing list archive at Nabble.com.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
