Replace username with display name in more places (#3040)

* Replace username with display name in more places * More readable spread operator and translator user magic
flarum · Aug 21, 2021 · aba6836 · aba6836
1 parent af89b23
commit aba6836
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/js/src/forum/utils/UserControls.js b/js/src/forum/utils/UserControls.js
@@ -124,13 +124,18 @@ export default {
    * @param {string} type
    */
   showDeletionAlert(user, type) {
-    const { username, email } = user.data.attributes;
     const message = {
       success: 'core.forum.user_controls.delete_success_message',
       error: 'core.forum.user_controls.delete_error_message',
     }[type];
 
-    app.alerts.show({ type }, app.translator.trans(message, { username, email }));
+    app.alerts.show(
+      { type },
+      app.translator.trans(message, {
+        user,
+        email: user.email(),
+      })
+    );
   },
 
   /**

diff --git a/src/Notification/NotificationMailer.php b/src/Notification/NotificationMailer.php
@@ -46,7 +46,7 @@ public function send(MailableInterface $blueprint, User $user)
             $blueprint->getEmailView(),
             compact('blueprint', 'user'),
             function (Message $message) use ($blueprint, $user) {
-                $message->to($user->email, $user->username)
+                $message->to($user->email, $user->display_name)
                         ->subject($blueprint->getEmailSubject($this->translator));
             }
         );

diff --git a/views/frontend/content/discussion.blade.php b/views/frontend/content/discussion.blade.php
@@ -5,7 +5,7 @@
         @foreach ($posts as $post)
             <div>
                 <?php $user = ! empty($post->relationships->user->data) ? $getResource($post->relationships->user->data) : null; ?>
-                <h3>{{ $user ? $user->attributes->username : $translator->trans('core.lib.username.deleted_text') }}</h3>
+                <h3>{{ $user ? $user->attributes->displayName : $translator->trans('core.lib.username.deleted_text') }}</h3>
                 <div class="Post-body">
                     {!! $post->attributes->contentHtml !!}
                 </div>