You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flex.apache.org by jm...@apache.org on 2015/11/18 05:21:44 UTC
[29/47] git commit: [flex-sdk] [refs/heads/develop] - FLEX-34778
CAUSE: When a node is opened,
HierarchicalCollectionView starts to listen to changes to its children
collection. And when it's closed,
it doesn't stop listening to these CollectionEvents. S
FLEX-34778
CAUSE: When a node is opened, HierarchicalCollectionView starts to listen to changes to its children collection. And when it's closed, it doesn't stop listening to these CollectionEvents. So when a closed node's child is replaced, nestedCollectionChangeHandler is triggered, even if the node itself is closed. Part of nestedCollectionChangeHandler's function is to dispatch a REMOVE CollectionEvent for all the nodes which were removed by the replacement (which is, all the descendants of the replaced - but closed! - node). Then, since the node that's being replaced is not accessible anymore (because its parent was closed), it's not added to convertedEvent.items. But the next lines assume that the replaced node will be in that array without mistake. Otherwise (as it happens in this bus) it goes into an infinite loop trying to find it.
SOLUTION: instead of looking for the node with an (indefinitely) incrementing index, we're now using Array.indexOf() to locate the node and remove it if it's found.
Project: http://git-wip-us.apache.org/repos/asf/flex-sdk/repo
Commit: http://git-wip-us.apache.org/repos/asf/flex-sdk/commit/d8c1a9fc
Tree: http://git-wip-us.apache.org/repos/asf/flex-sdk/tree/d8c1a9fc
Diff: http://git-wip-us.apache.org/repos/asf/flex-sdk/diff/d8c1a9fc
Branch: refs/heads/develop
Commit: d8c1a9fcaaecd3acd4e70bf7a7cd24133a2106b4
Parents: 83e4a9b
Author: Mihai Chira <mi...@apache.org>
Authored: Tue Mar 10 13:00:10 2015 +0100
Committer: Erik de Bruin <er...@ixsoftware.nl>
Committed: Fri Mar 20 09:51:21 2015 +0100
----------------------------------------------------------------------
.../src/mx/collections/HierarchicalCollectionView.as | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/flex-sdk/blob/d8c1a9fc/frameworks/projects/advancedgrids/src/mx/collections/HierarchicalCollectionView.as
----------------------------------------------------------------------
diff --git a/frameworks/projects/advancedgrids/src/mx/collections/HierarchicalCollectionView.as b/frameworks/projects/advancedgrids/src/mx/collections/HierarchicalCollectionView.as
index 6d4901c..d752e8e 100644
--- a/frameworks/projects/advancedgrids/src/mx/collections/HierarchicalCollectionView.as
+++ b/frameworks/projects/advancedgrids/src/mx/collections/HierarchicalCollectionView.as
@@ -1152,7 +1152,7 @@ public class HierarchicalCollectionView extends EventDispatcher
private function internalRefresh(dispatch:Boolean):Boolean
{
var obj:Object;
- var coll:ICollectionView
+ var coll:ICollectionView;
var needUpdate:Boolean = false;
// apply filter function to all the collections including the child collections
@@ -1531,9 +1531,9 @@ public class HierarchicalCollectionView extends EventDispatcher
for (i = 0; i < n; i++)
{
node = ce.items[i].oldValue;
- while (convertedEvent.items[j] != node)
- j++;
- convertedEvent.items.splice(j, 1);
+ var replacedNodePosition:int = convertedEvent.items.indexOf(node);
+ if(replacedNodePosition != -1)
+ convertedEvent.items.splice(replacedNodePosition, 1);
}
if (convertedEvent.items.length)
{
@@ -1654,9 +1654,10 @@ public class HierarchicalCollectionView extends EventDispatcher
changingNode = ce.items[i].oldValue;
if (changingNode is XML)
stopTrackUpdates(changingNode);
- while (convertedEvent.items[j] != changingNode)
- j++;
- convertedEvent.items.splice(j, 1);
+
+ var replacedNodePosition:int = convertedEvent.items.indexOf(changingNode);
+ if(replacedNodePosition != -1)
+ convertedEvent.items.splice(replacedNodePosition, 1);
}
if (convertedEvent.items.length)
{