From: Roland Häder <roland@mxchange.org>
Date: Sat, 21 Jan 2023 20:33:27 +0000 (+0100)
Subject: Continued:
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=0ddc9fd36e0998a36725a187d28b8e19d98b0956;p=juser-activity-core.git

Continued:
- added checks for parameters of public method
- tpzo fixed
---

diff --git a/src/org/mxchange/jusercore/model/user/activity/UserActivityLog.java b/src/org/mxchange/jusercore/model/user/activity/UserActivityLog.java
index 09627d6..87d1b75 100644
--- a/src/org/mxchange/jusercore/model/user/activity/UserActivityLog.java
+++ b/src/org/mxchange/jusercore/model/user/activity/UserActivityLog.java
@@ -252,7 +252,7 @@ public class UserActivityLog implements LogableUserActivity {
 	 * <p>
 	 * @param activityType      Type
 	 * @param activityUser      User instance
-	 * @param activityTimestamp imestamp
+	 * @param activityTimestamp Timestamp
 	 * @param principalName     Principal name
 	 */
 	public UserActivityLog (final String activityType, final User activityUser, final Date activityTimestamp, final String principalName) {
diff --git a/src/org/mxchange/jusercore/model/user/activity/comparator/UserActivityLogTimestampComparator.java b/src/org/mxchange/jusercore/model/user/activity/comparator/UserActivityLogTimestampComparator.java
index 09fd503..0885fa1 100644
--- a/src/org/mxchange/jusercore/model/user/activity/comparator/UserActivityLogTimestampComparator.java
+++ b/src/org/mxchange/jusercore/model/user/activity/comparator/UserActivityLogTimestampComparator.java
@@ -28,6 +28,16 @@ public class UserActivityLogTimestampComparator implements Comparator<LogableUse
 
 	@Override
 	public int compare (final LogableUserActivity userActivity1, final LogableUserActivity userActivity2) {
+		// Check if both parameters are given or below statement will cause a NPE
+		if (null == userActivity1) {
+			// Throw NPE
+			throw new NullPointerException("Parameter 'userActivity1' is null"); //NOI18N
+		} else if (null == userActivity2) {
+			// Throw NPE
+			throw new NullPointerException("Parameter 'userActivity2' is null"); //NOI18N
+		}
+
+		// Check all conditions
 		return (userActivity1.getActivityTimestamp().getTime() < userActivity2.getActivityTimestamp().getTime() ? -1 : userActivity1.getActivityTimestamp().getTime() == userActivity2.getActivityTimestamp().getTime() ? 0 : 1);
 	}