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

database/migrations/2024_08_11_023807_create_branches_table.php

@@ -1,39 +1,39 @@
 <?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 {
-        /**
-         * Run the migrations.
-         */
-        public function up()
-        : void
-        {
-            Schema::create('branches', function (Blueprint $table) {
-                $table->id();
-                $table->string('code', 9)->unique();
-                $table->string('name');
-                $table->boolean('status')->default(true)->nullable();
-                $table->timestamps();
-                $table->timestamp('authorized_at')->nullable();
-                $table->char('authorized_status', 1)->nullable();
-                $table->softDeletes();
+return new class extends Migration {
+    /**
+     * Run the migrations.
+     */
+    public function up(): void
+    {
+        Schema::create('branches', function (Blueprint $table) {
+            $table->id();
+            $table->string('code', 9)->unique();
+            $table->string('name');
+            $table->boolean('status')->default(true)->nullable();
+            $table->timestamps();
+            $table->timestamp('authorized_at')->nullable();
+            $table->char('authorized_status', 1)->nullable();
+            $table->softDeletes();
 
-                $table->unsignedBigInteger('created_by')->nullable();
-                $table->unsignedBigInteger('updated_by')->nullable();
-                $table->unsignedBigInteger('deleted_by')->nullable();
-                $table->unsignedBigInteger('authorized_by')->nullable();
-            });
-        }
+            $table->unsignedBigInteger('created_by')->nullable();
+            $table->unsignedBigInteger('updated_by')->nullable();
+            $table->unsignedBigInteger('deleted_by')->nullable();
+            $table->unsignedBigInteger('authorized_by')->nullable();
+        });
+    }
 
-        /**
-         * Reverse the migrations.
-         */
-        public function down()
-        : void
-        {
-            Schema::dropIfExists('branches');
-        }
-    };
+    /**
+     * Reverse the migrations.
+     */
+    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) {
-            $table->unsignedBigInteger('parent_id')->nullable()->after('name');
-            $table->foreign('parent_id')->references('id')->on('branches')->onDelete('set null');
+            // Tambah kolom parent_id jika belum ada
+            if (!Schema::hasColumn('branches', 'parent_id')) {
+                $table->unsignedBigInteger('parent_id')->nullable()->after('name');
+            }
         });
+
+        // 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('branches_parent_id_foreign');
+            });
+        }
+
+        // Drop kolom jika ada
         Schema::table('branches', function (Blueprint $table) {
-            $table->dropForeign(['parent_id']);
-            $table->dropColumn('parent_id');
+            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

@@ -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
@@ -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
@@ -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