Merge pull request #98 from ezsystems/ibx-1972-refactored-location-he…

…lper

IBX-1972: Refactored LocationHelper::getParentLocationPathString

src/lib/Helper/LocationHelper.php

@@ -48,20 +48,18 @@ public function areLocationsVisible(ContentInfo $contentInfo): bool
     /**
      * Returns location path string based on $contentId.
      *
-     * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
      * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      */
     public function getParentLocationPathString(int $contentId): ?string
     {
-        $content = $this->contentService->loadContent($contentId);
-        $mainLocationId = $content->contentInfo->mainLocationId;
-        if (null === $mainLocationId) {
-            return null;
-        }
-
         try {
-            $location = $this->locationService->loadLocation($mainLocationId);
-            $parentLocation = $location->getParentLocation();
+            $content = $this->contentService->loadContent($contentId);
+            $mainLocation = $content->contentInfo->getMainLocation();
+            if (null === $mainLocation) {
+                return null;
+            }
+
+            $parentLocation = $mainLocation->getParentLocation();
 
             return null !== $parentLocation ? $parentLocation->pathString : null;
         } catch (NotFoundException $exception) {