improve name validation of groups, users and repositories

This commit is contained in:
Sebastian Sdorra
2011-08-06 13:46:27 +02:00
parent 40bc8d88b2
commit 911eff0510
3 changed files with 5 additions and 3 deletions

View File

@@ -37,6 +37,7 @@ package sonia.scm.group;
import sonia.scm.ModelObject;
import sonia.scm.util.Util;
import sonia.scm.util.ValidationUtil;
//~--- JDK imports ------------------------------------------------------------
@@ -450,7 +451,7 @@ public class Group implements ModelObject, Iterable<String>
@Override
public boolean isValid()
{
return Util.isNotEmpty(name) && Util.isNotEmpty(type);
return ValidationUtil.isNameValid(name) && Util.isNotEmpty(type);
}
//~--- set methods ----------------------------------------------------------

View File

@@ -352,6 +352,7 @@ public class Repository implements ModelObject
*
* @return
*/
@Override
public String getId()
{
return id;
@@ -434,7 +435,7 @@ public class Repository implements ModelObject
@Override
public boolean isValid()
{
return Util.isNotEmpty(name) && Util.isNotEmpty(type)
return ValidationUtil.isNameValid(name) && Util.isNotEmpty(type)
&& ((Util.isEmpty(contact))
|| ValidationUtil.isMailAddressValid(contact));
}

View File

@@ -418,7 +418,7 @@ public class User implements Principal, ModelObject
@Override
public boolean isValid()
{
return Util.isNotEmpty(name) && Util.isNotEmpty(displayName)
return ValidationUtil.isUsernameValid(name) && Util.isNotEmpty(displayName)
&& Util.isNotEmpty(type)
&& ((Util.isEmpty(mail)) || ValidationUtil.isMailAddressValid(mail));
}