holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed problem with an container object returning 'true' for 'getAllSelected', even if it has no children.

Browse Source
This commit is contained in:
Andreas Stöckel 2011-05-31 17:02:51 +00:00
parent 574911d077
commit 36d2f605cb
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
phpgwapi/js/egw_action/egw_action.js
View File

@ -825,10 +825,11 @@ egwActionObject.prototype.getAllSelected = function()
if (!this.children[i].getAllSelected())
return false;
}
// This introduced various bugs inside the egwAction system and breaks the
// recursion abortion case of this function!
// if (this.children.length == 0) return false;
return true;
// If this element is an container *and* does not have any children, we
// should return false. If this element is not an container we have to
// return true has this is the recursion base case
return (!egwBitIsSet(this.flags, EGW_AO_FLAG_IS_CONTAINER)) ||
(this.children.length > 0);
}
return false;