You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-issues@incubator.apache.org by "Adam Winer (JIRA)" <ad...@incubator.apache.org> on 2006/05/08 17:41:21 UTC

[jira] Closed: (ADFFACES-1) tableSelectMany may break with non-string row keys

     [ http://issues.apache.org/jira/browse/ADFFACES-1?page=all ]
     
Adam Winer closed ADFFACES-1:
-----------------------------

    Resolution: Fixed

Fixed in SVN drop.

> tableSelectMany may break with non-string row keys
> --------------------------------------------------
>
>          Key: ADFFACES-1
>          URL: http://issues.apache.org/jira/browse/ADFFACES-1
>      Project: MyFaces ADF-Faces
>         Type: Bug

>  Environment: all
>     Reporter: Frank Felix Debatin
>     Assignee: Adam Winer

>
> from RowKeySet.java: the cast to String is unnecessary and wrong. (This would break if you clicked "Select All", then deselected a row, and had a non-string row key).
>  private Object _next()
>       {
>         while(rowKeyIterator.hasNext())
>         {
>           String rowKey = (String) rowKeyIterator.next(); // !? 
>           if (!set.contains(rowKey))
>             return rowKey;
>         }
>         return null;
>       }

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