From 0ddc9fd36e0998a36725a187d28b8e19d98b0956 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Roland=20H=C3=A4der?= <roland@mxchange.org>
Date: Sat, 21 Jan 2023 21:33:27 +0100
Subject: [PATCH] Continued: - added checks for parameters of public method -
 tpzo fixed

---
 .../jusercore/model/user/activity/UserActivityLog.java |  2 +-
 .../comparator/UserActivityLogTimestampComparator.java | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

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);
 	}
 
-- 
2.39.5