Fixed uniqueness

Author: rennokki

src/ChannelManagers/LocalChannelManager.php

@@ -323,7 +323,7 @@ public function getChannelMembers($appId, string $channel): PromiseInterface
 
         $members = collect($members)->map(function ($user) {
             return json_decode($user);
-        })->unique('id')->toArray();
+        })->unique('user_id')->toArray();
 
         return new FulfilledPromise($members);
     }

src/ChannelManagers/RedisChannelManager.php

@@ -350,7 +350,7 @@ public function getChannelMembers($appId, string $channel): PromiseInterface
                     ->map(function ($user) {
                         return json_decode($user);
                     })
-                    ->unique('id')
+                    ->unique('user_id')
                     ->toArray();
             });
     }

tests/PresenceChannelTest.php

@@ -73,6 +73,41 @@ public function test_connect_to_presence_channel_when_user_with_same_ids_is_alre
             ]);
         }
 
+        $rick->assertSentEvent('pusher_internal:subscription_succeeded', [
+            'channel' => 'presence-channel',
+            'data' => json_encode([
+                'presence' => [
+                    'ids' => ['1'],
+                    'hash' => ['1' => []],
+                    'count' => 1,
+                ],
+            ]),
+        ]);
+
+        $morty->assertSentEvent('pusher_internal:subscription_succeeded', [
+            'channel' => 'presence-channel',
+            'data' => json_encode([
+                'presence' => [
+                    'ids' => ['1', '2'],
+                    'hash' => ['1' => [], '2' => []],
+                    'count' => 2,
+                ],
+            ]),
+        ]);
+
+        // The duplicated-user_id connection should get basically the list of ids
+        // without dealing with duplicate user ids.
+        $pickleRick->assertSentEvent('pusher_internal:subscription_succeeded', [
+            'channel' => 'presence-channel',
+            'data' => json_encode([
+                'presence' => [
+                    'ids' => ['1', '2'],
+                    'hash' => ['1' => [], '2' => []],
+                    'count' => 2,
+                ],
+            ]),
+        ]);
+
         $this->channelManager
             ->getGlobalConnectionsCount('1234', 'presence-channel')
             ->then(function ($total) {