[jira] Commented: (WICKET-606) AbstractTextComponent#setConvertEmptyInputStringToNull(true) does not work with IObjectClassAwareModels (affects TextField, etc.)

["Johan Compagner (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WICKET-606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12500471 ] 

Johan Compagner commented on WICKET-606:
----------------------------------------

by the way. The problem with this is IF the developer overrides getConverter again
he has to honer this them selfs.



> 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.