From a723a7cba3cd863940ffbb6c9e658aab221e76ed Mon Sep 17 00:00:00 2001 From: "its.bread" <38575088+cyljacky02@users.noreply.github.com> Date: Fri, 5 Dec 2025 01:45:38 +0800 Subject: [PATCH] Fix MongoDB getMapBinding and clarify Javadoc (#624) --- .../java/net/william278/husksync/database/Database.java | 8 ++++---- .../net/william278/husksync/database/MongoDbDatabase.java | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/common/src/main/java/net/william278/husksync/database/Database.java b/common/src/main/java/net/william278/husksync/database/Database.java index 38e909ca..290ad9df 100644 --- a/common/src/main/java/net/william278/husksync/database/Database.java +++ b/common/src/main/java/net/william278/husksync/database/Database.java @@ -280,11 +280,11 @@ public abstract class Database { public abstract byte @Nullable [] getMapData(@NotNull String serverName, int mapId); /** - * Get a map server -> ID binding in the database + * Reverse lookup: given a local map binding, find the origin server and map ID. * - * @param serverName Name of the server the map originates from - * @param mapId Original map ID - * @return Map.Entry (key: server name, value: map ID) + * @param serverName Name of the local server (to_server_name in the binding) + * @param mapId Local map ID on this server (to_id in the binding) + * @return Map.Entry with origin server name (key) and origin map ID (value), or null if not found */ @Blocking public abstract @Nullable Map.Entry getMapBinding(@NotNull String serverName, int mapId); diff --git a/common/src/main/java/net/william278/husksync/database/MongoDbDatabase.java b/common/src/main/java/net/william278/husksync/database/MongoDbDatabase.java index 32987059..a2da42ff 100644 --- a/common/src/main/java/net/william278/husksync/database/MongoDbDatabase.java +++ b/common/src/main/java/net/william278/husksync/database/MongoDbDatabase.java @@ -399,8 +399,8 @@ public class MongoDbDatabase extends Database { final Document doc = iterable.first(); if (doc != null) { return new AbstractMap.SimpleImmutableEntry<>( - doc.getString("server_name"), - doc.getInteger("to_id") + doc.getString("from_server_name"), + doc.getInteger("from_id") ); } return null;