IT4Change/gradido
3
0
Fork 0
mirror of https://github.com/IT4Change/gradido.git synced 2025-12-13 07:45:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix language conflict between current user and email receiving user

Browse Source
This commit is contained in:
Wolfgang Huß 2022-11-11 12:10:26 +01:00
parent 5b5aa2268b
commit 5fcd684b6b
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
backend/src/emails/sendEmailTranslated.ts
View File

@ -14,14 +14,24 @@ export const sendEmailTranslated = async (params: {
template: string template: string
locals: Record<string, string> locals: Record<string, string>
}): Promise<boolean> => { }): Promise<boolean> => {
i18n.setLocale(params.locals.locale) // Wolle: test this …
// because language of receiver can differ from language of current user who triggers the sending
const rememberLocaleToRestore = i18n.getLocale()
// Wolle:
// console.log('sendEmailTranslated – i18n.getLocale, incoming from user: ', i18n.getLocale())
i18n.setLocale('en') // for logging
// Wolle:
// console.log('sendEmailTranslated – i18n.getLocale, logging: ', i18n.getLocale())
logger.info( logger.info(
`send Email: to=${params.receiver.to}` + `send Email: language=${params.locals.locale} to=${params.receiver.to}` +
(params.receiver.cc ? `, cc=${params.receiver.cc}` : '') + (params.receiver.cc ? `, cc=${params.receiver.cc}` : '') +
`, subject=${i18n.__('emails.' + params.template + '.subject')}`, `, subject=${i18n.__('emails.' + params.template + '.subject')}`,
) )
i18n.setLocale(params.locals.locale) // for email
// Wolle:
// console.log('sendEmailTranslated – i18n.getLocale, email: ', i18n.getLocale())
if (!CONFIG.EMAIL) { if (!CONFIG.EMAIL) {
logger.info(`Emails are disabled via config...`) logger.info(`Emails are disabled via config...`)
return false return false
@ -63,7 +73,9 @@ export const sendEmailTranslated = async (params: {
// i18n, // is only needed if you don't install i18n // i18n, // is only needed if you don't install i18n
}) })
email // TESTING: to send emails to yourself set .env "EMAIL_TEST_MODUS=true" and "EMAIL_TEST_RECEIVER" to your preferred email address
// ATTENTION: await is needed, because otherwise on send the email gets send in the language from the current user, because below the language gets reset to the current user
await email
.send({ .send({
template: path.join(__dirname, params.template), template: path.join(__dirname, params.template),
message: params.receiver, message: params.receiver,
@ -78,5 +90,10 @@ export const sendEmailTranslated = async (params: {
throw new Error('Error sending notification email!') throw new Error('Error sending notification email!')
}) })
// Wolle: !!! if we do this without an await on send the email gets send in the language from the current user !!!
i18n.setLocale(rememberLocaleToRestore)
// Wolle:
// console.log('sendEmailTranslated – i18n.getLocale, reset: ', i18n.getLocale())
return true return true
} }