From 977168fe209e67e5b12e9757dc537eaa005c4f2f Mon Sep 17 00:00:00 2001
From: Diamondback88 <23108617+Diamondback88@users.noreply.github.com>
Date: Thu, 19 Dec 2024 23:09:16 -0500
Subject: [PATCH] ty mojang

---
 .../modules/points/events/OnAdvancementDone.java            | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/modules/src/main/java/parallelmc/parallelutils/modules/points/events/OnAdvancementDone.java b/modules/src/main/java/parallelmc/parallelutils/modules/points/events/OnAdvancementDone.java
index d309c22e..758074bd 100644
--- a/modules/src/main/java/parallelmc/parallelutils/modules/points/events/OnAdvancementDone.java
+++ b/modules/src/main/java/parallelmc/parallelutils/modules/points/events/OnAdvancementDone.java
@@ -9,6 +9,7 @@
 import parallelmc.parallelutils.modules.parallelchat.ParallelChat;
 import parallelmc.parallelutils.modules.points.Points;
 
+import java.util.Objects;
 import java.util.logging.Level;
 
 public class OnAdvancementDone implements Listener {
@@ -21,8 +22,9 @@ public void onAdvancementDone(PlayerAdvancementDoneEvent event) {
         if (points == -1) {
             // If the advancement is a root advancement or recipes advancement, don't print an error message.
             // We can assume that advancements without displays are recipe advancements
-            // ParallelUtils.log(Level.WARNING, advancementTitle.toString());
-            if (advancement.getRoot() != advancement && advancement.getDisplay() != null) {
+            // Idk why I have to check that the displays are equal and not the advancement objects themselves, ty mojang
+            if (!(Objects.equals(advancement.getRoot().getDisplay(), advancement.getDisplay()))
+                    && advancement.getDisplay() != null) {
                 ParallelUtils.log(Level.WARNING, "Advancement " + advancement.getKey().asString() + " has no associated point value! Skipping...");
             }
             return;