Skip to content

Commit 3975fd8

Browse files
committed
Refactor MessageChannelDeliver to use instance variables for serviceRegistry and playerManager; update NavigationHelper to correct item assignments
1 parent afed486 commit 3975fd8

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

app/src/main/java/net/onelitefeather/titan/app/helper/NavigationHelper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,8 @@ private PersonalInventoryBuilder createPersonalInventoryBuilder(Player player) {
6161
finalLayout.setItems(LayoutCalculator.fillRow(InventoryType.CHEST_1_ROW), Items.NAVIGATOR_BLANK_ITEM_STACK);
6262
ThreadLocalUserProvider.bind(toUser(player));
6363
finalLayout.setItem(0, Items.NAVIGATOR_ELYTRA_ITEM_STACK, this::clickElytra);
64-
finalLayout.setItem(4, Items.NAVIGATOR_SLENDER_ITEM_STACK, this::clickSlender);
65-
finalLayout.setItem(5, Items.NAVIGATOR_SLENDER_ITEM_STACK, this::clickSurvival);
64+
finalLayout.setItem(4, Items.NAVIGATOR_SURVIVAL_ITEM_STACK, this::clickSurvival);
65+
finalLayout.setItem(5, Items.NAVIGATOR_SLENDER_ITEM_STACK, this::clickSlender);
6666
finalLayout.setItem(8, Items.NAVIGATOR_CREATIVE_ITEM_STACK, this::clickCreative);
6767
ThreadLocalUserProvider.release();
6868
return finalLayout;

common/src/main/java/net/onelitefeather/titan/common/deliver/MessageChannelDeliver.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,13 @@
1010

1111
public final class MessageChannelDeliver implements Deliver {
1212

13-
private static final ServiceRegistry serviceRegistry = InjectionLayer.ext().instance(ServiceRegistry.class);
14-
private static final PlayerManager playerManager = serviceRegistry.firstProvider(PlayerManager.class);
13+
private final ServiceRegistry serviceRegistry;
14+
private final PlayerManager playerManager;
15+
16+
public MessageChannelDeliver() {
17+
this.serviceRegistry = InjectionLayer.ext().instance(ServiceRegistry.class);
18+
this.playerManager = this.serviceRegistry.firstProvider(PlayerManager.class);
19+
}
1520

1621
@Override
1722
public void sendPlayer(Player player, DeliverComponent component) {

0 commit comments

Comments
 (0)