From e038aebfa091935c3916ddbfd817e7615e82cffd Mon Sep 17 00:00:00 2001 From: Toby Zerner Date: Fri, 21 Sep 2018 11:27:28 +0930 Subject: [PATCH] Remove notification pluralization logic Since we don't return an "unread count" in the API, this pluralization logic is useless at the moment. Related to #643 and #716 but really needs further thought in #868. --- .../js/src/forum/components/PostLikedNotification.js | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/extensions/likes/js/src/forum/components/PostLikedNotification.js b/extensions/likes/js/src/forum/components/PostLikedNotification.js index f399acfdf..34678eb2b 100644 --- a/extensions/likes/js/src/forum/components/PostLikedNotification.js +++ b/extensions/likes/js/src/forum/components/PostLikedNotification.js @@ -1,6 +1,4 @@ import Notification from 'flarum/components/Notification'; -import username from 'flarum/helpers/username'; -import punctuateSeries from 'flarum/helpers/punctuateSeries'; export default class PostLikedNotification extends Notification { icon() { @@ -14,15 +12,8 @@ export default class PostLikedNotification extends Notification { content() { const notification = this.props.notification; const user = notification.fromUser(); - const auc = notification.additionalUnreadCount(); - return app.translator.transChoice('flarum-likes.forum.notifications.post_liked_text', auc + 1, { - user, - username: auc ? punctuateSeries([ - username(user), - app.translator.transChoice('flarum-likes.forum.notifications.others_text', auc, {count: auc}) - ]) : undefined - }); + return app.translator.transChoice('flarum-likes.forum.notifications.post_liked_text', 1, {user}); } excerpt() {