Nemcio/SCM-Manager
1
0
Fork 0
mirror of https://github.com/scm-manager/scm-manager.git synced 2025-11-11 16:05:44 +01:00
Code Issues Packages Projects Releases Activity

merge with issue-470

Browse Source
This commit is contained in:
Sebastian Sdorra
2013-10-28 13:19:16 +01:00
parent 1682c2d1a1 004720660a
commit 0cdd2e8dd6
3 changed files with 17 additions and 41 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
scm-core/src/main/java/sonia/scm/util/ValidationUtil.java
View File

@@ -55,16 +55,13 @@ public final class ValidationUtil
"^[A-z0-9][\\w.-]*@[A-z0-9][\\w\\-\\.]+\\.[A-z0-9]{2,6}$";
/** Field description */
private static final String REGEX_NAME = "^[A-z0-9\\.\\-_]+$";
private static final String REGEX_NAME =
"^[A-z0-9\\.\\-_@]|[^ ]([A-z0-9\\.\\-_@ ]*[A-z0-9\\.\\-_@]|[^ ])?$";
/** Field description */
private static final String REGEX_REPOSITORYNAME =
"(?!^\\.\\.$)(?!^\\.$)(?!.*[\\\\\\[\\]])^[A-z0-9\\.][A-z0-9\\.\\-_/]*$";
/** Field description */
private static final String REGEX_USERNAME =
"^[A-z0-9\\.\\-_@]|[^ ]([A-z0-9\\.\\-_@ ]*[A-z0-9\\.\\-_@]|[^ ])?$";
//~--- constructors ---------------------------------------------------------
/**
@@ -185,10 +182,12 @@ public final class ValidationUtil
* @param username
*
* @return
* @deprecated use {@link #isNameValid(String)}
*/
@Deprecated
public static boolean isUsernameValid(String username)
{
return Util.isNotEmpty(username) && username.matches(REGEX_USERNAME);
return isNameValid(username);
}
/**

43
scm-core/src/test/java/sonia/scm/util/ValidationUtilTest.java
View File

@@ -102,14 +102,21 @@ public class ValidationUtilTest
assertTrue(ValidationUtil.isNameValid("test.git"));
assertTrue(ValidationUtil.isNameValid("Test123.git"));
assertTrue(ValidationUtil.isNameValid("Test123-git"));
assertTrue(ValidationUtil.isNameValid("Test_repository-123.git"));
assertTrue(ValidationUtil.isNameValid("Test_user-123.git"));
assertTrue(ValidationUtil.isNameValid("test@scm-manager.de"));
assertTrue(ValidationUtil.isNameValid("test 123"));
assertTrue(ValidationUtil.isNameValid("t"));
// false
assertFalse(ValidationUtil.isNameValid("test 123"));
assertFalse(ValidationUtil.isNameValid("test@123"));
assertFalse(ValidationUtil.isNameValid(" test 123"));
assertFalse(ValidationUtil.isNameValid(" test 123 "));
assertFalse(ValidationUtil.isNameValid("test 123 "));
assertFalse(ValidationUtil.isNameValid("test/123"));
assertFalse(ValidationUtil.isNameValid("test%123"));
assertFalse(ValidationUtil.isNameValid("test:123"));
assertFalse(ValidationUtil.isNameValid("t "));
assertFalse(ValidationUtil.isNameValid(" t"));
assertFalse(ValidationUtil.isNameValid(" t "));
}
/**
@@ -229,34 +236,4 @@ public class ValidationUtilTest
assertFalse(ValidationUtil.isRepositoryNameValid(path));
}
}
/**
* Method description
*
*/
@Test
public void testIsUsernameValid()
{
// true
assertTrue(ValidationUtil.isUsernameValid("test"));
assertTrue(ValidationUtil.isUsernameValid("test.git"));
assertTrue(ValidationUtil.isUsernameValid("Test123.git"));
assertTrue(ValidationUtil.isUsernameValid("Test123-git"));
assertTrue(ValidationUtil.isUsernameValid("Test_user-123.git"));
assertTrue(ValidationUtil.isUsernameValid("test@scm-manager.de"));
assertTrue(ValidationUtil.isUsernameValid("test 123"));
assertTrue(ValidationUtil.isUsernameValid("t"));
// false
assertFalse(ValidationUtil.isUsernameValid(" test 123"));
assertFalse(ValidationUtil.isUsernameValid(" test 123 "));
assertFalse(ValidationUtil.isUsernameValid("test 123 "));
assertFalse(ValidationUtil.isUsernameValid("test/123"));
assertFalse(ValidationUtil.isUsernameValid("test%123"));
assertFalse(ValidationUtil.isUsernameValid("test:123"));
assertFalse(ValidationUtil.isUsernameValid("t "));
assertFalse(ValidationUtil.isUsernameValid(" t"));
assertFalse(ValidationUtil.isUsernameValid(" t "));
}
}

4
scm-webapp/src/main/webapp/resources/js/override/ext.form.vtypes.js
View File

@@ -46,7 +46,7 @@ Ext.apply(Ext.form.VTypes, {
// name validator
name: function(val){
return /^[A-z0-9\.\-_]+$/.test(val);
return val.match(/^[^ ][A-z0-9\.\-_@ ]*[^ ]$/);
},
nameText: 'The name is invalid.',
@@ -75,7 +75,7 @@ Ext.apply(Ext.form.VTypes, {
// username validator
username: function(val){
return val.match(/^[^ ][A-z0-9\.\-_@ ]*[^ ]$/);
return name(val);
},
usernameText: 'The username is invalid.'