nodebb-plugin-poll 3.1.4 → 3.1.5

package/lib/nodebb.js

@@ -8,6 +8,7 @@ module.exports = {
 	Posts: require.main.require('./src/posts'),
 	Topics: require.main.require('./src/topics'),
 	Privileges: require.main.require('./src/privileges'),
+	Plugins: require.main.require('./src/plugins'),
 	PluginSockets: require.main.require('./src/socket.io/plugins'),
 	AdminSockets: require.main.require('./src/socket.io/admin').plugins,
 	SocketIndex: require.main.require('./src/socket.io/index'),

package/lib/poll.js

@@ -37,17 +37,17 @@ const Scheduler = require('./scheduler');
 				async.apply(async.series, [
 					// Save poll options in parallel
 					async.apply(
-						async.parallel, 
+						async.parallel,
 						pollData.options.map(
-							(option) => async.apply(NodeBB.db.setObject, `poll:${pollId}:options:${option.id}`, option)
+							option => async.apply(NodeBB.db.setObject, `poll:${pollId}:options:${option.id}`, option)
 						),
 					),
-					// Add poll option id to option list synchronously 
+					// Add poll option id to option list synchronously
 					// To avoid race condition causing duplicate error
 					async.apply(
 						async.series,
 						pollData.options.map(
-							(option) => async.apply(NodeBB.db.setAdd, `poll:${pollId}:options`, option.id)
+							option => async.apply(NodeBB.db.setAdd, `poll:${pollId}:options`, option.id)
 						),
 					),
 				]),
@@ -56,7 +56,6 @@ const Scheduler = require('./scheduler');
 				async.apply(NodeBB.db.listAppend, 'polls', pollId),
 				async.apply(NodeBB.db.setObjectField, `topic:${poll.tid}`, 'pollId', pollId),
 			], (err) => {
-
 				if (err) {
 					return callback(err);
 				}

package/lib/sockets.js

@@ -36,8 +36,7 @@ const Vote = require('./vote');
 			}
 
 			pollData.optionType = parseInt(pollData.settings.maxvotes, 10) > 1 ? 'checkbox' : 'radio';
-
-			return callback(null, pollData);
+			callback(null, pollData);
 		});
 	};
 
@@ -188,22 +187,34 @@ const Vote = require('./vote');
 		if (!socket.uid || !data || isNaN(parseInt(data.pollId, 10)) || isNaN(parseInt(data.optionId, 10))) {
 			return callback(new Error('Invalid request'));
 		}
-
-		Poll.getOption(data.pollId, data.optionId, true, (err, result) => {
+		async.parallel({
+			poll: function (next) {
+				Poll.getInfo(data.pollId, next);
+			},
+			option: function (next) {
+				Poll.getOption(data.pollId, data.optionId, true, next);
+			},
+		}, (err, result) => {
 			if (err) {
 				return callback(new Error('Something went wrong!'));
 			}
-
-			if (!result.votes || !result.votes.length) {
-				return callback(null, result);
+			const { poll, option } = result;
+			if (!option.votes || !option.votes.length) {
+				return callback(null, option);
 			}
 
-			NodeBB.User.getUsersFields(result.votes, ['uid', 'username', 'userslug', 'picture'], (err, userData) => {
+			NodeBB.User.getUsersFields(option.votes, ['uid', 'username', 'userslug', 'picture'], (err, userData) => {
 				if (err) {
 					console.error(err);
+					return callback(err);
 				}
-				result.votes = userData;
-				callback(null, result);
+				option.votes = userData;
+				NodeBB.Plugins.hooks.fire('filter:poll.getOptionDetails', {
+					poll,
+					option,
+				}, (err, result) => {
+					callback(err, result.option);
+				});
 			});
 		});
 	};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-poll",
-  "version": "3.1.4",
+  "version": "3.1.5",
   "description": "NodeBB Poll Plugin",
   "main": "library.js",
   "scripts": {

package/public/js/poll/main.js

@@ -16,6 +16,14 @@ window.Poll = {};
 		}
 	});
 
+	$(window).on('action:posts.loaded', function (ev, data) {
+		data.posts.forEach((post) => {
+			if (post.hasOwnProperty('pollId')) {
+				getPoll(post.pollId);
+			}
+		});
+	});
+
 	$(window).on('action:posts.edited', function (ev, data) {
 		if (data.post.hasOwnProperty('pollId')) {
 			getPoll(data.post.pollId);