Please cherry-pick:

- added assert
- renamed variables to managed user/contact as this reflects the correct JPA state

diff --git a/src/java/org/mxchange/jusercore/model/user/JobsAdminUserSessionBean.java b/src/java/org/mxchange/jusercore/model/user/JobsAdminUserSessionBean.java
index 62677607621049c3dbb01a15579e5e5b4938f3ef..ee00ea10445692e4a717f44e6041bdf821cd3a69 100644 (file)
--- a/src/java/org/mxchange/jusercore/model/user/JobsAdminUserSessionBean.java
+++ b/src/java/org/mxchange/jusercore/model/user/JobsAdminUserSessionBean.java
@@ -139,10 +139,13 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
                }
 
                // Try to find the contact
-               Contact foundContact = this.getEntityManager().getReference(user.getUserContact().getClass(), user.getUserContact().getContactId());
+               Contact managedContact = this.getEntityManager().find(user.getUserContact().getClass(), user.getUserContact().getContactId());
+
+               // Should be found!
+               assert (managedContact instanceof Contact) : MessageFormat.format("Contact with id {0} not found, but should be.", managedContact.getContactId()); //NOI18N
 
                // Set detached object in rexcruiter instance
-               user.setUserContact(foundContact);
+               user.setUserContact(managedContact);
 
                // Set timestamp
                user.setUserCreated(new GregorianCalendar());
@@ -213,7 +216,7 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
                user.setUserLastLockedReason(userLockReason);
 
                // Update user
-               User updatedUser = this.userBean.updateUserData(user);
+               User managedUser = this.userBean.updateUserData(user);
 
                // @TODO Create user lock history entry
                // Init variable
@@ -221,7 +224,7 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
 
                try {
                        // Create email address and set
-                       emailAddress = new InternetAddress(updatedUser.getUserContact().getContactEmailAddress());
+                       emailAddress = new InternetAddress(managedUser.getUserContact().getContactEmailAddress());
                } catch (final AddressException ex) {
                        // Throw again
                        throw new EJBException(ex);
@@ -229,13 +232,13 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
 
                // Send out email
                // @TODO externalize subject line
-               this.sendEmail("Account locked", "account_locked", emailAddress, updatedUser, baseUrl); //NOI18N
+               this.sendEmail("Account locked", "account_locked", emailAddress, managedUser, baseUrl); //NOI18N
 
                // Trace message
-               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.lockUserAccount: updatedUser={1} - EXIT!", this.getClass().getSimpleName(), updatedUser)); //NOI18N
+               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.lockUserAccount: managedUser={1} - EXIT!", this.getClass().getSimpleName(), managedUser)); //NOI18N
 
                // Return detached (and updated) user
-               return updatedUser;
+               return managedUser;
        }
 
        @Override
@@ -283,7 +286,7 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
                user.setUserAccountStatus(UserAccountStatus.CONFIRMED);
 
                // Update user
-               User updatedUser = this.userBean.updateUserData(user);
+               User managedUser = this.userBean.updateUserData(user);
 
                // @TODO Create user lock history entry
                // Init variable
@@ -291,7 +294,7 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
 
                try {
                        // Create email address and set
-                       emailAddress = new InternetAddress(updatedUser.getUserContact().getContactEmailAddress());
+                       emailAddress = new InternetAddress(managedUser.getUserContact().getContactEmailAddress());
                } catch (final AddressException ex) {
                        // Throw again
                        throw new EJBException(ex);
@@ -299,13 +302,13 @@ public class JobsAdminUserSessionBean extends BaseJobsDatabaseBean implements Ad
 
                // Send out email
                // @TODO externalize subject line
-               this.sendEmail("Account unlocked", "account_unlocked", emailAddress, updatedUser, baseUrl); //NOI18N
+               this.sendEmail("Account unlocked", "account_unlocked", emailAddress, managedUser, baseUrl); //NOI18N
 
                // Trace message
-               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.lockUserAccount: updatedUser={1} - EXIT!", this.getClass().getSimpleName(), updatedUser)); //NOI18N
+               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.lockUserAccount: managedUser={1} - EXIT!", this.getClass().getSimpleName(), managedUser)); //NOI18N
 
                // Return changed account
-               return updatedUser;
+               return managedUser;
        }
 
 }