Please cherry-pick:

- cannot copy whole user instance to RequestScoped bean as this is being
  forgotten on next request, SessionScoped will be a little overdose + it can
  be solved by taking the user instance already set in beanHelper bean

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

diff --git a/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperBean.java b/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperBean.java
index 41f3a04f0771b88998de275f2e63cbb1e5460b5f..31da70c525c6827351666bd12d7796964f4969b4 100644 (file)
--- a/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperBean.java
+++ b/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperBean.java
@@ -147,6 +147,7 @@ public class JobsWebViewHelperBean extends BaseJobsBean implements JobsWebViewHe
         * <p>
         * @return Contact instance
         */
+       @Override
        public Contact getContact () {
                return this.contact;
        }
@@ -199,6 +200,7 @@ public class JobsWebViewHelperBean extends BaseJobsBean implements JobsWebViewHe
         * <p>
         * @return Dialable fax number
         */
+       @Override
        public DialableFaxNumber getFaxNumber () {
                return this.faxNumber;
        }
@@ -217,6 +219,7 @@ public class JobsWebViewHelperBean extends BaseJobsBean implements JobsWebViewHe
         * <p>
         * @return Dialable land-line number
         */
+       @Override
        public DialableLandLineNumber getLandLineNumber () {
                return this.landLineNumber;
        }
@@ -235,6 +238,7 @@ public class JobsWebViewHelperBean extends BaseJobsBean implements JobsWebViewHe
         * <p>
         * @return Dialable mobile number
         */
+       @Override
        public DialableMobileNumber getMobileNumber () {
                return this.mobileNumber;
        }
@@ -253,6 +257,7 @@ public class JobsWebViewHelperBean extends BaseJobsBean implements JobsWebViewHe
         * <p>
         * @return User instance
         */
+       @Override
        public User getUser () {
                return this.user;
        }

diff --git a/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperController.java b/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperController.java
index eb5cb19207f1410f871607f962c282dd073405c8..bc85fb039f701a73bb3aa01c60b8c75a27cc0f59 100644 (file)
--- a/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperController.java
+++ b/src/java/org/mxchange/jjobs/beans/helper/JobsWebViewHelperController.java
@@ -17,6 +17,11 @@
 package org.mxchange.jjobs.beans.helper;
 
 import java.io.Serializable;
+import org.mxchange.jcontacts.model.contact.Contact;
+import org.mxchange.jphone.model.phonenumbers.fax.DialableFaxNumber;
+import org.mxchange.jphone.model.phonenumbers.landline.DialableLandLineNumber;
+import org.mxchange.jphone.model.phonenumbers.mobile.DialableMobileNumber;
+import org.mxchange.jusercore.model.user.User;
 
 /**
  * An interface for general bean helper
@@ -25,4 +30,39 @@ import java.io.Serializable;
  */
 public interface JobsWebViewHelperController extends Serializable {
 
+       /**
+        * Getter for contact instance
+        * <p>
+        * @return Contact instance
+        */
+       Contact getContact ();
+
+       /**
+        * Getter for dialable fax number
+        * <p>
+        * @return Dialable fax number
+        */
+       DialableFaxNumber getFaxNumber ();
+
+       /**
+        * Getter for dialable land-line number
+        * <p>
+        * @return Dialable land-line number
+        */
+       DialableLandLineNumber getLandLineNumber ();
+
+       /**
+        * Getter for dialable mobile number
+        * <p>
+        * @return Dialable mobile number
+        */
+       DialableMobileNumber getMobileNumber ();
+
+       /**
+        * Getter for user instance
+        * <p>
+        * @return User instance
+        */
+       User getUser ();
+
 }

diff --git a/src/java/org/mxchange/jjobs/beans/user/JobsAdminUserWebRequestBean.java b/src/java/org/mxchange/jjobs/beans/user/JobsAdminUserWebRequestBean.java
index 1704a3b10d94d9eb41948b19ddb9ef0c1de88f75..c83aba9427f000f11a74cb0de8de32276a6ab2e2 100644 (file)
--- a/src/java/org/mxchange/jjobs/beans/user/JobsAdminUserWebRequestBean.java
+++ b/src/java/org/mxchange/jjobs/beans/user/JobsAdminUserWebRequestBean.java
@@ -35,10 +35,10 @@ import org.mxchange.jjobs.beans.BaseJobsBean;
 import org.mxchange.jjobs.beans.contact.JobsAdminContactWebRequestController;
 import org.mxchange.jjobs.beans.contact.JobsContactWebRequestController;
 import org.mxchange.jjobs.beans.features.JobsFeaturesWebApplicationController;
+import org.mxchange.jjobs.beans.helper.FinancialsWebViewHelperController;
 import org.mxchange.jjobs.beans.user.list.JobsUserListWebViewController;
 import org.mxchange.jusercore.events.user.add.AdminAddedUserEvent;
 import org.mxchange.jusercore.events.user.add.ObservableAdminAddedUserEvent;
-import org.mxchange.jusercore.events.user.created.ObservableCreatedUserEvent;
 import org.mxchange.jusercore.events.user.delete.AdminDeletedUserEvent;
 import org.mxchange.jusercore.events.user.delete.ObservableAdminDeletedUserEvent;
 import org.mxchange.jusercore.events.user.linked.AdminLinkedUserEvent;
@@ -99,6 +99,12 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
        @EJB (lookup = "java:global/jjobs-ejb/adminUser!org.mxchange.jusercore.model.user.AdminUserSessionBeanRemote")
        private AdminUserSessionBeanRemote adminUserBean;
 
+       /**
+        * Bean helper
+        */
+       @Inject
+       private FinancialsWebViewHelperController beanHelper;
+
        /**
         * Contact instance
         */
@@ -130,11 +136,6 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
        @Any
        private Event<ObservableAdminPostUserDataUpdatedEvent> updatedUserDataEvent;
 
-       /**
-        * User instance
-        */
-       private User user;
-
        /**
         * General user EJB
         */
@@ -242,7 +243,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
 
                // Is the user name or email address used already?
                // @TODO Add password length check
-               if ((this.featureController.isFeatureEnabled("user_login_require_user_name")) && (this.userListController.isUserNameRegistered(newUser))) {
+               if ((this.featureController.isFeatureEnabled("user_login_require_user_name")) && (userListController.isUserNameRegistered(newUser))) {
                        // User name is already used
                        throw new FacesException(new UserNameAlreadyRegisteredException(newUser));
                } else if ((this.getContact() == null) && (this.contactController.isEmailAddressRegistered(newUser.getUserContact()))) {
@@ -277,7 +278,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                // Now, that all is set, call EJB
                if (this.getContact() instanceof Contact) {
                        // Fire event
-                       this.userLinkedEvent.fire(new AdminLinkedUserEvent(updatedUser));
+                       userLinkedEvent.fire(new AdminLinkedUserEvent(updatedUser));
                } else {
                        // Fire event
                        this.addedUserEvent.fire(new AdminAddedUserEvent(updatedUser));
@@ -289,33 +290,6 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                this.clear();
        }
 
-       /**
-        * Event observer for when a bean helper has successfully created a user
-        * instance, means the user exists. If the user does not exist, this event
-        * should not fire but instead a proper exception must be thrown.
-        * <p>
-        * @param event User created event
-        */
-       public void afterCreatedUserEvent (@Observes final ObservableCreatedUserEvent event) {
-               // Is the instance valid?
-               if (null == event) {
-                       // Throw NPE
-                       throw new NullPointerException("event is null"); //NOI18N
-               } else if (event.getCreatedUser() == null) {
-                       // Throw NPE again
-                       throw new NullPointerException("event.createdUser is null"); //NOI18N
-               } else if (event.getCreatedUser().getUserId() == null) {
-                       // Throw NPE again
-                       throw new NullPointerException("event.createdUser.userId is null"); //NOI18N
-               } else if (event.getCreatedUser().getUserId() < 1) {
-                       // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("event.createdUser.userId={0} is not valid", event.getCreatedUser().getUserId())); //NOI18N
-               }
-
-               // Set whole user
-               this.setUser(event.getCreatedUser());
-       }
-
        /**
         * Event observer for new user registrations
         * <p>
@@ -349,53 +323,59 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
         * Deletes given user account
         */
        public void deleteUserData () {
+               // Get user instance
+               final User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
-               if (this.getUser() == null) {
+               if (null == user) {
                        // Throw NPE
                        throw new NullPointerException("user is null"); //NOI18N
-               } else if (this.getUser().getUserId() == null) {
+               } else if (user.getUserId() == null) {
                        // Throw again
                        throw new NullPointerException("user.userId is null"); //NOI18N
-               } else if (this.getUser().getUserId() < 1) {
+               } else if (user.getUserId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("user.userId={0} is not valid", this.getUser().getUserId())); //NOI18N
+                       throw new IllegalArgumentException(MessageFormat.format("user.userId={0} is not valid", user.getUserId())); //NOI18N
                }
 
                try {
                        // All fine, delete it
-                       this.adminUserBean.deleteUser(this.getUser(), this.getUserDeleteReason());
+                       this.adminUserBean.deleteUser(user, this.getUserDeleteReason());
                } catch (final UserNotFoundException ex) {
                        // Should not happen, so throw again
                        throw new FacesException(ex);
                }
 
                // Fire event
-               this.deleteUserEvent.fire(new AdminDeletedUserEvent(this.getUser(), this.getUserDeleteReason()));
+               this.deleteUserEvent.fire(new AdminDeletedUserEvent(user, this.getUserDeleteReason()));
        }
 
        /**
         * Edits currently loaded user's data in database.
         */
        public void editUserData () {
+               // Get user instance
+               final User user = this.beanHelper.getUser();
+
                // Null password means not setting it
                String encryptedPassword = null;
 
                // Check if user instance is in helper and valid
-               if (this.getUser() == null) {
+               if (null == user) {
                        // Throw NPE
                        throw new NullPointerException("beanHelper.user is null"); //NOI18N
-               } else if (this.getUser().getUserId() == null) {
+               } else if (user.getUserId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("beanHelper.user.userId is null"); //NOI18N
-               } else if (this.getUser().getUserId() < 1) {
+               } else if (user.getUserId() < 1) {
                        // Invalid id
-                       throw new IllegalStateException(MessageFormat.format("beanHelper.user.userId={0} is invalid", this.getUser().getUserId())); //NOI18N
+                       throw new IllegalStateException(MessageFormat.format("beanHelper.user.userId={0} is invalid", user.getUserId())); //NOI18N
                } else if (this.getUserName() == null) {
                        // Not all required fields are set
-                       throw new NullPointerException("this.userName is null"); //NOI18N
+                       throw new NullPointerException("userName is null"); //NOI18N
                } else if (this.getUserName().isEmpty()) {
                        // Not all required fields are set
-                       throw new IllegalArgumentException("this.userName is empty"); //NOI18N
+                       throw new IllegalArgumentException("userName is empty"); //NOI18N
                } else if (((!this.getUserPassword().isEmpty()) || (!this.getUserPasswordRepeat().isEmpty())) && (!this.isSamePasswordEntered())) {
                        // Clear password fields
                        this.setUserPassword(null);
@@ -404,7 +384,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                        // Not same password entered
                        this.showFacesMessage("form_edit_user:userPassword", "ADMIN_USER_PASSWORD_REPEAT_DIFFERENT", FacesMessage.SEVERITY_INFO); //NOI18N
                        return;
-               } else if ((!Objects.equals(this.getUser().getUserName(), this.getUserName())) && (this.userBean.ifUserNameExists(this.getUserName()))) {
+               } else if ((!Objects.equals(user.getUserName(), this.getUserName())) && (userBean.ifUserNameExists(this.getUserName()))) {
                        // Clear all fields
                        this.clear();
 
@@ -413,7 +393,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                        return;
                } else if (this.isSamePasswordEntered()) {
                        // Same password entered, create container
-                       if ((Objects.equals(this.getUser().getUserMustChangePassword(), this.getUserMustChangePassword())) && (UserLoginUtils.ifPasswordMatches(new UserLoginContainer(this.getUser(), this.getUserPassword())))) {
+                       if ((Objects.equals(user.getUserMustChangePassword(), this.getUserMustChangePassword())) && (UserLoginUtils.ifPasswordMatches(new UserLoginContainer(user, this.getUserPassword())))) {
                                // Clear password fields
                                this.setUserPassword(null);
                                this.setUserPasswordRepeat(null);
@@ -428,13 +408,13 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                }
 
                // Set user name and flag
-               this.getUser().setUserName(this.getUserName());
-               this.getUser().setUserMustChangePassword(this.getUserMustChangePassword());
+               user.setUserName(this.getUserName());
+               user.setUserMustChangePassword(this.getUserMustChangePassword());
 
                // Is a password set?
                if (encryptedPassword != null) {
                        // Set it as well
-                       this.getUser().setUserEncryptedPassword(encryptedPassword);
+                       user.setUserEncryptedPassword(encryptedPassword);
                }
 
                // Init updated User instance
@@ -442,7 +422,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
 
                try {
                        // Call EJB for updating user data
-                       updatedUser = this.userBean.updateUserData(this.getUser());
+                       updatedUser = this.userBean.updateUserData(user);
                } catch (final UserNotFoundException ex) {
                        // Throw as cause
                        throw new FacesException(ex);
@@ -470,24 +450,6 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                this.contact = contact;
        }
 
-       /**
-        * Getter for user instance
-        * <p>
-        * @return User instance
-        */
-       public User getUser () {
-               return this.user;
-       }
-
-       /**
-        * Setter for user instance
-        * <p>
-        * @param user User instance
-        */
-       public void setUser (final User user) {
-               this.user = user;
-       }
-
        /**
         * Getter for user delete reason
         * <p>
@@ -621,28 +583,31 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
         * @return Redirect outcome
         */
        public String lockUserAccount () {
+               // Get user instance
+               final User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
-               if (this.getUser() == null) {
+               if (null == user) {
                        // Throw NPE
-                       throw new NullPointerException("this.user is null"); //NOI18N
-               } else if (this.getUser().getUserId() == null) {
+                       throw new NullPointerException("user is null"); //NOI18N
+               } else if (user.getUserId() == null) {
                        // Throw again
-                       throw new NullPointerException("this.user.userId is null"); //NOI18N
-               } else if (this.getUser().getUserId() < 1) {
+                       throw new NullPointerException("user.userId is null"); //NOI18N
+               } else if (user.getUserId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.user.userId={0} is not valid", this.getUser().getUserId())); //NOI18N
-               } else if (this.getUser().getUserAccountStatus() == UserAccountStatus.LOCKED) {
+                       throw new IllegalArgumentException(MessageFormat.format("user.userId={0} is not valid", user.getUserId())); //NOI18N
+               } else if (user.getUserAccountStatus() == UserAccountStatus.LOCKED) {
                        // User account is locked
-                       throw new FacesException(new UserStatusLockedException(this.getUser()));
-               } else if (this.getUser().getUserAccountStatus() == UserAccountStatus.UNCONFIRMED) {
+                       throw new FacesException(new UserStatusLockedException(user));
+               } else if (user.getUserAccountStatus() == UserAccountStatus.UNCONFIRMED) {
                        // User account is locked
-                       throw new FacesException(new UserStatusUnconfirmedException(this.getUser()));
+                       throw new FacesException(new UserStatusUnconfirmedException(user));
                } else if (this.getUserLockReason() == null) {
                        // Throw NPE again
-                       throw new NullPointerException("this.userLockReason is null"); //NOI18N
+                       throw new NullPointerException("userLockReason is null"); //NOI18N
                } else if (this.getUserLockReason().isEmpty()) {
                        // Empty lock reason
-                       throw new IllegalArgumentException("this.userLockReason is empty"); //NOI18N
+                       throw new IllegalArgumentException("userLockReason is empty"); //NOI18N
                }
 
                // Init updated user instance
@@ -653,7 +618,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                        final String baseUrl = FacesUtils.generateBaseUrl();
 
                        // Call EJB to lock account
-                       updatedUser = this.adminUserBean.lockUserAccount(this.getUser(), this.getUserLockReason(), baseUrl);
+                       updatedUser = this.adminUserBean.lockUserAccount(user, this.getUserLockReason(), baseUrl);
                } catch (final UserStatusLockedException | UserStatusUnconfirmedException | UserNotFoundException ex) {
                        // Throw again
                        throw new FacesException(ex);
@@ -676,22 +641,25 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
         * @return Redirect outcome
         */
        public String unlockUserAccount () {
+               // Get user instance
+               final User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
-               if (this.getUser() == null) {
+               if (null == user) {
                        // Throw NPE
-                       throw new NullPointerException("this.user is null"); //NOI18N
-               } else if (this.getUser().getUserId() == null) {
+                       throw new NullPointerException("user is null"); //NOI18N
+               } else if (user.getUserId() == null) {
                        // Throw again
-                       throw new NullPointerException("this.user.userId is null"); //NOI18N
-               } else if (this.getUser().getUserId() < 1) {
+                       throw new NullPointerException("user.userId is null"); //NOI18N
+               } else if (user.getUserId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.user.userId={0} is not valid", this.getUser().getUserId())); //NOI18N
-               } else if (this.getUser().getUserAccountStatus() == UserAccountStatus.CONFIRMED) {
+                       throw new IllegalArgumentException(MessageFormat.format("user.userId={0} is not valid", user.getUserId())); //NOI18N
+               } else if (user.getUserAccountStatus() == UserAccountStatus.CONFIRMED) {
                        // User account is locked
-                       throw new FacesException(new UserStatusConfirmedException(this.getUser()));
-               } else if (this.getUser().getUserAccountStatus() == UserAccountStatus.UNCONFIRMED) {
+                       throw new FacesException(new UserStatusConfirmedException(user));
+               } else if (user.getUserAccountStatus() == UserAccountStatus.UNCONFIRMED) {
                        // User account is locked
-                       throw new FacesException(new UserStatusUnconfirmedException(this.getUser()));
+                       throw new FacesException(new UserStatusUnconfirmedException(user));
                }
 
                // Init updated user instance
@@ -702,7 +670,7 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                        final String baseUrl = FacesUtils.generateBaseUrl();
 
                        // Call EJB to unlock account
-                       updatedUser = this.adminUserBean.unlockUserAccount(this.getUser(), baseUrl);
+                       updatedUser = this.adminUserBean.unlockUserAccount(user, baseUrl);
                } catch (final UserStatusConfirmedException | UserStatusUnconfirmedException | UserNotFoundException ex) {
                        // Throw again
                        throw new FacesException(ex);
@@ -725,7 +693,6 @@ public class JobsAdminUserWebRequestBean extends BaseJobsBean implements JobsAdm
                // Clear all data
                this.setContact(null);
                this.setUserDeleteReason(null);
-               this.setUser(null);
                this.setUserLockReason(null);
                this.setUserMustChangePassword(null);
                this.setUserName(null);

diff --git a/web/admin/user/admin_user_lock.xhtml b/web/admin/user/admin_user_lock.xhtml
index 93a7cb4a780b04aa539ebc3196b92d936dceb7fe..a0f535f3efb504bb79a7d6c4bca106f6e8a9c543 100644 (file)
--- a/web/admin/user/admin_user_lock.xhtml
+++ b/web/admin/user/admin_user_lock.xhtml
@@ -90,7 +90,9 @@
                                                type="submit"
                                                action="#{adminUserController.lockUserAccount()}"
                                                value="#{msg.BUTTON_ADMIN_LOCK_USER_ACCOUNT}"
-                                               />
+                                               >
+                                               <f:param name="userId" value="${beanHelper.user.userId}" />
+                                       </p:commandButton>
                                </p:panelGrid>
                        </h:panelGroup>
                </h:form>