Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

BUGFIX Aborting marking in Hierarchy->markPartialTree() before loadin…

…g nodes, if the expected node count would exceed the totals (regardless of the in-memory marking state)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@114133 467b73ca-7a2a-4603-9d3b-597d59a354a9
  • Loading branch information...
commit f1bf0d0fc4690e4598d9e761a3818a5a58984a71 1 parent 3ffa3bf
@chillu chillu authored
Showing with 4 additions and 0 deletions.
  1. +4 −0 core/model/Hierarchy.php
View
4 core/model/Hierarchy.php
@@ -100,6 +100,10 @@ public function markPartialTree($minNodeCount = 30, $context = null, $childrenMe
// foreach can't handle an ever-growing $nodes list
while(list($id, $node) = each($this->markedNodes)) {
+ // Inspect the number of potential nodes to mark - if its more than the minimum count,
+ // don't query them in the first place to inspect further marking state in-memory
+ if((sizeof($this->markedNodes) + $node->$numChildrenMethod()) >= $minNodeCount) break;
+
$this->markChildren($node, $context, $childrenMethod, $numChildrenMethod);
if($minNodeCount && sizeof($this->markedNodes) >= $minNodeCount) {
break;
Please sign in to comment.
Something went wrong with that request. Please try again.