Add a hook to ensure there is only one active ToU version

Author: dati18

app/TermsOfUseVersion.php

@@ -29,7 +29,26 @@ class TermsOfUseVersion extends Model {
         'active' => 'boolean',
     ];
 
+    /**
+     * Get the active ToU version.
+     */
     public static function latestActiveVersion(): ?self {
-        return self::query()->where('active', true)->latest()->first();
+        return self::query()->where('active', true)->first();
+    }
+
+    /**
+     * Ensure only one ToU version remains active after any save operation.
+     */
+    protected static function booted(): void {
+        static::saving(function (self $model): void {
+            if (!$model->active) {
+                return;
+            }
+
+            self::query()
+                ->where('version', '!=', $model->version)
+                ->where('active', true)
+                ->update(['active' => false]);
+        });
     }
 }

tests/Models/TermsOfUseVersionTest.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace Tests\Models;
+
+use App\TermsOfUseVersion;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Tests\TestCase;
+
+class TermsOfUseVersionTest extends TestCase {
+    use RefreshDatabase;
+
+    public function testSavingActiveVersionDeactivatesOtherVersions(): void {
+        $first = TermsOfUseVersion::create([
+            'version' => '2024-01-01',
+            'active' => true,
+        ]);
+
+        $second = TermsOfUseVersion::create([
+            'version' => '2025-01-01',
+            'active' => true,
+        ]);
+
+        $this->assertFalse($first->fresh()->active);
+        $this->assertTrue($second->fresh()->active);
+    }
+
+    public function testSavingInactiveVersionDoesNotAffectExistingActiveVersion(): void {
+        $active = TermsOfUseVersion::create([
+            'version' => '2024-03-01',
+            'active' => true,
+        ]);
+
+        TermsOfUseVersion::create([
+            'version' => '2024-04-01',
+            'active' => false,
+        ]);
+
+        $this->assertTrue($active->fresh()->active);
+    }
+}