Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-07 06:25:50 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: #8018, allow absolute urls in notification.path

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2019-11-01 21:01:46 -04:00
parent bfea9d3906
commit 0037a03886
4 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/notifications.js
View File

@@ -65,6 +65,9 @@ Notifications.getMultiple = async function (nids) {
notifications.forEach(function (notification, index) {
if (notification) {
if (notification.path && !notification.path.startsWith('http')) {
notification.path = nconf.get('relative_path') + notification.path;
}
notification.datetimeISO = utils.toISOString(notification.datetime);
if (notification.bodyLong) {
@@ -168,6 +171,7 @@ async function pushToUids(uids, notification) {
await async.eachLimit(uids, 3, function (uid, next) {
emailer.send('notification', uid, {
path: notification.path,
notification_url: notification.path.startsWith('http') ? notification.path : nconf.get('url') + notification.path,
subject: utils.stripHTMLTags(notification.subject || '[[notifications:new_notification]]'),
intro: utils.stripHTMLTags(notification.bodyShort),
body: notification.bodyLong || '',