[
https://issues.apache.org/jira/browse/WICKET-606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12501863
]
Johan Compagner commented on WICKET-606:
----------------------------------------
we always trimmed. So this is not new behavior this is behavior we always had
just because convertValue() wasn't called anymore for String values so it
didn't show this for a while in 1.3
i think in 99% of the cases i want trimming.
> 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
> Assignee: 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.
