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/jfinancials/validator/birthday/FinancialsBirthdayValidator.java b/src/java/org/mxchange/jfinancials/validator/birthday/FinancialsBirthdayValidator.java
index 521d43b027e352fc65252e3e3ca7374935868f23..cc663cbe0163d0089f5cd80490623ae6bbf1eee0 100644 (file)
--- a/src/java/org/mxchange/jfinancials/validator/birthday/FinancialsBirthdayValidator.java
+++ b/src/java/org/mxchange/jfinancials/validator/birthday/FinancialsBirthdayValidator.java
@@ -44,6 +44,7 @@ public class FinancialsBirthdayValidator extends BaseDateValidator implements Va
 
        /**
         * Default constructor
+        * @TODO Get rid of this
         */
        public FinancialsBirthdayValidator () {
                // Try to get it

diff --git a/src/java/org/mxchange/jfinancials/validator/emailaddress/FinancialsEmailAddressValidator.java b/src/java/org/mxchange/jfinancials/validator/emailaddress/FinancialsEmailAddressValidator.java
index d913605511c0ebea8e4335dbf516b1e7a6dc9d25..d55cc6d2f58767fecb9b2238f2c1c775eb62c05e 100644 (file)
--- a/src/java/org/mxchange/jfinancials/validator/emailaddress/FinancialsEmailAddressValidator.java
+++ b/src/java/org/mxchange/jfinancials/validator/emailaddress/FinancialsEmailAddressValidator.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;
@@ -114,7 +113,7 @@ public class FinancialsEmailAddressValidator extends BaseStringValidator impleme
                                FinancialsEmailAddressValidator.CONTACT_BEAN = (ContactSessionBeanRemote) initialContext.lookup("java:global/jfinancials-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/jfinancials/validator/phone/abroad/FinancialsAbroadDialValidator.java b/src/java/org/mxchange/jfinancials/validator/phone/abroad/FinancialsAbroadDialValidator.java
index 98a89d24774a1779d7e6ff6a76afee8c30a04e76..616640ddade295b43d2ea8e9a131af0d1e237cf8 100644 (file)
--- a/src/java/org/mxchange/jfinancials/validator/phone/abroad/FinancialsAbroadDialValidator.java
+++ b/src/java/org/mxchange/jfinancials/validator/phone/abroad/FinancialsAbroadDialValidator.java
@@ -47,14 +47,14 @@ public class FinancialsAbroadDialValidator extends BaseObjectValidator implement
                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) {