Skip to content
Snippets Groups Projects
Commit cf81f0fd authored by srosse's avatar srosse
Browse files

OO-4882: show the login name in specific mail templates

parent 97bbb281
No related branches found
No related tags found
No related merge requests found
......@@ -22,11 +22,13 @@ package org.olat.course.nodes.co;
import java.util.Locale;
import org.apache.velocity.VelocityContext;
import org.olat.basesecurity.BaseSecurity;
import org.olat.core.CoreSpringFactory;
import org.olat.core.helpers.Settings;
import org.olat.core.id.Identity;
import org.olat.core.id.User;
import org.olat.core.id.UserConstants;
import org.olat.core.util.StringHelper;
import org.olat.core.util.mail.MailTemplate;
import org.olat.repository.RepositoryEntry;
import org.olat.user.UserManager;
......@@ -61,6 +63,7 @@ public class CourseMailTemplate extends MailTemplate {
if(sender != null) {
User user = sender.getUser();
UserManager userManager = CoreSpringFactory.getImpl(UserManager.class);
BaseSecurity securityManager = CoreSpringFactory.getImpl(BaseSecurity.class);
vContext.put("firstname", user.getProperty(UserConstants.FIRSTNAME, null));
vContext.put(UserConstants.FIRSTNAME, user.getProperty(UserConstants.FIRSTNAME, null));
......@@ -71,7 +74,11 @@ public class CourseMailTemplate extends MailTemplate {
vContext.put("fullName", fullName);
vContext.put("mail", userManager.getUserDisplayEmail(user, locale));
vContext.put("email", userManager.getUserDisplayEmail(user, locale));
vContext.put("username", sender.getName());
String loginName = securityManager.findAuthenticationName(recipient);
if(!StringHelper.containsNonWhitespace(loginName)) {
loginName = recipient.getName();
}
vContext.put("username", loginName);
}
}
}
......@@ -298,6 +298,9 @@ public class ReminderServiceImpl implements ReminderService {
vContext.put("mail", email);
vContext.put("email", email);
String loginName = securityManager.findAuthenticationName(recipient);
if(!StringHelper.containsNonWhitespace(loginName)) {
loginName = recipient.getName();
}
vContext.put("username", loginName);
// Put variables from greater context
if(entry != null) {
......
......@@ -48,6 +48,7 @@ import org.olat.core.id.Identity;
import org.olat.core.id.User;
import org.olat.core.id.UserConstants;
import org.olat.core.logging.Tracing;
import org.olat.core.util.StringHelper;
import org.olat.core.util.Util;
import org.olat.core.util.i18n.I18nManager;
import org.olat.core.util.mail.MailBundle;
......@@ -383,7 +384,11 @@ public class UserLifecycleManagerImpl implements UserLifecycleManager {
vContext.put("fullName", fullName);
vContext.put("mail", userManager.getUserDisplayEmail(user, locale));
vContext.put("email", userManager.getUserDisplayEmail(user, locale));
vContext.put("username", recipient.getName());
String loginName = securityManager.findAuthenticationName(recipient);
if(!StringHelper.containsNonWhitespace(loginName)) {
loginName = recipient.getName();
}
vContext.put("username", loginName);
}
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment