From dd092eb279574c0a555c5eea243c328d4b97bddf Mon Sep 17 00:00:00 2001 From: Tim203 Date: Fri, 23 Apr 2021 23:41:39 +0200 Subject: [PATCH] Made combining GlobalLinking with own linking work again An instance of InjectorHolder was already bound to a child injector at the point that we request to inject GlobalPlayerLinking. This causes Guice to not create a new instance, because it was already bound somewhere in the injector. --- .../main/java/org/geysermc/floodgate/link/PlayerLinkLoader.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/src/main/java/org/geysermc/floodgate/link/PlayerLinkLoader.java b/common/src/main/java/org/geysermc/floodgate/link/PlayerLinkLoader.java index 87eba714..86a7995e 100644 --- a/common/src/main/java/org/geysermc/floodgate/link/PlayerLinkLoader.java +++ b/common/src/main/java/org/geysermc/floodgate/link/PlayerLinkLoader.java @@ -175,7 +175,7 @@ public final class PlayerLinkLoader { // we use our own internal PlayerLinking when global linking is enabled if (lConfig.isEnableGlobalLinking()) { - GlobalPlayerLinking linking = injector.getInstance(GlobalPlayerLinking.class); + GlobalPlayerLinking linking = linkInjector.getInstance(GlobalPlayerLinking.class); linking.setDatabaseImpl(instance); linking.load(); return linking;