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

uniqueId support in notifications

Browse Source
This commit is contained in:
Julian Lam
2013-05-23 13:48:42 -04:00
parent 681bf270a1
commit 7e46f8c6ce
2 changed files with 51 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
src/notifications.js
View File

@@ -5,13 +5,14 @@ var config = require('../config.js'),
(function(Notifications) { (function(Notifications) {
Notifications.get = function(nid, callback) { Notifications.get = function(nid, callback) {
RDB.hmget('notifications:' + nid, 'text', 'score', 'path', 'datetime', function(err, notification) { RDB.hmget('notifications:' + nid, 'text', 'score', 'path', 'datetime', 'uniqueId', function(err, notification) {
callback({ callback({
nid: nid, nid: nid,
text: notification[0], text: notification[0],
score: notification[1], score: notification[1],
path: notification[2], path: notification[2],
datetime: notification[3] datetime: notification[3],
uniqueId: notification[4]
}); });
}); });
} }
@@ -51,13 +52,52 @@ var config = require('../config.js'),
Notifications.get(nid, function(notif_data) { Notifications.get(nid, function(notif_data) {
for(x=0;x<numUids;x++) { for(x=0;x<numUids;x++) {
if (parseInt(uids[x]) > 0) { if (parseInt(uids[x]) > 0) {
RDB.zadd('uid:' + uids[x] + ':notifications:unread', notif_data.score, nid); (function(uid) {
if (callback) callback(true); Notifications.remove_by_uniqueId(notif_data.uniqueId, uid, function() {
RDB.zadd('uid:' + uid + ':notifications:unread', notif_data.score, nid);
if (callback) callback(true);
});
})(uids[x]);
} }
} }
}); });
} }
Notifications.remove_by_uniqueId = function(uniqueId, uid, callback) {
async.parallel([
function(next) {
RDB.zrange('uid:' + uid + ':notifications:unread', 0, -1, function(err, nids) {
if (nids && nids.length > 0) {
async.each(nids, function(nid, next) {
Notifications.get(nid, function(nid_info) {
if (nid_info.uniqueId === uniqueId) RDB.zrem('uid:' + uid + ':notifications:unread', nid);
next();
});
}, function(err) {
next();
});
} else next();
});
},
function(next) {
RDB.zrange('uid:' + uid + ':notifications:read', 0, -1, function(err, nids) {
if (nids && nids.length > 0) {
async.each(nids, function(nid, next) {
Notifications.get(nid, function(nid_info) {
if (nid_info.uniqueId === uniqueId) RDB.zrem('uid:' + uid + ':notifications:read', nid);
next();
});
}, function(err) {
next();
});
} else next();
});
}
], function(err) {
if (!err) callback(true);
});
}
Notifications.mark_read = function(nid, uid, callback) { Notifications.mark_read = function(nid, uid, callback) {
if (parseInt(uid) > 0) { if (parseInt(uid) > 0) {
Notifications.get(nid, function(notif_data) { Notifications.get(nid, function(notif_data) {

14
src/webserver.js
View File

@@ -240,14 +240,14 @@ var express = require('express'),
app.get('/api/:method/:id*', api_method); app.get('/api/:method/:id*', api_method);
app.get('/test', function(req, res) { app.get('/test', function(req, res) {
notifications.mark_read_multiple([1, 2], 1, function(success) { // notifications.remove_by_uniqueId('foobar', 1, function(success) {
res.send('mark: ' + success); // res.send('remove: ' + success);
});
// notifications.create('some text', 5, '/category/2/general-discussion', function(nid) {
// notifications.push(nid, 1, function() {
// res.send('nid: ', nid)
// });
// }); // });
notifications.create('a bunch more text', 5, '/category/2/general-discussion', 'foobar', function(nid) {
notifications.push(nid, 1, function() {
res.send('nid: ' + nid)
});
});
}); });