Please cherry-pick:

- h:outputText was wrong where it should be h:panelGroup, now the guest menue is being shown again
- feature controller is no longer causing and then catching NPE (bad code style)
- sorted web.xml
- reformated some files

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

diff --git a/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java b/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java
index c3918cdcc6df3e60dd89c4bc1925f8a31d415cb8..c51d5622245f822063ff150ecb31c5dceac2d623 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java
@@ -392,6 +392,7 @@ public interface PizzaContactWebSessionController extends Serializable {
         * Setter for controller type
         * <p>
         * @param controllerType Controller type
+        * <p>
         * @deprecated Don't use this method.
         */
        @Deprecated

diff --git a/src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestController.java b/src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestController.java
index ce26d7c98de3c05f4a02cd1bfb66462245360e97..da30752f639f96d88dafb4e62906ce0321abf2e1 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestController.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestController.java
@@ -51,8 +51,8 @@ public interface PizzaAdminContactPhoneWebRequestController extends Serializable
        String unlinkMobileContactData ();
 
        /**
-        * Links given contact instance with set fax number or entered data. If
-        * both is set, a proper exception is thrown as this is an invalid state.
+        * Links given contact instance with set fax number or entered data. If both
+        * is set, a proper exception is thrown as this is an invalid state.
         * <p>
         * @param contact Contact instance to link with existing/new data
         * <p>
@@ -61,9 +61,8 @@ public interface PizzaAdminContactPhoneWebRequestController extends Serializable
        String doLinkAddFaxNumber (final Contact contact);
 
        /**
-        * Links given contact instance with set land-line number or entered
-        * data. If both is set, a proper exception is thrown as this is an invalid
-        * state.
+        * Links given contact instance with set land-line number or entered data.
+        * If both is set, a proper exception is thrown as this is an invalid state.
         * <p>
         * @param contact Contact instance to link with existing/new data
         * <p>
@@ -72,8 +71,8 @@ public interface PizzaAdminContactPhoneWebRequestController extends Serializable
        String doLinkAddLandLineNumber (final Contact contact);
 
        /**
-        * Links given contact instance with set mobile or entered data. If both
-        * is set, a proper exception is thrown as this is an invalid state.
+        * Links given contact instance with set mobile or entered data. If both is
+        * set, a proper exception is thrown as this is an invalid state.
         * <p>
         * @param contact Contact instance to link with existing/new data
         * <p>

diff --git a/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java b/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java
index a4517c7e524f4d0782ecf3c2d92e82f3e2580605..f19b3bb66dbcbcba8d1438d6bb55f41cced7a0bc 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java
@@ -102,7 +102,7 @@ public interface PizzaAdminCountryWebRequestController extends Serializable {
         * Setter for whether the local dial prefix is required for local calls
         * <p>
         * @param countryIsLocalPrefixRequired Whether the local dial prefix is
-        * required
+        *                                     required
         */
        void setCountryIsLocalPrefixRequired (final Boolean countryIsLocalPrefixRequired);
 

diff --git a/src/java/org/mxchange/pizzaapplication/beans/features/PizzaFeatureWebApplicationBean.java b/src/java/org/mxchange/pizzaapplication/beans/features/PizzaFeatureWebApplicationBean.java
index 2a5b3266ea3263acd6073fa6c277e13dc0cb4ce8..c4a3f540a92a08de59c64096965f5fbf32e8277c 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/features/PizzaFeatureWebApplicationBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/features/PizzaFeatureWebApplicationBean.java
@@ -64,18 +64,17 @@ public class PizzaFeatureWebApplicationBean extends BasePizzaController implemen
                // Default is not enabled
                boolean isEnabled = false;
 
-               // Try it as an NPE may come
-               try {
-                       // Get value from property
-                       String value = this.getStringContextParameter(String.format("is_feature_%s_enabled", feature)); //NOI18N
+               // Get value from property
+               String contextParameter = this.getStringContextParameter(String.format("is_feature_%s_enabled", feature)); //NOI18N
+               //System.out.println(MessageFormat.format("isFeatureSet: feature={0},contextParameter[]={1}", feature, Objects.toString(contextParameter))); //NOI18N
 
+               // Is the context parameter found?
+               if (contextParameter instanceof String) {
                        // Is it set?
-                       isEnabled = (value.toLowerCase().equals("true")); //NOI18N
-               } catch (final NullPointerException ex) {
-                       // Ignored
+                       isEnabled = (Boolean.parseBoolean(contextParameter) == Boolean.TRUE);
                }
 
-               // Return value
+               // Return status
                return isEnabled;
        }
 

diff --git a/src/java/org/mxchange/pizzaapplication/beans/localization/PizzaLocalizationSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/localization/PizzaLocalizationSessionBean.java
index e652e5e462aab4a372c5767b21cc323502d859fe..a1da4c488ee33bad287e5a910709a6c516efc56f 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/localization/PizzaLocalizationSessionBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/localization/PizzaLocalizationSessionBean.java
@@ -28,8 +28,8 @@ import org.mxchange.jusercore.events.logout.ObservableUserLogoutEvent;
 import org.mxchange.pizzaapplication.beans.BasePizzaController;
 
 /**
- * A session-scoped bean for handling localization/internationalization changes. This
- * class is based on an example at [1] from mkyong.
+ * A session-scoped bean for handling localization/internationalization changes.
+ * This class is based on an example at [1] from mkyong.
  * <p>
  * 1: http://www.mkyong.com/jsf2/jsf-2-internationalization-example/
  * <p>

diff --git a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java
index 70e0acf1d4d996dbc06a3d6fc31386856d46189a..b4df84c7c9001086a3adecae6b1f94eba0333291 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java
@@ -87,17 +87,17 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
        private PizzaWebRequestHelperController beanHelper;
 
        /**
-        *  fax number
+        * fax number
         */
        private DialableFaxNumber choosenFaxNumber;
 
        /**
-        *  land-line number
+        * land-line number
         */
        private DialableLandLineNumber choosenLandLineNumber;
 
        /**
-        *  mobile number
+        * mobile number
         */
        private DialableMobileNumber choosenMobileNumber;
 

diff --git a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestController.java b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestController.java
index 9bc322b85087d407f5e5f75b4ccf03abc7ec8cbc..f8df058d057b1abc0de1af67dc57849654f23034 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestController.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestController.java
@@ -126,42 +126,42 @@ public interface PizzaAdminPhoneWebRequestController extends Serializable {
        /**
         * Getter for choosen fax number
         * <p>
-        * @return  fax number
+        * @return fax number
         */
        DialableFaxNumber getFaxNumber ();
 
        /**
         * Setter for choosen fax number
         * <p>
-        * @param choosenFaxNumber  fax number
+        * @param choosenFaxNumber fax number
         */
        void setFaxNumber (final DialableFaxNumber choosenFaxNumber);
 
        /**
         * Getter for choosen land-line number
         * <p>
-        * @return  land-line number
+        * @return land-line number
         */
        DialableLandLineNumber getLandLineNumber ();
 
        /**
         * Setter for choosen land-line number
         * <p>
-        * @param choosenLandLineNumber  land-line number
+        * @param choosenLandLineNumber land-line number
         */
        void setLandLineNumber (final DialableLandLineNumber choosenLandLineNumber);
 
        /**
         * Getter for choosen mobile number
         * <p>
-        * @return  mobile number
+        * @return mobile number
         */
        DialableMobileNumber getMobileNumber ();
 
        /**
         * Setter for choosen mobile number
         * <p>
-        * @param choosenMobileNumber  mobile number
+        * @param choosenMobileNumber mobile number
         */
        void setMobileNumber (final DialableMobileNumber choosenMobileNumber);
 

diff --git a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaPhoneWebApplicationBean.java b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaPhoneWebApplicationBean.java
index d1134bb38f5593cd057a5f4a309c61520165ae6b..19cc2bf244e688eb45e405453977e30b9cdd51aa 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaPhoneWebApplicationBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaPhoneWebApplicationBean.java
@@ -148,7 +148,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getDeletedFaxNumber()== null) {
+               } else if (event.getDeletedFaxNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.deletedFaxNumber is null"); //NOI18N
                } else if (event.getDeletedFaxNumber().getPhoneId() == null) {
@@ -172,7 +172,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getDeletedLandLineNumber()== null) {
+               } else if (event.getDeletedLandLineNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.deletedLandLineNumber is null"); //NOI18N
                } else if (event.getDeletedLandLineNumber().getPhoneId() == null) {
@@ -196,7 +196,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getDeletedMobileNumber()== null) {
+               } else if (event.getDeletedMobileNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.deletedMobileNumber is null"); //NOI18N
                } else if (event.getDeletedMobileNumber().getPhoneId() == null) {
@@ -228,7 +228,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactId is null");
                } else if (event.getContact().getContactId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactId="+ event.getContact().getContactId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactId=" + event.getContact().getContactId() + " is invalid");
                } else if (event.getContact().getContactFaxNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.contact.contactFaxNumber is null");
@@ -237,7 +237,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactFaxNumber.phoneId is null");
                } else if (event.getContact().getContactFaxNumber().getPhoneId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactFaxNumber.phoneId="+event.getContact().getContactFaxNumber().getPhoneId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactFaxNumber.phoneId=" + event.getContact().getContactFaxNumber().getPhoneId() + " is invalid");
                } else if (event.getLinkedFaxNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.linkedFaxNumer is null");
@@ -264,7 +264,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactId is null");
                } else if (event.getContact().getContactId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactId="+ event.getContact().getContactId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactId=" + event.getContact().getContactId() + " is invalid");
                } else if (event.getContact().getContactLandLineNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.contact.contactLandLineNumber is null");
@@ -273,7 +273,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactLandLineNumber.phoneId is null");
                } else if (event.getContact().getContactLandLineNumber().getPhoneId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactLandLineNumber.phoneId="+event.getContact().getContactLandLineNumber().getPhoneId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactLandLineNumber.phoneId=" + event.getContact().getContactLandLineNumber().getPhoneId() + " is invalid");
                } else if (event.getLinkedLandLineNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.linkedLandLineNumer is null");
@@ -300,7 +300,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactId is null");
                } else if (event.getContact().getContactId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactId="+ event.getContact().getContactId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactId=" + event.getContact().getContactId() + " is invalid");
                } else if (event.getContact().getContactMobileNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.contact.contactMobileNumber is null");
@@ -309,7 +309,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                        throw new NullPointerException("event.contact.contactMobileNumber.phoneId is null");
                } else if (event.getContact().getContactMobileNumber().getPhoneId() < 1) {
                        // Throw again ...
-                       throw new NullPointerException("event.contact.contactMobileNumber.phoneId="+event.getContact().getContactMobileNumber().getPhoneId() + " is invalid");
+                       throw new NullPointerException("event.contact.contactMobileNumber.phoneId=" + event.getContact().getContactMobileNumber().getPhoneId() + " is invalid");
                } else if (event.getLinkedMobileNumber() == null) {
                        // Throw again ...
                        throw new NullPointerException("event.linkedMobileNumer is null");
@@ -352,7 +352,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getUpdatedFaxNumber()== null) {
+               } else if (event.getUpdatedFaxNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.updatedFaxNumber is null"); //NOI18N
                } else if (event.getUpdatedFaxNumber().getPhoneId() == null) {
@@ -376,7 +376,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getUpdatedLandLineNumber()== null) {
+               } else if (event.getUpdatedLandLineNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.updatedLandLineNumber is null"); //NOI18N
                } else if (event.getUpdatedLandLineNumber().getPhoneId() == null) {
@@ -400,7 +400,7 @@ public class PizzaPhoneWebApplicationBean extends BasePizzaController implements
                if (null == event) {
                        // Throw NPE
                        throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getUpdatedMobileNumber()== null) {
+               } else if (event.getUpdatedMobileNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("event.updatedMobileNumber is null"); //NOI18N
                } else if (event.getUpdatedMobileNumber().getPhoneId() == null) {

diff --git a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java
index 19a864ce2e6076b6f41f651ca6473620ebc1901d..cecc3cf311f182ee95ffcfc2404523f07386536b 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java
+++ b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java
@@ -530,14 +530,12 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
 
                        // Debug message
                        //* NOISY-DEBUG: */ System.out.println(MessageFormat.format("{0}.createUserInstance: contact={1}", this.getClass().getSimpleName(), contact));
-
                        // Set contact in user
                        user.setUserContact(contact);
                }
 
                // Trace message
                //* NOISY-DEBUG: */ System.out.println(MessageFormat.format("{0}.createUserInstance: user={1} - EXIT!", this.getClass().getSimpleName(), user));
-
                // Return it
                return user;
        }
@@ -564,7 +562,6 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
 
                // Trace message
                //* NOISY-DEBUG */ System.out.println(MessageFormat.format("{0}.createUserLogin: user={1} - EXIT!", this.getClass().getSimpleName(), user));
-
                // Return the new instance
                return user;
        }

diff --git a/web/WEB-INF/templates/guest/guest_menu.tpl b/web/WEB-INF/templates/guest/guest_menu.tpl
index 8453b9e3117ada439e627409013372dfe6117311..1bc058f8a534ab161d6d9259133583958a2ed1e6 100644 (file)
--- a/web/WEB-INF/templates/guest/guest_menu.tpl
+++ b/web/WEB-INF/templates/guest/guest_menu.tpl
@@ -6,7 +6,7 @@
        xmlns:ui="http://xmlns.jcp.org/jsf/facelets">
 
        <div class="menu_outer">
-               <h:outputText styleClass="menu">
+               <h:panelGroup styleClass="menu" layout="block">
                        <div class="menu_header">
                                <h:outputText value="#{msg.MENU_HOME_TITLE}" />
                        </div>
@@ -46,6 +46,6 @@
                                        </li>
                                </ui:fragment>
                        </ul>
-               </h:outputText>
+               </h:panelGroup>
        </div>
 </ui:composition>

diff --git a/web/WEB-INF/web.xml b/web/WEB-INF/web.xml
index 44167044a16930556063b97f072ef11b807d2f85..9b8c778f3074f4d884f5669038ee89bdac8752f2 100644 (file)
--- a/web/WEB-INF/web.xml
+++ b/web/WEB-INF/web.xml
@@ -2,6 +2,11 @@
 <web-app version="2.5" xmlns="http://java.sun.com/xml/ns/javaee" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd">
     <description>An online address book application to share private and business memebers between all members. It is also possible that the user's profile can be made visible to outside.</description>
     <display-name>JLandingPage Application v1.0</display-name>
+    <context-param>
+        <description>Whether the date converter's default timezone is system's timezone.</description>
+        <param-name>javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE</param-name>
+        <param-value>true</param-value>
+    </context-param>
     <context-param>
         <description>Whether the multi-page registration page or a single registration page is active</description>
         <param-name>is_feature_user_register_multiple_page_enabled</param-name>
@@ -22,11 +27,6 @@
         <param-name>is_feature_user_resend_confirmation_link_enabled</param-name>
         <param-value>true</param-value>
     </context-param>
-    <context-param>
-        <description>Whether default timezon is set from system's timezone</description>
-        <param-name>javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE</param-name>
-        <param-value>true</param-value>
-    </context-param>
     <context-param>
         <description>Whether registration page is enabled.</description>
         <param-name>is_feature_user_registration_enabled</param-name>