Match Advancement Packet Behavior Towards Java (#4684)

* Send advancement packet regardless of current tab * Send advancement close packet when single-advancement form closed
GeyserMC · May 22, 2024 · 6f4c29c · 6f4c29c
1 parent 96bfda2
commit 6f4c29c
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/core/src/main/java/org/geysermc/geyser/session/cache/AdvancementsCache.java b/core/src/main/java/org/geysermc/geyser/session/cache/AdvancementsCache.java
@@ -91,11 +91,9 @@ public void buildAndShowMenuForm() {
         builder.validResultHandler((response) -> {
             String id = rootAdvancementIds.get(response.clickedButtonId());
             if (!id.equals("")) {
-                if (!id.equals(currentAdvancementCategoryId)) {
-                    // Send a packet indicating that we are opening this particular advancement window
-                    ServerboundSeenAdvancementsPacket packet = new ServerboundSeenAdvancementsPacket(id);
-                    session.sendDownstreamGamePacket(packet);
-                }
+                // Send a packet indicating that we are opening this particular advancement window
+                ServerboundSeenAdvancementsPacket packet = new ServerboundSeenAdvancementsPacket(id);
+                session.sendDownstreamGamePacket(packet);
                 currentAdvancementCategoryId = id;
                 buildAndShowListForm();
             }
@@ -188,6 +186,10 @@ public void buildAndShowInfoForm(GeyserAdvancement advancement) {
                         .content(content)
                         .button(GeyserLocale.getPlayerLocaleString("gui.back", language))
                         .validResultHandler((response) -> buildAndShowListForm())
+                        .closedResultHandler(() -> {
+                            // Indicate that we have closed the current advancement tab
+                            session.sendDownstreamGamePacket(new ServerboundSeenAdvancementsPacket());
+                        })
         );
     }