From: Roland Häder <roland@mxchange.org>
Date: Sun, 20 Aug 2017 19:49:49 +0000 (+0200)
Subject: Please cherry-pick:
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=4118b719cc73e598ce7c144559825000d787c98c;p=pizzaservice-war.git

Please cherry-pick:
- throwing a ConverterException in a validator is wrong, maybe copy-paste did it?
- added TODO for getting rid of wrong constructor in birthday validator
- renamed variable

Signed-off-by: Roland Häder <roland@mxchange.org>
---

diff --git a/src/java/org/mxchange/pizzaapplication/validator/birthday/PizzaBirthdayValidator.java b/src/java/org/mxchange/pizzaapplication/validator/birthday/PizzaBirthdayValidator.java
index 888b5b75..b1a3c9b0 100644
--- a/src/java/org/mxchange/pizzaapplication/validator/birthday/PizzaBirthdayValidator.java
+++ b/src/java/org/mxchange/pizzaapplication/validator/birthday/PizzaBirthdayValidator.java
@@ -44,6 +44,7 @@ public class PizzaBirthdayValidator extends BaseDateValidator implements Validat
 
 	/**
 	 * Default constructor
+	 * @TODO Get rid of this
 	 */
 	public PizzaBirthdayValidator () {
 		// Try to get it
diff --git a/src/java/org/mxchange/pizzaapplication/validator/emailaddress/PizzaEmailAddressValidator.java b/src/java/org/mxchange/pizzaapplication/validator/emailaddress/PizzaEmailAddressValidator.java
index 0c309f42..57bacdb2 100644
--- a/src/java/org/mxchange/pizzaapplication/validator/emailaddress/PizzaEmailAddressValidator.java
+++ b/src/java/org/mxchange/pizzaapplication/validator/emailaddress/PizzaEmailAddressValidator.java
@@ -21,7 +21,6 @@ import java.util.regex.Pattern;
 import javax.faces.application.FacesMessage;
 import javax.faces.component.UIComponent;
 import javax.faces.context.FacesContext;
-import javax.faces.convert.ConverterException;
 import javax.faces.validator.FacesValidator;
 import javax.faces.validator.Validator;
 import javax.faces.validator.ValidatorException;
@@ -131,7 +130,7 @@ public class PizzaEmailAddressValidator extends BaseStringValidator implements V
 				PizzaEmailAddressValidator.CONTACT_BEAN = (ContactSessionBeanRemote) initialContext.lookup("java:global/pizzaapplication-ejb/contact!org.mxchange.jcontacts.contact.ContactSessionBeanRemote"); //NOI18N
 			} catch (final NamingException ex) {
 				// Continue to throw it
-				throw new ConverterException(MessageFormat.format("initialContext.lookup() failed: {0}", ex.getMessage()), ex); //NOI18N
+				throw new ValidatorException(new FacesMessage(MessageFormat.format("initialContext.lookup() failed: {0}", ex.getMessage())), ex); //NOI18N
 			}
 		}
 
diff --git a/src/java/org/mxchange/pizzaapplication/validator/phone/abroad/PizzaAbroadDialValidator.java b/src/java/org/mxchange/pizzaapplication/validator/phone/abroad/PizzaAbroadDialValidator.java
index c2d9032c..fbce16bd 100644
--- a/src/java/org/mxchange/pizzaapplication/validator/phone/abroad/PizzaAbroadDialValidator.java
+++ b/src/java/org/mxchange/pizzaapplication/validator/phone/abroad/PizzaAbroadDialValidator.java
@@ -47,14 +47,14 @@ public class PizzaAbroadDialValidator extends BaseObjectValidator implements Val
 		super.preValidate(context, component, value, requiredFields, true);
 
 		// Parse value as string first
-		String dial = String.valueOf(value);
+		String dialNumber = String.valueOf(value);
 
 		// Is it not +?
-		if (!dial.equals("+")) { //NOI18N
+		if (!dialNumber.equals("+")) { //NOI18N
 			// No, then try to ...
 			try {
 				// ..parse as number
-				Long number = Long.parseLong(dial);
+				Long number = Long.parseLong(dialNumber);
 
 				// Not valid range? (1 - 99, very rude)
 				if (number < 1 || number > 99) {