From: Roland Häder <roland@mxchange.org>
Date: Wed, 24 Aug 2016 12:27:44 +0000 (+0200)
Subject: Please cherry-pick:
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=87f0cf1ef9159cb6ada442d41e01d36816302478;p=jjobs-ejb.git

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 6267760..ee00ea1 100644
--- 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;
 	}
 
 }