fixed a bug in large trees - see issue #210 (comment)

(cannot reproduce in tests, but can reproduce in https://plnkr.co/edit/rWgobIGvTF9rnzyI02Mu?p=preview)

Author: yoavaa

angular-tree-control.js

@@ -130,7 +130,7 @@ if (typeof module !== "undefined" && typeof exports !== "undefined" && module.ex
                     $scope.expandedNodes = $scope.expandedNodes || [];
                     $scope.expandedNodesMap = {};
                     for (var i=0; i < $scope.expandedNodes.length; i++) {
-                        $scope.expandedNodesMap[""+i] = $scope.expandedNodes[i];
+                        $scope.expandedNodesMap["a"+i] = $scope.expandedNodes[i];
                     }
                     $scope.parentScopeOfTree = $scope.$parent;
 
@@ -351,7 +351,7 @@ if (typeof module !== "undefined" && typeof exports !== "undefined" && module.ex
                                     }
                                 }
                                 if (!found)
-                                    newExpandedNodesMap[notFoundIds++] = newExNode;
+                                    newExpandedNodesMap['a' + notFoundIds++] = newExNode;
                             });
                             scope.expandedNodesMap = newExpandedNodesMap;
                         });

test/angular-tree-control-test.js

@@ -873,7 +873,9 @@ describe('treeControl', function() {
             element.find('li:eq(1) .tree-branch-head').click();
             expect($rootScope.expandedNodes).not.toContain($rootScope.treedata[1]);
         });
+    });
 
+    describe('expanded-nodes binding', function () {
         it('should retain expansions after full model refresh', function () {
             var testTree = createSubTree(2, 2);
             $rootScope.treedata = angular.copy(testTree);
@@ -888,6 +890,32 @@ describe('treeControl', function() {
             expect(element.find('li:eq(0)').hasClass('tree-expanded')).toBeTruthy();
         });
 
+        it('should support a large tree', function () {
+            var testTree = createSubTree(3, 10);
+            element = $compile('<treecontrol tree-model="treedata" expanded-nodes="expandedNodes">{{node.label}}</treecontrol>')($rootScope);
+
+            var expandedNodes = [];
+            function diveInto(parent) {
+                expandedNodes.push(parent);
+                if (Array.isArray(parent.children)) {
+                    parent.children.forEach(function(child) {
+                        diveInto(child);
+                    });
+                }
+            }
+            testTree.forEach(function(rootNode) {
+                diveInto(rootNode);
+            });
+
+            $rootScope.treedata = testTree;
+            $rootScope.$digest();
+            $rootScope.expandedNodes = expandedNodes;
+            $rootScope.$digest();
+
+            //console.log(element);
+            expect(element.find('li.tree-expanded').length + element.find('li.tree-leaf').length).toBe($rootScope.expandedNodes.length);
+        });
+
     });
 
     describe('external template', function () {