X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/65ac197be440b90deccfd13d4eb8e6bec11e23b0..refs/pull/5725/head:/app/Activity/Notifications/Messages/PageUpdateNotification.php diff --git a/app/Activity/Notifications/Messages/PageUpdateNotification.php b/app/Activity/Notifications/Messages/PageUpdateNotification.php index a303a7883..80ee378cc 100644 --- a/app/Activity/Notifications/Messages/PageUpdateNotification.php +++ b/app/Activity/Notifications/Messages/PageUpdateNotification.php @@ -2,9 +2,9 @@ namespace BookStack\Activity\Notifications\Messages; +use BookStack\Activity\Notifications\MessageParts\EntityLinkMessageLine; use BookStack\Activity\Notifications\MessageParts\ListMessageLine; use BookStack\Entities\Models\Page; -use BookStack\Entities\Models\Chapter; use BookStack\Users\Models\User; use Illuminate\Notifications\Messages\MailMessage; @@ -14,32 +14,19 @@ class PageUpdateNotification extends BaseActivityNotification { /** @var Page $page */ $page = $this->detail; - $book = $page->book; - $chapterId = $page->chapter_id; - $chapter = $chapterId ? Chapter::find($chapterId) : null; $locale = $notifiable->getLocale(); - $listMessageData = [ - $locale->trans('notifications.detail_page_name') => $page->name, - '' => '', - ]; - - if ($chapter) { - $listMessageData += [ - $locale->trans('notifications.detail_chapter_name') => $chapter->name, - ]; - } - - $listMessageData += [ - $locale->trans('notifications.detail_book_name') => $book->name, + $listLines = array_filter([ + $locale->trans('notifications.detail_page_name') => new EntityLinkMessageLine($page), + $locale->trans('notifications.detail_page_path') => $this->buildPagePathLine($page, $notifiable), $locale->trans('notifications.detail_updated_by') => $this->user->name, - ]; + ]); return $this->newMailMessage($locale) ->subject($locale->trans('notifications.updated_page_subject', ['pageName' => $page->getShortName()])) ->line($locale->trans('notifications.updated_page_intro', ['appName' => setting('app-name')])) - ->line(new ListMessageLine($listMessageData)) + ->line(new ListMessageLine($listLines)) ->line($locale->trans('notifications.updated_page_debounce')) ->action($locale->trans('notifications.action_view_page'), $page->getUrl()) ->line($this->buildReasonFooterLine($locale));