Merge pull request #12 from TappNetwork/option_to_add_forum_by_user

Ability to add forum

Author: andreia

README.md

@@ -99,6 +99,7 @@ The `ForumUser` trait provides:
 - `getMentionableUsers()` method - Customize which users are mentionable (see "Custom Mentionables" below)
 - `hasCustomForumSearch()`, `getForumSearchResults()`, `getForumOptionLabel()` methods - Custom search functionality (see "Custom User Model, Attribute, and Search Functionality" below)
 - `isForumAdmin()` method - Override to grant admin access to hidden forums (defaults to `false`)
+- `canCreateForum()` method - Controls forum creation permissions (see "Forum Creation Permissions" below)
 
 6. Add to your custom theme (usually`theme.css`) file:
 
@@ -370,6 +371,49 @@ public static function getMentionableUsers()
 
 Forums can be set as public (visible to all logged-in users) or hidden (only visible to assigned users). Forum admins can see all hidden forums regardless of assignment.
 
+### Forum Creation Permissions
+
+By default, forum creation is **disabled** for security. To enable forum creation, you must add the `ForumUser` trait to your User model and override the `canCreateForum()` method:
+
+```php
+// In your User model
+use Tapp\FilamentForum\Traits\ForumUser;
+
+class User extends Authenticatable
+{
+    use ForumUser;
+    
+    /**
+     * Determine if the user can create forums.
+     * Override this method to enable forum creation with custom logic.
+     */
+    public function canCreateForum(): bool
+    {
+        // Example: Using Spatie Laravel Permission
+        return $this->hasPermissionTo('create forum');
+        
+        // Example: Using roles
+        // return $this->hasRole(['Admin', 'Moderator']);
+        
+        // Example: Allow all authenticated users
+        // return true;
+        
+        // Example: Custom logic
+        // return $this->is_verified && $this->reputation_points >= 100;
+    }
+}
+```
+
+**Important**: 
+- Make sure to add the `ForumUser` trait to your User model (see step 5 in Installation)
+- The trait provides a default implementation that returns `false` for security
+- You must explicitly override `canCreateForum()` to enable forum creation
+
+The `canCreateForum()` method is:
+- **Checked automatically** when displaying the "Create" button on the forums list page
+- **Enforced** when accessing the forum creation page
+- **Secure by default** - forum creation is disabled unless explicitly enabled
+
 ### Setting Forum Access
 
 In the admin panel, you can set a forum as hidden by checking the "Hidden Forum" checkbox. Hidden forums will only be visible to:

database/migrations/create_forum_user_table.php.stub

@@ -13,8 +13,8 @@ return new class extends Migration
     {
         Schema::create('forum_user', function (Blueprint $table) {
             $table->id();
-            $table->foreignId('user_id')->constrained()->onDelete('cascade');
-            $table->foreignId('forum_id')->constrained('forums')->onDelete('cascade');
+            $table->foreignId('user_id')->constrained()->cascadeOnDelete();
+            $table->foreignId('forum_id')->constrained('forums')->cascadeOnDelete();
             $table->timestamps();
             $table->unique(['user_id', 'forum_id']);
         });

database/migrations/create_forums_table.php

@@ -22,7 +22,7 @@ public function up(): void
                     ->cascadeOnDelete();
             }
 
-            $table->foreignId('owner_id')->nullable()->constrained('users')->onDelete('set null');
+            $table->foreignId('owner_id')->nullable()->constrained('users')->nullOnDelete();
             $table->string('name');
             $table->text('description')->nullable();
             $table->timestamps();

resources/lang/en/filament-forum.php

@@ -13,6 +13,10 @@
     'forum.navigation-label' => 'Forums',
     'forum.view-posts' => 'View Posts',
     'forum.active' => 'Active',
+    'forum.delete.tooltip' => 'Delete',
+    'forum.delete.modal.heading' => 'Delete Forum',
+    'forum.delete.modal.description' => 'Are you sure you want to delete this forum? This action cannot be undone.',
+    'forum.delete.notification' => 'Forum deleted successfully',
 
     /*
     |--------------------------------------------------------------------------

resources/views/filament/tables/components/forum-card-column.blade.php

@@ -1,6 +1,8 @@
 @php
     use Tapp\FilamentForum\Filament\Resources\Forums\ForumResource;
     $record = $getRecord();
+    $livewire = $getLivewire();
+    $canDelete = auth()->user() && auth()->id() === $record->owner_id && $record->forumPosts()->count() === 0;
 @endphp
 
 <div class="w-full h-full flex flex-col">
@@ -13,6 +15,12 @@
                 class="w-full h-full object-cover"
             >
         @endif
+        
+        @if($canDelete)
+            <div class="absolute top-2 right-2">
+                {{ ($livewire->deleteAction)(['record' => $record->id]) }}
+            </div>
+        @endif
     </div>
 
     {{-- Content --}}

src/Filament/Resources/Forums/ForumResource.php

@@ -7,13 +7,16 @@
 use Filament\Resources\Resource;
 use Filament\Schemas\Components\Fieldset;
 use Filament\Schemas\Components\Section;
+use Filament\Schemas\Schema;
 use Filament\Support\Enums\Alignment;
 use Filament\Tables\Columns\Layout\Stack;
 use Filament\Tables\Table;
 use Illuminate\Database\Eloquent\Model;
+use Tapp\FilamentForum\Filament\Resources\Forums\Pages\CreateForum;
 use Tapp\FilamentForum\Filament\Resources\Forums\Pages\ListForums;
 use Tapp\FilamentForum\Filament\Resources\Forums\Pages\ManageForumPosts;
 use Tapp\FilamentForum\Filament\Resources\Forums\Pages\ViewForum;
+use Tapp\FilamentForum\Filament\Resources\Forums\Schemas\ForumForm;
 use Tapp\FilamentForum\Filament\Tables\Components\ForumCardColumn;
 use Tapp\FilamentForum\Models\Forum;
 
@@ -56,6 +59,11 @@ public static function getNavigationLabel(): string
         return __('filament-forum::filament-forum.forum.navigation-label');
     }
 
+    public static function form(Schema $schema): Schema
+    {
+        return ForumForm::configure($schema);
+    }
+
     public static function table(Table $table): Table
     {
         return $table
@@ -104,6 +112,7 @@ public static function getPages(): array
 
         return [
             'index' => ListForums::route('/'),
+            'create' => CreateForum::route('/create'),
             // 'view' => ViewForum::route('/{record}'),
             'forum-posts' => ManageForumPosts::route("/{record}/{$forumPostSlug}"),
         ];
@@ -116,6 +125,19 @@ public static function getSlug(?\Filament\Panel $panel = null): string
 
     public static function canCreate(): bool
     {
+        $user = auth()->user();
+
+        if (! $user) {
+            return false;
+        }
+
+        // Check if the user model has the canCreateForum method
+        if (method_exists($user, 'canCreateForum')) {
+            return $user->canCreateForum();
+        }
+
+        // Backward compatibility: return false if method doesn't exist
+        // This ensures existing installations don't suddenly allow forum creation
         return false;
     }
 

src/Filament/Resources/Forums/Pages/CreateForum.php

@@ -8,4 +8,12 @@
 class CreateForum extends CreateRecord
 {
     protected static string $resource = ForumResource::class;
+
+    protected function mutateFormDataBeforeCreate(array $data): array
+    {
+        // Automatically set the owner to the currently logged-in user
+        $data['owner_id'] = auth()->id();
+
+        return $data;
+    }
 }

src/Filament/Resources/Forums/Pages/ListForums.php

@@ -2,6 +2,8 @@
 
 namespace Tapp\FilamentForum\Filament\Resources\Forums\Pages;
 
+use Filament\Actions\CreateAction;
+use Filament\Actions\DeleteAction;
 use Filament\Resources\Pages\ListRecords;
 use Illuminate\Contracts\Support\Htmlable;
 use Illuminate\Support\Facades\Auth;
@@ -17,6 +19,35 @@ public function getTitle(): string|Htmlable
         return __('filament-forum::filament-forum.forum.title');
     }
 
+    public function deleteAction(): DeleteAction
+    {
+        return DeleteAction::make()
+            ->requiresConfirmation()
+            ->modalHeading(__('filament-forum::filament-forum.forum.delete.modal.heading'))
+            ->modalDescription(__('filament-forum::filament-forum.forum.delete.modal.description'))
+            ->successNotificationTitle(__('filament-forum::filament-forum.forum.delete.notification'))
+            ->iconButton()
+            ->icon('heroicon-o-trash')
+            ->color('danger')
+            ->tooltip(__('filament-forum::filament-forum.forum.delete.tooltip'))
+            ->size('sm')
+            ->extraAttributes(['class' => 'p-1'])
+            ->record(fn (array $arguments) => Forum::find($arguments['record']))
+            ->visible(
+                fn (array $arguments) => auth()->check() &&
+                Forum::find($arguments['record'])?->owner_id === auth()->id() &&
+                Forum::find($arguments['record'])?->forumPosts()->count() === 0
+            );
+    }
+
+    protected function getHeaderActions(): array
+    {
+        return [
+            CreateAction::make()
+                ->visible(fn (): bool => static::getResource()::canCreate()),
+        ];
+    }
+
     protected function getTableQuery(): \Illuminate\Database\Eloquent\Builder
     {
         $user = Auth::user();

src/Filament/Resources/Forums/Schemas/ForumForm.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace Tapp\FilamentForum\Filament\Resources\Forums\Schemas;
+
+use Filament\Forms\Components\Checkbox;
+use Filament\Forms\Components\SpatieMediaLibraryFileUpload;
+use Filament\Forms\Components\Textarea;
+use Filament\Forms\Components\TextInput;
+use Filament\Schemas\Components\Section;
+use Filament\Schemas\Schema;
+
+class ForumForm
+{
+    public static function configure(Schema $schema): Schema
+    {
+        return $schema
+            ->components([
+                Section::make()
+                    ->id('forumForm')
+                    ->columnSpanFull()
+                    ->columns(2)
+                    ->schema([
+                        TextInput::make('name')
+                            ->label(__('filament-forum::filament-forum.forum.form.label.name'))
+                            ->required()
+                            ->maxLength(255)
+                            ->columnSpanFull(),
+                        Textarea::make('description')
+                            ->label(__('filament-forum::filament-forum.forum.form.label.description'))
+                            ->required()
+                            ->columnSpanFull(),
+                        SpatieMediaLibraryFileUpload::make('image')
+                            ->label(__('filament-forum::filament-forum.forum.form.label.image'))
+                            ->collection('images')
+                            ->columnSpanFull(),
+                        Checkbox::make('is_hidden')
+                            ->label('Hidden Forum')
+                            ->helperText('If checked, only assigned users can view this forum. If unchecked, all logged in users can view it.')
+                            ->live(),
+                    ]),
+            ]);
+    }
+}

src/Traits/ForumUser.php

@@ -91,4 +91,23 @@ public function isForumAdmin(): bool
         // Override this in your User model to implement admin logic
         return false;
     }
+
+    /**
+     * Determine if the user can create forums.
+     *
+     * By default, this returns false for security. Override this method in your
+     * User model to enable forum creation with custom logic.
+     *
+     * Example:
+     * ```
+     * public function canCreateForum(): bool
+     * {
+     *     return $this->hasPermissionTo('create forum');
+     * }
+     * ```
+     */
+    public function canCreateForum(): bool
+    {
+        return false;
+    }
 }