combine 2 email exceptions into 1

Author: ddhuy

restcomm/restcomm.http/src/main/java/org/restcomm/connect/http/AccountsEndpoint.java

@@ -65,7 +65,7 @@
 import org.restcomm.connect.http.exceptions.AuthorizationException;
 import org.restcomm.connect.http.exceptions.EmailAlreadyExisted;
 import org.restcomm.connect.http.exceptions.InsufficientPermission;
-import org.restcomm.connect.http.exceptions.InvalidEmailFormat;
+import org.restcomm.connect.http.exceptions.InvalidEmailException;
 import org.restcomm.connect.http.exceptions.PasswordTooWeak;
 import org.restcomm.connect.http.exceptions.RcmlserverNotifyError;
 import org.restcomm.connect.identity.EmailValidator;
@@ -465,7 +465,7 @@ private Client createClientFrom(final Sid accountSid, final MultivaluedMap<Strin
      * @throws AccountAlreadyClosed
      */
     private Account prepareAccountForUpdate(final Account account, final MultivaluedMap<String, String> data)
-            throws AccountAlreadyClosed, PasswordTooWeak, EmailAlreadyExisted, InvalidEmailFormat {
+            throws AccountAlreadyClosed, PasswordTooWeak, EmailAlreadyExisted, InvalidEmailException {
         Account result = account;
         boolean isPasswordReset = false;
         Account.Status newStatus = null;
@@ -513,14 +513,14 @@ private Account prepareAccountForUpdate(final Account account, final Multivalued
             if (data.containsKey("EmailAddress")) {
                 String newEmailAddress = data.getFirst("EmailAddress").toLowerCase();
                 if (!EmailValidator.isValidEmailFormat(newEmailAddress)) {
-                    throw new InvalidEmailFormat();
+                    throw new InvalidEmailException();
                 }
                 if (accountsDao.getAccount(newEmailAddress) != null) {
                     throw new EmailAlreadyExisted();
                 }
                 result = result.setEmailAddress(newEmailAddress);
             }
-        } catch (AuthorizationException | AccountAlreadyClosed | PasswordTooWeak | EmailAlreadyExisted | InvalidEmailFormat e) {
+        } catch (AuthorizationException | AccountAlreadyClosed | PasswordTooWeak | EmailAlreadyExisted | InvalidEmailException e) {
             // some exceptions should reach outer layers and result in 403
             throw e;
         } catch (Exception e) {

restcomm/restcomm.http/src/main/java/org/restcomm/connect/http/EmailMessagesEndpoint.java

@@ -26,6 +26,7 @@
 import org.restcomm.connect.email.api.Mail;
 import org.restcomm.connect.http.converter.EmailMessageConverter;
 import org.restcomm.connect.http.converter.RestCommResponseConverter;
+import org.restcomm.connect.http.exceptions.InvalidEmailException;
 import org.restcomm.connect.identity.EmailValidator;
 
 import javax.annotation.PostConstruct;
@@ -258,16 +259,6 @@ public void onReceive(final Object message) throws Exception {
         }
     }
 
-    private static class InvalidEmailException extends Exception {
-        //Parameterless Constructor
-        public InvalidEmailException() {}
-
-        //Constructor that accepts a message
-        public InvalidEmailException(String message) {
-            super(message);
-        }
-    }
-
 }
 
 

…/http/exceptions/InvalidEmailFormat.java …tp/exceptions/InvalidEmailException.javarestcomm/restcomm.http/src/main/java/org/restcomm/connect/http/exceptions/InvalidEmailFormat.java renamed to restcomm/restcomm.http/src/main/java/org/restcomm/connect/http/exceptions/InvalidEmailException.java restcomm/restcomm.http/src/main/java/org/restcomm/connect/http/exceptions/InvalidEmailFormat.java renamed to restcomm/restcomm.http/src/main/java/org/restcomm/connect/http/exceptions/InvalidEmailException.java

@@ -20,11 +20,16 @@
 
 package org.restcomm.connect.http.exceptions;
 
-import org.restcomm.connect.commons.exceptions.RestcommRuntimeException;
-
 
 /**
  * @author ddh.huy@gmail.com (Huy Dang)
  */
-public class InvalidEmailFormat extends RestcommRuntimeException {
+public class InvalidEmailException extends RuntimeException {
+    //Parameterless Constructor
+    public InvalidEmailException() {}
+
+    //Constructor that accepts a message
+    public InvalidEmailException(String message) {
+        super(message);
+    }
 }