From 5c8332445a92e6df96c46220aabcf48ecd1fbc07 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roland=20H=C3=A4der?= Date: Thu, 22 Sep 2022 19:20:23 +0200 Subject: [PATCH] Please cherry-pick: - registerBean is no longer used here (old, removed code) - baseUrl should never be NULL or empty - user.userContact must be valid MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Roland Häder --- .../model/user/PizzaAdminUserSessionBean.java | 6 ++++++ .../jusercore/model/user/PizzaUserSessionBean.java | 12 ++---------- 2 files changed, 8 insertions(+), 10 deletions(-) diff --git a/src/java/org/mxchange/jusercore/model/user/PizzaAdminUserSessionBean.java b/src/java/org/mxchange/jusercore/model/user/PizzaAdminUserSessionBean.java index 3e80bb0..d9d768b 100644 --- a/src/java/org/mxchange/jusercore/model/user/PizzaAdminUserSessionBean.java +++ b/src/java/org/mxchange/jusercore/model/user/PizzaAdminUserSessionBean.java @@ -70,6 +70,12 @@ public class PizzaAdminUserSessionBean extends BasePizzaEnterpriseBean implement } else if (user.getUserId() instanceof Long) { // Not allowed here throw new IllegalStateException(MessageFormat.format("user.userId must be null, is: {0}", user.getUserId())); //NOI18N + } else if (null == user.getUserContact()) { + // Abort here + throw new NullPointerException("user.contact is null"); //NOI18N + } else if (user.getUserContact().getContactId() != null) { + // Not allowed here + throw new IllegalStateException(MessageFormat.format("user.userContact.contactId must be null, is: {0}", user.getUserId())); //NOI18N } // Check if user is registered diff --git a/src/java/org/mxchange/jusercore/model/user/PizzaUserSessionBean.java b/src/java/org/mxchange/jusercore/model/user/PizzaUserSessionBean.java index 4a1980d..83bba18 100644 --- a/src/java/org/mxchange/jusercore/model/user/PizzaUserSessionBean.java +++ b/src/java/org/mxchange/jusercore/model/user/PizzaUserSessionBean.java @@ -20,7 +20,6 @@ import java.text.MessageFormat; import java.util.Date; import java.util.List; import java.util.Objects; -import javax.ejb.EJB; import javax.ejb.EJBException; import javax.ejb.Stateless; import javax.persistence.Query; @@ -39,7 +38,6 @@ import org.mxchange.jusercore.exceptions.UserStatusUnconfirmedException; import org.mxchange.jusercore.model.user.password_history.PasswordHistory; import org.mxchange.jusercore.model.user.password_history.UserPasswordHistory; import org.mxchange.jusercore.model.user.status.UserAccountStatus; -import org.mxchange.juserlogincore.model.user.register.UserRegistrationSessionBeanRemote; import org.mxchange.pizzaaplication.enterprise.BasePizzaEnterpriseBean; /** @@ -55,12 +53,6 @@ public class PizzaUserSessionBean extends BasePizzaEnterpriseBean implements Use */ private static final long serialVersionUID = 542_145_347_916L; - /** - * Registration EJB - */ - @EJB (lookup = "java:global/pizzaservice-ejb/userRegistration!org.mxchange.juserlogincore.model.user.register.UserRegistrationSessionBeanRemote") - private UserRegistrationSessionBeanRemote registerBean; - /** * Default constructor */ @@ -97,7 +89,7 @@ public class PizzaUserSessionBean extends BasePizzaEnterpriseBean implements Use // Throw it again throw new NullPointerException("baseUrl is null"); //NOI18N } else if (baseUrl.isEmpty()) { - // Invalid parameter + // Throw IAE throw new IllegalArgumentException("baseUrl is empty"); //NOI18N } @@ -366,7 +358,7 @@ public class PizzaUserSessionBean extends BasePizzaEnterpriseBean implements Use managedUser.setUserEntryUpdated(new Date()); // Create history entry - PasswordHistory entry = new UserPasswordHistory(user.getUserEncryptedPassword(), managedUser); + final PasswordHistory entry = new UserPasswordHistory(user.getUserEncryptedPassword(), managedUser); // Set created timestamp entry.setUserPasswordHistoryCreated(new Date()); -- 2.39.5