fix: reduce section seeding, add WYSIWYG to short description, allow slug auto-generation

Author: thapacodes4u

database/seeders/CmsSeeder.php

@@ -30,7 +30,7 @@ protected function seedForTenant($tenant): void
     {
         $sections = Section::factory()
             ->forSite($tenant)
-            ->count(5)
+            ->count(2)
             ->create();
 
         $sections->each(function (Section $section): void {
@@ -48,7 +48,7 @@ protected function seedForTenant($tenant): void
     protected function seedWithoutTenancy(): void
     {
         $sections = Section::factory()
-            ->count(5)
+            ->count(2)
             ->create();
 
         $sections->each(function (Section $section): void {

src/Admin/Filament/Resources/BannerPositionResource/RelationManagers/BannerRelationManager.php

@@ -4,7 +4,6 @@
 
 use Eclipse\Cms\Models\Banner;
 use Eclipse\Cms\Rules\BannerImageDimensionRule;
-use Eclipse\Common\Filament\Tables\Columns\ImageColumn;
 use Eclipse\Common\Helpers\MediaHelper;
 use Filament\Actions;
 use Filament\Forms;
@@ -14,6 +13,7 @@
 use Filament\Schemas\Components\Utilities\Get;
 use Filament\Schemas\Schema;
 use Filament\Tables;
+use Filament\Tables\Columns\ImageColumn;
 use Filament\Tables\Table;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\SoftDeletingScope;
@@ -45,7 +45,6 @@ protected function getDynamicImageColumns(): array
         return $imageTypes->map(function ($imageType) {
             return ImageColumn::make("image_type_{$imageType->id}")
                 ->label($imageType->name)
-                ->preview()
                 ->getStateUsing(function (Banner $record) use ($imageType) {
                     $locale = $this->activeLocale ?? app()->getLocale();
                     $image = $record->images->where('type_id', $imageType->id)->first();
@@ -64,12 +63,6 @@ protected function getDynamicImageColumns(): array
 
                     return null;
                 })
-                ->title(function (Banner $record) use ($imageType) {
-                    $locale = $this->activeLocale ?? app()->getLocale();
-
-                    return $record->getTranslation('name', $locale).' - '.$imageType->name;
-                })
-                ->link(fn (Banner $record) => $record->link ?? '#')
                 ->sortable(false);
         })->toArray();
     }

src/Admin/Filament/Resources/PageResource.php

@@ -13,7 +13,6 @@
 use Filament\Forms\Components\Placeholder;
 use Filament\Forms\Components\RichEditor;
 use Filament\Forms\Components\Select;
-use Filament\Forms\Components\Textarea;
 use Filament\Forms\Components\TextInput;
 use Filament\Resources\Resource;
 use Filament\Schemas\Components\Section;
@@ -77,9 +76,9 @@ public static function form(Schema $schema): Schema
 
                         TextInput::make('sef_key')
                             ->label('URL Slug')
-                            ->required()
                             ->maxLength(255)
                             ->placeholder('auto-generated-from-title')
+                            ->helperText('Leave empty to auto-generate from title')
                             ->columnSpan(2),
 
                         Select::make('status')
@@ -102,10 +101,15 @@ public static function form(Schema $schema): Schema
                 Section::make('Content')
                     ->columnSpanFull()
                     ->schema([
-                        Textarea::make('short_text')
+                        RichEditor::make('short_text')
                             ->label('Short Description')
                             ->placeholder('Brief summary or excerpt...')
-                            ->rows(3)
+                            ->toolbarButtons([
+                                'bold',
+                                'italic',
+                                'underline',
+                                'link',
+                            ])
                             ->columnSpanFull(),
 
                         RichEditor::make('long_text')

src/Models/Menu/Item.php

@@ -11,6 +11,7 @@
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
+use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use SolutionForest\FilamentTree\Concern\ModelTree;
@@ -82,6 +83,13 @@ public function parent(): BelongsTo
         return $this->belongsTo(Item::class, 'parent_id');
     }
 
+    public function children(): HasMany
+    {
+        return $this->hasMany(static::class, $this->determineParentColumnName())
+            ->with('children')
+            ->orderBy($this->determineOrderColumnName());
+    }
+
     public function linkable(): MorphTo
     {
         return $this->morphTo('linkable', 'linkable_class', 'linkable_id');

tests/Unit/MenuItemTest.php

@@ -236,9 +236,9 @@
 
 it('deleting parent item cascades to delete children', function () {
     $menu = Menu::factory()->create();
-    $parent = Item::factory()->create(['menu_id' => $menu->id]);
-    $child1 = Item::factory()->create(['menu_id' => $menu->id, 'parent_id' => $parent->id]);
-    $child2 = Item::factory()->create(['menu_id' => $menu->id, 'parent_id' => $parent->id]);
+    $parent = Item::factory()->active()->create(['menu_id' => $menu->id]);
+    $child1 = Item::factory()->active()->create(['menu_id' => $menu->id, 'parent_id' => $parent->id]);
+    $child2 = Item::factory()->active()->create(['menu_id' => $menu->id, 'parent_id' => $parent->id]);
 
     expect(Item::withoutGlobalScopes()->where('parent_id', $parent->id)->count())->toBe(2);
     expect($child1->trashed())->toBeFalse();
@@ -269,10 +269,10 @@
 
 it('cascading delete only affects children, not siblings', function () {
     $menu = Menu::factory()->create();
-    $parent1 = Item::factory()->create(['menu_id' => $menu->id]);
-    $parent2 = Item::factory()->create(['menu_id' => $menu->id]);
-    $child1 = Item::factory()->create(['menu_id' => $menu->id, 'parent_id' => $parent1->id]);
-    $child2 = Item::factory()->create(['menu_id' => $menu->id, 'parent_id' => $parent2->id]);
+    $parent1 = Item::factory()->active()->create(['menu_id' => $menu->id]);
+    $parent2 = Item::factory()->active()->create(['menu_id' => $menu->id]);
+    $child1 = Item::factory()->active()->create(['menu_id' => $menu->id, 'parent_id' => $parent1->id]);
+    $child2 = Item::factory()->active()->create(['menu_id' => $menu->id, 'parent_id' => $parent2->id]);
 
     $parent1->delete();