Continued with rewrite:

- introduced showFacesMessage()
- rewrote to FacesMessage and not ugly exception 500 error
- rewrote more messages + fixed form ids to naming convention

diff --git a/src/java/org/mxchange/pizzaapplication/beans/BasePizzaController.java b/src/java/org/mxchange/pizzaapplication/beans/BasePizzaController.java
index dd426872868bfed6996d53e3af81ea3072b2b520..69a0379f96fabc909778e35b3ac7c5aec3afb4f2 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/BasePizzaController.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/BasePizzaController.java
@@ -17,6 +17,8 @@
 package org.mxchange.pizzaapplication.beans;
 
 import java.io.Serializable;
+import java.text.MessageFormat;
+import javax.faces.application.FacesMessage;
 import javax.faces.context.FacesContext;
 
 /**
@@ -58,4 +60,19 @@ public abstract class BasePizzaController implements Serializable {
                return isEnabled;
        }
 
+       /**
+        * Shows a faces message for given causing exception. The message from the
+        * exception is being inserted into the message.
+        * <p>
+        * @param clientId Client id to send message to
+        * @param cause Causing exception
+        */
+       protected void showFacesMessage (final String clientId, final Throwable cause) {
+               // Trace message
+               System.out.println(MessageFormat.format("showFacesMessage: clientId={0},cause={1} - CALLED!", clientId, cause));
+
+               // Get context and add message
+               FacesContext.getCurrentInstance().addMessage(clientId, new FacesMessage(cause.getMessage()));
+       }
+
 }

diff --git a/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionBean.java
index 3ed20d676fd8de8aa60bbe95e849337873f86206..07400a0c24268584cda8f2f35c78fb9cc935f206 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionBean.java
@@ -565,7 +565,8 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                        throw new FaceletException("Not all required fields are set."); //NOI18N
                } else if (!this.userLoginController.ifCurrentPasswordMatches()) {
                        // Password not matching
-                       throw new FaceletException(new UserPasswordMismatchException(this.userLoginController.getLoggedInUser()));
+                       this.showFacesMessage("login_change_personal_form:currentPassword", new UserPasswordMismatchException(this.userLoginController.getLoggedInUser())); //NOI18N
+                       return ""; //NOI18N
                }
 
                // Get contact instance

diff --git a/src/java/org/mxchange/pizzaapplication/beans/email_address/PizzaEmailChangeWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/email_address/PizzaEmailChangeWebSessionBean.java
index ec382e601190e1efe804452bc36402c2dd3ad561..a783581ab9aed4b4cd22510243224d6d5986c5ba 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/email_address/PizzaEmailChangeWebSessionBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/email_address/PizzaEmailChangeWebSessionBean.java
@@ -110,7 +110,8 @@ public class PizzaEmailChangeWebSessionBean extends BasePizzaController implemen
                        throw new FaceletException("Email address 1/2 are mismatching."); //NOI18N
                } else if (!this.loginController.ifCurrentPasswordMatches()) {
                        // Password not matching
-                       throw new FaceletException(new UserPasswordMismatchException(this.loginController.getLoggedInUser()));
+                       this.showFacesMessage("login_change_email_address_form:currentPassword", new UserPasswordMismatchException(this.loginController.getLoggedInUser())); //NOI18N
+                       return ""; //NOI18N
                }
 
                // Get user instance

diff --git a/src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestBean.java b/src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestBean.java
index 8ba9165ea934a3fb84bd4ebc1b56de6d8227d4d0..d384acce65e51c41c3dc4d32187ccef8b4aadf65 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestBean.java
@@ -114,7 +114,8 @@ public class PizzaAdminMobileProviderWebRequestBean extends BasePizzaController
                // Is the provider already created?
                if (this.isMobileProviderCreated(mobileProvider)) {
                        // Then throw exception
-                       throw new FaceletException(new MobileProviderAlreadyAddedException(mobileProvider));
+                       this.showFacesMessage("add_mobile_provider_form:providerDialPrefix", new MobileProviderAlreadyAddedException(mobileProvider)); //NOI18N
+                       return ""; //NOI18N
                }
 
                // Init variable

diff --git a/src/java/org/mxchange/pizzaapplication/beans/resendlink/PizzaResendLinkWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/resendlink/PizzaResendLinkWebSessionBean.java
index 2e345fac8705734473d6040faf442e70e8a2b540..f70623861db28783eb8e83f2a8477a50d0613c2a 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/resendlink/PizzaResendLinkWebSessionBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/resendlink/PizzaResendLinkWebSessionBean.java
@@ -102,17 +102,19 @@ public class PizzaResendLinkWebSessionBean extends BasePizzaController implement
                        // Is the email address really not used?
                        user = this.userController.lookupUserByEmailAddress(this.getEmailAddress());
                } catch (final UserEmailAddressNotFoundException ex) {
-                       // Not found, should not happen as the registeredvalidator should find it
+                       // Not found, should not happen as the registered validator should find it
                        throw new FaceletException(MessageFormat.format("this.emailAddress={0} should be resolveable into User instance.", this.getEmailAddress()), ex); //NOI18N
                }
 
                // Is the user account already confirmed?
                if (user.getUserAccountStatus() == UserAccountStatus.CONFIRMED) {
                        // Then abort here
-                       throw new FaceletException(new UserStatusConfirmedException(user));
+                       this.showFacesMessage("form_resend_link:resendEmailAddress", new UserStatusConfirmedException(user)); //NOI18N
+                       return ""; //NOI18N
                } else if (user.getUserAccountStatus() == UserAccountStatus.LOCKED) {
                        // User account is locked
-                       throw new FaceletException(new UserStatusLockedException(user));
+                       this.showFacesMessage("form_resend_link:resendEmailAddress", new UserStatusLockedException(user)); //NOI18N
+                       return ""; //NOI18N
                } else if (user.getUserConfirmKey() == null) {
                        // Status is UNCONFIRMED but confirmation key is NULL
                        throw new NullPointerException("user.userConfirmKey is null"); //NOI18N

diff --git a/web/WEB-INF/templates/admin/mobile_provider/admin_form_mobile_provider.tpl b/web/WEB-INF/templates/admin/mobile_provider/admin_form_mobile_provider.tpl
index 9bbaf3d1f10c9bcaadaea76a768eff9f360c1259..e6a87ddb1fdd212d0f9b0eb7dd3764ce8f3151f0 100644 (file)
--- a/web/WEB-INF/templates/admin/mobile_provider/admin_form_mobile_provider.tpl
+++ b/web/WEB-INF/templates/admin/mobile_provider/admin_form_mobile_provider.tpl
@@ -27,6 +27,8 @@
                                <div class="clear"></div>
                        </div>
 
+                       <h:message for="providerDialPrefix" errorClass="errors" fatalClass="errors" warnClass="errors" />
+
                        <div class="table_row">
                                <div class="table_left_medium">
                                        <h:outputLabel for="providerMailPattern" value="#{msg.ADMIN_ENTER_MOBILE_PROVIDER_PATTERN}" />

diff --git a/web/admin/mobile_provider/admin_mobile_provider_list.xhtml b/web/admin/mobile_provider/admin_mobile_provider_list.xhtml
index dbd277520f138b8e95d2d9c5b8f42636fe0c0a8f..fdd1d473bed5e0fc4567d73cea93a4ed0148bbb4 100644 (file)
--- a/web/admin/mobile_provider/admin_mobile_provider_list.xhtml
+++ b/web/admin/mobile_provider/admin_mobile_provider_list.xhtml
@@ -62,7 +62,7 @@
                                </h:column>
                        </h:dataTable>
 
-                       <h:form id="form_add_mobile_provider">
+                       <h:form id="add_mobile_provider_form">
                                <div class="table_medium">
                                        <div class="table_header">
                                                <h:outputText value="#{msg.ADMIN_ADD_MOBILE_PROVIDER_TITLE}" />

diff --git a/web/user/login_change_email_address.xhtml b/web/user/login_change_email_address.xhtml
index 4d70ccb0b27f1d5978fd6d39f5c6fe0b7812225c..f6bf9b9769fbd27fa6b488bc57c31d621526129d 100644 (file)
--- a/web/user/login_change_email_address.xhtml
+++ b/web/user/login_change_email_address.xhtml
@@ -22,7 +22,7 @@
                                                #{msg.LOGIN_CHANGE_EMAIL_ADDRESS_TITLE}
                                        </div>
 
-                                       <h:form id="login_form">
+                                       <h:form id="login_change_email_address_form">
                                                <div class="para">
                                                        <fieldset id="change_email">
                                                                <legend title="#{msg.LOGIN_CHANGE_EMAIL_LEGEND_TITLE}">#{msg.LOGIN_CHANGE_EMAIL_LEGEND}</legend>