Merge pull request AY2324S1-CS2103-F13-2#205 from nixonwidjaja/master

Fix phone bug
nixonwidjaja · Nov 13, 2023 · 913403c · 913403c
2 parents 9dd49ac + ac83f53
commit 913403c
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/main/java/seedu/address/model/person/Phone.java b/src/main/java/seedu/address/model/person/Phone.java
@@ -31,11 +31,14 @@ public Phone(String phone) {
      * Returns true if a given string is a valid phone number.
      */
     public static boolean isValidPhone(String test) {
-        if (test.length() < 3 || test.length() > 20) {
+        if (test.length() < 3) {
             return false;
         }
         if (test.charAt(0) == '+') {
-            return test.substring(1).matches(VALIDATION_REGEX);
+            test = test.substring(1);
+        }
+        if (test.length() < 3 || test.length() > 20) {
+            return false;
         }
         return test.matches(VALIDATION_REGEX);
     }