[2.x] Fix queue worker TypeError on null connection name

illuminate/queue 13.15.0 type-hinted WorkerIdle::$connectionName as
`string`. Flarum's queue worker resolves its connection name from
config('queue.default') (the WorkCommand argument fallback), which Flarum
never set, so the worker loop dispatched WorkerIdle(null, ...) and threw
a TypeError on every idle tick.

Set config('queue.default') to 'flarum' and give the queue connection a
name via setConnectionName('flarum'), so the worker always receives a
non-null string. This is a code fix rather than a version constraint:
it works on illuminate/queue both before and after 13.15.0.

Author: imorland

framework/core/src/Foundation/InstalledSite.php

@@ -159,7 +159,16 @@ protected function getIlluminateConfig(): ConfigRepository
                 'lifetime' => 120,
                 'files' => $this->paths->storage.'/sessions',
                 'cookie' => 'session'
-            ]
+            ],
+            // Flarum uses a single queue connection bound as 'flarum.queue.connection'
+            // rather than Laravel's named connections. The queue worker command falls
+            // back to config('queue.default') when no connection argument is given, so
+            // we provide a name here. Without it the name is null, which since
+            // illuminate/queue 13.15.0 throws a TypeError from WorkerIdle's typed
+            // string $connectionName in the worker loop.
+            'queue' => [
+                'default' => 'flarum',
+            ],
         ]);
     }
 

framework/core/src/Queue/QueueServiceProvider.php

@@ -73,6 +73,12 @@ public function register(): void
 
             $queue->setContainer($container);
 
+            // Give the connection a name so events that carry it (e.g. WorkerIdle)
+            // receive a string rather than null. illuminate/queue 13.15.0 type-hinted
+            // WorkerIdle::$connectionName as `string`, so a null name now throws a
+            // TypeError in the worker loop.
+            $queue->setConnectionName('flarum');
+
             return $queue;
         });
 

framework/core/tests/integration/queue/QueueServiceProviderTest.php

@@ -91,6 +91,39 @@ public function it_registers_queue_commands_for_database_driver()
         $this->assertContains(\Illuminate\Queue\Console\RetryCommand::class, $commandNames);
     }
 
+    /**
+     * Regression test: the queue connection must have a non-null string name.
+     *
+     * illuminate/queue 13.15.0 type-hinted WorkerIdle::$connectionName as
+     * `string`, so the worker loop throws a TypeError when the connection name
+     * is null. The name flows from the queue connection / config('queue.default').
+     */
+    #[Test]
+    public function queue_connection_has_a_non_null_string_name()
+    {
+        $this->config('queue', ['driver' => 'database']);
+
+        $this->app();
+
+        $queue = $this->app()->getContainer()->make(Queue::class);
+
+        $this->assertIsString($queue->getConnectionName());
+        $this->assertNotEmpty($queue->getConnectionName());
+    }
+
+    #[Test]
+    public function default_queue_connection_name_is_configured()
+    {
+        $this->app();
+
+        $default = $this->app()->getContainer()->make('config')->get('queue.default');
+
+        // The worker command falls back to this when no connection argument is
+        // given; it must be a non-null string to satisfy WorkerIdle.
+        $this->assertIsString($default);
+        $this->assertNotEmpty($default);
+    }
+
     #[Test]
     public function it_uses_null_failed_job_provider_for_sync_queue()
     {