Mark usernames with @.

Part-DB · Feb 6, 2020 · 7434234 · 7434234
1 parent 73c2aa2
commit 7434234
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Entity/UserSystem/User.php b/src/Entity/UserSystem/User.php
@@ -513,7 +513,7 @@ public function setPwResetExpires(DateTime $pw_reset_expires): self
     public function getFullName(bool $including_username = false): string
     {
         if ($including_username) {
-            return sprintf('%s %s (%s)', $this->getFirstName(), $this->getLastName(), $this->getName());
+            return sprintf('%s %s (@%s)', $this->getFirstName(), $this->getLastName(), $this->getName());
         }
 
         return sprintf('%s %s', $this->getFirstName(), $this->getLastName());

diff --git a/templates/_navbar.html.twig b/templates/_navbar.html.twig
@@ -33,7 +33,7 @@
                     {% if app.user %}<i class="fa fa-user" aria-hidden="true"></i>{% else %}<i class="far fa-user" aria-hidden="true"></i>{% endif %} <span class="caret"></span></a>
                 <ul class="dropdown-menu dropdown-menu-right" id="login-menu">
                     {% if app.user %}
-                        <a class="dropdown-item disabled" href="#" >{% trans %}user.loggedin.label{% endtrans %} {{ app.user.firstName }} {{app.user.lastName}} ({{app.user.name}})</a>
+                        <a class="dropdown-item disabled" href="#" >{% trans %}user.loggedin.label{% endtrans %} <b>{{ app.user.firstName }} {{app.user.lastName}}</b> (@{{app.user.name}})</a>
                         <a class="dropdown-item" href="{{ path("user_settings") }}"><i class="fa fa-cogs fa-fw" aria-hidden="true"></i> {% trans %}user.settings.label{% endtrans %}</a>
                         <a class="dropdown-item" href="{{ path("user_info_self") }}"><i class="fa fa-info-circle fa-fw" aria-hidden="true"></i> {% trans %}user.info.label{% endtrans %}</a>
                         <li role="separator" class="dropdown-divider"></li>

diff --git a/tests/Entity/UserSystem/UserTest.php b/tests/Entity/UserSystem/UserTest.php
@@ -39,7 +39,7 @@ public function testGetFullName(): void
         $user->setLastName('Doe');
 
         $this->assertSame('John Doe', $user->getFullName(false));
-        $this->assertSame('John Doe (username)', $user->getFullName(true));
+        $this->assertSame('John Doe (@username)', $user->getFullName(true));
     }
 
     public function googleAuthenticatorEnabledDataProvider(): array