fix: enhance action URL handling in relation managers

Author: cklei-carly

src/Base/Filament/RelationManagers/BaseChildrenRelationManager.php

@@ -2,8 +2,10 @@
 
 namespace SolutionForest\InspireCms\Base\Filament\RelationManagers;
 
+use Filament\Actions\Action;
 use Filament\Actions\CreateAction;
 use Filament\Actions\EditAction;
+use Filament\Actions\ViewAction;
 use Filament\Resources\RelationManagers\RelationManager;
 use Filament\Schemas\Schema;
 use Filament\Tables\Table;
@@ -32,7 +34,9 @@ public function table(Table $table): Table
             ->pluralModelLabel(lcfirst(__('inspirecms::inspirecms.children.plural')))
             ->emptyStateHeading(null)
             ->headerActions([
-                CreateAction::make(),
+                CreateAction::make()
+                    ->slideOver()
+                    ->modalWidth('7xl'),
             ]);
     }
 
@@ -41,44 +45,55 @@ public static function getTitle(Model $ownerRecord, string $pageClass): string
         return __('inspirecms::inspirecms.children.plural');
     }
 
-    protected function configureCreateAction(CreateAction $action): void
+    public function getDefaultActionUrl(Action $action): ?string
     {
-        parent::configureCreateAction($action);
+        $relatedResource = static::getRelatedResource() ?? $this->getPageClass()::getResource();
 
-        if ($this->isRedirectToCreatePage()) {
+        if (! $relatedResource) {
+            return null;
+        }
 
-            $resource = $this->getPageClass()::getResource();
+        if (
+            ($action instanceof CreateAction)
+        ) {
 
-            $parameters = ['parent' => $this->getOwnerRecord()->getKey()];
+            if ($this->isRedirectToCreatePage()) {
 
-            $url = FilamentResourceHelper::attemptToGetUrl($resource, ['create'], $parameters, false);
+                $parameters = ['parent' => $this->getOwnerRecord()->getKey()];
 
-            if ($url) {
-                $action->url($url);
+                return FilamentResourceHelper::attemptToGetUrl($relatedResource, ['create'], $parameters, false);
             }
 
+            if ($relatedResource::hasPage('create')) {
+                return $relatedResource::getUrl('create', shouldGuessMissingParameters: true);
+            }
         }
 
-        $action
-            ->slideOver()
-            ->modalWidth('7xl');
-    }
-
-    protected function configureEditAction(EditAction $action): void
-    {
-        parent::configureEditAction($action);
+        if ($action instanceof EditAction) {
+            
+            if ($this->isRedirectToDetailPage()) {
+                return FilamentResourceHelper::attemptToGetUrl($relatedResource, 'edit', ['record' => $action->getRecord()], false);
+            } elseif ($relatedResource::hasPage('edit')) {
+                return $relatedResource::getUrl('edit', ['record' => $action->getRecord()], shouldGuessMissingParameters: true);
+            }
 
-        if ($this->isRedirectToDetailPage()) {
-            $resource = $this->getPageClass()::getResource();
+        }
 
-            $action->url(
-                fn ($record) => FilamentResourceHelper::attemptToGetUrl($resource, 'edit', ['record' => $record], false)
-            );
+        // if (
+        //     ($action instanceof EditAction) &&
+        //     ($relatedResource::hasPage('edit'))
+        // ) {
+        //     return $relatedResource::getUrl('edit', ['record' => $action->getRecord()], shouldGuessMissingParameters: true);
+        // }
+
+        if (
+            ($action instanceof ViewAction) &&
+            ($relatedResource::hasPage('view'))
+        ) {
+            return $relatedResource::getUrl('view', ['record' => $action->getRecord()], shouldGuessMissingParameters: true);
         }
 
-        $action
-            ->slideOver()
-            ->modalWidth('7xl');
+        return null;
     }
 
     protected function isRedirectToDetailPage(): bool

src/Filament/Resources/ContentResource/RelationManagers/ChildrenRelationManager.php

@@ -2,6 +2,7 @@
 
 namespace SolutionForest\InspireCms\Filament\Resources\ContentResource\RelationManagers;
 
+use Filament\Actions\Action;
 use Filament\Actions\EditAction;
 use Filament\Actions\ViewAction;
 use Filament\Tables\Table;
@@ -69,26 +70,21 @@ public function getParentKey(): string | int | null
         return $this->getOwnerRecord()->parent_id;
     }
 
-    protected function configureEditAction(EditAction $action): void
+    public function getDefaultActionUrl(Action $action): ?string
     {
-        parent::configureEditAction($action);
-
+        $base = parent::getDefaultActionUrl($action);
+        
         $resource = $this->getPageClass()::getResource();
 
-        $action->url(
-            fn ($record) => FilamentResourceHelper::attemptToGetUrl($resource, 'edit', ['record' => $record, ...$this->getRedirectUrlParameters()], false)
-        );
-    }
-
-    protected function configureViewAction(ViewAction $action): void
-    {
-        parent::configureViewAction($action);
+        if ($action instanceof EditAction) {
+            return FilamentResourceHelper::attemptToGetUrl($resource, 'edit', ['record' => $action->getRecord(), ...$this->getRedirectUrlParameters()], false);
+        }
 
-        $resource = $this->getPageClass()::getResource();
+        if ($action instanceof ViewAction) {
+            return FilamentResourceHelper::attemptToGetUrl($resource, 'view', ['record' => $action->getRecord(), ...$this->getRedirectUrlParameters()], false);
+        }
 
-        $action->url(
-            fn ($record) => FilamentResourceHelper::attemptToGetUrl($resource, 'view', ['record' => $record, ...$this->getRedirectUrlParameters()], false)
-        );
+        return $base;
     }
 
     protected function getRedirectUrlParameters(): array