Refactor UserTermsOfUseAcceptance and TermsOfUseVersion Models

Bug: T407722

Author: dati18

app/TermsOfUseVersion.php

@@ -13,7 +13,10 @@
 class TermsOfUseVersion extends Model {
     use HasFactory;
 
+    protected $primaryKey = 'version';
+    protected $keyType = 'string';
     protected $table = 'tou_versions';
+    public $incrementing = false;
 
     const FIELDS = [
         'version',
@@ -32,4 +35,17 @@ class TermsOfUseVersion extends Model {
     public static function latestActiveVersion(): ?self {
         return self::query()->where('active', true)->latest()->first();
     }
+
+    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]);
+        });
+    }
 }

app/UserTermsOfUseAcceptance.php

@@ -9,6 +9,10 @@
 class UserTermsOfUseAcceptance extends Model {
     use HasFactory;
 
+    public $incrementing = false;
+    protected $primaryKey = null;
+    protected $keyType = 'string';
+
     public const FIELDS = [
         'user_id',
         'tou_version',
@@ -29,4 +33,10 @@ class UserTermsOfUseAcceptance extends Model {
     public function user(): BelongsTo {
         return $this->belongsTo(User::class);
     }
+
+    protected function setKeysForSaveQuery($query) {
+        return $query
+            ->where('user_id', $this->getAttribute('user_id'))
+            ->where('tou_version', $this->getAttribute('tou_version'));
+    }
 }

database/migrations/2025_09_29_194758_tou_acceptances.php

@@ -10,9 +10,8 @@
      */
     public function up(): void {
         Schema::create('tou_acceptances', function (Blueprint $table) {
-            $table->id();
             $table->unsignedInteger('user_id');
-            $table->string('tou_version', 10);
+            $table->string('tou_version');
             $table->timestamp('tou_accepted_at');
             $table->timestamps();
             $table->unique(['user_id', 'tou_version']);

database/migrations/2025_10_14_091126_tou_versions.php

@@ -10,7 +10,6 @@
      */
     public function up(): void {
         Schema::create('tou_versions', function (Blueprint $table) {
-            $table->id();
             $table->string('version')->unique();
             $table->boolean('active')->default(false);
             $table->timestamps();