test: fix

flarum · Oct 19, 2024 · 17ec156 · 17ec156
1 parent e3f9695
commit 17ec156
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 6 deletions.
diff --git a/extensions/messages/composer.json b/extensions/messages/composer.json
@@ -25,6 +25,9 @@
         "flarum-extension": {
             "title": "Messages",
             "category": "feature",
+            "optional-dependencies": [
+                "flarum/tags"
+            ],
             "icon": {
                 "name": "fas fa-envelope-open",
                 "color": "#ffffff",

diff --git a/...nsions/messages/migrations/2024_10_19_000002_create_dialog_message_mentions_tag_table.php b/...nsions/messages/migrations/2024_10_19_000002_create_dialog_message_mentions_tag_table.php
@@ -9,16 +9,20 @@
 
 use Flarum\Database\Migration;
 use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Schema\Builder;
 
 return Migration::createTable(
     'dialog_message_mentions_tag',
-    function (Blueprint $table) {
+    function (Blueprint $table, Builder $schema) {
         $table->unsignedInteger('dialog_message_id');
         $table->unsignedInteger('mentions_tag_id');
         $table->dateTime('created_at')->nullable()->useCurrent();
 
         $table->primary(['dialog_message_id', 'mentions_tag_id']);
         $table->foreign('dialog_message_id')->references('id')->on('dialog_messages')->cascadeOnDelete();
-        $table->foreign('mentions_tag_id')->references('id')->on('tags')->cascadeOnDelete();
+
+        if ($schema->hasTable('tags')) {
+            $table->foreign('mentions_tag_id')->references('id')->on('tags')->cascadeOnDelete();
+        }
     }
 );
diff --git a/framework/core/src/Database/Migration.php b/framework/core/src/Database/Migration.php
@@ -25,8 +25,8 @@ public static function createTable(string $name, callable $definition): array
     {
         return [
             'up' => function (Builder $schema) use ($name, $definition) {
-                $schema->create($name, function (Blueprint $table) use ($definition) {
-                    $definition($table);
+                $schema->create($name, function (Blueprint $table) use ($definition, $schema) {
+                    $definition($table, $schema);
                 });
             },
             'down' => function (Builder $schema) use ($name) {
@@ -40,8 +40,8 @@ public static function createTableIfNotExists(string $name, callable $definition
         return [
             'up' => function (Builder $schema) use ($name, $definition) {
                 if (! $schema->hasTable($name)) {
-                    $schema->create($name, function (Blueprint $table) use ($definition) {
-                        $definition($table);
+                    $schema->create($name, function (Blueprint $table) use ($definition, $schema) {
+                        $definition($table, $schema);
                     });
                 }
             },