Merge pull request #3866 from neos/task/4634-make-content-repository-service-internal

Task/4634 make content repository service internal

Author: mhsdesign

Classes/Controller/BackendController.php

@@ -151,14 +151,11 @@ public function indexAction(string $node = null)
         }
         $contentGraph = $contentRepository->getContentGraph($workspace->workspaceName);
 
-        $backendControllerInternals = $this->contentRepositoryRegistry->buildService(
-            $siteDetectionResult->contentRepositoryId,
-            new BackendControllerInternalsFactory()
-        );
-        $defaultDimensionSpacePoint = $backendControllerInternals->getDefaultDimensionSpacePoint();
+        $rootDimensionSpacePoints = $contentRepository->getVariationGraph()->getRootGeneralizations();
+        $arbitraryRootDimensionSpacePoint = array_shift($rootDimensionSpacePoints);
 
         $subgraph = $contentGraph->getSubgraph(
-            $nodeAddress ? $nodeAddress->dimensionSpacePoint : $defaultDimensionSpacePoint,
+            $nodeAddress ? $nodeAddress->dimensionSpacePoint : $arbitraryRootDimensionSpacePoint,
             VisibilityConstraints::withoutRestrictions()
         );
 
@@ -170,7 +167,7 @@ public function indexAction(string $node = null)
         if (!$rootNodeAggregate) {
             throw new \RuntimeException(sprintf('No sites root node found in content repository "%s", while fetching site node "%s"', $contentRepository->id->value, $siteDetectionResult->siteNodeName->value), 1724849303);
         }
-        $rootNode = $rootNodeAggregate->getNodeByCoveredDimensionSpacePoint($defaultDimensionSpacePoint);
+        $rootNode = $rootNodeAggregate->getNodeByCoveredDimensionSpacePoint($arbitraryRootDimensionSpacePoint);
 
         $siteNode = $subgraph->findNodeByPath(
             $siteDetectionResult->siteNodeName->toNodeName(),

Classes/Controller/BackendControllerInternals.php

@@ -36,10 +36,8 @@ class ContentDimensionsHelper implements ProtectedContextAwareInterface
      */
     public function contentDimensionsByName(ContentRepositoryId $contentRepositoryId): array
     {
-        $contentDimensionHelperInternals = $this->contentRepositoryRegistry->buildService($contentRepositoryId, new ContentDimensionsHelperInternalsFactory());
-        assert($contentDimensionHelperInternals instanceof ContentDimensionsHelperInternals);
-        $contentDimensionSource = $contentDimensionHelperInternals->contentDimensionSource;
-
+        $contentDimensionSource = $this->contentRepositoryRegistry->get($contentRepositoryId)
+            ->getContentDimensionSource();
         $dimensions = $contentDimensionSource->getContentDimensionsOrderedByPriority();
 
         $result = [];
@@ -72,8 +70,8 @@ public function contentDimensionsByName(ContentRepositoryId $contentRepositoryId
      */
     public function allowedPresetsByName(DimensionSpacePoint $dimensions, ContentRepositoryId $contentRepositoryId): array|object
     {
-        $contentDimensionHelperInternals = $this->contentRepositoryRegistry->buildService($contentRepositoryId, new ContentDimensionsHelperInternalsFactory());
-        $contentDimensionSource = $contentDimensionHelperInternals->contentDimensionSource;
+        $contentDimensionSource = $this->contentRepositoryRegistry->get($contentRepositoryId)
+            ->getContentDimensionSource();
 
         // TODO: re-implement this here; currently EVERYTHING is allowed!!
         $allowedPresets = [];