Filament•5d ago

Populate Select with only a select subset of an enum

What I am trying to achieve is populating the Select with only certain values of an Enum based on permissions. It is likely I am approaching this entirely the wrong way, and have tried various solutions without success. Although this does not work, it shows what I am trying to achieve. The ArticleStatus is a backed Enum and the value of that enum is stored in the status attribute (it is casted). Everything works properly if I want to load all the values of the enum (->options(ArticleStatus::class)). Running the code below gives this error:

Filament\Forms\Components\Select::isOptionDisabled(): Argument #2 ($label) must be of type string, App\Enums\ArticleStatus given

Select::make('status')
  ->selectablePlaceholder(false)
  ->options(fn (?Article $record, $operation) => Article::getStatusSelectBuilder($record, $operation))
  ->getOptionLabelUsing(fn ($value): ?string => ArticleStatus::from($value)->getLabel())
  ->required(),

Select::make('status')
  ->selectablePlaceholder(false)
  ->options(fn (?Article $record, $operation) => Article::getStatusSelectBuilder($record, $operation))
  ->getOptionLabelUsing(fn ($value): ?string => ArticleStatus::from($value)->getLabel())
  ->required(),

Solution:

So you could map the allowed enum cases to key/value pairs (enum value as key, and label as value) for ->options/icons/colors: ```php Select::make('status') ->selectablePlaceholder(false)...

Jump to solution

4 Replies

BatmanOP•5d ago

public static function getStatusSelectBuilder(?Article $record, string $operation = 'create'): array
    {
        $allowedStatus = [];

        if ($operation === 'edit') {
            $allowedStatus[] = $record->status;
        }

        if (Auth::user()->can('draft', Article::class)) {
            $allowedStatus[] = ArticleStatus::DRAFT;
        }

        if (Auth::user()->can('ready-for-review', Article::class)) {
            $allowedStatus[] = ArticleStatus::READY_FOR_REVIEW;
        }

        if (Auth::user()->can('pending-review', Article::class)) {
            $allowedStatus[] = ArticleStatus::PENDING_REVIEW;
        }

        if (Auth::user()->can('approve', Article::class)) {
            $allowedStatus[] = ArticleStatus::APPROVED;
        }

        if (Auth::user()->can('reject', Article::class)) {
            $allowedStatus[] = ArticleStatus::REJECTED;
        }

        if (Auth::user()->can('archive', Article::class)) {
            $allowedStatus[] = ArticleStatus::ARCHIVED;
        }

        return $allowedStatus;
    }

public static function getStatusSelectBuilder(?Article $record, string $operation = 'create'): array
    {
        $allowedStatus = [];

        if ($operation === 'edit') {
            $allowedStatus[] = $record->status;
        }

        if (Auth::user()->can('draft', Article::class)) {
            $allowedStatus[] = ArticleStatus::DRAFT;
        }

        if (Auth::user()->can('ready-for-review', Article::class)) {
            $allowedStatus[] = ArticleStatus::READY_FOR_REVIEW;
        }

        if (Auth::user()->can('pending-review', Article::class)) {
            $allowedStatus[] = ArticleStatus::PENDING_REVIEW;
        }

        if (Auth::user()->can('approve', Article::class)) {
            $allowedStatus[] = ArticleStatus::APPROVED;
        }

        if (Auth::user()->can('reject', Article::class)) {
            $allowedStatus[] = ArticleStatus::REJECTED;
        }

        if (Auth::user()->can('archive', Article::class)) {
            $allowedStatus[] = ArticleStatus::ARCHIVED;
        }

        return $allowedStatus;
    }

ashattack•5d ago

So I made a similar post about this here: https://discord.com/channels/883083792112300104/1353819807941660723/1353819807941660723 For me, to get around the error naively I did something like this (note here i'm just rejecting a single case):

    ->options(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getLabel()])
        ->toArray()
    )

    ->options(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getLabel()])
        ->toArray()
    )

In that case i also wanted it to render icons and colors from the enum, so i disgustingly got it working like so:

    ->options(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getLabel()])
        ->toArray()
    )
    ->icons(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getIcon()])
        ->toArray())
    ->colors(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getColor()])
        ->toArray())

    ->options(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getLabel()])
        ->toArray()
    )
    ->icons(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getIcon()])
        ->toArray())
    ->colors(fn () => collect(enumClassWeMade::cases())
        ->reject(fn ($case) => $case === enumClassWeMade::RANDOM_CASE_ONE)
        ->mapWithKeys(fn ($case) => [$case->value => $case->getColor()])
        ->toArray())

So under the same logic, you could just wrap the result of the permission check in a collection before converting it to the array:

->options(fn (?Article $record, $operation) => Article::getStatusSelectBuilder($record, $operation))
->getOptionLabelUsing(fn ($value): ?string => ArticleStatus::from($value)->getLabel())

->options(fn (?Article $record, $operation) => Article::getStatusSelectBuilder($record, $operation))
->getOptionLabelUsing(fn ($value): ?string => ArticleStatus::from($value)->getLabel())

Solution

ashattack•5d ago

So you could map the allowed enum cases to key/value pairs (enum value as key, and label as value) for ->options/icons/colors:

Select::make('status')
    ->selectablePlaceholder(false)
    ->options(fn (?Article $record, $operation) => 
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getLabel()])
            ->toArray()
    )
    ->icons(fn (?Article $record, $operation) =>
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getIcon()])
            ->toArray()
    )
    ->colors(fn (?Article $record, $operation) =>
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getColor()])
            ->toArray()
    )
    ->default(ArticleStatus::DRAFT->value)
    ->required();

Select::make('status')
    ->selectablePlaceholder(false)
    ->options(fn (?Article $record, $operation) => 
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getLabel()])
            ->toArray()
    )
    ->icons(fn (?Article $record, $operation) =>
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getIcon()])
            ->toArray()
    )
    ->colors(fn (?Article $record, $operation) =>
        collect(Article::getStatusSelectBuilder($record, $operation))
            ->mapWithKeys(fn($status) => [$status->value => $status->getColor()])
            ->toArray()
    )
    ->default(ArticleStatus::DRAFT->value)
    ->required();

Just doesn't seem too DRY for my liking, unless you make some sort of macro, or some helper. Gonna eventually dive into options() to improve this but gotta move on to the next feature now lol.

BatmanOP•5d ago

@Ash Thank you! Keywords I was searching did not bring up yours unfortunately.

Gaming

Programming

Populate Select with only a select subset of an enum

Did you find this page helpful?