Skip to content

Commit

Permalink
TASK: Make NodeAggregate::getNodes internal
Browse files Browse the repository at this point in the history
  • Loading branch information
mhsdesign committed May 13, 2024
1 parent 3febbd6 commit b205bf0
Showing 1 changed file with 11 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -91,16 +91,6 @@ public function occupiesDimensionSpacePoint(OriginDimensionSpacePoint $originDim
return $this->occupiedDimensionSpacePoints->contains($originDimensionSpacePoint);
}

/**
* Returns the nodes belonging to this aggregate, i.e. the "real materialized" node rows.
*
* @return iterable<int,Node>
*/
public function getNodes(): iterable
{
return array_values($this->nodesByOccupiedDimensionSpacePoint);
}

public function getNodeByOccupiedDimensionSpacePoint(
OriginDimensionSpacePoint $occupiedDimensionSpacePoint
): Node {
Expand Down Expand Up @@ -168,4 +158,15 @@ public function getDimensionSpacePointsTaggedWith(SubtreeTag $subtreeTag): Dimen
{
return $this->dimensionSpacePointsBySubtreeTags->forSubtreeTag($subtreeTag);
}

/**
* Returns the nodes belonging to this aggregate, i.e. the "real materialized" node rows.
*
* @internal Using this method to access all occupied nodes or possibly extract a single arbitrary node is not intended for use outside the core.
* @return iterable<int,Node>
*/
public function getNodes(): iterable
{
return array_values($this->nodesByOccupiedDimensionSpacePoint);
}
}

0 comments on commit b205bf0

Please sign in to comment.