Skip to content

Commit e4c4b1c

Browse files
committed
[MP-1010] Fix ampersand in email_subject_room_name
1 parent 10af010 commit e4c4b1c

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

app/lib/server/functions/notifications/email.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ export function getEmailData({
134134
const emailSubject = Mailer.replace(settings.get(subjectKey), {
135135
user: username,
136136
room: roomTypes.getRoomName(room.t, room),
137-
});
137+
}, ['room']);
138138
const content = getEmailContent({
139139
message,
140140
user: receiver,

app/mailer/server/api.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ settings.get('Language', (key, value) => {
2323
lng = value || 'en';
2424
});
2525

26-
export const replacekey = (str, key, value = '') => str.replace(new RegExp(`(\\[${ key }\\]|__${ key }__)`, 'igm'), s.escapeHTML(value));
26+
export const replacekey = (str, key, value = '', nonEscapeKeys = []) => str.replace(new RegExp(`(\\[${ key }\\]|__${ key }__)`, 'igm'), nonEscapeKeys && nonEscapeKeys.includes(key) ? value : s.escapeHTML(value));
2727
export const translate = (str) => str.replace(/\{ ?([^\} ]+)(( ([^\}]+))+)? ?\}/gmi, (match, key) => TAPi18n.__(key, { lng }));
28-
export const replace = function replace(str, data = {}) {
28+
export const replace = function replace(str, data = {}, nonEscapeKeys = []) {
2929
if (!str) {
3030
return '';
3131
}
@@ -39,7 +39,7 @@ export const replace = function replace(str, data = {}) {
3939
},
4040
...data,
4141
};
42-
return Object.entries(options).reduce((ret, [key, value]) => replacekey(ret, key, value), translate(str));
42+
return Object.entries(options).reduce((ret, [key, value]) => replacekey(ret, key, value, nonEscapeKeys), translate(str));
4343
};
4444

4545
const nonEscapeKeys = ['room_path'];

0 commit comments

Comments
 (0)