From 986dd1b14ec9f2a549fccec5f09ca98b0b5fd0b5 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 --- .../JobsContactWebSessionController.java | 1 + ...AdminContactPhoneWebRequestController.java | 13 ++++----- .../JobsAdminCountryWebRequestController.java | 2 +- .../JobsFeatureWebApplicationBean.java | 13 ++++----- .../JobsLocalizationSessionBean.java | 4 +-- .../phone/JobsAdminPhoneWebRequestBean.java | 6 ++-- .../JobsAdminPhoneWebRequestController.java | 12 ++++---- .../phone/JobsPhoneWebApplicationBean.java | 24 +++++++-------- .../beans/user/JobsUserWebSessionBean.java | 3 -- .../user/JobsUserWebSessionController.java | 3 +- web/WEB-INF/templates/guest/guest_menu.tpl | 4 +-- web/WEB-INF/web.xml | 29 ++++--------------- 12 files changed, 46 insertions(+), 68 deletions(-) diff --git a/src/java/org/mxchange/jjobs/beans/contact/JobsContactWebSessionController.java b/src/java/org/mxchange/jjobs/beans/contact/JobsContactWebSessionController.java index 8ad0e645..0ec3dad7 100644 --- a/src/java/org/mxchange/jjobs/beans/contact/JobsContactWebSessionController.java +++ b/src/java/org/mxchange/jjobs/beans/contact/JobsContactWebSessionController.java @@ -392,6 +392,7 @@ public interface JobsContactWebSessionController extends Serializable { * Setter for controller type *

* @param controllerType Controller type + *

* @deprecated Don't use this method. */ @Deprecated diff --git a/src/java/org/mxchange/jjobs/beans/contact/phone/JobsAdminContactPhoneWebRequestController.java b/src/java/org/mxchange/jjobs/beans/contact/phone/JobsAdminContactPhoneWebRequestController.java index 8941d0ff..05e4f607 100644 --- a/src/java/org/mxchange/jjobs/beans/contact/phone/JobsAdminContactPhoneWebRequestController.java +++ b/src/java/org/mxchange/jjobs/beans/contact/phone/JobsAdminContactPhoneWebRequestController.java @@ -51,8 +51,8 @@ public interface JobsAdminContactPhoneWebRequestController 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 JobsAdminContactPhoneWebRequestController 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 JobsAdminContactPhoneWebRequestController 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/jjobs/beans/country/JobsAdminCountryWebRequestController.java b/src/java/org/mxchange/jjobs/beans/country/JobsAdminCountryWebRequestController.java index 55866bd8..16dba6a5 100644 --- a/src/java/org/mxchange/jjobs/beans/country/JobsAdminCountryWebRequestController.java +++ b/src/java/org/mxchange/jjobs/beans/country/JobsAdminCountryWebRequestController.java @@ -102,7 +102,7 @@ public interface JobsAdminCountryWebRequestController 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/jjobs/beans/features/JobsFeatureWebApplicationBean.java b/src/java/org/mxchange/jjobs/beans/features/JobsFeatureWebApplicationBean.java index c08cef03..5a1edcee 100644 --- a/src/java/org/mxchange/jjobs/beans/features/JobsFeatureWebApplicationBean.java +++ b/src/java/org/mxchange/jjobs/beans/features/JobsFeatureWebApplicationBean.java @@ -64,18 +64,17 @@ public class JobsFeatureWebApplicationBean extends BaseJobsController implements // Default is not enabled boolean isEnabled = false; - // Try it as an NPE may come - try { - // Get value from property - String contextParameter = 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 = (Boolean.parseBoolean(contextParameter) == Boolean.TRUE); - } catch (final NullPointerException ex) { - // Ignored } - // Return value + // Return status return isEnabled; } diff --git a/src/java/org/mxchange/jjobs/beans/localization/JobsLocalizationSessionBean.java b/src/java/org/mxchange/jjobs/beans/localization/JobsLocalizationSessionBean.java index bdc63b6d..8b699463 100644 --- a/src/java/org/mxchange/jjobs/beans/localization/JobsLocalizationSessionBean.java +++ b/src/java/org/mxchange/jjobs/beans/localization/JobsLocalizationSessionBean.java @@ -28,8 +28,8 @@ import org.mxchange.jusercore.events.login.ObservableUserLoggedInEvent; import org.mxchange.jusercore.events.logout.ObservableUserLogoutEvent; /** - * 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/jjobs/beans/phone/JobsAdminPhoneWebRequestBean.java b/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestBean.java index e36220e2..07c4ec98 100644 --- a/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestBean.java +++ b/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestBean.java @@ -87,17 +87,17 @@ public class JobsAdminPhoneWebRequestBean extends BaseJobsController implements private JobsWebRequestHelperController 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/jjobs/beans/phone/JobsAdminPhoneWebRequestController.java b/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestController.java index f305b631..2b4a56e5 100644 --- a/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestController.java +++ b/src/java/org/mxchange/jjobs/beans/phone/JobsAdminPhoneWebRequestController.java @@ -126,42 +126,42 @@ public interface JobsAdminPhoneWebRequestController 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/jjobs/beans/phone/JobsPhoneWebApplicationBean.java b/src/java/org/mxchange/jjobs/beans/phone/JobsPhoneWebApplicationBean.java index 16d6130b..357d9932 100644 --- a/src/java/org/mxchange/jjobs/beans/phone/JobsPhoneWebApplicationBean.java +++ b/src/java/org/mxchange/jjobs/beans/phone/JobsPhoneWebApplicationBean.java @@ -148,7 +148,7 @@ public class JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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 JobsPhoneWebApplicationBean extends BaseJobsController implements J 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/jjobs/beans/user/JobsUserWebSessionBean.java b/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionBean.java index 6bb8bd11..bfe5eee7 100644 --- a/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionBean.java +++ b/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionBean.java @@ -530,14 +530,12 @@ public class JobsUserWebSessionBean extends BaseJobsController implements JobsUs // 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 JobsUserWebSessionBean extends BaseJobsController implements JobsUs // 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/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionController.java b/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionController.java index 087cea46..e3a64f8d 100644 --- a/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionController.java +++ b/src/java/org/mxchange/jjobs/beans/user/JobsUserWebSessionController.java @@ -96,7 +96,8 @@ public interface JobsUserWebSessionController extends Serializable { *

* @return User instance *

- * @throws UserEmailAddressNotFoundException If the user's email address is not found + * @throws UserEmailAddressNotFoundException If the user's email address is + * not found */ User lookupUserByEmailAddress (final String emailAddress) throws UserEmailAddressNotFoundException; 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 5f8b5567..8b472b45 100644 --- a/web/WEB-INF/web.xml +++ b/web/WEB-INF/web.xml @@ -7,6 +7,11 @@ javax.faces.PROJECT_STAGE Development + + 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,26 +27,6 @@ is_feature_user_resend_confirmation_link_enabled true - - Whether default timezone is set from system's timezone - javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE - true - - - Whether the guest menu is enabled/shown to the user. - is_feature_guest_menu_enabled - true - - - Whether recovery of user passwords is enabled. - is_feature_user_password_recovery_enabled - true - - - Wether the gender is required for using the general contact controller. - is_feature_general_gender_enabled - true - Whether registration page is enabled. is_feature_user_registration_enabled @@ -137,10 +122,6 @@ is_debug_register_enabled false - - javax.faces.DATETIMECONVERTER_DEFAULT_TIMEZONE_IS_SYSTEM_TIMEZONE - true - Faces Servlet javax.faces.webapp.FacesServlet -- 2.39.5