Notifications: Do not use rich text for board/card

Signed-off-by: Julius Härtl <jus@bitgrid.net>

lib/Notification/Notifier.php

@@ -49,8 +49,7 @@ class Notifier implements INotifier {
 		IURLGenerator $url,
 		IUserManager $userManager,
 		CardMapper $cardMapper,
-		BoardMapper $boardMapper,
+		BoardMapper $boardMapper
-		Definitions $definitions
 	) {
 		$this->l10nFactory = $l10nFactory;
 		$this->url = $url;
@@ -67,10 +66,10 @@ class Notifier implements INotifier {
 	 * @since 9.0.0
 	 */
 	public function prepare(INotification $notification, $languageCode) {
+		$l = $this->l10nFactory->get('deck', $languageCode);
 		if($notification->getApp() !== 'deck') {
 			throw new \InvalidArgumentException();
 		}
-		$l = $this->l10nFactory->get('deck', $languageCode);
 		$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath('deck', 'deck-dark.svg')));
 		$params = $notification->getSubjectParameters();
 
@@ -79,28 +78,13 @@ class Notifier implements INotifier {
 				$cardId = $notification->getObjectId();
 				$boardId = $this->cardMapper->findBoardId($cardId);
 				$notification->setParsedSubject(
-					(string) $l->t('The card "%s" on "%s" has reached its due date.', $notification->getSubjectParameters())
+					(string) $l->t('The card "%s" on "%s" has reached its due date.', $params)
-				);
-				$notification->setRichSubject(
-					(string) $l->t('The card {card} on {board} has reached its due date.'),
-					[
-						'card' => [
-							'id' => null,
-							'type' => 'announcement',
-							'name' => $params[0],
-						],
-						'board' => [
-							'id' => null,
-							'type' => 'announcement',
-							'name' => $params[1],
-						],
-					]
 				);
 				$notification->setLink($this->url->linkToRouteAbsolute('deck.page.index') . '#!/board/'.$boardId.'//card/'.$cardId.'');
 				break;
 			case 'board-shared':
 				$boardId = $notification->getObjectId();
-				$initiator = \OC::$server->getUserManager()->get($params[1]);
+				$initiator = $this->userManager->get($params[1]);
 				if($initiator !== null) {
 					$dn = $initiator->getDisplayName();
 				} else {
@@ -108,19 +92,15 @@ class Notifier implements INotifier {
 				}
 				$notification->setParsedSubject(
 					(string) $l->t('The board "%s" has been shared with you by %s.', [$params[0], $dn])
-				)->setRichSubject(
+				);
-					(string) $l->t('{user} has shared the board {board} with you.'),
+				$notification->setRichSubject(
+					(string) $l->t('{user} has shared the board %s with you.', [$params[0]]),
 					[
 						'user' => [
 							'type' => 'user',
 							'id' => $params[1],
 							'name' => $dn,
-						],
+						]
-						'board' => [
-							'id' => null,
-							'type' => 'announcement',
-							'name' => $params[0],
-						],
 					]
 				);
 				$notification->setLink($this->url->linkToRouteAbsolute('deck.page.index') . '#!/board/'.$boardId.'/');

tests/unit/Notification/NotifierTest.php

@@ -0,0 +1,7 @@
+<?php
+/**
+ * Created by PhpStorm.
+ * User: jus
+ * Date: 01.10.17
+ * Time: 17:23
+ */