Notification missing setParsedSubject

Rello · Dec 15, 2019 · c05d747 · c05d747
1 parent badce4c
commit c05d747
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,10 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## 1.2.2 - 2019-12-15
+### Fixed
+- Notification missing setParsedSubject
+
 ## 1.2.1 - 2019-12-15
 ### Fixed
 - Activity not always reported

diff --git a/lib/Notification/Notifier.php b/lib/Notification/Notifier.php
@@ -87,7 +87,7 @@ public function prepare(INotification $notification, string $languageCode): INot
         $l = $this->l10nFactory->get('analytics', $languageCode);
         $parsedSubject = '';
 
-        //$this->logger->error('Notifier 89: '.$notification->getSubject());
+        //$this->logger->error('Notifier 90: '.$notification->getSubject());
         switch ($notification->getSubject()) {
             case NotificationManager::SUBJECT_THRESHOLD:
                 $parsedSubject = $l->t("Exception in Report '{report}'. The value of '{subject}' reached the threshold of '{rule} {value}'");
@@ -123,6 +123,7 @@ public function prepare(INotification $notification, string $languageCode): INot
             ]
         );
         $notification->setIcon($this->urlGenerator->getAbsoluteURL($this->urlGenerator->imagePath('analytics', 'app-dark.svg')));
+        $notification->setParsedSubject('analytics');
         return $notification;
     }
 }