merge user dan role conflict resolved
This commit is contained in:
@@ -1,6 +1,10 @@
|
|||||||
{
|
{
|
||||||
"permissions": {
|
"permissions": {
|
||||||
"allow": [
|
"allow": [
|
||||||
|
<<<<<<< HEAD
|
||||||
|
=======
|
||||||
|
"mcp__laravel-boost__database-schema",
|
||||||
|
>>>>>>> role-module
|
||||||
"Bash(php artisan *)",
|
"Bash(php artisan *)",
|
||||||
"Bash(vendor/bin/pint --dirty --format agent)",
|
"Bash(vendor/bin/pint --dirty --format agent)",
|
||||||
"mcp__laravel-boost__search-docs"
|
"mcp__laravel-boost__search-docs"
|
||||||
|
|||||||
36
app/Http/Controllers/RoleController.php
Normal file
36
app/Http/Controllers/RoleController.php
Normal file
@@ -0,0 +1,36 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Http\Controllers;
|
||||||
|
|
||||||
|
use App\Http\Requests\StoreRoleRequest;
|
||||||
|
use App\Models\Role;
|
||||||
|
use Illuminate\Http\RedirectResponse;
|
||||||
|
use Illuminate\Http\Request;
|
||||||
|
use Illuminate\View\View;
|
||||||
|
|
||||||
|
class RoleController extends Controller
|
||||||
|
{
|
||||||
|
public function index(Request $request)
|
||||||
|
{
|
||||||
|
$roles = Role::orderBy('name')->paginate(10);
|
||||||
|
|
||||||
|
if ($request->ajax()) {
|
||||||
|
return view('roles._table', compact('roles'));
|
||||||
|
}
|
||||||
|
|
||||||
|
return view('roles.index', compact('roles'));
|
||||||
|
}
|
||||||
|
|
||||||
|
public function create(): View
|
||||||
|
{
|
||||||
|
return view('roles.create');
|
||||||
|
}
|
||||||
|
|
||||||
|
public function store(StoreRoleRequest $request): RedirectResponse
|
||||||
|
{
|
||||||
|
Role::create($request->validated());
|
||||||
|
|
||||||
|
return redirect()->route('roles.index')
|
||||||
|
->with('status', 'role-created');
|
||||||
|
}
|
||||||
|
}
|
||||||
28
app/Http/Requests/StoreRoleRequest.php
Normal file
28
app/Http/Requests/StoreRoleRequest.php
Normal file
@@ -0,0 +1,28 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Http\Requests;
|
||||||
|
|
||||||
|
use Illuminate\Contracts\Validation\ValidationRule;
|
||||||
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
|
|
||||||
|
class StoreRoleRequest extends FormRequest
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* Determine if the user is authorized to make this request.
|
||||||
|
*/
|
||||||
|
public function authorize(): bool
|
||||||
|
{
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @return array<string, ValidationRule|array<mixed>|string>
|
||||||
|
*/
|
||||||
|
public function rules(): array
|
||||||
|
{
|
||||||
|
return [
|
||||||
|
'name' => ['required', 'string', 'max:255', 'unique:roles,name'],
|
||||||
|
'description' => ['nullable', 'string', 'max:500'],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
}
|
||||||
15
app/Models/Role.php
Normal file
15
app/Models/Role.php
Normal file
@@ -0,0 +1,15 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Models;
|
||||||
|
|
||||||
|
use Database\Factories\RoleFactory;
|
||||||
|
use Illuminate\Database\Eloquent\Factories\HasFactory;
|
||||||
|
use Illuminate\Database\Eloquent\Model;
|
||||||
|
|
||||||
|
class Role extends Model
|
||||||
|
{
|
||||||
|
/** @use HasFactory<RoleFactory> */
|
||||||
|
use HasFactory;
|
||||||
|
|
||||||
|
protected $fillable = ['name', 'description'];
|
||||||
|
}
|
||||||
25
database/factories/RoleFactory.php
Normal file
25
database/factories/RoleFactory.php
Normal file
@@ -0,0 +1,25 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace Database\Factories;
|
||||||
|
|
||||||
|
use App\Models\Role;
|
||||||
|
use Illuminate\Database\Eloquent\Factories\Factory;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @extends Factory<Role>
|
||||||
|
*/
|
||||||
|
class RoleFactory extends Factory
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* Define the model's default state.
|
||||||
|
*
|
||||||
|
* @return array<string, mixed>
|
||||||
|
*/
|
||||||
|
public function definition(): array
|
||||||
|
{
|
||||||
|
return [
|
||||||
|
'name' => fake()->unique()->word(),
|
||||||
|
'description' => fake()->sentence(),
|
||||||
|
];
|
||||||
|
}
|
||||||
|
}
|
||||||
29
database/migrations/2026_05_11_042341_create_roles_table.php
Normal file
29
database/migrations/2026_05_11_042341_create_roles_table.php
Normal file
@@ -0,0 +1,29 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
use Illuminate\Database\Migrations\Migration;
|
||||||
|
use Illuminate\Database\Schema\Blueprint;
|
||||||
|
use Illuminate\Support\Facades\Schema;
|
||||||
|
|
||||||
|
return new class extends Migration
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* Run the migrations.
|
||||||
|
*/
|
||||||
|
public function up(): void
|
||||||
|
{
|
||||||
|
Schema::create('roles', function (Blueprint $table) {
|
||||||
|
$table->id();
|
||||||
|
$table->string('name')->unique();
|
||||||
|
$table->string('description')->nullable();
|
||||||
|
$table->timestamps();
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Reverse the migrations.
|
||||||
|
*/
|
||||||
|
public function down(): void
|
||||||
|
{
|
||||||
|
Schema::dropIfExists('roles');
|
||||||
|
}
|
||||||
|
};
|
||||||
22
database/seeders/RoleSeeder.php
Normal file
22
database/seeders/RoleSeeder.php
Normal file
@@ -0,0 +1,22 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace Database\Seeders;
|
||||||
|
|
||||||
|
use App\Models\Role;
|
||||||
|
use Illuminate\Database\Seeder;
|
||||||
|
|
||||||
|
class RoleSeeder extends Seeder
|
||||||
|
{
|
||||||
|
public function run(): void
|
||||||
|
{
|
||||||
|
$roles = [
|
||||||
|
['name' => 'Admin', 'description' => 'Full access to all resources'],
|
||||||
|
['name' => 'Editor', 'description' => 'Can create and edit content'],
|
||||||
|
['name' => 'Viewer', 'description' => 'Read-only access'],
|
||||||
|
];
|
||||||
|
|
||||||
|
foreach ($roles as $role) {
|
||||||
|
Role::firstOrCreate(['name' => $role['name']], $role);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -17,6 +17,8 @@
|
|||||||
</x-nav-link>
|
</x-nav-link>
|
||||||
<x-nav-link :href="route('users.index')" :active="request()->routeIs('users.index')">
|
<x-nav-link :href="route('users.index')" :active="request()->routeIs('users.index')">
|
||||||
{{ __('Users') }}
|
{{ __('Users') }}
|
||||||
|
<x-nav-link :href="route('roles.index')" :active="request()->routeIs('roles.index')">
|
||||||
|
{{ __('Roles') }}
|
||||||
</x-nav-link>
|
</x-nav-link>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
@@ -75,6 +77,8 @@
|
|||||||
</x-responsive-nav-link>
|
</x-responsive-nav-link>
|
||||||
<x-responsive-nav-link :href="route('users.index')" :active="request()->routeIs('users.index')">
|
<x-responsive-nav-link :href="route('users.index')" :active="request()->routeIs('users.index')">
|
||||||
{{ __('Users') }}
|
{{ __('Users') }}
|
||||||
|
<x-responsive-nav-link :href="route('roles.index')" :active="request()->routeIs('roles.index')">
|
||||||
|
{{ __('Roles') }}
|
||||||
</x-responsive-nav-link>
|
</x-responsive-nav-link>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
|||||||
34
resources/views/roles/_table.blade.php
Normal file
34
resources/views/roles/_table.blade.php
Normal file
@@ -0,0 +1,34 @@
|
|||||||
|
<table class="w-full text-sm text-left text-gray-900 dark:text-gray-100">
|
||||||
|
<thead class="text-xs text-gray-700 dark:text-gray-400 uppercase bg-gray-50 dark:bg-gray-700">
|
||||||
|
<tr>
|
||||||
|
<th class="px-6 py-3">#</th>
|
||||||
|
<th class="px-6 py-3">{{ __('Name') }}</th>
|
||||||
|
<th class="px-6 py-3">{{ __('Description') }}</th>
|
||||||
|
<th class="px-6 py-3">{{ __('Created') }}</th>
|
||||||
|
</tr>
|
||||||
|
</thead>
|
||||||
|
<tbody>
|
||||||
|
@foreach ($roles as $role)
|
||||||
|
<tr class="{{ $loop->even ? 'bg-gray-50 dark:bg-gray-700' : 'bg-white dark:bg-gray-800' }} border-b border-gray-200 dark:border-gray-600">
|
||||||
|
<td class="px-6 py-4">{{ $roles->firstItem() + $loop->index }}</td>
|
||||||
|
<td class="px-6 py-4 font-medium">{{ $role->name }}</td>
|
||||||
|
<td class="px-6 py-4">{{ $role->description ?? '—' }}</td>
|
||||||
|
<td class="px-6 py-4">{{ $role->created_at->format('d M Y') }}</td>
|
||||||
|
</tr>
|
||||||
|
@endforeach
|
||||||
|
|
||||||
|
@if ($roles->isEmpty())
|
||||||
|
<tr>
|
||||||
|
<td colspan="4" class="px-6 py-4 text-center text-gray-500 dark:text-gray-400">
|
||||||
|
{{ __('No roles found.') }}
|
||||||
|
</td>
|
||||||
|
</tr>
|
||||||
|
@endif
|
||||||
|
</tbody>
|
||||||
|
</table>
|
||||||
|
|
||||||
|
@if ($roles->hasPages())
|
||||||
|
<div class="mt-4">
|
||||||
|
{{ $roles->links() }}
|
||||||
|
</div>
|
||||||
|
@endif
|
||||||
56
resources/views/roles/create.blade.php
Normal file
56
resources/views/roles/create.blade.php
Normal file
@@ -0,0 +1,56 @@
|
|||||||
|
<x-app-layout>
|
||||||
|
<x-slot name="header">
|
||||||
|
<h2 class="font-semibold text-xl text-gray-800 dark:text-gray-200 leading-tight">
|
||||||
|
{{ __('New Role') }}
|
||||||
|
</h2>
|
||||||
|
</x-slot>
|
||||||
|
|
||||||
|
<div class="py-12">
|
||||||
|
<div class="max-w-7xl mx-auto sm:px-6 lg:px-8">
|
||||||
|
<div class="p-4 sm:p-8 bg-white dark:bg-gray-800 shadow sm:rounded-lg">
|
||||||
|
<div class="max-w-xl">
|
||||||
|
<section>
|
||||||
|
<header>
|
||||||
|
<h2 class="text-lg font-medium text-gray-900 dark:text-gray-100">
|
||||||
|
{{ __('Role Details') }}
|
||||||
|
</h2>
|
||||||
|
|
||||||
|
<p class="mt-1 text-sm text-gray-600 dark:text-gray-400">
|
||||||
|
{{ __('Create a new role to assign to users in the system.') }}
|
||||||
|
</p>
|
||||||
|
</header>
|
||||||
|
|
||||||
|
<form method="post" action="{{ route('roles.store') }}" class="mt-6 space-y-6">
|
||||||
|
@csrf
|
||||||
|
|
||||||
|
<div>
|
||||||
|
<x-input-label for="name" :value="__('Name')" />
|
||||||
|
<x-text-input id="name" name="name" type="text" class="mt-1 block w-full" :value="old('name')" required autofocus autocomplete="off" />
|
||||||
|
<x-input-error class="mt-2" :messages="$errors->get('name')" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div>
|
||||||
|
<x-input-label for="description" :value="__('Description')" />
|
||||||
|
<textarea
|
||||||
|
id="description"
|
||||||
|
name="description"
|
||||||
|
rows="3"
|
||||||
|
class="mt-1 block w-full border-gray-300 dark:border-gray-700 dark:bg-gray-900 dark:text-gray-300 focus:border-indigo-500 dark:focus:border-indigo-600 focus:ring-indigo-500 dark:focus:ring-indigo-600 rounded-md shadow-sm"
|
||||||
|
>{{ old('description') }}</textarea>
|
||||||
|
<x-input-error class="mt-2" :messages="$errors->get('description')" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="flex items-center gap-4">
|
||||||
|
<x-primary-button>{{ __('Create Role') }}</x-primary-button>
|
||||||
|
|
||||||
|
<a href="{{ route('roles.index') }}" class="text-sm text-gray-600 dark:text-gray-400 hover:text-gray-900 dark:hover:text-gray-100 underline">
|
||||||
|
{{ __('Cancel') }}
|
||||||
|
</a>
|
||||||
|
</div>
|
||||||
|
</form>
|
||||||
|
</section>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</x-app-layout>
|
||||||
59
resources/views/roles/index.blade.php
Normal file
59
resources/views/roles/index.blade.php
Normal file
@@ -0,0 +1,59 @@
|
|||||||
|
<x-app-layout>
|
||||||
|
<x-slot name="header">
|
||||||
|
<div class="flex items-center justify-between">
|
||||||
|
<h2 class="font-semibold text-xl text-gray-800 dark:text-gray-200 leading-tight">
|
||||||
|
{{ __('Roles') }}
|
||||||
|
</h2>
|
||||||
|
<a href="{{ route('roles.create') }}">
|
||||||
|
<x-primary-button>{{ __('New Role') }}</x-primary-button>
|
||||||
|
</a>
|
||||||
|
</div>
|
||||||
|
</x-slot>
|
||||||
|
|
||||||
|
<div class="py-12">
|
||||||
|
<div class="max-w-7xl mx-auto sm:px-6 lg:px-8">
|
||||||
|
@if (session('status') === 'role-created')
|
||||||
|
<div
|
||||||
|
x-data="{ show: true }"
|
||||||
|
x-show="show"
|
||||||
|
x-transition
|
||||||
|
x-init="setTimeout(() => show = false, 3000)"
|
||||||
|
class="mb-4 p-4 bg-green-100 dark:bg-green-900 text-green-800 dark:text-green-200 text-sm rounded-lg"
|
||||||
|
>
|
||||||
|
{{ __('Role created successfully.') }}
|
||||||
|
</div>
|
||||||
|
@endif
|
||||||
|
|
||||||
|
<div class="bg-white dark:bg-gray-800 overflow-hidden shadow-sm sm:rounded-lg">
|
||||||
|
<div
|
||||||
|
class="p-6"
|
||||||
|
x-data="{
|
||||||
|
loading: false,
|
||||||
|
async paginate(url) {
|
||||||
|
this.loading = true;
|
||||||
|
const response = await fetch(url, {
|
||||||
|
headers: { 'X-Requested-With': 'XMLHttpRequest' }
|
||||||
|
});
|
||||||
|
this.$refs.tableContainer.innerHTML = await response.text();
|
||||||
|
this.loading = false;
|
||||||
|
}
|
||||||
|
}"
|
||||||
|
@click.prevent="
|
||||||
|
const link = $event.target.closest('a[href]');
|
||||||
|
if (link && $refs.tableContainer.contains(link)) {
|
||||||
|
paginate(link.href);
|
||||||
|
}
|
||||||
|
"
|
||||||
|
>
|
||||||
|
<div x-show="loading" class="text-center py-4 text-gray-500 dark:text-gray-400">
|
||||||
|
{{ __('Loading...') }}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div x-ref="tableContainer" x-show="!loading">
|
||||||
|
@include('roles._table')
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</x-app-layout>
|
||||||
@@ -2,6 +2,7 @@
|
|||||||
|
|
||||||
use App\Http\Controllers\ProfileController;
|
use App\Http\Controllers\ProfileController;
|
||||||
use App\Http\Controllers\UserController;
|
use App\Http\Controllers\UserController;
|
||||||
|
use App\Http\Controllers\RoleController;
|
||||||
use Illuminate\Support\Facades\Route;
|
use Illuminate\Support\Facades\Route;
|
||||||
|
|
||||||
Route::get('/', function () {
|
Route::get('/', function () {
|
||||||
@@ -14,6 +15,9 @@ Route::get('/dashboard', function () {
|
|||||||
|
|
||||||
Route::middleware('auth')->group(function () {
|
Route::middleware('auth')->group(function () {
|
||||||
Route::get('/users', [UserController::class, 'index'])->name('users.index');
|
Route::get('/users', [UserController::class, 'index'])->name('users.index');
|
||||||
|
Route::get('/roles', [RoleController::class, 'index'])->name('roles.index');
|
||||||
|
Route::get('/roles/create', [RoleController::class, 'create'])->name('roles.create');
|
||||||
|
Route::post('/roles', [RoleController::class, 'store'])->name('roles.store');
|
||||||
Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit');
|
Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit');
|
||||||
Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update');
|
Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update');
|
||||||
Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy');
|
Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy');
|
||||||
|
|||||||
41
tests/Feature/RoleIndexTest.php
Normal file
41
tests/Feature/RoleIndexTest.php
Normal file
@@ -0,0 +1,41 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
use App\Models\Role;
|
||||||
|
use App\Models\User;
|
||||||
|
|
||||||
|
test('guests are redirected to login from roles page', function () {
|
||||||
|
$this->get('/roles')->assertRedirect('/login');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('authenticated users can view the roles page', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->get('/roles')
|
||||||
|
->assertOk()
|
||||||
|
->assertViewIs('roles.index');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('roles page passes paginated roles to view', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->get('/roles')
|
||||||
|
->assertOk()
|
||||||
|
->assertViewHas('roles');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('ajax request returns roles table partial', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->withHeader('X-Requested-With', 'XMLHttpRequest')
|
||||||
|
->get('/roles')
|
||||||
|
->assertOk()
|
||||||
|
->assertViewIs('roles._table');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('ajax pagination returns correct page', function () {
|
||||||
|
Role::factory()->count(15)->create();
|
||||||
|
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->withHeader('X-Requested-With', 'XMLHttpRequest')
|
||||||
|
->get('/roles?page=2')
|
||||||
|
->assertOk()
|
||||||
|
->assertViewIs('roles._table')
|
||||||
|
->assertViewHas('roles', fn ($roles) => $roles->currentPage() === 2);
|
||||||
|
});
|
||||||
52
tests/Feature/StoreRoleTest.php
Normal file
52
tests/Feature/StoreRoleTest.php
Normal file
@@ -0,0 +1,52 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
use App\Models\Role;
|
||||||
|
use App\Models\User;
|
||||||
|
|
||||||
|
test('guests cannot access the create role page', function () {
|
||||||
|
$this->get('/roles/create')->assertRedirect('/login');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('authenticated users can access the create role page', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->get('/roles/create')
|
||||||
|
->assertOk()
|
||||||
|
->assertViewIs('roles.create');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('authenticated users can create a role', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->post('/roles', ['name' => 'Manager', 'description' => 'Manages things'])
|
||||||
|
->assertRedirect(route('roles.index'))
|
||||||
|
->assertSessionHas('status', 'role-created');
|
||||||
|
|
||||||
|
$this->assertDatabaseHas('roles', ['name' => 'Manager', 'description' => 'Manages things']);
|
||||||
|
});
|
||||||
|
|
||||||
|
test('role name is required', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->post('/roles', ['name' => '', 'description' => 'Some description'])
|
||||||
|
->assertSessionHasErrors('name');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('role name must be unique', function () {
|
||||||
|
Role::factory()->create(['name' => 'Admin']);
|
||||||
|
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->post('/roles', ['name' => 'Admin'])
|
||||||
|
->assertSessionHasErrors('name');
|
||||||
|
});
|
||||||
|
|
||||||
|
test('description is optional', function () {
|
||||||
|
$this->actingAs(User::factory()->create())
|
||||||
|
->post('/roles', ['name' => 'Viewer'])
|
||||||
|
->assertRedirect(route('roles.index'));
|
||||||
|
|
||||||
|
$this->assertDatabaseHas('roles', ['name' => 'Viewer', 'description' => null]);
|
||||||
|
});
|
||||||
|
|
||||||
|
test('guests cannot create a role', function () {
|
||||||
|
$this->post('/roles', ['name' => 'Admin'])->assertRedirect('/login');
|
||||||
|
|
||||||
|
$this->assertDatabaseMissing('roles', ['name' => 'Admin']);
|
||||||
|
});
|
||||||
Reference in New Issue
Block a user