Skip to content

Commit

Permalink
use role contract
Browse files Browse the repository at this point in the history
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
  • Loading branch information
lloricode committed Aug 11, 2024
1 parent 4fed2e8 commit d51b585
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 90 deletions.
22 changes: 0 additions & 22 deletions src/Models/EloquentBuilder/RoleEloquentBuilder.php

This file was deleted.

56 changes: 0 additions & 56 deletions src/Models/Role.php

This file was deleted.

6 changes: 3 additions & 3 deletions src/Resources/RoleResource/Pages/EditRole.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@
use Illuminate\Database\Eloquent\Model;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Actions\EditRoleAction;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Data\RoleData;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Models\Role;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Resources\RoleResource;
use Spatie\Permission\Contracts\Role as RoleContract;

/**
* @property-read Role $record
* @property-read RoleContract&Model $record
*/
class EditRole extends EditRecord
{
Expand All @@ -30,7 +30,7 @@ protected function getHeaderActions(): array
#[\Override]
protected function handleRecordUpdate(Model $record, array $data): Model
{
/** @var Role $record */
/** @var RoleContract&Model $record */
return app(EditRoleAction::class)
->execute($record, new RoleData(...$data));
}
Expand Down
18 changes: 9 additions & 9 deletions src/Resources/RoleResource/Schema/PermissionSchema.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@
use Filament\Forms\Set;
use Illuminate\Support\Collection as CollectionSupport;
use Illuminate\Support\Str;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Models\Role;
use Lloricode\FilamentSpatieLaravelPermissionPlugin\Resources\RoleResource\Support\PermissionData;
use Spatie\Permission\Contracts\Permission as PermissionContract;
use Spatie\Permission\Contracts\Role as RoleContract;
use Spatie\Permission\PermissionRegistrar;

final class PermissionSchema
Expand All @@ -38,7 +38,7 @@ public static function schema(?string $guardName): array
return [

Forms\Components\Hidden::make('permissions')
->afterStateHydrated(function (Forms\Components\Hidden $component, ?Role $record): void {
->afterStateHydrated(function (Forms\Components\Hidden $component, ?RoleContract $record): void {
$component->state($record ? $record->permissions->pluck('name') : []);
})
->dehydrateStateUsing(
Expand Down Expand Up @@ -72,7 +72,7 @@ function (
->afterStateUpdated(function (Get $get, Set $set, bool $state): void {
self::updatedToggleSelectAllState(get: $get, set: $set, state: $state);
})
->afterStateHydrated(function (Forms\Components\Toggle $component, ?Role $record): void {
->afterStateHydrated(function (Forms\Components\Toggle $component, ?RoleContract $record): void {

if ($record === null) {
$component->state(false);
Expand All @@ -84,7 +84,7 @@ function (

foreach (self::permissions() as $permissionData) {

if (! $record->hasPermissionTo($permissionData->name)) {
if (! $record->hasPermissionTo($permissionData->name, self::$guardName)) {
$all = false;

break;
Expand All @@ -109,15 +109,15 @@ function (
->offIcon('heroicon-s-lock-closed')
->reactive()
->afterStateHydrated(
function (Forms\Components\Toggle $component, ?Role $record) use (
function (Forms\Components\Toggle $component, ?RoleContract $record) use (
$parentPermission
): void {
if ($record === null) {
$component->state(false);

return;
}
$component->state($record->hasPermissionTo($parentPermission));
$component->state($record->hasPermissionTo($parentPermission, self::$guardName));
}
)
->afterStateUpdated(
Expand Down Expand Up @@ -152,7 +152,7 @@ function (Set $set, Get $get, bool $state) use (
->columns(2)
->reactive()
->afterStateHydrated(
function (Forms\Components\CheckboxList $component, ?Role $record) use (
function (Forms\Components\CheckboxList $component, ?RoleContract $record) use (
$permissionsDatas,
$parentPermission
): void {
Expand All @@ -163,7 +163,7 @@ function (Forms\Components\CheckboxList $component, ?Role $record) use (
return;
}

if ($record->hasPermissionTo($parentPermission)) {
if ($record->hasPermissionTo($parentPermission, self::$guardName)) {
$component->state($permissionsDatas->pluck('name')->toArray());

return;
Expand All @@ -173,7 +173,7 @@ function (Forms\Components\CheckboxList $component, ?Role $record) use (

foreach ($permissionsDatas as $permissionData) {

if ($record->hasPermissionTo($permissionData->name)) {
if ($record->hasPermissionTo($permissionData->name, self::$guardName)) {
$names[] = $permissionData->name;
}
}
Expand Down

0 comments on commit d51b585

Please sign in to comment.