From 952d25ac2d32ca9191d8d3ee16df357ec5174d61 Mon Sep 17 00:00:00 2001
From: Angel de la Torre <angel.dlt.1996@gmail.com>
Date: Wed, 25 Feb 2026 10:25:19 -0700
Subject: [PATCH] fix: update group migrations to use soft deletes

The migration 2024_09_16_093650_inactive_groups is querying the groups
table with a deleted_at column that doesn't exist yet.
This is a migration ordering issue the model (i.e. current code)
uses SoftDeletes but the column hasn't been added at the time this migration
runs, its added in the 2026_02_18_000000_add_soft_deletes_to_groups_table migration.

As such, we need to update the prior migration to not use the Eloquent
model and use the query builder instead.

TL;DR The ORM did not match the DB schema during migrations. The fix is
to not use ORMs to query the DB during migrations.
---
 .../2024_09_16_093650_inactive_groups.php     | 27 ++++++++++++-------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/database/migrations/2024_09_16_093650_inactive_groups.php b/database/migrations/2024_09_16_093650_inactive_groups.php
index ec2273890e..e4ad4cbc8d 100644
--- a/database/migrations/2024_09_16_093650_inactive_groups.php
+++ b/database/migrations/2024_09_16_093650_inactive_groups.php
@@ -2,6 +2,7 @@
 
 use Illuminate\Database\Migrations\Migration;
 use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Schema;
 
 return new class extends Migration
@@ -11,18 +12,25 @@
      */
     public function up(): void
     {
-        $groups = \App\Models\Group::join('grouptags_groups', 'groups.idgroups', '=', 'grouptags_groups.group')
+        // Use query builder instead of Eloquent model to avoid SoftDeletes
+        // scope referencing a deleted_at column that doesn't exist yet.
+        $groups = DB::table('groups')
+            ->join('grouptags_groups', 'groups.idgroups', '=', 'grouptags_groups.group')
             ->join('group_tags', 'grouptags_groups.group_tag', '=', 'group_tags.id')
             ->where('group_tags.id', \App\Models\GroupTags::INACTIVE)
+            ->select('groups.*')
             ->get();
 
         foreach ($groups as $group) {
-            $group->archived_at = $group->updated_at;
+            $name = str_replace('[INACTIVE] ', '', $group->name);
+            $name = str_replace('[INACTIVE]', '', $name);
 
-            // Remove [INACTIVE] from the group name - this is now indicated via archived_at.
-            $group->name = str_replace('[INACTIVE] ', '', $group->name);
-            $group->name = str_replace('[INACTIVE]', '', $group->name);
-            $group->save();
+            DB::table('groups')
+                ->where('idgroups', $group->idgroups)
+                ->update([
+                    'archived_at' => $group->updated_at,
+                    'name' => $name,
+                ]);
         }
     }
 
@@ -32,11 +40,12 @@ public function up(): void
     public function down(): void
     {
         // Add [INACTIVE] into all groups with archived_at.
-        $groups = \App\Models\Group::whereNotNull('archived_at')->get();
+        $groups = DB::table('groups')->whereNotNull('archived_at')->get();
 
         foreach ($groups as $group) {
-            $group->name = '[INACTIVE] ' . $group->name;
-            $group->save();
+            DB::table('groups')
+                ->where('idgroups', $group->idgroups)
+                ->update(['name' => '[INACTIVE] ' . $group->name]);
         }
     }
 };