OO-1038: restraints character boundaries

src/main/java/org/olat/core/util/StringHelper.java

@@ -404,7 +404,7 @@ public class StringHelper {
 		char[] charArr = string.toCharArray();
 		for(int i=charArr.length; i-->stop; ) {
 			char ch = charArr[i];
-			if(ch < 47 || ch > 58) {
+			if(ch < 48 || ch > 57) {
 				return false;
 			}
 		}

src/test/java/org/olat/core/util/StringHelperTest.java

@@ -112,12 +112,20 @@ public class StringHelperTest {
 	@Test
 	public void isLong() {
 		Assert.assertTrue(StringHelper.isLong("234"));
+		Assert.assertTrue(StringHelper.isLong("0123456789"));
 		Assert.assertTrue(StringHelper.isLong("9223372036854775807"));
 		Assert.assertTrue(StringHelper.isLong("-9223372036854775807"));
 
+		//check some unacceptable strings
 		Assert.assertFalse(StringHelper.isLong("10223372036854775807"));
 		Assert.assertFalse(StringHelper.isLong("-dru"));
 		Assert.assertFalse(StringHelper.isLong("OpenOLAT"));
 		Assert.assertFalse(StringHelper.isLong("A very long number with a lot of characters"));
+		
+		//check ascii range
+		Assert.assertFalse(StringHelper.isLong("/"));
+		Assert.assertFalse(StringHelper.isLong(":"));
+		Assert.assertFalse(StringHelper.isLong("."));
+		Assert.assertFalse(StringHelper.isLong(";"));
 	}
 }