Children lazy empty nodes remain checked when parent is unchecked with hierarchical multi-selection

Close mar10#250

Author: mar10

CHANGELOG.md

@@ -1,6 +1,7 @@
 # 2.2.0 / Unreleased
   * [Improved] #245 tree.generateInput() now returns data using PHPs array
-    convention, i.e. by appending '[]' to the name.
+    convention, i.e. by appending brackets to the name: 'ft_1[]'.
+  * [Fixed] #250: Children lazy empty nodes remain checked when parent is unchecked with hierarchical multi-selection
 
 
 # 2.1.0 / 2014-05-29

demo/sample-select.html

@@ -40,9 +40,9 @@
 				}
 			]
 		},
-		{title: "Document with some children (expanded on init)", key: "id4", expand: true,
+		{title: "Document with some children (expanded on init)", key: "id4", expanded: true,
 			children: [
-				{title: "Sub-item 4.1 (active on init)", activate: true,
+				{title: "Sub-item 4.1 (active on init)", active: true,
 					children: [
 						{title: "Sub-item 4.1.1", key: "id4.1.1" },
 						{title: "Sub-item 4.1.2", key: "id4.1.2" }
@@ -57,7 +57,8 @@
 				{title: "Sub-item 4.3 (hideCheckbox)", hideCheckbox: true },
 				{title: "Sub-item 4.4 (unselectable)", unselectable: true }
 			]
-		}
+		},
+		{title: "Lazy folder", folder: true, lazy: true }
 	];
 	$(function(){
 
@@ -123,6 +124,12 @@
 			checkbox: true,
 			selectMode: 3,
 			source: treeData,
+			lazyLoad: function(event, ctx) {
+   				ctx.result = {url: "ajax-sub2.json", debugDelay: 1000};
+   			},
+			loadChildren: function(event, ctx) {
+   				ctx.node.fixSelection3AfterClick();
+   			},
 			select: function(event, data) {
 				// Get a list of all selected nodes, and convert to a key array:
 				var selKeys = $.map(data.tree.getSelectedNodes(), function(node){

src/jquery.fancytree.js

@@ -614,7 +614,7 @@ FancytreeNode.prototype = /** @lends FancytreeNode# */{
 			var i, l, child, s, state, allSelected,someSelected,
 				children = node.children;
 
-			if( children ){
+			if( children && children.length ){
 				// check all children recursively
 				allSelected = true;
 				someSelected = false;