From c1131258567d127f6995bcd5615ab1c6d620843b Mon Sep 17 00:00:00 2001
From: Daniel Wilkowski <wilkowski.kontakt@gmail.com>
Date: Wed, 9 Oct 2024 11:24:21 +0200
Subject: [PATCH] Make viewers avatars link to user profile

---
 app/Domain/Online/ViewerUser.php        | 1 +
 app/Domain/Online/ViewersStore.php      | 1 +
 resources/views/components/viewers.twig | 8 ++++----
 tests/Unit/OnlineUsers/ViewersTest.php  | 2 +-
 4 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/app/Domain/Online/ViewerUser.php b/app/Domain/Online/ViewerUser.php
index 369020468..037c990b9 100644
--- a/app/Domain/Online/ViewerUser.php
+++ b/app/Domain/Online/ViewerUser.php
@@ -7,6 +7,7 @@ public function __construct(
         public string  $name,
         public ?string $groupName,
         public ?string $avatarUrl,
+        public string  $profileUrl,
     )
     {
     }
diff --git a/app/Domain/Online/ViewersStore.php b/app/Domain/Online/ViewersStore.php
index 5c08a49be..24b156980 100644
--- a/app/Domain/Online/ViewersStore.php
+++ b/app/Domain/Online/ViewersStore.php
@@ -28,6 +28,7 @@ private function viewerUser(User $user): ViewerUser
             $user->name,
             $user->group_name,
             $user->photo->getFilename(),
+            \route('profile', [$user->id]),
         );
     }
 }
diff --git a/resources/views/components/viewers.twig b/resources/views/components/viewers.twig
index 042d50ba9..eaa4ebd2a 100644
--- a/resources/views/components/viewers.twig
+++ b/resources/views/components/viewers.twig
@@ -12,7 +12,7 @@
 <div class="viewers-online">
   <div class="mb-1 viewers-users-group">
     {% for user in usersWithGroup %}
-      <div class="viewer-pill d-inline-flex mb-1" title="{{ user.name }} (online)">
+      <a class="viewer-pill d-inline-flex mb-1" title="{{ user.name }} (online)" style="color:inherit;" href="{{ user.profileUrl }}">
         <div class="viewer-pill-avatar d-inline-block">
           {{ user_avatar(user.avatarUrl, user.name) }}
         </div>
@@ -21,14 +21,14 @@
           <br>
           <small>{{ user.groupName }}</small>
         </span>
-      </div>
+      </a>
     {% endfor %}
   </div>
   <div class="viewers-users">
     {% for user in usersWithoutGroup %}
-      <span class="circle d-inline-block" title="{{ user.name }} (online)">
+      <a class="circle d-inline-block" title="{{ user.name }} (online)" style="color:inherit;" href="{{ user.profileUrl }}">
         {{ user_avatar(user.avatarUrl, user.name) }}
-      </span>
+      </a>
     {% endfor %}
     {% if superfluousCount > 0 %}
       <span class="circle circle-number d-inline-block" title="Jeszcze {{ superfluousCount }} innych użytkowników (online)">
diff --git a/tests/Unit/OnlineUsers/ViewersTest.php b/tests/Unit/OnlineUsers/ViewersTest.php
index 53033f573..aae54e2ed 100644
--- a/tests/Unit/OnlineUsers/ViewersTest.php
+++ b/tests/Unit/OnlineUsers/ViewersTest.php
@@ -46,6 +46,6 @@ private function usersOfSize(int $size): array
 
     private function user(string $name = null, string $group = null): ViewerUser
     {
-        return new ViewerUser($name ?? '', $group, null);
+        return new ViewerUser($name ?? '', $group, null, '');
     }
 }