Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 08:36:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: js error when clicking a notification

Browse Source 
that sends you to same page
This commit is contained in:
Barış Soner Uşaklı
2024-12-23 17:15:38 -05:00
parent 1cb7bb6a72
commit d40762b7d0
2 changed files with 17 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
public/src/client/header/notifications.js
View File

@@ -6,14 +6,19 @@ define('forum/header/notifications', function () {
notifications.prepareDOM = function () {
const notifTrigger = $('[component="notifications"] [data-bs-toggle="dropdown"]');
notifTrigger.on('show.bs.dropdown', (ev) => {
requireAndCall('loadNotifications', $(ev.target).parent().find('[component="notifications/list"]'));
notifTrigger.on('show.bs.dropdown', async (ev) => {
const notifications = await app.require('notifications');
const triggerEl = $(ev.target);
notifications.loadNotifications(triggerEl, triggerEl.parent().find('[component="notifications/list"]'));
});
notifTrigger.each((index, el) => {
const dropdownEl = $(el).parent().find('.dropdown-menu');
const triggerEl = $(el);
const dropdownEl = triggerEl.parent().find('.dropdown-menu');
if (dropdownEl.hasClass('show')) {
requireAndCall('loadNotifications', dropdownEl.find('[component="notifications/list"]'));
app.require('notifications').then((notifications) => {
notifications.loadNotifications(triggerEl, dropdownEl.find('[component="notifications/list"]'));
});
}
});
@@ -24,18 +29,14 @@ define('forum/header/notifications', function () {
socket.on('event:notifications.updateCount', onUpdateCount);
};
function onNewNotification(data) {
requireAndCall('onNewNotification', data);
async function onNewNotification(data) {
const notifications = await app.require('notifications');
notifications.onNewNotification(data);
}
function onUpdateCount(data) {
requireAndCall('updateNotifCount', data);
}
function requireAndCall(method, param) {
require(['notifications'], function (notifications) {
notifications[method](param);
});
async function onUpdateCount(data) {
const notifications = await app.require('notifications');
notifications.updateNotifCount(data);
}
return notifications;

4
public/src/modules/notifications.js
View File

@@ -28,7 +28,7 @@ define('notifications', [
});
hooks.on('filter:notifications.load', _addTimeagoString);
Notifications.loadNotifications = function (notifList, callback) {
Notifications.loadNotifications = function (triggerEl, notifList, callback) {
callback = callback || function () {};
socket.emit('notifications.get', null, function (err, data) {
if (err) {
@@ -47,7 +47,7 @@ define('notifications', [
if (scrollToPostIndexIfOnPage(notifEl)) {
ev.stopPropagation();
ev.preventDefault();
components.get('notifications/list').dropdown('toggle');
triggerEl.dropdown('toggle');
}
const unread = notifEl.hasClass('unread');