From 1d451138ff6e126c87e1335655300b70145053ad Mon Sep 17 00:00:00 2001
From: =?utf8?q?Roland=20H=C3=A4der?= <roland@mxchange.org>
Date: Sat, 6 Aug 2016 09:55:12 +0200
Subject: [PATCH] Continued a bit: (please cherry-pick) - don't wrap command
 buttons in rendered="" conditions and not with ui:fragment (please name me a
 full replacement for all the other cases where I'm using it) - this makes it
 impossible for the button's action (controller method) being executed as this
 depends on the life-cycle of the JSF - it may work in viewing the JSF page
 but not when you submit it. - rewrote parameter 'user' to beanHelper.user as
 this was already being used - show proper lock/unlock link depending on
 user's account status - added navigation rule for new user lock JSF page
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: Roland Häder <roland@mxchange.org>
---
 nbproject/faces-config.NavData                | 133 +++++++++---------
 .../localization/bundle_de_DE.properties      |   1 +
 .../localization/bundle_en_US.properties      |   1 +
 web/WEB-INF/faces-config.xml                  |   4 +
 .../templates/admin/user/admin_user_links.tpl |  18 ++-
 web/admin/user/admin_user_lock.xhtml          |  10 +-
 web/admin/user/admin_user_show.xhtml          |   4 +-
 web/admin/user/admin_user_unlock.xhtml        |  10 +-
 8 files changed, 93 insertions(+), 88 deletions(-)

diff --git a/nbproject/faces-config.NavData b/nbproject/faces-config.NavData
index ea458f2c..2c9cd953 100644
--- a/nbproject/faces-config.NavData
+++ b/nbproject/faces-config.NavData
@@ -2,73 +2,74 @@
 <Scene Scope="Project" version="2">
     <Scope Scope="Faces Configuration Only"/>
     <Scope Scope="Project">
-        <Node id="admin/cellphone/admin_contact_cellphone_list.xhtml" x="400" y="600" zoom="true"/>
-        <Node id="user/login_logout.xhtml" x="1150" y="900" zoom="true"/>
-        <Node id="admin/contact/admin_contact_show.xhtml" x="400" y="1050" zoom="true"/>
-        <Node id="privacy.xhtml" x="400" y="450" zoom="true"/>
-        <Node id="admin/mobile_provider/admin_mobile_provider_show.xhtml" x="150" y="1200" zoom="true"/>
-        <Node id="guest/user/show_addressbook.xhtml" x="400" y="1350" zoom="true"/>
-        <Node id="admin/user/admin_user_show.xhtml" x="900" y="1200" zoom="true"/>
-        <Node id="admin/index.xhtml" x="650" y="450" zoom="true"/>
-        <Node id="user/login_own_addressbooks.xhtml" x="1400" y="600" zoom="true"/>
-        <Node id="user/login_user_data_saved.xhtml" x="1900" y="300" zoom="true"/>
-        <Node id="user/index.xhtml" x="900" y="900" zoom="true"/>
-        <Node id="admin/admin_logout.xhtml" x="650" y="600" zoom="true"/>
-        <Node id="user/login_data_saved.xhtml" x="900" y="600" zoom="true"/>
-        <Node id="admin/user/admin_user_unlock.xhtml" x="150" y="150" zoom="true"/>
-        <Node id="user/login_other_addressbooks.xhtml" x="150" y="750" zoom="true"/>
-        <Node id="user/login_start_sharing_addressbook.xhtml" x="900" y="750" zoom="true"/>
-        <Node id="index.xhtml" x="2650" y="150" zoom="true"/>
-        <Node id="guest/user/user_list.xhtml" x="400" y="1500" zoom="true"/>
-        <Node id="user/login_edit_user_data.xhtml" x="2400" y="300" zoom="true"/>
-        <Node id="admin/admin_category_delete.xhtml" x="1650" y="150" zoom="true"/>
-        <Node id="*" x="1650" y="600" zoom="true"/>
-        <Node id="user/login_index.xhtml" x="2400" y="150" zoom="true"/>
-        <Node id="admin/cellphone/admin_contact_cellphone_edit.xhtml" x="1900" y="450" zoom="true"/>
-        <Node id="guest/user/lost_passwd.xhtml" x="150" y="1650" zoom="true"/>
-        <Node id="guest/user/register_done.xhtml" x="650" y="1050" zoom="true"/>
-        <Node id="guest/user/show_addressbook_entries.xhtml" x="1900" y="150" zoom="true"/>
-        <Node id="user/login.xhtml" x="150" y="450" zoom="true"/>
-        <Node id="login/login_edit_user_data.xhtml" x="150" y="300" zoom="true"/>
-        <Node id="admin/cellphone/admin_contact_cellphone_unlink.xhtml" x="2150" y="150" zoom="true"/>
-        <Node id="admin/country/admin_country_list.xhtml" x="1650" y="750" zoom="true"/>
-        <Node id="guest/user/login_error.xhtml" x="2150" y="450" zoom="true"/>
-        <Node id="admin/user/admin_user_export.xhtml" x="650" y="1350" zoom="true"/>
-        <Node id="admin/mobile_provider/admin_mobile_provider_delete.xhtml" x="1150" y="450" zoom="true"/>
-        <Node id="guest/user/resend_done.xhtml" x="1150" y="750" zoom="true"/>
-        <Node id="user/login_change_password.xhtml" x="400" y="750" zoom="true"/>
-        <Node id="admin/mobile_provider/admin_mobile_provider_list.xhtml" x="150" y="900" zoom="true"/>
-        <Node id="admin/contact/admin_contact_export.xhtml" x="1650" y="450" zoom="true"/>
-        <Node id="guest/user/user_profile.xhtml" x="2900" y="150" zoom="true"/>
-        <Node id="guest/user/confirm_account.xhtml" x="1150" y="600" zoom="true"/>
-        <Node id="admin/cellphone/admin_contact_cellphone_delete.xhtml" x="650" y="300" zoom="true"/>
-        <Node id="admin/user/admin_user_delete.xhtml" x="900" y="300" zoom="true"/>
-        <Node id="exception.xhtml" x="2150" y="300" zoom="true"/>
-        <Node id="user/login_list_sharing_addressbooks.xhtml" x="900" y="1050" zoom="true"/>
-        <Node id="login/login_index.xhtml" x="150" y="1050" zoom="true"/>
-        <Node id="admin/user/admin_user_edit.xhtml" x="400" y="300" zoom="true"/>
-        <Node id="guest/user/register_page2.xhtml" x="650" y="750" zoom="true"/>
-        <Node id="terms.xhtml" x="400" y="150" zoom="true"/>
-        <Node id="admin/contact/admin_contact_edit.xhtml" x="650" y="1200" zoom="true"/>
-        <Node id="admin/contact/admin_contact_list.xhtml" x="1400" y="900" zoom="true"/>
-        <Node id="admin/user/admin_user_list.xhtml" x="150" y="1500" zoom="true"/>
+        <Node id="admin/cellphone/admin_contact_cellphone_list.xhtml" x="650" y="1350" zoom="true"/>
+        <Node id="user/login_logout.xhtml" x="2900" y="150" zoom="true"/>
+        <Node id="admin/contact/admin_contact_show.xhtml" x="1150" y="1050" zoom="true"/>
+        <Node id="privacy.xhtml" x="1900" y="450" zoom="true"/>
+        <Node id="admin/mobile_provider/admin_mobile_provider_show.xhtml" x="400" y="600" zoom="true"/>
+        <Node id="guest/user/show_addressbook.xhtml" x="2150" y="450" zoom="true"/>
+        <Node id="admin/user/admin_user_show.xhtml" x="2150" y="300" zoom="true"/>
+        <Node id="admin/index.xhtml" x="650" y="150" zoom="true"/>
+        <Node id="user/login_own_addressbooks.xhtml" x="650" y="600" zoom="true"/>
+        <Node id="user/login_user_data_saved.xhtml" x="650" y="900" zoom="true"/>
+        <Node id="user/index.xhtml" x="400" y="300" zoom="true"/>
+        <Node id="admin/admin_logout.xhtml" x="900" y="600" zoom="true"/>
+        <Node id="user/login_data_saved.xhtml" x="1400" y="150" zoom="true"/>
+        <Node id="admin/user/admin_user_unlock.xhtml" x="1400" y="450" zoom="true"/>
+        <Node id="user/login_other_addressbooks.xhtml" x="1650" y="750" zoom="true"/>
+        <Node id="user/login_start_sharing_addressbook.xhtml" x="150" y="1350" zoom="true"/>
+        <Node id="guest/user/user_list.xhtml" x="150" y="1050" zoom="true"/>
+        <Node id="index.xhtml" x="650" y="1200" zoom="true"/>
+        <Node id="user/login_edit_user_data.xhtml" x="1150" y="750" zoom="true"/>
+        <Node id="*" x="900" y="150" zoom="true"/>
+        <Node id="admin/admin_category_delete.xhtml" x="400" y="1500" zoom="true"/>
+        <Node id="user/login_index.xhtml" x="400" y="450" zoom="true"/>
+        <Node id="admin/cellphone/admin_contact_cellphone_edit.xhtml" x="1650" y="300" zoom="true"/>
+        <Node id="guest/user/lost_passwd.xhtml" x="1900" y="600" zoom="true"/>
+        <Node id="guest/user/register_done.xhtml" x="400" y="750" zoom="true"/>
+        <Node id="guest/user/show_addressbook_entries.xhtml" x="150" y="1650" zoom="true"/>
+        <Node id="user/login.xhtml" x="900" y="450" zoom="true"/>
+        <Node id="login/login_edit_user_data.xhtml" x="400" y="150" zoom="true"/>
+        <Node id="admin/country/admin_country_list.xhtml" x="1150" y="150" zoom="true"/>
+        <Node id="admin/cellphone/admin_contact_cellphone_unlink.xhtml" x="900" y="300" zoom="true"/>
+        <Node id="guest/user/login_error.xhtml" x="400" y="1200" zoom="true"/>
+        <Node id="admin/user/admin_user_export.xhtml" x="1900" y="300" zoom="true"/>
+        <Node id="admin/mobile_provider/admin_mobile_provider_delete.xhtml" x="400" y="1050" zoom="true"/>
+        <Node id="guest/user/resend_done.xhtml" x="650" y="300" zoom="true"/>
+        <Node id="user/login_change_password.xhtml" x="650" y="450" zoom="true"/>
+        <Node id="admin/mobile_provider/admin_mobile_provider_list.xhtml" x="900" y="900" zoom="true"/>
+        <Node id="admin/contact/admin_contact_export.xhtml" x="150" y="600" zoom="true"/>
+        <Node id="guest/user/user_profile.xhtml" x="900" y="750" zoom="true"/>
+        <Node id="guest/user/confirm_account.xhtml" x="1400" y="900" zoom="true"/>
+        <Node id="admin/user/admin_user_delete.xhtml" x="150" y="450" zoom="true"/>
+        <Node id="exception.xhtml" x="150" y="1200" zoom="true"/>
+        <Node id="admin/cellphone/admin_contact_cellphone_delete.xhtml" x="1150" y="900" zoom="true"/>
+        <Node id="user/login_list_sharing_addressbooks.xhtml" x="150" y="1500" zoom="true"/>
+        <Node id="login/login_index.xhtml" x="1650" y="150" zoom="true"/>
+        <Node id="admin/user/admin_user_edit.xhtml" x="650" y="1050" zoom="true"/>
+        <Node id="guest/user/register_page2.xhtml" x="2400" y="300" zoom="true"/>
+        <Node id="terms.xhtml" x="150" y="900" zoom="true"/>
+        <Node id="admin/contact/admin_contact_list.xhtml" x="650" y="750" zoom="true"/>
+        <Node id="admin/contact/admin_contact_edit.xhtml" x="1400" y="750" zoom="true"/>
+        <Node id="admin/user/admin_user_lock.xhtml" x="2650" y="300" zoom="true"/>
+        <Node id="admin/user/admin_user_list.xhtml" x="150" y="750" zoom="true"/>
         <Node id="admin/country/admin_country_delete.xhtml" x="1150" y="300" zoom="true"/>
-        <Node id="user/login_change_personal_data.xhtml" x="1150" y="150" zoom="true"/>
-        <Node id="logout.xhtml" x="150" y="600" zoom="true"/>
-        <Node id="admin/country/admin_country_edit.xhtml" x="1400" y="750" zoom="true"/>
-        <Node id="user/login_add_addressbook.xhtml" x="1400" y="150" zoom="true"/>
-        <Node id="guest/user/resend_link.xhtml" x="400" y="1200" zoom="true"/>
-        <Node id="user/login_contact_data_saved.xhtml" x="900" y="450" zoom="true"/>
-        <Node id="admin/contact/admin_contact_delete.xhtml" x="1400" y="300" zoom="true"/>
-        <Node id="user/user_profile.xhtml" x="1400" y="450" zoom="true"/>
-        <Node id="imprint.xhtml" x="900" y="150" zoom="true"/>
-        <Node id="user/login_shared_addressbooks.xhtml" x="650" y="150" zoom="true"/>
-        <Node id="admin/mobile_provider/admin_mobile_provider_edit.xhtml" x="650" y="900" zoom="true"/>
-        <Node id="admin/cellphone/admin_contact_cellphone_show.xhtml" x="1650" y="300" zoom="true"/>
-        <Node id="guest/user/login.xhtml" x="150" y="1350" zoom="true"/>
-        <Node id="admin/admin_product_delete.xhtml" x="1900" y="600" zoom="true"/>
-        <Node id="guest/user/register.xhtml" x="1150" y="1050" zoom="true"/>
-        <Node id="user/login_change_email_address.xhtml" x="400" y="900" zoom="true"/>
+        <Node id="user/login_change_personal_data.xhtml" x="2650" y="150" zoom="true"/>
+        <Node id="logout.xhtml" x="1650" y="600" zoom="true"/>
+        <Node id="admin/country/admin_country_edit.xhtml" x="900" y="1050" zoom="true"/>
+        <Node id="user/login_add_addressbook.xhtml" x="150" y="150" zoom="true"/>
+        <Node id="guest/user/resend_link.xhtml" x="2400" y="150" zoom="true"/>
+        <Node id="user/login_contact_data_saved.xhtml" x="1400" y="300" zoom="true"/>
+        <Node id="user/user_profile.xhtml" x="400" y="900" zoom="true"/>
+        <Node id="admin/contact/admin_contact_delete.xhtml" x="1400" y="600" zoom="true"/>
+        <Node id="imprint.xhtml" x="400" y="1350" zoom="true"/>
+        <Node id="user/login_shared_addressbooks.xhtml" x="1650" y="450" zoom="true"/>
+        <Node id="admin/mobile_provider/admin_mobile_provider_edit.xhtml" x="150" y="300" zoom="true"/>
+        <Node id="admin/cellphone/admin_contact_cellphone_show.xhtml" x="2150" y="150" zoom="true"/>
+        <Node id="guest/user/login.xhtml" x="1150" y="450" zoom="true"/>
+        <Node id="admin/admin_product_delete.xhtml" x="1150" y="600" zoom="true"/>
+        <Node id="guest/user/register.xhtml" x="900" y="1200" zoom="true"/>
+        <Node id="user/login_change_email_address.xhtml" x="1900" y="150" zoom="true"/>
     </Scope>
     <Scope Scope="All Faces Configurations"/>
 </Scene>
diff --git a/src/java/org/mxchange/localization/bundle_de_DE.properties b/src/java/org/mxchange/localization/bundle_de_DE.properties
index 38a48a43..5dc86c49 100644
--- a/src/java/org/mxchange/localization/bundle_de_DE.properties
+++ b/src/java/org/mxchange/localization/bundle_de_DE.properties
@@ -630,3 +630,4 @@ BUTTON_ADMIN_UNLOCK_USER_ACCOUNT=Benutzeraccount entsperren
 ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED=Unbest\u00e4tigte Benutzeraccounts k\u00f6nnen nicht gesperrt werden.
 PAGE_TITLE_ADMIN_LOCK_USER=Benutzeraccount sperren
 CONTENT_TITLE_ADMIN_LOCK_USER=Benutzeraccount sperren:
+ERROR_BEAN_HELPER_USER_NOT_SET=Fehler: Instanz 'user' im Bean-Helper nicht gesetzt.
diff --git a/src/java/org/mxchange/localization/bundle_en_US.properties b/src/java/org/mxchange/localization/bundle_en_US.properties
index 6d95931b..cd9dee42 100644
--- a/src/java/org/mxchange/localization/bundle_en_US.properties
+++ b/src/java/org/mxchange/localization/bundle_en_US.properties
@@ -627,3 +627,4 @@ BUTTON_ADMIN_UNLOCK_USER_ACCOUNT=Unlock user account
 ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED=Unconfirmed user accounts cannot be locked.
 PAGE_TITLE_ADMIN_LOCK_USER=Lock user account
 CONTENT_TITLE_ADMIN_LOCK_USER=Lock user account:
+ERROR_BEAN_HELPER_USER_NOT_SET=Error: Instance 'user' not set in bean helper.
diff --git a/web/WEB-INF/faces-config.xml b/web/WEB-INF/faces-config.xml
index 6d3fcbd0..5f435f73 100644
--- a/web/WEB-INF/faces-config.xml
+++ b/web/WEB-INF/faces-config.xml
@@ -275,6 +275,10 @@
 			<from-outcome>admin_delete_user</from-outcome>
 			<to-view-id>/admin/user/admin_user_delete.xhtml</to-view-id>
 		</navigation-case>
+		<navigation-case>
+			<from-outcome>admin_lock_user</from-outcome>
+			<to-view-id>/admin/user/admin_user_lock.xhtml</to-view-id>
+		</navigation-case>
 		<navigation-case>
 			<from-outcome>admin_unlock_user</from-outcome>
 			<to-view-id>/admin/user/admin_user_unlock.xhtml</to-view-id>
diff --git a/web/WEB-INF/templates/admin/user/admin_user_links.tpl b/web/WEB-INF/templates/admin/user/admin_user_links.tpl
index 0c66c586..ef30a769 100644
--- a/web/WEB-INF/templates/admin/user/admin_user_links.tpl
+++ b/web/WEB-INF/templates/admin/user/admin_user_links.tpl
@@ -5,9 +5,9 @@
 	xmlns:h="http://java.sun.com/jsf/html"
 	xmlns:ui="http://xmlns.jcp.org/jsf/facelets">
 
-	<h:outputText styleClass="errors" value="#{msg.ERROR_PARAMETER_USER_NOT_SET}" rendered="#{empty user}" />
+	<h:outputText styleClass="errors" value="#{msg.ERROR_BEAN_HELPER_USER_NOT_SET}" rendered="#{empty beanHelper.user}" />
 
-	<ui:fragment rendered="#{not empty user}">
+	<ui:fragment rendered="#{not empty beanHelper.user}">
 		<div class="table">
 			<div class="table_header">
 				<h:outputText value="#{msg.ADMIN_LINKS_HEADER}" />
@@ -17,26 +17,30 @@
 				<ul>
 					<li>
 						<h:link outcome="admin_edit_user" value="#{msg.ADMIN_LINK_EDIT_USER}" title="#{msg.ADMIN_LINK_EDIT_USER_TITLE}">
-							<f:param name="userId" value="#{user.userId}" />
+							<f:param name="userId" value="#{beanHelper.user.userId}" />
 						</h:link>
 					</li>
 
 					<li>
 						<h:link outcome="admin_edit_contact" value="#{msg.ADMIN_LINK_EDIT_CONTACT}" title="#{msg.ADMIN_LINK_EDIT_CONTACT_TITLE}">
-							<f:param name="contactId" value="#{user.userContact.contactId}" />
+							<f:param name="contactId" value="#{beanHelper.user.userContact.contactId}" />
 						</h:link>
 					</li>
 
 					<li>
-						<h:link outcome="admin_unlock_user" value="#{msg.ADMIN_LINK_UNLOCK_USER}" title="#{msg.ADMIN_LINK_UNLOCK_USER_TITLE}">
-							<f:param name="userId" value="#{user.userId}" />
+						<h:link outcome="admin_unlock_user" value="#{msg.ADMIN_LINK_UNLOCK_USER}" title="#{msg.ADMIN_LINK_UNLOCK_USER_TITLE}" rendered="#{beanHelper.user.userAccountStatus == 'LOCKED'}">
+							<f:param name="userId" value="#{beanHelper.user.userId}" />
+						</h:link>
+
+						<h:link outcome="admin_lock_user" value="#{msg.ADMIN_LINK_LOCK_USER}" title="#{msg.ADMIN_LINK_LOCK_USER_TITLE}" rendered="#{beanHelper.user.userAccountStatus == 'CONFIRMED'}">
+							<f:param name="userId" value="#{beanHelper.user.userId}" />
 						</h:link>
 					</li>
 
 					<li>
 						<h:link outcome="admin_delete_user" title="#{msg.ADMIN_LINK_DELETE_USER_TITLE}">
 							<h:outputText styleClass="delete_link" value="#{msg.ADMIN_LINK_DELETE_USER}" />
-							<f:param name="userId" value="#{user.userId}" />
+							<f:param name="userId" value="#{beanHelper.user.userId}" />
 						</h:link>
 					</li>
 				</ul>
diff --git a/web/admin/user/admin_user_lock.xhtml b/web/admin/user/admin_user_lock.xhtml
index f08c7d82..e074bcb7 100644
--- a/web/admin/user/admin_user_lock.xhtml
+++ b/web/admin/user/admin_user_lock.xhtml
@@ -23,7 +23,7 @@
 		<ui:define name="content">
 			<h:outputText styleClass="errors" value="#{msg.ERROR_USER_ID_NOT_FOUND}" rendered="#{empty beanHelper.user}" />
 
-			<h:form id="form_admin_user_unlock" rendered="#{not empty beanHelper.user}">
+			<h:form id="form_admin_user_unlock" rendered="#{not empty beanHelper.user and beanHelper.user.userAccountStatus == 'CONFIRMED'}">
 				<div class="table_medium">
 					<div class="table_header">
 						<h:outputFormat value="#{msg.ADMIN_LOCK_USER_TITLE}">
@@ -77,14 +77,12 @@
 					<div class="table_footer">
 						<h:commandButton styleClass="reset" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
 
-						<ui:fragment rendered="#{beanHelper.user.userAccountStatus == 'CONFIRMED'}">
-							<h:commandButton styleClass="submit" type="submit" action="#{adminUserController.lockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_LOCK_USER_ACCOUNT}" />
-						</ui:fragment>
-
-						<h:outputText styleClass="errors" value="#{msg.ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED}" rendered="#{beanHelper.user.userAccountStatus == 'UNCONFIRMED'}" />
+						<h:commandButton styleClass="submit" type="submit" action="#{adminUserController.lockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_LOCK_USER_ACCOUNT}" />
 					</div>
 				</div>
 			</h:form>
+
+			<h:outputText styleClass="errors" value="#{msg.ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED}" rendered="#{beanHelper.user.userAccountStatus == 'UNCONFIRMED'}" />
 		</ui:define>
 	</ui:composition>
 </html>
diff --git a/web/admin/user/admin_user_show.xhtml b/web/admin/user/admin_user_show.xhtml
index 17df6277..f9c9b9cd 100644
--- a/web/admin/user/admin_user_show.xhtml
+++ b/web/admin/user/admin_user_show.xhtml
@@ -180,9 +180,7 @@
 			</h:panelGrid>
 
 			<div>
-				<ui:include src="/WEB-INF/templates/admin/user/admin_user_links.tpl">
-					<ui:param name="user" value="#{beanHelper.user}" />
-				</ui:include>
+				<ui:include src="/WEB-INF/templates/admin/user/admin_user_links.tpl" />
 			</div>
 
 			<div>
diff --git a/web/admin/user/admin_user_unlock.xhtml b/web/admin/user/admin_user_unlock.xhtml
index 8c9e71f0..fe8dc182 100644
--- a/web/admin/user/admin_user_unlock.xhtml
+++ b/web/admin/user/admin_user_unlock.xhtml
@@ -22,7 +22,7 @@
 		<ui:define name="content">
 			<h:outputText styleClass="errors" value="#{msg.ERROR_USER_ID_NOT_FOUND}" rendered="#{empty beanHelper.user}" />
 
-			<h:form id="form_admin_user_unlock" rendered="#{not empty beanHelper.user}">
+			<h:form id="form_admin_user_unlock" rendered="#{not empty beanHelper.user and beanHelper.user.userAccountStatus == 'LOCKED'}">
 				<div class="table_medium">
 					<div class="table_header">
 						<h:outputFormat value="#{msg.ADMIN_UNLOCK_USER_TITLE}">
@@ -76,14 +76,12 @@
 					<div class="table_footer">
 						<h:commandButton styleClass="reset" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
 
-						<ui:fragment rendered="#{beanHelper.user.userAccountStatus == 'LOCKED'}">
-							<h:commandButton styleClass="submit" type="submit" action="#{adminUserController.unlockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_UNLOCK_USER_ACCOUNT}" />
-						</ui:fragment>
-
-						<h:outputText styleClass="errors" value="#{msg.ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED}" rendered="#{beanHelper.user.userAccountStatus == 'UNCONFIRMED'}" />
+						<h:commandButton styleClass="submit" type="submit" action="#{adminUserController.unlockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_UNLOCK_USER_ACCOUNT}" />
 					</div>
 				</div>
 			</h:form>
+
+			<h:outputText styleClass="errors" value="#{msg.ERROR_CANNOT_UN_LOCK_USER_ACCOUNT_UNCONFIRMED}" rendered="#{beanHelper.user.userAccountStatus == 'UNCONFIRMED'}" />
 		</ui:define>
 	</ui:composition>
 </html>
-- 
2.39.5