[jira] [Commented] (FLEX-20313) SelectItem on AdvancedDataGrid use weird code

["Mihai Chira (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FLEX-20313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142663#comment-15142663 ] 

Mihai Chira commented on FLEX-20313:
------------------------------------

Main issue was already resolved on the first Apache Flex release (4.8).
I also just made the asdoc changes suggested by the previous comment.

> SelectItem on AdvancedDataGrid use weird code
> ---------------------------------------------
>
>                 Key: FLEX-20313
>                 URL: https://issues.apache.org/jira/browse/FLEX-20313
>             Project: Apache Flex
>          Issue Type: Bug
>          Components: mx: DataGrid
>    Affects Versions: Adobe Flex SDK 3.2 (Release)
>         Environment: Affected OS(s): All OS Platforms
> Affected OS(s): All OS Platforms
> Language Found: English
>            Reporter: Adobe JIRA
>            Assignee: Mihai Chira
>             Fix For: Apache Flex 4.8 (parity release)
>
>
> In HierarchicalCollectionViewCursor, lines 400 to 426:
> public function findAny(values:Object):Boolean
>     {
>         seek(CursorBookmark.FIRST);
>         
>         var done:Boolean = false;
>         while (!done)
>         {
>             var o:Object = hierarchicalData.getData(current);
>             
>             var matches:Boolean = true;
>             for (var p:String in values)
>             {
>                 if (o[p] != values[p])
>                 {
>                     matches = false;
>                     break;
>                 }
>             }
>             if (matches)
>                 return true;
>             done = moveNext(); // Here is the problem
>         }
>         return false;
>     }
>  
> moveNext returns true if it was possible to move to the next position and false otherwise, so done should be not moveNext.
> Each time this method is called, it jumps after the first item and return false.
> This code was not tested ... no ?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)