Apply fixes from StyleCI (#448)

src/HttpApi/Controllers/FetchChannelsController.php

@@ -2,13 +2,13 @@
 
 namespace BeyondCode\LaravelWebSockets\HttpApi\Controllers;
 
+use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
+use BeyondCode\LaravelWebSockets\WebSockets\Channels\ChannelManager;
+use BeyondCode\LaravelWebSockets\WebSockets\Channels\PresenceChannel;
 use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Str;
 use Symfony\Component\HttpKernel\Exception\HttpException;
-use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
-use BeyondCode\LaravelWebSockets\WebSockets\Channels\ChannelManager;
-use BeyondCode\LaravelWebSockets\WebSockets\Channels\PresenceChannel;
 
 class FetchChannelsController extends Controller
 {
@@ -45,7 +45,7 @@ class FetchChannelsController extends Controller
         // We want to get the channel user count all in one shot when
         // using a replication backend rather than doing individual queries.
         // To do so, we first collect the list of channel names.
-        $channelNames = $channels->map(function (PresenceChannel $channel) use ($request) {
+        $channelNames = $channels->map(function (PresenceChannel $channel) {
             return $channel->getChannelName();
         })->toArray();
 

src/PubSub/Broadcasters/RedisPusherBroadcaster.php

@@ -2,12 +2,12 @@
 
 namespace BeyondCode\LaravelWebSockets\PubSub\Broadcasters;
 
-use Pusher\Pusher;
-use Illuminate\Support\Arr;
-use Illuminate\Support\Str;
-use Illuminate\Contracts\Redis\Factory as Redis;
 use Illuminate\Broadcasting\Broadcasters\Broadcaster;
 use Illuminate\Broadcasting\Broadcasters\UsePusherChannelConventions;
+use Illuminate\Contracts\Redis\Factory as Redis;
+use Illuminate\Support\Arr;
+use Illuminate\Support\Str;
+use Pusher\Pusher;
 use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
 
 class RedisPusherBroadcaster extends Broadcaster

src/PubSub/Drivers/LocalClient.php

@@ -2,11 +2,11 @@
 
 namespace BeyondCode\LaravelWebSockets\PubSub\Drivers;
 
-use stdClass;
+use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
 use React\EventLoop\LoopInterface;
 use React\Promise\FulfilledPromise;
 use React\Promise\PromiseInterface;
-use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
+use stdClass;
 
 class LocalClient implements ReplicationInterface
 {
@@ -23,7 +23,7 @@ class LocalClient implements ReplicationInterface
      * @param LoopInterface $loop
      * @return self
      */
-    public function boot(LoopInterface $loop) : ReplicationInterface
+    public function boot(LoopInterface $loop): ReplicationInterface
     {
         return $this;
     }
@@ -36,7 +36,7 @@ class LocalClient implements ReplicationInterface
      * @param stdClass $payload
      * @return bool
      */
-    public function publish(string $appId, string $channel, stdClass $payload) : bool
+    public function publish(string $appId, string $channel, stdClass $payload): bool
     {
         // Nothing to do, nobody to publish to
         return true;
@@ -49,7 +49,7 @@ class LocalClient implements ReplicationInterface
      * @param string $channel
      * @return bool
      */
-    public function subscribe(string $appId, string $channel) : bool
+    public function subscribe(string $appId, string $channel): bool
     {
         return true;
     }
@@ -61,7 +61,7 @@ class LocalClient implements ReplicationInterface
      * @param string $channel
      * @return bool
      */
-    public function unsubscribe(string $appId, string $channel) : bool
+    public function unsubscribe(string $appId, string $channel): bool
     {
         return true;
     }
@@ -103,7 +103,7 @@ class LocalClient implements ReplicationInterface
      * @param string $channel
      * @return PromiseInterface
      */
-    public function channelMembers(string $appId, string $channel) : PromiseInterface
+    public function channelMembers(string $appId, string $channel): PromiseInterface
     {
         $members = $this->channelData["$appId:$channel"] ?? [];
 
@@ -122,7 +122,7 @@ class LocalClient implements ReplicationInterface
      * @param array $channelNames
      * @return PromiseInterface
      */
-    public function channelMemberCounts(string $appId, array $channelNames) : PromiseInterface
+    public function channelMemberCounts(string $appId, array $channelNames): PromiseInterface
     {
         $results = [];
 

src/PubSub/Drivers/RedisClient.php

@@ -2,14 +2,14 @@
 
 namespace BeyondCode\LaravelWebSockets\PubSub\Drivers;
 
-use stdClass;
-use Illuminate\Support\Str;
-use Clue\React\Redis\Client;
-use Clue\React\Redis\Factory;
-use React\EventLoop\LoopInterface;
-use React\Promise\PromiseInterface;
 use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
 use BeyondCode\LaravelWebSockets\WebSockets\Channels\ChannelManager;
+use Clue\React\Redis\Client;
+use Clue\React\Redis\Factory;
+use Illuminate\Support\Str;
+use React\EventLoop\LoopInterface;
+use React\Promise\PromiseInterface;
+use stdClass;
 
 class RedisClient implements ReplicationInterface
 {

src/PubSub/ReplicationInterface.php

@@ -2,9 +2,9 @@
 
 namespace BeyondCode\LaravelWebSockets\PubSub;
 
-use stdClass;
 use React\EventLoop\LoopInterface;
 use React\Promise\PromiseInterface;
+use stdClass;
 
 interface ReplicationInterface
 {

src/WebSocketsServiceProvider.php

@@ -17,13 +17,13 @@ use BeyondCode\LaravelWebSockets\Statistics\Http\Controllers\WebSocketStatistics
 use BeyondCode\LaravelWebSockets\Statistics\Http\Middleware\Authorize as AuthorizeStatistics;
 use BeyondCode\LaravelWebSockets\WebSockets\Channels\ChannelManager;
 use BeyondCode\LaravelWebSockets\WebSockets\Channels\ChannelManagers\ArrayChannelManager;
-use Pusher\Pusher;
-use Psr\Log\LoggerInterface;
 use Illuminate\Broadcasting\BroadcastManager;
 use Illuminate\Support\Facades\Gate;
 use Illuminate\Support\Facades\Route;
-use Illuminate\Support\ServiceProvider;
 use Illuminate\Support\Facades\Schema;
+use Illuminate\Support\ServiceProvider;
+use Psr\Log\LoggerInterface;
+use Pusher\Pusher;
 
 class WebSocketsServiceProvider extends ServiceProvider
 {

tests/Channels/ChannelReplicationTest.php

@@ -8,7 +8,7 @@ class ChannelReplicationTest extends ChannelTest
 {
     use TestsReplication;
 
-    public function setUp() : void
+    public function setUp(): void
     {
         parent::setUp();
 

tests/Channels/PresenceChannelReplicationTest.php

@@ -8,7 +8,7 @@ class PresenceChannelReplicationTest extends PresenceChannelTest
 {
     use TestsReplication;
 
-    public function setUp() : void
+    public function setUp(): void
     {
         parent::setUp();
 

tests/HttpApi/FetchChannelReplicationTest.php

@@ -8,7 +8,7 @@ class FetchChannelReplicationTest extends FetchChannelTest
 {
     use TestsReplication;
 
-    public function setUp() : void
+    public function setUp(): void
     {
         parent::setUp();
 

tests/HttpApi/FetchChannelsReplicationTest.php

@@ -8,7 +8,7 @@ class FetchChannelsReplicationTest extends FetchChannelsTest
 {
     use TestsReplication;
 
-    public function setUp() : void
+    public function setUp(): void
     {
         parent::setUp();
 

tests/HttpApi/FetchUsersReplicationTest.php

@@ -8,7 +8,7 @@ class FetchUsersReplicationTest extends FetchUsersTest
 {
     use TestsReplication;
 
-    public function setUp() : void
+    public function setUp(): void
     {
         parent::setUp();
 

tests/Statistics/Controllers/WebSocketsStatisticsControllerTest.php

@@ -23,7 +23,7 @@ class WebSocketsStatisticsControllerTest extends TestCase
         $this->assertCount(1, $entries);
 
         $actual = $entries->first()->attributesToArray();
-      
+
         foreach ($this->payload() as $key => $value) {
             $this->assertArrayHasKey($key, $actual);
             $this->assertSame($value, $actual[$key]);

tests/TestsReplication.php

@@ -2,9 +2,9 @@
 
 namespace BeyondCode\LaravelWebSockets\Tests;
 
-use Illuminate\Support\Facades\Config;
 use BeyondCode\LaravelWebSockets\PubSub\Drivers\LocalClient;
 use BeyondCode\LaravelWebSockets\PubSub\ReplicationInterface;
+use Illuminate\Support\Facades\Config;
 
 trait TestsReplication
 {