[
https://issues.apache.org/jira/browse/WICKET-606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12500501
]
Johan Compagner commented on WICKET-606:
----------------------------------------
i think that would work yes
It should be allowed to set the type yourself (but that is still the case)
because i could think of some string->string conversions (mask textfields and
so on)
and getConverter is i think only used when a type is set.
> AbstractTextComponent#setConvertEmptyInputStringToNull(true) does not work
> with IObjectClassAwareModels (affects TextField, etc.)
> ---------------------------------------------------------------------------------------------------------------------------------
>
> Key: WICKET-606
> URL: https://issues.apache.org/jira/browse/WICKET-606
> Project: Wicket
> Issue Type: Bug
> Affects Versions: trunk
> Reporter: Alastair Maw
> Priority: Blocker
> Fix For: 1.3.0-beta2
>
>
> The unit tests ought to cover this, but apparently don't. A TextField should
> convert empty strings to nulls if you call
> setConvertEmptyInputStringToNull(true) on it (which the constructors do by
> default).
> However, this doesn't currently work:
> - AbstractTextComponent#getConvertEmptyInputStringToNull() is only called
> from AbstractTextComponent#convertValue(String[]).
> - AbstractTextComponent#convertValue(String[]) is called from
> FormComponent#convert(), but only if the component doesn't have a type
> assigned to it.
> - As soon as you use a PropertyModel (which implements
> IObjectClassAwareModel), this means empty strings are no longer converted to
> nulls.
> This is obviously a great big blocker and needs fixing ASAP.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
