From 73562b86271ef24b41eebe5b78c3083ebe11620b Mon Sep 17 00:00:00 2001
From: Roland Haeder <roland@mxchange.org>
Date: Tue, 20 Oct 2015 10:52:21 +0200
Subject: [PATCH] =?utf8?q?Continued:=20-=20rewrote=20isProfileLinkVisible(?=
 =?utf8?q?)=20to=20not=20accept=20an=20instance=20and=20use=20the=20curren?=
 =?utf8?q?t=20(in=20bean=20set)=20instance=20instead=20-=20well,=20don't?=
 =?utf8?q?=20use=20c:bla=20in=20your=20JSF=20pages=20-=20updated=20jar(s)?=
 =?utf8?q?=20Signed-off-by:Roland=20H=C3=A4der=20<roland@mxchange.org>?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 lib/jcontacts-lib.jar                         | Bin 660 -> 363 bytes
 .../profile/UserProfileWebRequestBean.java    |  47 +++++++++------
 .../UserProfileWebRequestController.java      |   6 +-
 .../templates/generic/user_profile_link.tpl   |  26 +++------
 .../login_start_sharing_addressbook.xhtml     |  55 ++++++++----------
 5 files changed, 64 insertions(+), 70 deletions(-)

diff --git a/lib/jcontacts-lib.jar b/lib/jcontacts-lib.jar
index 4bcab60630b0e9a2e35ba531a7fc870b20c03b36..811eebfffcb34d12c74b016b7ce39f88ce4471d9 100644
GIT binary patch
delta 169
zcmbQj`kIM1z?+$ci-CcIf#E@D$V6TZo4cVQ?uSLv=cY3-Fo3WK5c;}?IO=-(x#{~l
z`gyv!28ZbRx-l{cfaQTo;Pk{;V{sE+F6W}u#FEq$U8hPb1w%b^J%e~N^NAbec^R2R
hnBjI!j$^doMX_x1Vn%HdRyL4<Oh8x*q;G&Y3;-QcBCY@c

literal 660
zcmWIWW@h1HVBlb2u$dX`&VU3s8CV#6T|*poJ^kGD|D9rB2!JcK1}e4Pr17W#s2qev
zfEcdS*U`_@%{4eg&)3a2F)uSMwYWq#EVZaOGe6Ht!BEeDm&-8^NyM=rF*zeu0VoEP
zwA3@><#H}cO)N=G(RHc>skhKGh&MCS&CDxFEy_#G(M>WiRM7A*N=(j4RdCKPD#$NN
zECCv=$;-vdg>WIdXIOwP%r8pU$E|`Fsv@@{IU_MIJr$2qadf3w$@zIDiOD6!KuwHH
zBFwl06sQvj1Q^~rf@pY{L4p^d6<Yv<G=RX8Mr|Mo*NPO}_$&tn00e+6KMQ0+f&*bE
xdXRwv1Oh-h-UFE+9Z120TPrAlAONIQiV;mKa&WP-ffTX=p*~QVKM*r80049<d7J<M

diff --git a/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestBean.java b/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestBean.java
index 5e7d87c3..0869a84b 100644
--- a/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestBean.java
+++ b/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestBean.java
@@ -69,28 +69,41 @@ public class UserProfileWebRequestBean implements UserProfileWebRequestControlle
 	}
 
 	@Override
-	public boolean isProfileLinkVisible (final User user) {
+	public boolean isProfileLinkVisible () {
 		// Check on user
-		if (null == user) {
-			// Throw NPE
-			throw new NullPointerException("user is null"); //NOI18N
-		} else if (user.getUserId() == null) {
-			// Throw NPE again
-			throw new NullPointerException("user.userId is null"); //NOI18N
-		} else if (user.getUserId() < 1) {
-			// Invalid id
-			throw new IllegalArgumentException(MessageFormat.format("user.userId={0} is invalid", user.getUserId())); //NOI18N
-		} else if (user.getUserProfileMode() == null) {
-			// And another NPE ...
-			throw new NullPointerException("user.userProfileMode is null"); //NOI18N
+		if (null == this.getUser()) {
+			/*
+			 * Not set, means wrong invocation of this method as the user
+			 * instance needs to be set first.
+			 */
+			throw new NullPointerException("this.user is null"); //NOI18N
+		} else if (this.getUser().getUserId() == null) {
+			/*
+			 * If the id number is not set it means that the user instance has
+			 * not been persisted and the JPA has not been flushed. Or a
+			 * "virgin" instance (e.g. from registration) has been used.
+			 */
+			throw new NullPointerException("this.user.userId is null"); //NOI18N
+		} else if (this.getUser().getUserId() < 1) {
+			/*
+			 * The id number is set invalid for an unknown reason.
+			 */
+			throw new IllegalArgumentException(MessageFormat.format("this.user.userId={0} is invalid", this.getUser().getUserId())); //NOI18N
+		} else if (this.getUser().getUserProfileMode() == null) {
+			/*
+			 * Possibly an out-dated user profile is being used. This should not
+			 * happen.
+			 */
+			throw new NullPointerException("this.user.userProfileMode is null"); //NOI18N
 		}
 
-		// Default profile mode is not visible
-		ProfileMode profileMode = user.getUserProfileMode();
+		// Get profile mode from user instance (safe now)
+		ProfileMode profileMode = this.getUser().getUserProfileMode();
 
 		// Check all conditions (except for admin)
 		// TODO: Add admin role somehow?
-		return ((profileMode.equals(ProfileMode.PUBLIC)) || (this.loginController.isUserLoggedIn()) && (profileMode.equals(ProfileMode.MEMBERS)));
+		return ((profileMode.equals(ProfileMode.PUBLIC))
+				|| (this.loginController.isUserLoggedIn()) && (profileMode.equals(ProfileMode.MEMBERS)));
 	}
 
 	@Override
@@ -116,6 +129,6 @@ public class UserProfileWebRequestBean implements UserProfileWebRequestControlle
 		}
 
 		// Ask other method
-		return this.isProfileLinkVisible(u);
+		return this.isProfileLinkVisible();
 	}
 }
diff --git a/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestController.java b/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestController.java
index 5da13a3a..12e04ebd 100644
--- a/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestController.java
+++ b/src/java/org/mxchange/addressbook/beans/profile/UserProfileWebRequestController.java
@@ -27,13 +27,11 @@ import org.mxchange.jusercore.model.user.User;
 public interface UserProfileWebRequestController extends Serializable {
 
 	/**
-	 * Checks if the user profile link is visible
-	 * <p>
-	 * @param user User instance
+	 * Checks if the current user profile link is visible
 	 * <p>
 	 * @return Whether the profile link is visible
 	 */
-	boolean isProfileLinkVisible (final User user);
+	boolean isProfileLinkVisible ();
 
 	/**
 	 * Checks if the user profile link is visible
diff --git a/web/WEB-INF/templates/generic/user_profile_link.tpl b/web/WEB-INF/templates/generic/user_profile_link.tpl
index bef7fcd5..379c763e 100644
--- a/web/WEB-INF/templates/generic/user_profile_link.tpl
+++ b/web/WEB-INF/templates/generic/user_profile_link.tpl
@@ -3,24 +3,12 @@
 	xmlns="http://www.w3.org/1999/xhtml"
 	xmlns:f="http://java.sun.com/jsf/core"
 	xmlns:h="http://java.sun.com/jsf/html"
-	xmlns:ui="http://java.sun.com/jsf/facelets"
-	xmlns:c="http://xmlns.jcp.org/jsp/jstl/core">
+	xmlns:ui="http://java.sun.com/jsf/facelets">
 
-	<h:outputText value="#{msg.ERROR_USER_INSTANCE_NOT_SET}" rendered="#{not empty loginController.}"
-	<c:choose>
-		<c:when test="#{user == null}">
-			<h:outputText class="notice" value="#{msg.ERROR_USER_INSTANCE_NOT_SET}" />
-		</c:when>
-
-		<c:when test="#{profileController.isProfileLinkVisible(user)}">
-			<h:link id="userProfileLink" outcome="user_profile" title="#{msg.LINK_USER_PROFILE_TITLE}">
-				<h:outputText id="userName" value="#{user.userName}" />
-				<f:param name="userId" value="#{user.userId}" />
-			</h:link>
-		</c:when>
-
-		<c:otherwise>
-			<h:outputText class="notice" value="#{msg.USER_PROFILE_NOT_PUBLICLY_VISIBLE}" />
-		</c:otherwise>
-	</c:choose>
+	<h:outputText styleClass="notice" value="#{msg.ERROR_USER_INSTANCE_NOT_SET}" rendered="#{empty profileController.user}" />
+	<h:link id="userProfileLink" outcome="user_profile" title="#{msg.LINK_USER_PROFILE_TITLE}" rendered="#{not empty profileController.user and profileController.isProfileLinkVisible()}">
+		<h:outputText id="userName" value="#{profileController.user.userName}" />
+		<f:param name="userId" value="#{profileController.user.userId}" />
+	</h:link>
+	<h:outputText class="notice" value="#{msg.USER_PROFILE_NOT_PUBLICLY_VISIBLE}" />
 </ui:composition>
diff --git a/web/login/login_start_sharing_addressbook.xhtml b/web/login/login_start_sharing_addressbook.xhtml
index d9c930b2..6704fb30 100644
--- a/web/login/login_start_sharing_addressbook.xhtml
+++ b/web/login/login_start_sharing_addressbook.xhtml
@@ -22,36 +22,31 @@
 				</div>
 
 				<div class="para">
-					<c:choose>
-						<c:when test="#{loginController.isUserLoggedIn() and not loginController.isInvisible()}">
-							<h:dataTable id="userList" headerClass="table_header_column" var="user" value="#{addressbookController.allUsersNotSharing()}">
-								<h:column>
-									<f:facet name="header">#{msg.USER_ID}</f:facet>
-									<h:outputText value="#{user.userId}" />
-								</h:column>
-
-								<h:column>
-									<f:facet name="header">#{msg.USER_NAME}</f:facet>
-									<ui:include src="/WEB-INF/templates/generic/user_profile_link.tpl">
-										<ui:param name="user" value="#{user}" />
-									</ui:include>
-								</h:column>
-
-								<h:column>
-									<f:facet name="header">#{msg.LOGIN_START_SHARING_TITLE}</f:facet>
-									<h:form acceptcharset="utf-8" id="startSharing" rendered="#{profileController.isProfileLinkVisible(user)}">
-										<h:commandButton class="submit" id="submit" value="#{msg.LOGIN_START_SHARING_BUTTON}" action="#{shareController.startSharing(user, addressbookController.addressbook)}" title="#{msg.LOGIN_START_SHARING_BUTTON_TITLE}" />
-									</h:form>
-								</h:column>
-							</h:dataTable>
-						</c:when>
-						<c:when test="#{not loginController.isUserLoggedIn()}">
-								<h:outputText id="ownProfileInvible" class="notice" value="#{msg.USER_NOT_LOGGED_IN}" />
-						</c:when>
-						<c:otherwise>
-							<h:outputText id="ownProfileInvible" class="notice" value="#{msg.LOGIN_OWN_PROFILE_INVISIBLE}" />
-						</c:otherwise>
-					</c:choose>
+					<h:dataTable id="userList" headerClass="table_header_column" var="user" value="#{addressbookController.allUsersNotSharing()}" rendered="#{loginController.isUserLoggedIn() and not loginController.isInvisible()}">
+						<f:setPropertyActionListener target="#{user}" value="#{profileController.user}" />
+						<h:column>
+							<f:facet name="header">#{msg.USER_ID}</f:facet>
+							<h:outputText value="#{user.userId}" />
+						</h:column>
+
+						<h:column>
+							<f:facet name="header">#{msg.USER_NAME}</f:facet>
+							<ui:include src="/WEB-INF/templates/generic/user_profile_link.tpl">
+								<ui:param name="user" value="#{user}" />
+							</ui:include>
+						</h:column>
+
+						<h:column>
+							<f:facet name="header">#{msg.LOGIN_START_SHARING_TITLE}</f:facet>
+							<h:form acceptcharset="utf-8" id="startSharing" rendered="#{profileController.isProfileLinkVisible()}">
+								<h:commandButton class="submit" id="submit" value="#{msg.LOGIN_START_SHARING_BUTTON}" action="#{shareController.startSharing(user, addressbookController.addressbook)}" title="#{msg.LOGIN_START_SHARING_BUTTON_TITLE}" />
+							</h:form>
+						</h:column>
+					</h:dataTable>
+
+					<h:outputText id="ownProfileInvible" class="notice" value="#{msg.USER_NOT_LOGGED_IN}" rendered="#{not loginController.isUserLoggedIn()}" />
+
+					<h:outputText id="ownProfileInvible" class="notice" value="#{msg.LOGIN_OWN_PROFILE_INVISIBLE}" />
 				</div>
 
 				<div class="table_footer">
-- 
2.39.5