[ 
http://issues.apache.org/jira/browse/TAPESTRY-636?page=comments#action_12358215 
] 

Jesse Kuhnert commented on TAPESTRY-636:
----------------------------------------

This is more or less what was recently (last 48 hours or so ;) ) being worked 
on in tacos. It of course relied on just bypassing client-side validation, 
while server-side errors were left to their own devices...

I like the state tracking being broken out of the validation portion idea very 
much. Implementation might be harder than thought though if people start doing 
double-duty on things like parsing/formatting incoming data for both validation 
and data capture. Maybe field tracking already handling exceptions thrown and 
data values separately (more or less) provides enough of an existing model to 
make this feasible without any new perf hits? 

> field input value is not used if the field is not in error
> ----------------------------------------------------------
>
>          Key: TAPESTRY-636
>          URL: http://issues.apache.org/jira/browse/TAPESTRY-636
>      Project: Tapestry
>         Type: Bug
>   Components: Framework
>     Versions: 4.0
>  Environment: Windows XP
>     Reporter: Kent Tong
>     Assignee: Howard M. Lewis Ship
>      Fix For: 4.0

>
> This bug is related to bug TAPESTRY-549. After calling clearErrors(), the 
> errors are cleared and the input field values are still kept. When the 
> TextField renders the next, it should display its existing input field value, 
> but currently it will do so only when it is in error. The expected behavior 
> is that it should display its existing input field value as long as there is 
> a tracking for it.
> The bug is in TranslatedFieldSupportImpl:
> public class TranslatedFieldSupportImpl implements TranslatedFieldSupport
> {
>     public String format(TranslatedField field, Object object)
>     {
>         IValidationDelegate delegate = field.getForm().getDelegate();
>        //BUG: uses field input value only if it is in error
>         return delegate.isInError() ? delegate.getFieldInputValue() : 
> field.getTranslator().format(
>                 field,
>                 _threadLocale.getLocale(),
>                 object);
>     }
> }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to