From: Roland Häder <roland@mxchange.org>
Date: Wed, 20 May 2020 16:42:59 +0000 (+0200)
Subject: Please cherry-pick:
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=7e5ad3519764af5829470ca24c8bfb9bce5fc2c6;p=jfinancials-war.git

Please cherry-pick:
- renamed userCreated -> userEntryCreated
- renamed userUpdated -> userEntryUpdated
- updateUserData() may throw a checked UserNotFoundException, let's catch it
  here and throw a FacesException with it as cause
- same with confirmAccount()

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

diff --git a/lib/jbonuscard-core.jar b/lib/jbonuscard-core.jar
new file mode 100644
index 00000000..2aa8f1a6
Binary files /dev/null and b/lib/jbonuscard-core.jar differ
diff --git a/src/java/org/mxchange/jfinancials/beans/user/FinancialsAdminUserWebRequestBean.java b/src/java/org/mxchange/jfinancials/beans/user/FinancialsAdminUserWebRequestBean.java
index f8cbb418..37af0ee8 100644
--- a/src/java/org/mxchange/jfinancials/beans/user/FinancialsAdminUserWebRequestBean.java
+++ b/src/java/org/mxchange/jfinancials/beans/user/FinancialsAdminUserWebRequestBean.java
@@ -431,8 +431,16 @@ public class FinancialsAdminUserWebRequestBean extends BaseFinancialsBean implem
 			this.getUser().setUserEncryptedPassword(encryptedPassword);
 		}
 
-		// Call EJB for updating user data
-		final User updatedUser = this.userBean.updateUserData(this.getUser());
+		// Init updated User instance
+		final User updatedUser;
+
+		try {
+			// Call EJB for updating user data
+			updatedUser = this.userBean.updateUserData(this.getUser());
+		} catch (final UserNotFoundException ex) {
+			// Throw as cause
+			throw new FacesException(ex);
+		}
 
 		// Fire event
 		this.updatedUserDataEvent.fire(new AdminPostUserDataUpdatedEvent(updatedUser));
diff --git a/src/java/org/mxchange/jfinancials/beans/user/confirmlink/FinancialsConfirmationLinkWebRequestBean.java b/src/java/org/mxchange/jfinancials/beans/user/confirmlink/FinancialsConfirmationLinkWebRequestBean.java
index ab615016..0025e5b0 100644
--- a/src/java/org/mxchange/jfinancials/beans/user/confirmlink/FinancialsConfirmationLinkWebRequestBean.java
+++ b/src/java/org/mxchange/jfinancials/beans/user/confirmlink/FinancialsConfirmationLinkWebRequestBean.java
@@ -32,6 +32,7 @@ import org.mxchange.jfinancials.beans.BaseFinancialsBean;
 import org.mxchange.jfinancials.beans.user.list.FinancialsUserListWebViewController;
 import org.mxchange.jusercore.events.user.created.CreatedUserEvent;
 import org.mxchange.jusercore.events.user.created.ObservableCreatedUserEvent;
+import org.mxchange.jusercore.exceptions.UserNotFoundException;
 import org.mxchange.jusercore.exceptions.UserStatusConfirmedException;
 import org.mxchange.jusercore.exceptions.UserStatusLockedException;
 import org.mxchange.jusercore.model.user.User;
@@ -183,7 +184,7 @@ public class FinancialsConfirmationLinkWebRequestBean extends BaseFinancialsBean
 
 			// Confirm account
 			updatedUser = this.userBean.confirmAccount(user, baseUrl);
-		} catch (final UserStatusConfirmedException | UserStatusLockedException ex) {
+		} catch (final UserStatusConfirmedException | UserStatusLockedException | UserNotFoundException ex) {
 			// Something unexpected happened
 			throw new FacesException(MessageFormat.format("Cannot confirm user account {0}", user.getUserName()), ex); //NOI18N
 		}
diff --git a/web/WEB-INF/templates/admin/user/admin_show_user_data.tpl b/web/WEB-INF/templates/admin/user/admin_show_user_data.tpl
index 28f60d9e..e114c2d9 100644
--- a/web/WEB-INF/templates/admin/user/admin_show_user_data.tpl
+++ b/web/WEB-INF/templates/admin/user/admin_show_user_data.tpl
@@ -37,17 +37,17 @@
 		</ui:fragment>
 
 		<p:column>
-			<p:outputLabel for="userCreated" styleClass="table-data-label" value="#{msg.ADMIN_USER_CREATED}" />
+			<p:outputLabel for="userEntryCreated" styleClass="table-data-label" value="#{msg.ADMIN_USER_CREATED}" />
 
-			<h:outputText id="userCreated" styleClass="table-data-field" value="#{beanHelper.user.userCreated}">
+			<h:outputText id="userEntryCreated" styleClass="table-data-field" value="#{beanHelper.user.userEntryCreated}">
 				<f:convertDateTime type="both" />
 			</h:outputText>
 		</p:column>
 
 		<p:column>
-			<p:outputLabel for="userUpdated" styleClass="table-data-label" value="#{msg.ADMIN_USER_UPDATED}" />
+			<p:outputLabel for="userEntryUpdated" styleClass="table-data-label" value="#{msg.ADMIN_USER_UPDATED}" />
 
-			<h:outputText id="userUpdated" styleClass="table-data-field" value="#{beanHelper.user.userUpdated}">
+			<h:outputText id="userEntryUpdated" styleClass="table-data-field" value="#{beanHelper.user.userEntryUpdated}">
 				<f:convertDateTime type="both" />
 			</h:outputText>
 		</p:column>
diff --git a/web/WEB-INF/templates/admin/user/admin_show_user_data_mini.tpl b/web/WEB-INF/templates/admin/user/admin_show_user_data_mini.tpl
index 23db73bd..b6a0db4d 100644
--- a/web/WEB-INF/templates/admin/user/admin_show_user_data_mini.tpl
+++ b/web/WEB-INF/templates/admin/user/admin_show_user_data_mini.tpl
@@ -46,9 +46,9 @@
 		</ui:fragment>
 
 		<p:column>
-			<p:outputLabel for="userCreated" styleClass="table-data-label" value="#{msg.ADMIN_USER_CREATED}" />
+			<p:outputLabel for="userEntryCreated" styleClass="table-data-label" value="#{msg.ADMIN_USER_CREATED}" />
 
-			<h:outputText id="userCreated" styleClass="table-data-field" value="#{beanHelper.user.userCreated}">
+			<h:outputText id="userEntryCreated" styleClass="table-data-field" value="#{beanHelper.user.userEntryCreated}">
 				<f:convertDateTime type="both" />
 			</h:outputText>
 		</p:column>
diff --git a/web/admin/user/admin_user_activity_log.xhtml b/web/admin/user/admin_user_activity_log.xhtml
index 63b7cde7..e0ec4c6d 100644
--- a/web/admin/user/admin_user_activity_log.xhtml
+++ b/web/admin/user/admin_user_activity_log.xhtml
@@ -117,7 +117,7 @@
 					<h:outputText value="#{msg.ADMIN_LIST_USER_ACTIVITY_LOG_TIMESTAMP}" />
 				</f:facet>
 
-				<h:outputText id="userCreated" value="#{userActivity.activityTimestamp}">
+				<h:outputText value="#{userActivity.activityTimestamp}">
 					<f:convertDateTime type="both" timeStyle="short" dateStyle="short" />
 				</h:outputText>
 			</p:column>
diff --git a/web/admin/user/admin_user_list.xhtml b/web/admin/user/admin_user_list.xhtml
index a3fed78d..425ae0d4 100644
--- a/web/admin/user/admin_user_list.xhtml
+++ b/web/admin/user/admin_user_list.xhtml
@@ -241,20 +241,20 @@
 
 				<p:column
 					headerText="#{msg.ENTRY_CREATED_HEADER}"
-					sortBy="#{user.userCreated}"
+					sortBy="#{user.userEntryCreated}"
 					filterable="false"
 					>
-					<h:outputText value="#{user.userCreated}">
+					<h:outputText value="#{user.userEntryCreated}">
 						<f:convertDateTime type="both" timeStyle="short" dateStyle="short" />
 					</h:outputText>
 				</p:column>
 
 				<p:column
 					headerText="#{msg.ENTRY_UPDATED_HEADER}"
-					sortBy="#{user.userUpdated}"
+					sortBy="#{user.userEntryUpdated}"
 					filterable="false"
 					>
-					<h:outputText value="#{user.userUpdated}">
+					<h:outputText value="#{user.userEntryUpdated}">
 						<f:convertDateTime type="both" timeStyle="short" dateStyle="short" />
 					</h:outputText>
 				</p:column>
@@ -358,7 +358,6 @@
 
 						<p:commandButton
 							type="submit"
-							id="submit_add_user"
 							value="#{msg.BUTTON_ADMIN_ADD_USER}"
 							action="#{adminUserController.addUser()}"
 							update="form-list-users:userList"
diff --git a/web/guest/user/user_list.xhtml b/web/guest/user/user_list.xhtml
index e602e32e..43392523 100644
--- a/web/guest/user/user_list.xhtml
+++ b/web/guest/user/user_list.xhtml
@@ -56,7 +56,7 @@
 							<h:outputText value="#{msg.USER_CREATED}" />
 						</f:facet>
 
-						<h:outputFormat id="userCreated" value="#{user.userCreated.time}" title="#{msg.USER_CREATED_TITLE}" />
+						<h:outputFormat id="userEntryCreated" value="#{user.userEntryCreated.time}" title="#{msg.USER_CREATED_TITLE}" />
 					</p:column>
 				</ui:fragment>
 			</p:dataTable>