From ab9dd2be1847f4bf9f5946d82a8d70352d9ddd86 Mon Sep 17 00:00:00 2001
From: Florian Gnaegi - frentix GmbH <gnaegi@frentix.com>
Date: Thu, 27 Feb 2014 13:48:14 +0100
Subject: [PATCH] OO-1003 fix issue with trailing white space when setting
email via RESt
---
.../java/org/olat/user/propertyhandlers/EmailProperty.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/main/java/org/olat/user/propertyhandlers/EmailProperty.java b/src/main/java/org/olat/user/propertyhandlers/EmailProperty.java
index 83d8489db39..e2bc3f7af06 100644
--- a/src/main/java/org/olat/user/propertyhandlers/EmailProperty.java
+++ b/src/main/java/org/olat/user/propertyhandlers/EmailProperty.java
@@ -59,9 +59,9 @@ public class EmailProperty extends Generic127CharTextPropertyHandler {
@Override
protected void setInternalValue(User user, String mail) {
- // save mail addresses always lowercase
+ // save mail addresses always lower case and remove trailing whitespace
if (mail != null) {
- super.setInternalValue(user, mail.toLowerCase());
+ super.setInternalValue(user, mail.toLowerCase().trim());
} else {
super.setInternalValue(user, null);
}
@@ -132,6 +132,7 @@ public class EmailProperty extends Generic127CharTextPropertyHandler {
String value = textElement.getValue();
if (StringHelper.containsNonWhitespace(value)) {
+ value = value.toLowerCase().trim();
// check mail address syntax
if (!MailHelper.isValidEmailAddress(value)) {
textElement.setErrorKey(i18nFormElementLabelKey() + ".error.valid", null);
@@ -159,6 +160,7 @@ public class EmailProperty extends Generic127CharTextPropertyHandler {
}
if (StringHelper.containsNonWhitespace(value)) {
+ value = value.toLowerCase().trim();
// check mail address syntax
if ( ! MailHelper.isValidEmailAddress(value)) {
validationError.setErrorKey(i18nFormElementLabelKey() + ".error.valid");
--
GitLab