Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-02 20:16:04 +01:00
Code Issues Packages Projects Releases Wiki Activity

changed user posted notification to include title

Browse Source 
dont send 2 notifications if you are already following topic
This commit is contained in:
barisusakli
2014-07-21 17:43:38 -04:00
parent 67c5477fb2
commit c9fb5b3e11
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/user/notifications.js
View File

@@ -156,7 +156,7 @@ var async = require('async'),
async.parallel({
username: async.apply(user.getUserField, uid, 'username'),
topic: async.apply(topics.getTopicFields, tid, ['slug', 'cid']),
topic: async.apply(topics.getTopicFields, tid, ['slug', 'cid', 'title']),
postIndex: async.apply(posts.getPidIndex, pid),
postContent: function(next) {
async.waterfall([
@@ -165,14 +165,21 @@ var async = require('async'),
postTools.parse(content, next);
}
], next);
},
topicFollowers: function(next) {
db.isSetMembers('tid:' + tid + ':followers', followers, next);
}
}, function(err, results) {
if (err) {
return;
}
followers = followers.filter(function(value, index) {
return !results.topicFollowers[index];
});
notifications.create({
bodyShort: '[[notifications:user_made_post, ' + results.username + ']]',
bodyShort: '[[notifications:user_posted_to, ' + results.username + ', ' + results.topic.title + ']]',
bodyLong: results.postContent,
path: nconf.get('relative_path') + '/topic/' + results.topic.slug + '/' + results.postIndex,
uniqueId: 'topic:' + tid,