Merge pull request #23 from zmanowar/old-profile-lastspawned-fix

zmanowar · web-flow · commit 65b9c0fb1778 · 2024-03-02T11:41:04.000-05:00
Returns Instant.now instead of null when old profiles are used
diff --git a/src/main/java/com/randomEventAnalytics/RandomEventAnalyticsPlugin.java b/src/main/java/com/randomEventAnalytics/RandomEventAnalyticsPlugin.java
@@ -211,7 +211,9 @@ private Instant getLastRandomSpawnInstant()
 		RandomEventRecord record = localStorage.getMostRecentRandom();
 		if (record.spawnedTime < 0)
 		{
-			return null;
+			// This assumes the profile failed to load. eg. malformed data, old profile that never had a random spawn, etc.
+			// we'll reset the timer.
+			return Instant.now();
 		}
 
 		return Instant.ofEpochMilli(record.spawnedTime);

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,9 @@ private Instant getLastRandomSpawnInstant()`
`211`	`211`	`RandomEventRecord record = localStorage.getMostRecentRandom();`
`212`	`212`	`if (record.spawnedTime < 0)`
`213`	`213`	`{`
`214`		`- return null;`
	`214`	`+ // This assumes the profile failed to load. eg. malformed data, old profile that never had a random spawn, etc.`
	`215`	`+ // we'll reset the timer.`
	`216`	`+ return Instant.now();`
`215`	`217`	`}`
`216`	`218`
`217`	`219`	`return Instant.ofEpochMilli(record.spawnedTime);`