IBX-6554: Refactored location and content property accessors for better consistency and null safety

Author: tbialcz

src/bundle/Controller/ContentViewController.php

@@ -160,16 +160,16 @@ public function __construct(
     public function locationViewAction(Request $request, ContentView $view): ContentView
     {
         $location = $view->getLocation();
-        if ($location === null || $location->id === null) {
-            $contentId = $view->getContent()->id ?? 'unknown';
+        if ($location === null) {
+            $contentId = $view->getContent()->getId();
             throw new NotFoundException('Location', "content ID {$contentId}");
         }
 
         // We should not cache ContentView because we use forms with CSRF tokens in template
         // JIRA ref: https://issues.ibexa.co/browse/EZP-28190
         $view->setCacheEnabled(false);
 
-        if (!$view->getContent()->contentInfo->isTrashed()) {
+        if (!$view->getContent()->getContentInfo()->isTrashed()) {
             $this->supplyPathLocations($view);
             $this->subitemsContentViewParameterSupplier->supply($view);
             $this->supplyContentActionForms($view);

src/lib/Menu/Voter/LocationVoter.php

@@ -37,7 +37,7 @@ public function matchItem(ItemInterface $item): ?bool
                 if ($contentView instanceof ContentView) {
                     $location = $contentView->getLocation();
                     if ($location !== null) {
-                        $path = $location->path ?? [$location->id];
+                        $path = $location->getPath();
 
                         if (in_array($locationId, $path, true)) {
                             return true;