X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2Fjava%2Forg%2Fmxchange%2Fpizzaapplication%2Fbeans%2Fuser%2FPizzaUserWebRequestBean.java;h=cbde14a2ec7f1b2a1646f21f981d9cf3463b14ac;hb=aa445e02ba07fa1e2ff0bcc030b6a32a550d075a;hp=a86c718694a0cc8c2dd92d5c9a320f00a1d287c8;hpb=8fd877f0aeabd727ec35ec9f88738b134a324613;p=pizzaservice-war.git diff --git a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebRequestBean.java b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebRequestBean.java index a86c7186..cbde14a2 100644 --- a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebRequestBean.java +++ b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebRequestBean.java @@ -1,5 +1,5 @@ /* - * Copyright (C) 2016, 2017 Roland Häder + * Copyright (C) 2016, 2020 Free Software Foundation * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Affero General Public License as @@ -59,7 +59,7 @@ import org.mxchange.juserlogincore.events.registration.ObservableUserRegisteredE import org.mxchange.juserlogincore.events.user.password_change.ObservableUpdatedUserPasswordEvent; import org.mxchange.juserlogincore.exceptions.UserPasswordMismatchException; import org.mxchange.juserlogincore.login.UserLoginUtils; -import org.mxchange.pizzaapplication.beans.BasePizzaController; +import org.mxchange.pizzaapplication.beans.BasePizzaBean; import org.mxchange.pizzaapplication.beans.contact.PizzaContactWebRequestController; import org.mxchange.pizzaapplication.beans.features.PizzaFeaturesWebApplicationController; import org.mxchange.pizzaapplication.beans.localization.PizzaLocalizationSessionController; @@ -72,7 +72,7 @@ import org.mxchange.pizzaapplication.beans.user.login.PizzaUserLoginWebSessionCo */ @Named ("userController") @RequestScoped -public class PizzaUserWebRequestBean extends BasePizzaController implements PizzaUserWebRequestController { +public class PizzaUserWebRequestBean extends BasePizzaBean implements PizzaUserWebRequestController { /** * Serial number @@ -112,15 +112,15 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz /** * Remote user bean */ - @EJB (lookup = "java:global/jfinancials-ejb/user!org.mxchange.jusercore.model.user.UserSessionBeanRemote") + @EJB (lookup = "java:global/pizzaservice-ejb/user!org.mxchange.jusercore.model.user.UserSessionBeanRemote") private UserSessionBeanRemote userBean; /** * A list of all user profiles */ @Inject - @NamedCache (cacheName = "userCache", managementEnabled = true) - private transient Cache userCache; + @NamedCache (cacheName = "userCache") + private Cache userCache; /** * User id @@ -142,8 +142,8 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz * User name list */ @Inject - @NamedCache (cacheName = "userNameCache", managementEnabled = true) - private transient Cache userNameCache; + @NamedCache (cacheName = "userNameCache") + private Cache userNameCache; /** * User password (clear-text from web form) @@ -360,7 +360,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz } // Get user instance - User user = event.getCreatedUser(); + final User user = event.getCreatedUser(); // Set all fields here this.copyUser(user); @@ -433,7 +433,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz } // "Cache" user instance - User loggedInUser = event.getLoggedInUser(); + final User loggedInUser = event.getLoggedInUser(); // Copy all data to this bean this.copyUser(loggedInUser); @@ -484,7 +484,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz } // Get user instance - User registeredUser = event.getRegisteredUser(); + final User registeredUser = event.getRegisteredUser(); // Copy all data from registered->user this.copyUser(registeredUser); @@ -557,10 +557,10 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz @SuppressWarnings ("ReturnOfCollectionOrArrayField") public List allUsers () { // Init list - List list = new LinkedList<>(); + final List list = new LinkedList<>(); // Get iterator - Iterator> iterator = this.userCache.iterator(); + final Iterator> iterator = this.userCache.iterator(); // Loop over all while (iterator.hasNext()) { @@ -609,14 +609,11 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz @Override public User createUserInstance (final boolean createContactData) { - // Trace message - //* NOISY-DEBUG: */ System.out.println(MessageFormat.format("{0}.createUserInstance: CALLED!", this.getClass().getSimpleName())); - // Required personal data must be set assert (this.isRequiredPersonalDataSet()) : "not all personal data is set"; //NOI18N // Create new user instance - User user = new LoginUser(); + final User user = new LoginUser(); // Is user name required? if (!this.isUserNameRequired()) { @@ -643,25 +640,18 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz // Is multiple registration page if ((createContactData) || (!this.featureController.isFeatureEnabled("user_register_multiple_page"))) { //NOI18N // Create contact instance - Contact contact = this.contactController.createContactInstance(); + final Contact contact = this.contactController.createContactInstance(); - // 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; } @Override public User createUserLogin () { - // Trace message - //* NOISY-DEBUG */ System.out.println(MessageFormat.format("{0}.createUserLogin: CALLED!", this.getClass().getSimpleName())); - // Is all data set? if (this.getUserName() == null) { // Throw NPE @@ -672,13 +662,11 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz } // Create new user instance - User user = new LoginUser(); + final User user = new LoginUser(); // Update all data ... user.setUserName(this.getUserName()); - // Trace message - //* NOISY-DEBUG */ System.out.println(MessageFormat.format("{0}.createUserLogin: user={1} - EXIT!", this.getClass().getSimpleName(), user)); // Return the new instance return user; } @@ -701,7 +689,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz } // Get user instance - User user = this.userLoginController.getLoggedInUser(); + final User user = this.userLoginController.getLoggedInUser(); // Copy contact data to contact instance this.contactController.updateContactDataFromController(user.getUserContact()); @@ -718,7 +706,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz user.setUserProfileMode(this.getUserProfileMode()); // Send it to the EJB - User updatedUser = this.userBean.updateUserPersonalData(user); + final User updatedUser = this.userBean.updateUserPersonalData(user); // Fire event this.updatedPersonalDataEvent.fire(new UpdatedUserPersonalDataEvent(updatedUser)); @@ -829,7 +817,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz // Is cache there? if (!this.userCache.iterator().hasNext()) { // Get whole list - List list = this.userBean.allUsers(); + final List list = this.userBean.allUsers(); // Add all for (final Iterator iterator = list.iterator(); iterator.hasNext();) { @@ -861,12 +849,12 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz boolean isFound = false; // Get iterator - Iterator iterator = this.allUsers().iterator(); + final Iterator iterator = this.allUsers().iterator(); // Loop through all entries while (iterator.hasNext()) { // Get user - User next = iterator.next(); + final User next = iterator.next(); // Compare both objects if (Objects.equals(contact, next.getUserContact())) { @@ -883,10 +871,10 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz @Override public boolean isPublicUserProfileEnabled () { // Get context parameter - String contextParameter = FacesContext.getCurrentInstance().getExternalContext().getInitParameter("is_public_profile_enabled"); //NOI18N + final String contextParameter = FacesContext.getCurrentInstance().getExternalContext().getInitParameter("is_public_profile_enabled"); //NOI18N // Is it set? - boolean isEnabled = ((contextParameter instanceof String) && (contextParameter.toLowerCase().equals("true"))); //NOI18N + final boolean isEnabled = ((contextParameter instanceof String) && (contextParameter.toLowerCase().equals("true"))); //NOI18N // This requires user names being enabled, too. if ((isEnabled) && (!this.isUserNameRequired())) { @@ -938,10 +926,10 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz @Override public boolean isUserNameRequired () { // Get context parameter - String contextParameter = FacesContext.getCurrentInstance().getExternalContext().getInitParameter("is_user_login_require_user_name"); //NOI18N + final String contextParameter = FacesContext.getCurrentInstance().getExternalContext().getInitParameter("is_user_login_require_user_name"); //NOI18N // Is it set? - boolean isRequired = ((contextParameter instanceof String) && (contextParameter.toLowerCase().equals("true"))); //NOI18N + final boolean isRequired = ((contextParameter instanceof String) && (contextParameter.toLowerCase().equals("true"))); //NOI18N // Return value return isRequired; @@ -964,7 +952,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz // Try to lookup it in visible user list for (final Iterator> iterator = this.userCache.iterator(); iterator.hasNext();) { // Get next user - Cache.Entry next = iterator.next(); + final Cache.Entry next = iterator.next(); // Contact should be set if (next.getValue().getUserContact() == null) { @@ -1010,7 +998,7 @@ public class PizzaUserWebRequestBean extends BasePizzaController implements Pizz // Try to lookup it in visible user list for (final Iterator> iterator = this.userCache.iterator(); iterator.hasNext();) { // Get next user - Cache.Entry next = iterator.next(); + final Cache.Entry next = iterator.next(); // Is the user id found? if (Objects.equals(next.getKey(), userId)) {