From dc990d8086886d0b245d5c734729ad4cf6e9f305 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roland=20H=C3=A4der?= Date: Sun, 11 Jun 2017 13:38:16 +0200 Subject: [PATCH] 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 MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Roland Häder --- .../PizzaContactWebSessionController.java | 1 + ...AdminContactPhoneWebRequestController.java | 13 +++++----- ...PizzaAdminCountryWebRequestController.java | 2 +- .../PizzaFeatureWebApplicationBean.java | 15 ++++++------ .../PizzaLocalizationSessionBean.java | 4 ++-- .../phone/PizzaAdminPhoneWebRequestBean.java | 6 ++--- .../PizzaAdminPhoneWebRequestController.java | 12 +++++----- .../phone/PizzaPhoneWebApplicationBean.java | 24 +++++++++---------- .../beans/user/PizzaUserWebSessionBean.java | 3 --- web/WEB-INF/templates/guest/guest_menu.tpl | 4 ++-- web/WEB-INF/web.xml | 10 ++++---- 11 files changed, 45 insertions(+), 49 deletions(-) diff --git a/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java b/src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java index c3918cdc..c51d5622 100644 --- 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 *

* @param controllerType Controller type + *

* @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 ce26d7c9..da30752f 100644 --- 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. *

* @param contact Contact instance to link with existing/new data *

@@ -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. *

* @param contact Contact instance to link with existing/new data *

@@ -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. *

* @param contact Contact instance to link with existing/new data *

diff --git a/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java b/src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java index a4517c7e..f19b3bb6 100644 --- 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 *

* @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 2a5b3266..c4a3f540 100644 --- 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 e652e5e4..a1da4c48 100644 --- 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. *

* 1: http://www.mkyong.com/jsf2/jsf-2-internationalization-example/ *

diff --git a/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java b/src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java index 70e0acf1..b4df84c7 100644 --- 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 9bc322b8..f8df058d 100644 --- 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 *

- * @return fax number + * @return fax number */ DialableFaxNumber getFaxNumber (); /** * Setter for choosen fax number *

- * @param choosenFaxNumber fax number + * @param choosenFaxNumber fax number */ void setFaxNumber (final DialableFaxNumber choosenFaxNumber); /** * Getter for choosen land-line number *

- * @return land-line number + * @return land-line number */ DialableLandLineNumber getLandLineNumber (); /** * Setter for choosen land-line number *

- * @param choosenLandLineNumber land-line number + * @param choosenLandLineNumber land-line number */ void setLandLineNumber (final DialableLandLineNumber choosenLandLineNumber); /** * Getter for choosen mobile number *

- * @return mobile number + * @return mobile number */ DialableMobileNumber getMobileNumber (); /** * Setter for choosen mobile number *

- * @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 d1134bb3..19cc2bf2 100644 --- 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 19a864ce..cecc3cf3 100644 --- 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 8453b9e3..1bc058f8 100644 --- 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">

- +
@@ -46,6 +46,6 @@ - +
diff --git a/web/WEB-INF/web.xml b/web/WEB-INF/web.xml index 44167044..9b8c778f 100644 --- a/web/WEB-INF/web.xml +++ b/web/WEB-INF/web.xml @@ -2,6 +2,11 @@ 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. JLandingPage Application v1.0 + + Whether the date converter's default timezone is system's timezone. + javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE + true + Whether the multi-page registration page or a single registration page is active is_feature_user_register_multiple_page_enabled @@ -22,11 +27,6 @@ is_feature_user_resend_confirmation_link_enabled true - - Whether default timezon is set from system's timezone - javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE - true - Whether registration page is enabled. is_feature_user_registration_enabled -- 2.39.5