Refactor: Using right URL for dynamic section & Page filtering accordingly & test cases

Author: thapacodes4u

src/Admin/Filament/Resources/PageResource.php

@@ -30,6 +30,7 @@
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\SoftDeletingScope;
 use Illuminate\Support\Str;
+use Livewire\Attributes\Url;
 
 class PageResource extends Resource
 {
@@ -47,6 +48,12 @@ class PageResource extends Resource
 
     protected static ?string $navigationLabel = 'Pages';
 
+    // /**
+    //  * @var array<string, mixed> | null
+    //  */
+    // #[Url()]
+    // public ?array $tableFilters = null;
+
     public static function form(Form $form): Form
     {
         return $form
@@ -163,6 +170,10 @@ public static function form(Form $form): Form
     public static function table(Table $table): Table
     {
         return $table
+            ->modifyQueryUsing(fn (Builder $query) => $query->when(
+                request()->get('section'),
+                fn (Builder $q, $sectionId) => $q->where('section_id', $sectionId)
+            ))
             ->columns([
                 TextColumn::make('title')
                     ->label('Page Title')
@@ -216,11 +227,6 @@ public static function table(Table $table): Table
                     ->toggleable(),
             ])
             ->filters([
-                SelectFilter::make('section')
-                    ->relationship('section', 'name')
-                    ->searchable()
-                    ->preload(),
-
                 SelectFilter::make('status')
                     ->options(PageStatus::class),
 

src/CmsPlugin.php

@@ -2,7 +2,7 @@
 
 namespace Eclipse\Cms;
 
-use Eclipse\Cms\Admin\Filament\Resources\SectionResource;
+use Eclipse\Cms\Admin\Filament\Resources\PageResource;
 use Eclipse\Cms\Models\Section;
 use Eclipse\Common\Foundation\Plugins\Plugin;
 use Exception;
@@ -27,8 +27,8 @@ public function getSectionNavigationItems(): array
                 ->get()
                 ->map(fn (Section $section): NavigationItem => NavigationItem::make($section->getTranslation('name', app()->getLocale()))
                     ->url(
-                        fn (): string => SectionResource::getUrl('edit', [
-                            'record' => $section->id,
+                        fn (): string => PageResource::getUrl('index', [
+                            'section' => $section->id,
                         ])
                     )
                     ->icon('heroicon-o-arrow-turn-down-right')

tests/Feature/Filament/Resources/PageResourceTest.php

@@ -1,6 +1,7 @@
 <?php
 
 use Eclipse\Cms\Admin\Filament\Resources\PageResource;
+use Eclipse\Cms\CmsPlugin;
 use Eclipse\Cms\Enums\PageStatus;
 use Eclipse\Cms\Models\Page;
 use Eclipse\Cms\Models\Section;
@@ -152,3 +153,31 @@
 
     expect($page->fresh()->trashed())->toBeTrue();
 });
+
+test('pages can be filtered by section via URL parameter', function () {
+    $section1 = Section::factory()->create(['name' => ['en' => 'Section 1']]);
+    $section2 = Section::factory()->create(['name' => ['en' => 'Section 2']]);
+
+    $page1 = Page::factory()->forSection($section1)->create();
+    $page2 = Page::factory()->forSection($section2)->create();
+
+    $response = $this->get(PageResource::getUrl('index').'?section='.$section1->id);
+
+    $response->assertSuccessful();
+    $response->assertSee($page1->title);
+    $response->assertDontSee($page2->title);
+});
+
+test('section navigation items generate correct URLs', function () {
+    $section = Section::factory()->create(['name' => ['en' => 'Test Section']]);
+
+    $plugin = new CmsPlugin;
+    $navigationItems = $plugin->getSectionNavigationItems();
+
+    expect($navigationItems)->toHaveCount(1);
+
+    $item = $navigationItems[0];
+    expect($item->getLabel())->toBe('Test Section');
+    expect($item->getUrl())->toContain('section='.$section->id);
+    expect($item->getGroup())->toBe('CMS');
+});