Worked on PR review changes

app-modules/knowledge-base/database/migrations/2024_10_30_184250_update_slug_column_to_not_nullable_in_knowledge_base_categories_table.php

@@ -0,0 +1,21 @@
+<?php
+
+use Illuminate\Support\Facades\Schema;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Migrations\Migration;
+
+return new class () extends Migration {
+    public function up(): void
+    {
+        Schema::table('knowledge_base_categories', function (Blueprint $table) {
+            $table->string('slug')->nullable(false)->change();
+        });
+    }
+
+    public function down(): void
+    {
+        Schema::table('knowledge_base_categories', function (Blueprint $table) {
+            $table->string('slug')->nullable()->change();
+        });
+    }
+};

app-modules/knowledge-base/database/migrations/2024_10_29_183334_data_add_remainig_slug_in_knowledge_base_categories_table.php → app-modules/knowledge-base/database/migrations/2024_10_30_184951_data_add_remainig_slug_in_knowledge_base_categories_table.php

@@ -2,10 +2,9 @@
 
 use Illuminate\Support\Str;
 use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Log;
-use Illuminate\Database\QueryException;
 use Illuminate\Database\Migrations\Migration;
 use AidingApp\KnowledgeBase\Models\KnowledgeBaseCategory;
+use Illuminate\Database\UniqueConstraintViolationException;
 
 return new class () extends Migration {
     public function up(): void
@@ -27,20 +26,15 @@ public function up(): void
                     DB::commit();
 
                     break;
-                } catch (QueryException $e) {
+                } catch (UniqueConstraintViolationException $e) {
                     DB::rollBack();
-
-                    if ($e->errorInfo[1] === 7) {
-                        $attempts++;
-                        $slug = $baseSlug . '-' . $attempts;
-                    } else {
-                        throw $e;
-                    }
+                    $attempts++;
+                    $slug = $baseSlug . '-' . $attempts;
                 }
             }
 
             if ($attempts == 15) {
-                Log::info("Failed to generate a unique slug after 15 attempts for category ID {$category->name}\n");
+                throw new Exception("Failed to generate a unique slug after 15 attempts for category ID {$category->name}\n");
             }
         }
     }