generate notification path on demand

public/src/modules/notifications.js

@@ -66,26 +66,28 @@ define('notifications', ['sounds', 'translator', 'components'], function(sound,
 
 		socket.on('event:new_notification', function(notifData) {
 			// If a path is defined, show notif data, otherwise show generic data
-			var payload;
+			var payload = {
-			if (notifData.path) {
-				payload = {
 				alert_id: 'new_notif',
 				title: '[[notifications:new_notification]]',
-					message: notifData.bodyShort,
-					type: 'info',
-					timeout: 2000,
-					clickfn: function() {
-						ajaxify.go(notifData.path);
-					}
-				};
-			} else {
-				payload = {
-					alert_id: 'new_notif',
-					title: '[[notifications:new_notification]]',
-					message: '[[notifications:you_have_unread_notifications]]',
-					type: 'warning',
 				timeout: 2000
 			};
+
+			if (notifData.path) {
+				payload.message = notifData.bodyShort;
+				payload.type = 'info';
+				payload.clickfn = function() {
+					socket.emit('notifications.generatePath', notifData.nid, function(err, path) {
+						if (err) {
+							return app.alertError(err.message);
+						}
+						if (path) {
+							ajaxify.go(path);
+						}
+					});
+				};
+			} else {
+				payload.message: '[[notifications:you_have_unread_notifications]]';
+				payload.type = 'warning';
 			}
 
 			app.alert(payload);

src/notifications.js

@@ -195,11 +195,8 @@ var async = require('async'),
 
 			var websockets = require('./socket.io');
 			if (websockets.server) {
-				// Add notification paths to sent notification object as well
+				uids.forEach(function(uid) {
-				async.eachLimit(uids, 50, function(uid, next) {
+					websockets.in('uid_' + uid).emit('event:new_notification', notification);
-					User.notifications.generateNotificationPaths([notification], uid, function(err, notifications) {
-						websockets.in('uid_' + uid).emit('event:new_notification', notifications[0]);
-					});
 				});
 			}
 
@@ -386,7 +383,7 @@ var async = require('async'),
 						var usernames = set.map(function(notifObj) {
 							return notifObj.user.username;
 						}).filter(function(username, idx, array) {
-							return array.indexOf(username) === idx
+							return array.indexOf(username) === idx;
 						});
 						var numUsers = usernames.length;
 

src/socket.io/notifications.js

@@ -1,8 +1,10 @@
 "use strict";
 
-var	user = require('../user'),
+var async = require('async');
-	notifications = require('../notifications'),
+var	user = require('../user');
-	SocketNotifs = {};
+var notifications = require('../notifications');
+
+var SocketNotifs = {};
 
 SocketNotifs.get = function(socket, data, callback) {
 	if (data && Array.isArray(data.nids) && socket.uid) {
@@ -53,4 +55,28 @@ SocketNotifs.markAllRead = function(socket, data, callback) {
 	notifications.markAllRead(socket.uid, callback);
 };
 
+SocketNotifs.generatePath = function(socket, nid, callback) {
+	if (!socket.uid) {
+		return;
+	}
+	async.waterfall([
+		function (next) {
+			notifications.get(nid, next);
+		},
+		function (notification, next) {
+			if (!notification) {
+				return next(null, '');
+			}
+			user.notifications.generateNotificationPaths([notification], socket.uid, next);
+		},
+		function (notificationsData, next) {
+			if (notificationsData && notificationsData.length) {
+				next(null, notificationsData[0].path);
+			} else {
+				next();
+			}
+		}
+	], callback);
+};
+
 module.exports = SocketNotifs;