Correctly set the subject, message and icon

Signed-off-by: Joas Schilling <coding@schilljs.com>

Author: nickvergessen

lib/NotificationGenerator.php

@@ -63,7 +63,11 @@ private function getNotificationForEvent(IEvent $event, int $activityId): INotif
 		$notification->setObject('activity_notification', (string)$activityId);
 		$notification->setSubject($event->getSubject());
 
-		if ($notification->getRichMessage()) {
+		if ($event->getRichSubject()) {
+			$notification->setRichSubject($event->getRichSubject(), $event->getRichSubjectParameters());
+		}
+
+		if ($event->getRichMessage()) {
 			$notification->setRichMessage($event->getRichMessage(), $event->getRichMessageParameters());
 		}
 
@@ -117,11 +121,15 @@ private function getDisplayNotificationForEvent(IEvent $event, int $activityId):
 		$notification->setRichSubject($event->getRichSubject(), $event->getRichSubjectParameters());
 		$notification->setParsedSubject($event->getParsedSubject());
 
-		if ($notification->getRichMessage()) {
+		if ($event->getIcon()) {
+			$notification->setIcon($event->getIcon());
+		}
+
+		if ($event->getRichMessage()) {
 			$notification->setRichMessage($event->getRichMessage(), $event->getRichMessageParameters());
 		}
 
-		if ($notification->getParsedMessage()) {
+		if ($event->getParsedMessage()) {
 			$notification->setParsedSubject($event->getParsedSubject());
 		}