hiro/Vulcan
1
0
Fork 0
mirror of https://github.com/vale981/Vulcan synced 2025-03-06 10:01:40 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Revert "Workaround for Issue #1580"

Browse source 
This reverts commit b46e6d4fa0.
This commit is contained in:
Dominic Tracey 2017-03-27 14:48:24 -04:00
parent b46e6d4fa0
commit 1892a3cf58
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
packages/vulcan-notifications/lib/callbacks.js
View file

@ -18,7 +18,7 @@ if (!!Posts) {
createNotification(post.userId, 'postApproved', notificationData); createNotification(post.userId, 'postApproved', notificationData);
} }
addCallback("posts.approve.async", PostsApprovedNotification); addCallback("posts.approve.async", PostsApprovedNotification);
/** /**
* @summary Add new post notification callback on post submit * @summary Add new post notification callback on post submit
*/ */
@ -26,7 +26,7 @@ if (!!Posts) {
let adminIds = _.pluck(Users.adminUsers({fields: {_id:1}}), '_id'); let adminIds = _.pluck(Users.adminUsers({fields: {_id:1}}), '_id');
let notifiedUserIds = _.pluck(Users.find({'notifications_posts': true}, {fields: {_id:1}}).fetch(), '_id'); let notifiedUserIds = _.pluck(Users.find({'notifications_posts': true}, {fields: {_id:1}}).fetch(), '_id');
const notificationData = { const notificationData = {
post: _.pick(post, '_id', 'userId', 'title', 'url', 'slug') post: _.pick(post, '_id', 'userId', 'title', 'url', 'slug')
}; };
@ -42,7 +42,7 @@ if (!!Posts) {
// if post is approved, notify everybody // if post is approved, notify everybody
createNotification(notifiedUserIds, 'newPost', notificationData); createNotification(notifiedUserIds, 'newPost', notificationData);
} }
} }
addCallback("posts.new.async", PostsNewNotifications); addCallback("posts.new.async", PostsNewNotifications);
} }
@ -65,7 +65,7 @@ if (!!Comments) {
// 1. Notify author of post (if they have new comment notifications turned on) // 1. Notify author of post (if they have new comment notifications turned on)
// but do not notify author of post if they're the ones posting the comment // but do not notify author of post if they're the ones posting the comment
if (Users.getSetting(postAuthor, "notifications_comments", false) && comment.userId !== postAuthor._id) { if (Users.getSetting(postAuthor, "notifications_comments", true) && comment.userId !== postAuthor._id) {
createNotification(post.userId, 'newComment', notificationData); createNotification(post.userId, 'newComment', notificationData);
userIdsNotified.push(post.userId); userIdsNotified.push(post.userId);
} }
@ -82,7 +82,7 @@ if (!!Comments) {
const parentCommentAuthor = Users.findOne(parentComment.userId); const parentCommentAuthor = Users.findOne(parentComment.userId);
// do not notify parent comment author if they have reply notifications turned off // do not notify parent comment author if they have reply notifications turned off
if (Users.getSetting(parentCommentAuthor, "notifications_replies", false)) { if (Users.getSetting(parentCommentAuthor, "notifications_replies", true)) {
// add parent comment to notification data // add parent comment to notification data
notificationData.parentComment = _.pick(parentComment, '_id', 'userId', 'author', 'htmlBody'); notificationData.parentComment = _.pick(parentComment, '_id', 'userId', 'author', 'htmlBody');
@ -93,9 +93,9 @@ if (!!Comments) {
} }
} }
} }
} }
addCallback("comments.new.async", CommentsNewNotifications); addCallback("comments.new.async", CommentsNewNotifications);
} }