UserName VO fix (between 2 and 32 chars)

chonsser · Jun 15, 2017 · 509ccfc · 509ccfc
1 parent 98ee5fe
commit 509ccfc
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/Provider/ValueObject/UserName.php b/src/Provider/ValueObject/UserName.php
@@ -44,18 +44,18 @@ public function __toString()
      */
     private function guardUserName($user_name)
     {
-        if(is_string($user_name) === false){
+        if (is_string($user_name) === false) {
             throw new InvalidUserNameException("Username has to be a string");
         }
 
         $name_length = strlen($user_name);
 
-        if ($name_length > self::MAX_LENGTH) {
-            throw new InvalidUserNameException("Username must have less than 100 characters");
+        if ($name_length >= self::MAX_LENGTH) {
+            throw new InvalidUserNameException("Username must have less than " . self::MAX_LENGTH . " characters");
         }
 
-        if ($name_length < self::MIN_LENGTH) {
-            throw new InvalidUserNameException("Username must have more than 2 characters");
+        if ($name_length <= self::MIN_LENGTH) {
+            throw new InvalidUserNameException("Username must have more than " . self::MIN_LENGTH . " characters");
         }
     }