nodebb-plugin-mentions 4.0.1 → 4.0.3

package/library.js

@@ -96,12 +96,12 @@ Mentions.notify = async function (data) {
 		return;
 	}
 
-	const [topic, author, topicFollowers] = await Promise.all([
+	const [topic, userData, topicFollowers] = await Promise.all([
 		Topics.getTopicFields(postData.tid, ['title', 'cid']),
-		User.getUserField(postData.uid, 'username'),
+		User.getUserFields(postData.uid, ['username']),
 		Mentions._settings.disableFollowedTopics === 'on' ? Topics.getFollowers(postData.tid) : [],
 	]);
-
+	const { displayname } = userData;
 	const title = entitiesDecode(topic.title);
 	const titleEscaped = title.replace(/%/g, '%').replace(/,/g, ',');
 
@@ -129,7 +129,7 @@ Mentions.notify = async function (data) {
 
 	const filteredUids = await filterUidsAlreadyMentioned(uids, postData.pid);
 	if (filteredUids.length) {
-		await sendNotificationToUids(postData, filteredUids, 'user', `[[notifications:user_mentioned_you_in, ${author}, ${titleEscaped}]]`);
+		await sendNotificationToUids(postData, filteredUids, 'user', `[[notifications:user_mentioned_you_in, ${displayname}, ${titleEscaped}]]`);
 		await db.setAdd(`mentions:pid:${postData.pid}:uids`, filteredUids);
 	}
 
@@ -139,7 +139,7 @@ Mentions.notify = async function (data) {
 			const groupName = groupsToNotify[i].name;
 			const groupMentionSent = await db.isSetMember(`mentions:pid:${postData.pid}:groups`, groupName);
 			if (!groupMentionSent && memberUids.length) {
-				await sendNotificationToUids(postData, memberUids, groupName, `[[notifications:user_mentioned_group_in, ${author} , ${groupName}, ${titleEscaped}]]`);
+				await sendNotificationToUids(postData, memberUids, groupName, `[[notifications:user_mentioned_group_in, ${displayname} , ${groupName}, ${titleEscaped}]]`);
 				await db.setAdd(`mentions:pid:${postData.pid}:groups`, groupName);
 			}
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-mentions",
-  "version": "4.0.1",
+  "version": "4.0.3",
   "description": "NodeBB Plugin that allows users to mention other users by prepending an '@' sign to their username",
   "main": "library.js",
   "scripts": {
@@ -31,9 +31,9 @@
     "xregexp": "^5.1.0"
   },
   "devDependencies": {
-    "mocha": "10.0.0",
-    "eslint": "8.23.0",
-    "eslint-config-nodebb": "0.1.1",
-    "eslint-plugin-import": "2.26.0"
+    "mocha": "10.2.0",
+    "eslint": "8.32.0",
+    "eslint-config-nodebb": "0.2.1",
+    "eslint-plugin-import": "2.27.5"
   }
 }

package/static/admin.js

@@ -1,10 +1,7 @@
 'use strict';
 
-/* globals $, app, socket, define */
-
 define('admin/plugins/mentions', ['settings', 'alerts'], function (Settings, alerts) {
-
-	var ACP = {};
+	const ACP = {};
 
 	ACP.init = function () {
 		Settings.load('mentions', $('.mentions-settings'));
@@ -18,9 +15,10 @@ define('admin/plugins/mentions', ['settings', 'alerts'], function (Settings, ale
 					alert_id: 'mentions-saved',
 					title: 'Settings Saved',
 					message: 'Please reload your NodeBB to apply these settings',
+					timeout: 5000,
 					clickfn: function () {
 						socket.emit('admin.reload');
-					}
+					},
 				});
 			});
 		});
@@ -34,4 +32,4 @@ define('admin/plugins/mentions', ['settings', 'alerts'], function (Settings, ale
 	}
 
 	return ACP;
-});
+});

package/static/autofill.js

@@ -83,7 +83,7 @@ $(document).ready(function () {
 			}
 
 			// Format suggestions as 'avatar username (fullname)'
-			const avatar = helpers.buildAvatar(user, 'sm');
+			const avatar = helpers.buildAvatar(user, '24px', true);
 			const fullname = user.fullname ? `(${user.fullname})` : '';
 			carry.push(`${avatar} ${user.username} ${helpers.escape(fullname)}`);
 

package/templates/admin/plugins/mentions.tpl

@@ -56,6 +56,4 @@
 	</div>
 </form>
 
-<button id="save" class="floating-button mdl-button mdl-js-button mdl-button--fab mdl-js-ripple-effect mdl-button--colored">
-	<i class="material-icons">save</i>
-</button>
+<!-- IMPORT admin/partials/save_button.tpl -->

package/upgrades/mentions_delete_mentions_set_zset.js

@@ -1,23 +1,24 @@
-'use strict';
-
-
-const db = module.parent.require('./database');
-const batch = module.parent.require('./batch');
-module.exports = {
-	name: 'Delete mentions:sent:<pid> sorted sets',
-	timestamp: Date.UTC(2021, 10, 2),
-	method: async function () {
-        const { progress } = this;
-        const nextPid = await db.getObjectField('global', 'nextPid');
-        const allPids = [];
-        for (let pid = 1; pid < nextPid; ++ pid) {
-            allPids.push(pid);
-        }
-        await batch.processArray(allPids, async (pids) => {
-            progress.incr(pids.length);
-            await db.deleteAll(pids.map(pid => `mentions:sent:${pid}`));
-        }, {
-            batch: 500,
-        });
-	},
+'use strict';
+
+
+const db = module.parent.require('./database');
+const batch = module.parent.require('./batch');
+module.exports = {
+	name: 'Delete mentions:sent:<pid> sorted sets',
+	timestamp: Date.UTC(2021, 10, 2),
+	method: async function () {
+		const { progress } = this;
+		const nextPid = await db.getObjectField('global', 'nextPid');
+		const allPids = [];
+		for (let pid = 1; pid < nextPid; ++pid) {
+			allPids.push(pid);
+		}
+		progress.total = allPids.length;
+		await batch.processArray(allPids, async (pids) => {
+			progress.incr(pids.length);
+			await db.deleteAll(pids.map(pid => `mentions:sent:${pid}`));
+		}, {
+			batch: 500,
+		});
+	},
 };