You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Jesse Kuhnert (JIRA)" <ta...@jakarta.apache.org> on 2006/12/16 21:10:25 UTC

[jira] Resolved: (TAPESTRY-732) LabeledPropertySelectionModel should support null String values

     [ http://issues.apache.org/jira/browse/TAPESTRY-732?page=all ]

Jesse Kuhnert resolved TAPESTRY-732.
------------------------------------

    Resolution: Fixed

Not sure when it was fixed but I think this is being handled now.

> LabeledPropertySelectionModel should support null String values
> ---------------------------------------------------------------
>
>                 Key: TAPESTRY-732
>                 URL: http://issues.apache.org/jira/browse/TAPESTRY-732
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: Framework
>    Affects Versions: 4.0
>            Reporter: Leonardo Quijano Vincenzi
>            Priority: Minor
>             Fix For: 4.1.1
>
>
> Just for robustness, LabeledPropertySelectionModel (and similar implementations of IPropertySelectionModel), should support null values in translateField:
>     public Object translateValue(String value)
>     {
>         return value.equals(_value) ? _option : _model.translateValue(value);
>     }
> This is throwing NPEs when value is null. I think null could be a valid value in some cases, but in any case, it doesn't hurt to handle nulls in this method (or at least throw a more verbose NPE?).

-- 
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: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org