nodebb-plugin-poll 1.1.7 → 2.0.2

package/.eslintrc

@@ -0,0 +1,116 @@
+{
+	"extends": "airbnb-base",
+	"ignorePatterns": ["**/vendor/*.js"],
+	"parserOptions": {
+		"sourceType": "script"
+	},
+
+	"rules": {
+		// === Configure rules for our style ===
+		// imports must be resolvable
+		"import/no-unresolved": "error",
+		// use single quotes,
+		// unless a different style allows avoiding escapes
+		"quotes": ["error", "single", {
+			"avoidEscape": true,
+			"allowTemplateLiterals": true
+		}],
+		// allow else-if return
+		"no-else-return": [ "error", { "allowElseIf": true } ],
+		// expressions split over multiple lines
+		// should break after the operator
+		"operator-linebreak": [ "error", "after" ],
+		// require arrow parens only when needed
+		// and whenever the body is a block
+		"arrow-parens": ["error", "as-needed", { "requireForBlockBody": true }],
+		// what variables are errors in callbacks
+		"handle-callback-err": [ "error","^(e$|(e|(.*(_e|E)))rr)" ],
+		// allow dangling commas in functions
+		// require them everywhere else
+		"comma-dangle": ["error", {
+			"arrays": "always-multiline",
+			"objects": "always-multiline",
+			"imports": "always-multiline",
+			"exports": "always-multiline",
+			"functions": "only-multiline"
+		}],
+		// we actually encourage `return await`
+		"no-return-await": "off",
+		// allow `while (true)`
+		"no-constant-condition": ["error", { "checkLoops": false }],
+		// allow ignoring an error with `catch`
+		"no-empty": ["error", { "allowEmptyCatch": true }],
+		// allow `3 + 5 - 1`, but not `3 * 5 - 1`
+		"no-mixed-operators": ["error", { "allowSamePrecedence": true }],
+		// require `'use strict';`
+		"strict": ["error", "global"],
+		// we actually use tabs for indentation
+		"indent": ["error", "tab", { "SwitchCase": 1 }],
+		"no-tabs": "off",
+		// we want `== null` to also handle undefined
+		"no-eq-null": "off",
+		// allow `for (..; i++)`
+		"no-plusplus": ["error", { "allowForLoopAfterthoughts": true }],
+		// allow using functions defined later
+		"no-use-before-define": ["error", "nofunc"],
+		// require consistent newlines before and after braces
+		// if contents are multiline
+		"object-curly-newline": ["error", { "consistent": true, "multiline": true }],
+		// require consistent linebreaks inline function parenthesis (arguments or params)
+		"function-paren-newline": ["error", "consistent"],
+		// only require const if all parts of destructuring can be const
+		"prefer-const": ["error", { "destructuring": "all" }],
+		// don't require destructuring for arrays or assignment
+		"prefer-destructuring": ["error", {
+			"VariableDeclarator": { "array": false, "object": true },
+			"AssignmentExpression": { "array": false, "object": false }
+		}],
+		// identical to airbnb rule, except for allowing for..of, because we want to use it
+		"no-restricted-syntax": [
+			"error",
+			{
+				"selector": "ForInStatement",
+				"message": "for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array."
+			},
+			{
+				"selector": "LabeledStatement",
+				"message": "Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand."
+			},
+			{
+				"selector": "WithStatement",
+				"message": "`with` is disallowed in strict mode because it makes code impossible to predict and optimize."
+			}
+		],
+		// allow lines of up to 120 characters
+		"max-len": ["error", { "code": 120, "tabWidth": 2, "ignoreUrls": true, "ignoreStrings": true, "ignoreTemplateLiterals": true, "ignoreRegExpLiterals": true }],
+
+		// === Disable rules ===
+		// more liberal naming
+		"camelcase": "off",
+		"no-underscore-dangle": "off",
+		// don't require anonymous function names
+		"func-names": "off",
+		// allow console
+		"no-console": "off",
+		// allow new for side effects
+		// allow new with non-capitalized
+		"no-new": "off",
+		"new-cap": "off",
+		// allow shadowing variables (usually callbacks)
+		"no-shadow": "off",
+		// allow multiple empty lines in a row
+		"no-multiple-empty-lines": "off",
+		// allow not using object shorthand
+		"object-shorthand": "off",
+
+		// TODO
+		"consistent-return": "off",
+		"no-restricted-globals": "off",
+		"no-prototype-builtins": "off",
+		"import/no-extraneous-dependencies": "off",
+		"import/no-dynamic-require": "off",
+		"global-require": "off",
+		"no-param-reassign": "off",
+		"default-case": "off"
+	}
+}

package/languages/ru/poll.json

@@ -1,14 +1,14 @@
 {
   "poll": "Опрос",
-  "toggles": "Переключает",
-  "allow_guests": "Разрешить гостям Результаты опроса",
+  "toggles": "Переключатели",
+  "allow_guests": "Разрешить гостям видеть результаты опроса",
   "limits": "Ограничения",
   "max_options": "Максимальное количество вариантов в опросе",
-  "defaults": "значение по умолчанию",
-  "default_title": "По умолчанию Опрос Имя",
+  "defaults": "Значение по умолчанию",
+  "default_title": "Название опроса по умолчанию",
   "max_votes": "Максимальное количество голосов в опросе",
   "disallow_vote_update": "Запретить переголосование",
-  "info_choices": "число, большее 1 позволит с несколькими вариантами опросы.",
+  "info_choices": "Число большее 1 позволит проводить опросы с несколькими вариантами ответа.",
   "settings": "Настройки",
   "save": "Сохранить",
   "reset": "Сброс",
@@ -16,29 +16,29 @@
   "creator_title": "Создать опрос",
   "poll_title": "Название опроса",
   "poll_title_placeholder": "Введите название опроса",
-  "options_title": "Опции",
-  "options_add": "Добавить вариант",
+  "options_title": "Варианты ответов",
+  "options_add": "Добавить вариант ответа",
   "max_votes_placeholder": "Введите максимальное кол-во голосов",
   "auto_end_title": "Автоматически закончить опрос",
   "auto_end_placeholder": "Нажмите, чтобы ввести дату и время",
-  "auto_end_help": "Оставляя данное поле пустым, опрос будет всегда открыт.",
+  "auto_end_help": "Если оставить данное поле пустым, то опрос будет всегда открыт.",
 
   "error.max_options": "Вы можете создать только %d опций.",
-  "error.no_options": "Создайте хотя бы один вариант.",
-  "error.valid_date": "Введите корректное число.",
+  "error.no_options": "Создайте хотя бы один вариант ответа.",
+  "error.valid_date": "Введите корректную дату.",
 
-  "error.not_main": "Опрос доступен при создании темы",
+  "error.not_main": "Опрос доступен только при создании темы",
   "error.privilege.create": "У вас недостаточно прав для создания опроса",
 
-  "vote": "Голосовать",
+  "vote": "Проголосовать",
   "update_vote": "Переголосовать",
   "remove_vote": "Удалить голос",
-  "to_voting": "Для голосования",
+  "to_voting": "Проголосовать",
   "to_results": "Результаты",
   "voting_unavailable_title": "Голосование недоступно",
   "voting_unavailable_message": "Этот опрос завершен или был помечен как удаленный. Вы все еще можете просмотреть результаты.",
-  "voting_update_disallowed_message": "Переголосование недоступно",
-  "vote_is_final": "Опрос уже завершен",
+  "voting_update_disallowed_message": "Вы уже проголосовали, и изменение вашего голоса не допускается для этого голосования.",
+  "vote_is_final": "В данном опросе нельзя переголосовать.",
   "vote_count": "пользователей проголосовали за этот вариант",
   "votes": "голосов"
 }

package/lib/config.js

@@ -1,49 +1,48 @@
-"use strict";
+'use strict';
 
-var NodeBB = require('./nodebb'),
+const NodeBB = require('./nodebb');
 
-	packageInfo = require('../package.json'),
-	pluginInfo = require('../plugin.json'),
-	pluginId = pluginInfo.id.replace('nodebb-plugin-', '');
+const packageInfo = require('../package.json');
+const pluginInfo = require('../plugin.json');
 
-(function(Config) {
+const pluginId = pluginInfo.id.replace('nodebb-plugin-', '');
 
+(function (Config) {
 	Config.plugin = {
 		name: pluginInfo.name,
 		id: pluginId,
 		version: packageInfo.version,
 		description: packageInfo.description,
-		icon: 'fa-bar-chart-o'
+		icon: 'fa-bar-chart-o',
 	};
 
 	Config.defaults = {
 		toggles: {
-			allowAnon: false
+			allowAnon: false,
 		},
 		limits: {
-			maxOptions: 10
+			maxOptions: 10,
 		},
 		defaults: {
 			title: 'Poll',
 			maxvotes: 1,
 			disallowVoteUpdate: 0,
-			end: 0
-		}
+			end: 0,
+		},
 	};
 
 	Config.settings = {};
 
-	Config.init = function(callback) {
+	Config.init = function (callback) {
 		Config.settings = new NodeBB.Settings(Config.plugin.id, Config.plugin.version, Config.defaults, callback);
 	};
 
 	Config.adminSockets = {
-		sync: function() {
+		sync: function () {
 			Config.settings.sync();
 		},
-		getDefaults: function(socket, data, callback) {
+		getDefaults: function (socket, data, callback) {
 			callback(null, Config.settings.createDefaultWrapper());
-		}
+		},
 	};
-
-})(exports);
+}(exports));

package/lib/hooks.js

@@ -1,29 +1,29 @@
-var NodeBB = require('./nodebb'),
-
-	Config = require('./config'),
-	Poll = require('./poll'),
-	Serializer = require('./serializer');
+'use strict';
 
 const util = require('util');
+const NodeBB = require('./nodebb');
+
+const Config = require('./config');
+const Poll = require('./poll');
+const Serializer = require('./serializer');
 
-(function(Hooks) {
 
+(function (Hooks) {
 	Hooks.filter = {};
 	Hooks.action = {};
 
-	Hooks.filter.parseRaw = function(raw, callback) {
+	Hooks.filter.parseRaw = function (raw, callback) {
 		callback(null, Serializer.removeMarkup(raw, '[Poll]'));
 	};
 
 	Hooks.filter.postCreate = async function (obj) {
 		if (Serializer.hasMarkup(obj.post.content) && obj.data.isMain) {
 			return await savePoll(obj);
-		} else {
-			return obj;
 		}
+		return obj;
 	};
 
-	Hooks.filter.postEdit = async function(obj) {
+	Hooks.filter.postEdit = async function (obj) {
 		const { tid, pollId } = await NodeBB.Posts.getPostFields(obj.data.pid, ['tid', 'pollId']);
 		if (pollId || !Serializer.hasMarkup(obj.post.content)) {
 			return obj;
@@ -56,41 +56,52 @@ const util = require('util');
 		return obj;
 	};
 
-	Hooks.filter.topicPost = async function (data, callback) {
+	Hooks.filter.topicPost = async function (data) {
 		if (Serializer.hasMarkup(data.content)) {
 			await canCreate(data.cid, data.uid);
 			return data;
-		} else {
-			return data;
 		}
+		return data;
 	};
 
-	Hooks.action.postDelete = function(data) {
-		Poll.getPollIdByPid(data.post.pid, function(err, pollId) {
+	Hooks.action.postDelete = function (data) {
+		Poll.getPollIdByPid(data.post.pid, (err, pollId) => {
+			if (err) {
+				console.error(err);
+			}
 			if (pollId) {
 				Poll.delete(pollId);
 			}
 		});
 	};
 
-	Hooks.action.postRestore = function(data) {
-		Poll.getPollIdByPid(data.post.pid, function(err, pollId) {
+	Hooks.action.postRestore = function (data) {
+		Poll.getPollIdByPid(data.post.pid, (err, pollId) => {
+			if (err) {
+				console.error(err);
+			}
 			if (pollId) {
 				Poll.restore(pollId);
 			}
 		});
 	};
 
-	Hooks.action.topicDelete = function(data) {
-		Poll.getPollIdByTid(data.topic.tid, function(err, pollId) {
+	Hooks.action.topicDelete = function (data) {
+		Poll.getPollIdByTid(data.topic.tid, (err, pollId) => {
+			if (err) {
+				console.error(err);
+			}
 			if (pollId) {
 				Poll.delete(pollId);
 			}
 		});
 	};
 
-	Hooks.action.topicRestore = function(data) {
-		Poll.getPollIdByTid(data.topic.tid, function(err, pollId) {
+	Hooks.action.topicRestore = function (data) {
+		Poll.getPollIdByTid(data.topic.tid, (err, pollId) => {
+			if (err) {
+				console.error(err);
+			}
 			if (pollId) {
 				Poll.restore(pollId);
 			}
@@ -105,8 +116,8 @@ const util = require('util');
 	}
 
 	async function savePoll(obj) {
-		var postobj = obj.post ? obj.post : obj;
-		var pollData = Serializer.serialize(postobj.content, Config.settings.get());
+		const postobj = obj.post ? obj.post : obj;
+		const pollData = Serializer.serialize(postobj.content, Config.settings.get());
 
 		if (!pollData || !pollData.options.length) {
 			return obj;
@@ -120,4 +131,4 @@ const util = require('util');
 
 		return obj;
 	}
-})(exports);
+}(exports));

package/lib/nodebb.js

@@ -1,3 +1,4 @@
+'use strict';
 
 module.exports = {
 	db: require.main.require('./src/database'),
@@ -12,5 +13,5 @@ module.exports = {
 	SocketIndex: require.main.require('./src/socket.io/index'),
 	Translator: require.main.require('./public/src/modules/translator'),
 	winston: require.main.require('winston'),
-	app: null
+	app: null,
 };