chore: eslint function-paren-newline

.eslintrc

@@ -57,8 +57,7 @@
 		// "object-curly-newline": ["error", { "consistent": true, "multiline": true }],
 		"object-curly-newline": "off",
 		// require consistent linebreaks inline function parenthesis (arguments or params)
-		// "function-paren-newline": ["error", "consistent"],
-		"function-paren-newline": "off",
+		"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 when assigning

public/src/admin/manage/users.js

@@ -305,13 +305,15 @@ define('admin/manage/users', [
 
 			bootbox.confirm(confirmMsg, function (confirm) {
 				if (confirm) {
-					Promise.all(uids.map(uid => api.del(`/users/${uid}${path}`, {})
-						.then(() => {
+					Promise.all(
+						uids.map(
+							uid => api.del(`/users/${uid}${path}`, {}).then(() => {
 								if (path !== '/content') {
 									removeRow(uid);
 								}
 							})
-					)).then(() => {
+						)
+					).then(() => {
 						if (path !== '/content') {
 							app.alertSuccess('[[admin/manage/users:alerts.delete-success]]');
 						} else {

public/src/client/topic/threadTools.js

@@ -321,10 +321,11 @@ define('forum/topic/threadTools', [
 		var icon = $('.topic-header [component="topic/pinned"]');
 		icon.toggleClass('hidden', !data.pinned);
 		if (data.pinned) {
-			icon.translateAttr('title', data.pinExpiry && data.pinExpiryISO ?
+			icon.translateAttr('title', (
+				data.pinExpiry && data.pinExpiryISO ?
 					'[[topic:pinned-with-expiry, ' + data.pinExpiryISO + ']]' :
 					'[[topic:pinned]]'
-			);
+			));
 		}
 		ajaxify.data.pinned = data.pinned;
 

public/src/modules/api.js

@@ -12,10 +12,11 @@ define('api', () => {
 		function doAjax(cb) {
 			$.ajax(options)
 				.done((res) => {
-					cb(null,
-						res && res.hasOwnProperty('status') && res.hasOwnProperty('response') ?
-							res.response : (res || {})
-					);
+					cb(null, (
+						res &&
+						res.hasOwnProperty('status') &&
+						res.hasOwnProperty('response') ? res.response : (res || {})
+					));
 				})
 				.fail((ev) => {
 					let errMessage;

src/api/posts.js

@@ -137,7 +137,8 @@ async function deleteOrRestore(caller, data, params) {
 async function deleteOrRestoreTopicOf(command, pid, caller) {
 	const topic = await posts.getTopicFields(pid, ['tid', 'cid', 'deleted']);
 	// command: delete/restore
-	await apiHelpers.doTopicAction(command,
+	await apiHelpers.doTopicAction(
+		command,
 		topic.deleted ? 'event:topic_restored' : 'event:topic_deleted',
 		caller,
 		{ tids: [topic.tid], cid: topic.cid }
@@ -178,7 +179,9 @@ postsAPI.purge = async function (caller, data) {
 	});
 
 	if (isMainAndLast) {
-		await apiHelpers.doTopicAction('purge', 'event:topic_purged',
+		await apiHelpers.doTopicAction(
+			'purge',
+			'event:topic_purged',
 			caller,
 			{ tids: [postData.tid], cid: topicData.cid }
 		);

src/controllers/admin/groups.js

@@ -59,12 +59,13 @@ groupsController.get = async function (req, res, next) {
 
 async function getGroupNames() {
 	const groupNames = await db.getSortedSetRange('groups:createtime', 0, -1);
-	return groupNames.filter(name => name !== 'registered-users' &&
+	return groupNames.filter(name => (
+		name !== 'registered-users' &&
 		name !== 'verified-users' &&
 		name !== 'unverified-users' &&
 		name !== groups.BANNED_USERS &&
 		!groups.isPrivilegeGroup(name)
-	);
+	));
 }
 
 groupsController.getCSV = async function (req, res) {

src/database/postgres.js

@@ -368,8 +368,8 @@ postgresModule.info = async function (db) {
 	const res = await db.query(`
 		SELECT true "postgres",
 		   current_setting('server_version') "version",
-		   EXTRACT(EPOCH FROM NOW() - pg_postmaster_start_time()) * 1000 "uptime"`
-	);
+			 EXTRACT(EPOCH FROM NOW() - pg_postmaster_start_time()) * 1000 "uptime"
+	`);
 	return res.rows[0];
 };
 

src/groups/delete.js

@@ -18,7 +18,8 @@ module.exports = function (Groups) {
 		}
 		const keys = [];
 		groupNames.forEach((groupName) => {
-			keys.push(`group:${groupName}`,
+			keys.push(
+				`group:${groupName}`,
 				`group:${groupName}:members`,
 				`group:${groupName}:pending`,
 				`group:${groupName}:invited`,

src/groups/join.js

@@ -55,7 +55,8 @@ module.exports = function (Groups) {
 		const visibleGroups = groupData.filter(groupData => groupData && !groupData.hidden);
 
 		if (visibleGroups.length) {
-			await db.sortedSetAdd('groups:visible:memberCount',
+			await db.sortedSetAdd(
+				'groups:visible:memberCount',
 				visibleGroups.map(groupData => groupData.memberCount),
 				visibleGroups.map(groupData => groupData.name)
 			);

src/groups/leave.js

@@ -43,7 +43,9 @@ module.exports = function (Groups) {
 			promises.push(Groups.destroy, emptyPrivilegeGroups);
 		}
 		if (visibleGroups.length) {
-			promises.push(db.sortedSetAdd, 'groups:visible:memberCount',
+			promises.push(
+				db.sortedSetAdd,
+				'groups:visible:memberCount',
 				visibleGroups.map(groupData => groupData.memberCount),
 				visibleGroups.map(groupData => groupData.name)
 			);

src/upgrades/1.14.1/readd_deleted_recent_topics.js

@@ -25,22 +25,26 @@ module.exports = {
 				}
 			});
 
-			await db.sortedSetAdd('topics:recent',
+			await db.sortedSetAdd(
+				'topics:recent',
 				topicData.map(t => t.lastposttime || 0),
 				topicData.map(t => t.tid)
 			);
 
-			await db.sortedSetAdd('topics:views',
+			await db.sortedSetAdd(
+				'topics:views',
 				topicData.map(t => t.viewcount || 0),
 				topicData.map(t => t.tid)
 			);
 
-			await db.sortedSetAdd('topics:posts',
+			await db.sortedSetAdd(
+				'topics:posts',
 				topicData.map(t => t.postcount || 0),
 				topicData.map(t => t.tid)
 			);
 
-			await db.sortedSetAdd('topics:votes',
+			await db.sortedSetAdd(
+				'topics:votes',
 				topicData.map(t => t.votes || 0),
 				topicData.map(t => t.tid)
 			);

src/user/jobs/export-profile.js

@@ -79,7 +79,9 @@ async function getRoomMessages(uid, roomId) {
 	let data = [];
 	await batch.processSortedSet(`uid:${uid}:chat:room:${roomId}:mids`, async (mids) => {
 		const messageData = await db.getObjects(mids.map(mid => `message:${mid}`));
-		data = data.concat(messageData.filter(m => m && m.fromuid === uid && !m.system)
+		data = data.concat(
+			messageData
+				.filter(m => m && m.fromuid === uid && !m.system)
 				.map(m => ({ content: m.content, timestamp: m.timestamp }))
 		);
 	}, { batch: 500, interval: 1000 });