wip

Author: StanBarrows

api.json

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Exceptions\Api;
+
+class TimeEntryInvoicedApiException extends ApiException
+{
+    public const string KEY = 'time_entry_invoiced';
+}

app/Exceptions/Api/TimeEntryInvoicedApiException.php

@@ -10,6 +10,7 @@
 use App\Exceptions\Api\OverlappingTimeEntryApiException;
 use App\Exceptions\Api\PdfRendererIsNotConfiguredException;
 use App\Exceptions\Api\TimeEntryCanNotBeRestartedApiException;
+use App\Exceptions\Api\TimeEntryInvoicedApiException;
 use App\Exceptions\Api\TimeEntryStillRunningApiException;
 use App\Http\Requests\V1\TimeEntry\TimeEntryAggregateExportRequest;
 use App\Http\Requests\V1\TimeEntry\TimeEntryAggregateRequest;
@@ -101,6 +102,11 @@ protected function checkPermission(Organization $organization, string $permissio
         if ($timeEntry !== null && $timeEntry->organization_id !== $organization->getKey()) {
             throw new AuthorizationException('Time entry does not belong to organization');
         }
+        if ($timeEntry !== null && $timeEntry->invoiced_at !== null) {
+            if (in_array($permission, ['time-entries:update:own', 'time-entries:update:all', 'time-entries:delete:own', 'time-entries:delete:all'], true)) {
+                throw new TimeEntryInvoicedApiException;
+            }
+        }
     }
 
     /**
@@ -628,9 +634,9 @@ public function update(Organization $organization, TimeEntry $timeEntry, TimeEnt
         /** @var Member|null $member */
         $member = $request->has('member_id') ? Member::query()->findOrFail($request->input('member_id')) : null;
         if ($timeEntry->member->user_id === Auth::id() && ($member === null || $member->user_id === Auth::id())) {
-            $this->checkPermission($organization, 'time-entries:update:own');
+            $this->checkPermission($organization, 'time-entries:update:own', $timeEntry);
         } else {
-            $this->checkPermission($organization, 'time-entries:update:all');
+            $this->checkPermission($organization, 'time-entries:update:all', $timeEntry);
         }
 
         if ($timeEntry->end !== null && $request->has('end') && $request->input('end') === null) {
@@ -744,6 +750,11 @@ public function updateMultiple(Organization $organization, TimeEntryUpdateMultip
                 continue;
 
             }
+            if ($timeEntry->invoiced_at !== null) {
+                $error->push($id);
+
+                continue;
+            }
             $oldProject = $timeEntry->project;
             $oldTask = $timeEntry->task;
 
@@ -851,6 +862,11 @@ public function destroyMultiple(Organization $organization, TimeEntryDestroyMult
                 continue;
 
             }
+            if ($timeEntry->invoiced_at !== null) {
+                $error->push($id);
+
+                continue;
+            }
 
             $project = $timeEntry->project;
             $task = $timeEntry->task;

app/Http/Controllers/Api/V1/TimeEntryController.php

@@ -47,6 +47,8 @@ public function toArray(Request $request): array
             'tags' => $this->resource->tags ?? [],
             /** @var bool $billable Whether time entry is billable */
             'billable' => $this->resource->billable,
+            /** @var string|null $invoiced_at When the time entry was invoiced (ISO 8601 format, UTC timezone, example: 2024-02-26T17:17:17Z) */
+            'invoiced_at' => $this->formatDateTime($this->resource->invoiced_at),
         ];
     }
 }

app/Http/Resources/V1/TimeEntry/TimeEntryResource.php

@@ -33,6 +33,7 @@
  * @property string $member_id
  * @property bool $is_imported
  * @property Carbon|null $still_active_email_sent_at
+ * @property Carbon|null $invoiced_at
  * @property Carbon|null $created_at
  * @property Carbon|null $updated_at
  * @property-read User $user
@@ -75,6 +76,7 @@ class TimeEntry extends Model implements AuditableContract
         'billable_rate' => 'int',
         'is_imported' => 'bool',
         'still_active_email_sent_at' => 'datetime',
+        'invoiced_at' => 'datetime',
     ];
 
     public const array SELECT_COLUMNS = [
@@ -95,6 +97,7 @@ class TimeEntry extends Model implements AuditableContract
         'client_id',
         'is_imported',
         'still_active_email_sent_at',
+        'invoiced_at',
     ];
 
     /**

app/Models/TimeEntry.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+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::table('time_entries', function (Blueprint $table): void {
+            $table->dateTime('invoiced_at')->nullable()->after('still_active_email_sent_at');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     */
+    public function down(): void
+    {
+        Schema::table('time_entries', function (Blueprint $table): void {
+            $table->dropColumn('invoiced_at');
+        });
+    }
+};