Merge pull request 'feat(migration): add parent_id column and foreign key to branches table with checks' (#7) from shola into master

Reviewed-on: #7

app/Models/Branch.php

@@ -1,9 +1,9 @@
 <?php
 
-    namespace Modules\Basicdata\Models;
+namespace Modules\Basicdata\Models;
 
-    class Branch extends Base
-    {
+class Branch extends Base
+{
     protected $table = 'branches';
     protected $fillable = [
         'code',
@@ -40,4 +40,10 @@
     {
         return $this->hasMany(Branch::class, 'parent_id');
     }
+
+
+    public function users()
+    {
+        return $this->belongsToMany(Modules\Usermanagement\Models\User::class, 'user_branches', 'branch_id', 'user_id');
     }
+}

database/migrations/2024_08_11_023807_create_branches_table.php

@@ -1,15 +1,14 @@
 <?php
 
-    use Illuminate\Database\Migrations\Migration;
-    use Illuminate\Database\Schema\Blueprint;
-    use Illuminate\Support\Facades\Schema;
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
 
-    return new class extends Migration {
+return new class extends Migration {
     /**
      * Run the migrations.
      */
-        public function up()
-        : void
+    public function up(): void
     {
         Schema::create('branches', function (Blueprint $table) {
             $table->id();
@@ -31,9 +30,10 @@
     /**
      * Reverse the migrations.
      */
-        public function down()
-        : void
+    public function down(): void
     {
+        Schema::disableForeignKeyConstraints();
         Schema::dropIfExists('branches');
+        Schema::enableForeignKeyConstraints();
     }
-    };
+};

database/migrations/2025_05_18_074721_add_parent_id_to_branches_table.php

@@ -3,18 +3,30 @@
 use Illuminate\Database\Migrations\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
+use Illuminate\Support\Facades\DB;
 
-return new class extends Migration
-{
+return new class extends Migration {
     /**
      * Run the migrations.
      */
     public function up(): void
     {
         Schema::table('branches', function (Blueprint $table) {
+            // Tambah kolom parent_id jika belum ada
+            if (!Schema::hasColumn('branches', 'parent_id')) {
                 $table->unsignedBigInteger('parent_id')->nullable()->after('name');
-            $table->foreign('parent_id')->references('id')->on('branches')->onDelete('set null');
+            }
         });
+
+        // Tambah foreign key jika belum ada
+        if (!$this->foreignKeyExists('branches', 'branches_parent_id_foreign')) {
+            Schema::table('branches', function (Blueprint $table) {
+                $table->foreign('parent_id', 'branches_parent_id_foreign')
+                    ->references('id')
+                    ->on('branches')
+                    ->onDelete('set null');
+            });
+        }
     }
 
     /**
@@ -22,9 +34,39 @@ return new class extends Migration
      */
     public function down(): void
     {
+        // Drop foreign key jika ada
+        if ($this->foreignKeyExists('branches', 'branches_parent_id_foreign')) {
             Schema::table('branches', function (Blueprint $table) {
-            $table->dropForeign(['parent_id']);
-            $table->dropColumn('parent_id');
+                $table->dropForeign('branches_parent_id_foreign');
             });
         }
+
+        // Drop kolom jika ada
+        Schema::table('branches', function (Blueprint $table) {
+            if (Schema::hasColumn('branches', 'parent_id')) {
+                $table->dropColumn('parent_id');
+            }
+        });
+    }
+
+    /**
+     * Cek apakah foreign key exists
+     */
+    private function foreignKeyExists(string $table, string $name): bool
+    {
+        $conn = Schema::getConnection();
+        $dbName = $conn->getDatabaseName();
+
+        $result = DB::select(
+            "SELECT CONSTRAINT_NAME
+             FROM information_schema.TABLE_CONSTRAINTS
+             WHERE TABLE_SCHEMA = ?
+             AND TABLE_NAME = ?
+             AND CONSTRAINT_NAME = ?
+             AND CONSTRAINT_TYPE = 'FOREIGN KEY'",
+            [$dbName, $table, $name]
+        );
+
+        return count($result) > 0;
+    }
 };

database/seeders/BranchesSeeder.php

@@ -18,7 +18,7 @@ class BranchesSeeder extends Seeder
         $branches = [
             [
                 'code' => 'ID0010001',
-                'name' => 'PT. Bank Artha Graha',
+                'name' => 'KPNO',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -186,7 +186,7 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010031',
-                'name' => 'BKR Bandung - KCP',
+                'name' => 'BKR BANDUNG - KCP',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -235,14 +235,14 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010039',
-                'name' => 'Cimahi - KCP',
+                'name' => 'CIMAHI - KCP',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
             ],
             [
                 'code' => 'ID0010050',
-                'name' => 'Ir.Soekarno(MERR)-KCP',
+                'name' => 'IR.SOEKARNO(MERR)-KCP',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -319,7 +319,7 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010063',
-                'name' => 'Diponegoro Denpasar KK',
+                'name' => 'DIPONEGORO DENPASAR KK',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -494,7 +494,7 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010108',
-                'name' => 'Tzu Chi - KK',
+                'name' => 'TZU CHI - KK',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -557,7 +557,7 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010131',
-                'name' => 'Veteran Makassar - KCP',
+                'name' => 'VETERAN MAKASSAR - KCP',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now
@@ -704,7 +704,7 @@ class BranchesSeeder extends Seeder
             ],
             [
                 'code' => 'ID0010272',
-                'name' => 'Setiabudi Kuningan KK',
+                'name' => 'SETIABUDI KUNINGAN KK',
                 'status' => true,
                 'created_at' => $now,
                 'updated_at' => $now