From b68f55529b99a77695babd791f54bf41a27cd7fd Mon Sep 17 00:00:00 2001 From: Roland Haeder Date: Mon, 18 Apr 2016 21:18:42 +0200 Subject: [PATCH] Continued a bit: - Use Objects.equals() as it is more safe (NPE-free) - add updated user instance to full admin list --- .../beans/user/PizzaAdminUserWebSessionBean.java | 3 +++ .../pizzaapplication/beans/user/PizzaUserWebSessionBean.java | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebSessionBean.java index d3239f9e..37053fb4 100644 --- a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebSessionBean.java +++ b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebSessionBean.java @@ -350,6 +350,9 @@ public class PizzaAdminUserWebSessionBean implements PizzaAdminUserWebSessionCon // Fire event this.addedUserEvent.fire(new AdminUserAddedEvent(updatedUser)); + // Add user to local list + this.userList.add(updatedUser); + // Clear all this.clear(); } diff --git a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java index b568ae42..1b6bb0c2 100644 --- a/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java +++ b/src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java @@ -261,7 +261,7 @@ public class PizzaUserWebSessionBean implements PizzaUserWebSessionController { if (null == event) { // Throw NPE throw new NullPointerException("event is null"); //NOI18N - } else if (event.getRegisteredUser()== null) { + } else if (event.getRegisteredUser() == null) { // Throw NPE again throw new NullPointerException("event.user is null"); //NOI18N } else if (event.getRegisteredUser().getUserId() == null) { @@ -291,7 +291,7 @@ public class PizzaUserWebSessionBean implements PizzaUserWebSessionController { this.setUserId(registeredUser.getUserId()); // Is the account public? - if (registeredUser.getUserProfileMode().equals(ProfileMode.PUBLIC)) { + if (Objects.equals(registeredUser.getUserProfileMode(), ProfileMode.PUBLIC)) { // Also add it to this list this.visibleUserList.add(registeredUser); } -- 2.39.5