refactor players page and add tabs

Author: Boy132

player-counter/plugin.json

@@ -2,7 +2,7 @@
     "id": "player-counter",
     "name": "Player Counter",
     "author": "Boy132",
-    "version": "1.0.3",
+    "version": "1.1.0",
     "description": "Show the amount of connected players to the server",
     "category": "plugin",
     "url": "https://hub.pelican.dev/plugins/player-counter",

player-counter/src/Filament/Server/Pages/PlayersPage.php

@@ -13,7 +13,9 @@
 use Filament\Facades\Filament;
 use Filament\Notifications\Notification;
 use Filament\Pages\Page;
+use Filament\Resources\Concerns\HasTabs;
 use Filament\Schemas\Components\EmbeddedTable;
+use Filament\Schemas\Components\Tabs\Tab;
 use Filament\Schemas\Schema;
 use Filament\Support\Facades\FilamentView;
 use Filament\Tables\Columns\ImageColumn;
@@ -23,10 +25,12 @@
 use Filament\Tables\Contracts\HasTable;
 use Filament\Tables\Table;
 use Illuminate\Pagination\LengthAwarePaginator;
+use Livewire\Attributes\Url;
 
 class PlayersPage extends Page implements HasTable
 {
     use BlockAccessInConflict;
+    use HasTabs;
     use InteractsWithTable;
 
     protected static string|\BackedEnum|null $navigationIcon = 'tabler-users-group';
@@ -35,6 +39,20 @@ class PlayersPage extends Page implements HasTable
 
     protected static ?int $navigationSort = 30;
 
+    #[Url(as: 'tab')]
+    public ?string $activeTab = null;
+
+    public bool $isMinecraft = false;
+
+    /** @var array<string, mixed> */
+    public array $players = [];
+
+    /** @var string[] */
+    public array $whitelist = [];
+
+    /** @var string[] */
+    public array $ops = [];
+
     public static function canAccess(): bool
     {
         /** @var Server $server */
@@ -72,57 +90,67 @@ public function getTitle(): string
         return static::getNavigationLabel();
     }
 
-    /**
-     * @throws Exception
-     */
-    public function table(Table $table): Table
+    public function mount(): void
+    {
+        $this->loadPlayersData();
+
+        $this->loadDefaultActiveTab();
+    }
+
+    protected function loadPlayersData(): void
     {
         /** @var Server $server */
         $server = Filament::getTenant();
 
         /** @var ?GameQuery $gameQuery */
         $gameQuery = $server->egg->gameQuery; // @phpstan-ignore property.notFound
 
-        $isMinecraft = $gameQuery?->query_type === 'minecraft_java';
+        $this->isMinecraft = $gameQuery?->query_type === 'minecraft_java';
 
-        $whitelist = [];
-        $ops = [];
+        $this->whitelist = [];
+        $this->ops = [];
 
-        if ($isMinecraft) {
+        if ($this->isMinecraft) {
             $fileRepository = (new DaemonFileRepository())->setServer($server);
 
             try {
                 $whitelist = json_decode($fileRepository->getContent('whitelist.json'), true, 512, JSON_THROW_ON_ERROR);
-                $whitelist = array_unique(array_map(fn ($data) => $data['name'], $whitelist));
+                $this->whitelist = array_unique(array_map(fn ($data) => $data['name'], $whitelist));
             } catch (Exception $exception) {
                 report($exception);
             }
 
             try {
                 $ops = json_decode($fileRepository->getContent('ops.json'), true, 512, JSON_THROW_ON_ERROR);
-                $ops = array_unique(array_map(fn ($data) => $data['name'], $ops));
+                $this->ops = array_unique(array_map(fn ($data) => $data['name'], $ops));
             } catch (Exception $exception) {
                 report($exception);
             }
         }
 
-        return $table
-            ->records(function (?string $search, int $page, int $recordsPerPage) {
-                /** @var Server $server */
-                $server = Filament::getTenant();
-
-                $players = [];
+        $this->players = [];
 
-                /** @var ?GameQuery $gameQuery */
-                $gameQuery = $server->egg->gameQuery; // @phpstan-ignore property.notFound
+        if ($gameQuery) {
+            $data = $gameQuery->runQuery($server);
 
-                if ($gameQuery) {
-                    $data = $gameQuery->runQuery($server);
+            if ($data) {
+                $this->players = $data['players'] ?? [];
+            }
+        }
+    }
 
-                    if ($data) {
-                        $players = $data['players'] ?? [];
-                    }
-                }
+    /**
+     * @throws Exception
+     */
+    public function table(Table $table): Table
+    {
+        return $table
+            ->records(function (?string $search, int $page, int $recordsPerPage) {
+                $players = match ($this->activeTab) {
+                    'whitelist' => array_map(fn ($player) => ['name' => $player], $this->whitelist),
+                    'ops' => array_map(fn ($player) => ['name' => $player], $this->ops),
+                    default => $this->players,
+                };
 
                 if ($search) {
                     $players = array_filter($players, fn ($player) => str($player['name'])->contains($search, true));
@@ -134,38 +162,38 @@ public function table(Table $table): Table
             ->contentGrid([
                 'default' => 1,
                 'lg' => 2,
-                'xl' => $isMinecraft ? 2 : 3,
+                'xl' => $this->isMinecraft ? 2 : 3,
             ])
             ->columns([
                 Split::make([
                     ImageColumn::make('avatar')
-                        ->visible(fn () => $isMinecraft)
-                        ->state(fn (array $record) => 'https://cravatar.eu/helmhead/' . $record['id'] . '/256.png')
+                        ->visible(fn () => $this->isMinecraft)
+                        ->state(fn (array $record) => 'https://cravatar.eu/helmhead/' . (array_key_exists('id', $record) ? $record['id'] : $record['name']) . '/256.png')
                         ->grow(false),
                     TextColumn::make('name')
                         ->label('Name')
                         ->tooltip(fn (array $record) => array_key_exists('id', $record) ? $record['id'] : null)
                         ->searchable(),
                     TextColumn::make('is_whitelisted')
-                        ->visible(fn () => $isMinecraft)
+                        ->visible(fn () => $this->isMinecraft)
                         ->badge()
                         ->grow(false)
-                        ->state(fn (array $record) => in_array($record['name'], $whitelist) ? trans('player-counter::query.whitelisted') : null),
+                        ->state(fn (array $record) => in_array($record['name'], $this->whitelist) ? trans('player-counter::query.whitelisted') : null),
                     TextColumn::make('is_op')
-                        ->visible(fn () => $isMinecraft)
+                        ->visible(fn () => $this->isMinecraft)
                         ->badge()
                         ->grow(false)
-                        ->state(fn (array $record) => in_array($record['name'], $ops) ? trans('player-counter::query.op') : null),
+                        ->state(fn (array $record) => in_array($record['name'], $this->ops) ? trans('player-counter::query.op') : null),
                     TextColumn::make('time')
-                        ->hidden(fn () => $isMinecraft)
+                        ->hidden(fn () => $this->isMinecraft)
                         ->badge()
                         ->grow(false)
                         ->formatStateUsing(fn ($state) => $state ? CarbonInterval::seconds($state)->cascade()->forHumans() : null),
                 ]),
             ])
             ->recordActions([
                 Action::make('exclude_kick')
-                    ->visible(fn () => $isMinecraft)
+                    ->visible(fn () => !$this->activeTab || $this->activeTab === 'online')
                     ->label(trans('player-counter::query.kick'))
                     ->icon('tabler-door-exit')
                     ->color('danger')
@@ -194,7 +222,7 @@ public function table(Table $table): Table
                         }
                     }),
                 Action::make('exclude_ban')
-                    ->visible(fn () => $isMinecraft)
+                    ->visible(fn () => !$this->activeTab || $this->activeTab === 'online')
                     ->label(trans('player-counter::query.ban'))
                     ->icon('tabler-hammer')
                     ->color('danger')
@@ -224,16 +252,16 @@ public function table(Table $table): Table
                         }
                     }),
                 Action::make('exclude_whitelist')
-                    ->visible(fn () => $isMinecraft)
-                    ->label(fn (array $record) => in_array($record['name'], $whitelist) ? trans('player-counter::query.remove_from_whitelist') : trans('player-counter::query.add_to_whitelist'))
-                    ->icon(fn (array $record) => in_array($record['name'], $whitelist) ? 'tabler-playlist-x' : 'tabler-playlist-add')
-                    ->color(fn (array $record) => in_array($record['name'], $whitelist) ? 'danger' : 'success')
-                    ->action(function (array $record) use ($whitelist) {
+                    ->visible(fn () => $this->isMinecraft)
+                    ->label(fn (array $record) => in_array($record['name'], $this->whitelist) ? trans('player-counter::query.remove_from_whitelist') : trans('player-counter::query.add_to_whitelist'))
+                    ->icon(fn (array $record) => in_array($record['name'], $this->whitelist) ? 'tabler-playlist-x' : 'tabler-playlist-add')
+                    ->color(fn (array $record) => in_array($record['name'], $this->whitelist) ? 'danger' : 'success')
+                    ->action(function (array $record) {
                         /** @var Server $server */
                         $server = Filament::getTenant();
 
                         try {
-                            $action = in_array($record['name'], $whitelist) ? 'remove' : 'add';
+                            $action = in_array($record['name'], $this->whitelist) ? 'remove' : 'add';
 
                             $server->send('whitelist ' . $action . ' ' . $record['name']);
 
@@ -255,16 +283,16 @@ public function table(Table $table): Table
                         }
                     }),
                 Action::make('exclude_op')
-                    ->visible(fn () => $isMinecraft)
-                    ->label(fn (array $record) => in_array($record['name'], $ops) ? trans('player-counter::query.remove_from_ops') : trans('player-counter::query.add_to_ops'))
-                    ->icon(fn (array $record) => in_array($record['name'], $ops) ? 'tabler-shield-minus' : 'tabler-shield-plus')
-                    ->color(fn (array $record) => in_array($record['name'], $ops) ? 'warning' : 'success')
-                    ->action(function (array $record) use ($ops) {
+                    ->visible(fn () => $this->isMinecraft)
+                    ->label(fn (array $record) => in_array($record['name'], $this->ops) ? trans('player-counter::query.remove_from_ops') : trans('player-counter::query.add_to_ops'))
+                    ->icon(fn (array $record) => in_array($record['name'], $this->ops) ? 'tabler-shield-minus' : 'tabler-shield-plus')
+                    ->color(fn (array $record) => in_array($record['name'], $this->ops) ? 'warning' : 'success')
+                    ->action(function (array $record) {
                         /** @var Server $server */
                         $server = Filament::getTenant();
 
                         try {
-                            $action = in_array($record['name'], $ops) ? 'deop' : 'op';
+                            $action = in_array($record['name'], $this->ops) ? 'deop' : 'op';
 
                             $server->send($action  . ' ' . $record['name']);
 
@@ -287,6 +315,10 @@ public function table(Table $table): Table
                     }),
             ])
             ->emptyStateHeading(function () {
+                if ($this->activeTab && $this->activeTab !== 'online') {
+                    return trans('player-counter::query.table.no_players');
+                }
+
                 /** @var Server $server */
                 $server = Filament::getTenant();
 
@@ -297,6 +329,10 @@ public function table(Table $table): Table
                 return trans('player-counter::query.table.no_players');
             })
             ->emptyStateDescription(function () {
+                if ($this->activeTab && $this->activeTab !== 'online') {
+                    return null;
+                }
+
                 /** @var Server $server */
                 $server = Filament::getTenant();
 
@@ -308,10 +344,32 @@ public function table(Table $table): Table
             });
     }
 
+    public function getTabs(): array
+    {
+        if (!$this->isMinecraft) {
+            return [];
+        }
+
+        return [
+            'online' => Tab::make('online')
+                ->label('Online')
+                ->badge(fn () => count($this->players)),
+
+            'whitelist' => Tab::make('whitelist')
+                ->label('Whitelist')
+                ->badge(fn () => count($this->whitelist)),
+
+            'ops' => Tab::make('ops')
+                ->label('OPs')
+                ->badge(fn () => count($this->ops)),
+        ];
+    }
+
     public function content(Schema $schema): Schema
     {
         return $schema
             ->components([
+                $this->getTabsContentComponent(),
                 EmbeddedTable::make(),
             ]);
     }