Only call the selectCallback once, call it when setAllSelected is called and select the whole row when the checkbox is pre-checked

phpgwapi/js/egw_action/egw_action.js

@@ -1108,6 +1108,9 @@ egwActionObject.prototype.setAllSelected = function(_selected, _informParent)
 			this.selectedChildren.push(this.children[i]);
 		}
 	}
+
+	// Call the setSelectedCallback
+	egwQueueCallback(this.setSelectedCallback, [], this, "setSelectedCallback");
 }
 
 
@@ -1138,10 +1141,8 @@ egwActionObject.prototype.updateSelectedChildren = function(_child, _selected)
 		this.parent.updateSelectedChildren(this, wasEmpty);
 	}
 
-	if (this.setSelectedCallback)
+	// Call the setSelectedCallback
-	{
+	egwQueueCallback(this.setSelectedCallback, [], this, "setSelectedCallback");
-		this.setSelectedCallback.call(this); //TODO: Not called, when non-selected elements are made visible (treeview)
-	}
 }
 
 /**

phpgwapi/js/egw_action/egw_action_common.js

@@ -140,6 +140,31 @@ function egwArraysEqual(_ar1, _ar2)
 	return result;
 }
 
+var _egwQueuedCallbacks = {};
+function egwQueueCallback(_proc, _args, _context, _id)
+{
+	if (_proc)
+	{
+		var cur_id = 0;
+		if (typeof _egwQueuedCallbacks[_id] == "undefined")
+		{
+			cur_id = _egwQueuedCallbacks[_id] = 1;
+		}
+		else
+		{
+			cur_id = ++_egwQueuedCallbacks[_id];
+		}
+
+		window.setTimeout(function() {
+			if (_egwQueuedCallbacks[_id] == cur_id)
+			{
+				_proc.apply(_context, _args);
+				delete _egwQueuedCallbacks[_id];
+			}
+		}, 0);
+	}
+}
+
 /**
 sprintf() for JavaScript 0.6
 

phpgwapi/js/egw_action/egw_grid.js

@@ -52,7 +52,7 @@ function egwGrid(_parentNode, _columns, _objectManager, _fetchCallback, _columnC
 
 			if (self.selectedChangeCallback)
 			{
-				self.selectedChangeCallback.call(self.context, allSelected)
+				self.selectedChangeCallback.call(self.context, allSelected);
 			}
 		}
 	};

phpgwapi/js/egw_action/egw_grid_view.js

@@ -1688,10 +1688,11 @@ function egwGridViewRow_doUpdateData(_immediate)
 				}
 				else if (col.type == EGW_COL_TYPE_CHECKBOX)
 				{
-					this.checkbox.attr("checked",
+					var checked = (data[col.id].data === 0) ?
-						(data[col.id].data === 0) ?
 						egwBitIsSet(this.aoi.getState(), EGW_AO_STATE_SELECTED) :
-							data[col.id].data);
+						data[col.id].data;
+					this.checkbox.attr("checked", checked);
+					this.item.actionObject.setSelected(checked);
 				}
 				else
 				{

phpgwapi/js/egw_action/test/test_grid_view.html

@@ -38,6 +38,10 @@
 
 			var columns = 
 				[
+					{
+						"id": "check",
+						"type": EGW_COL_TYPE_CHECKBOX
+					},
 					{
 						"id": "name",
 						"caption": "Name",
@@ -210,6 +214,7 @@
 							data["ctime"] = (new Date).toUTCString();
 							data["owner"] = "as";
 							data["group"] = "stylitedevs";
+							data["check"] = true;
 
 							result.push({
 								"id": _elems[i],