feat: prevent duplicate pending leave requests and restrict submission UI when a pending request exists
This commit is contained in:
parent
46cb2c3588
commit
c140192453
@ -2,6 +2,7 @@
|
||||
|
||||
namespace App\Http\Controllers\Admin\Hr;
|
||||
|
||||
use App\Enums\LeaveRequestStatus;
|
||||
use App\Enums\Permission;
|
||||
use App\Http\Controllers\Concerns\FlashesEntityMessage;
|
||||
use App\Http\Controllers\Concerns\ParsesDataTableQuery;
|
||||
@ -29,12 +30,21 @@ public function index(Request $request): Response
|
||||
$tableQuery = $this->parseDataTableQuery($request);
|
||||
$user = $request->user();
|
||||
|
||||
$hasPending = false;
|
||||
if ($user?->employee !== null) {
|
||||
$hasPending = LeaveRequest::query()
|
||||
->where('employee_id', $user->employee->id)
|
||||
->where('status', LeaveRequestStatus::PENDING)
|
||||
->exists();
|
||||
}
|
||||
|
||||
return Inertia::render('admin/hr/leave-requests/Index', [
|
||||
'leaveRequests' => $this->leaveRequestService->paginateForIndex($tableQuery),
|
||||
'authEmployeeId' => $user?->employee?->id,
|
||||
'canSubmit' => $user?->can(Permission::LEAVE_REQUESTS_CREATE->value)
|
||||
&& ! $user->can(Permission::LEAVE_REQUESTS_VERIFY->value)
|
||||
&& $user->employee !== null,
|
||||
'hasPending' => $hasPending,
|
||||
'filters' => $this->dataTableFilters($tableQuery),
|
||||
]);
|
||||
}
|
||||
|
||||
@ -51,6 +51,17 @@ public function create(array $validated, User $user): void
|
||||
{
|
||||
$employee = $this->resolveAuthEmployee($user);
|
||||
|
||||
$hasPending = LeaveRequest::query()
|
||||
->where('employee_id', $employee->id)
|
||||
->where('status', LeaveRequestStatus::PENDING)
|
||||
->exists();
|
||||
|
||||
if ($hasPending) {
|
||||
throw ValidationException::withMessages([
|
||||
'start_date' => 'Anda masih memiliki pengajuan cuti yang aktif (menunggu persetujuan).',
|
||||
]);
|
||||
}
|
||||
|
||||
$startDate = Carbon::parse($validated['start_date'])->startOfDay();
|
||||
$endDate = Carbon::parse($validated['end_date'])->startOfDay();
|
||||
|
||||
@ -107,7 +118,7 @@ public function approve(LeaveRequest $leaveRequest, User $user): void
|
||||
|
||||
DB::transaction(function () use ($leaveRequest, $user): void {
|
||||
$leaveRequest->status = LeaveRequestStatus::APPROVED;
|
||||
$leaveRequest->verified_at = now();
|
||||
$leaveRequest->verified_at = Carbon::now();
|
||||
$leaveRequest->verified_by_id = $user->id;
|
||||
$leaveRequest->save();
|
||||
});
|
||||
@ -129,7 +140,7 @@ public function reject(LeaveRequest $leaveRequest, string $reason, User $user):
|
||||
|
||||
DB::transaction(function () use ($leaveRequest, $user, $reason): void {
|
||||
$leaveRequest->status = LeaveRequestStatus::REJECTED;
|
||||
$leaveRequest->verified_at = now();
|
||||
$leaveRequest->verified_at = Carbon::now();
|
||||
$leaveRequest->verified_by_id = $user->id;
|
||||
$leaveRequest->save();
|
||||
|
||||
@ -157,7 +168,7 @@ private function calculateTotalDays(Carbon $startDate, Carbon $endDate): int
|
||||
|
||||
private function ensureMinimumLeadTime(Carbon $startDate): void
|
||||
{
|
||||
$minimumStartDate = now()->startOfDay()->addDay();
|
||||
$minimumStartDate = Carbon::now()->startOfDay()->addDay();
|
||||
|
||||
if ($startDate->lt($minimumStartDate)) {
|
||||
throw ValidationException::withMessages([
|
||||
|
||||
@ -93,7 +93,7 @@ watch(
|
||||
</h2>
|
||||
</div>
|
||||
|
||||
<Button v-if="canSubmit" class="shrink-0 self-start sm:self-center" @click="openCreateModal">
|
||||
<Button v-if="canSubmit && !hasPending" class="shrink-0 self-start sm:self-center" @click="openCreateModal">
|
||||
<Plus class="size-4" />
|
||||
Ajukan
|
||||
</Button>
|
||||
|
||||
@ -37,6 +37,7 @@ export type LeaveRequestPageProps = {
|
||||
leaveRequests: PaginatedLeaveRequests;
|
||||
authEmployeeId: number | null;
|
||||
canSubmit: boolean;
|
||||
hasPending: boolean;
|
||||
filters: {
|
||||
search: string;
|
||||
sort?: string;
|
||||
|
||||
Loading…
Reference in New Issue
Block a user