From 558681a2f2cfce6dfe854e76ab5cc7c26a2878e1 Mon Sep 17 00:00:00 2001 From: garbagemule Date: Sat, 19 Mar 2016 22:59:54 +0100 Subject: [PATCH] Remove redundant setContents() calls from class chest handlers. Before, these redundant calls didn't really do anything. Now they mess everything up, because setContents() doesn't set inventory contents only, but the entire inventory instead, because that's so super useful. Well done, boys! #notsalty --- src/main/java/com/garbagemule/MobArena/ArenaListener.java | 1 - .../com/garbagemule/MobArena/commands/user/PickClassCommand.java | 1 - 2 files changed, 2 deletions(-) diff --git a/src/main/java/com/garbagemule/MobArena/ArenaListener.java b/src/main/java/com/garbagemule/MobArena/ArenaListener.java index ba2ff31..293bbf6 100644 --- a/src/main/java/com/garbagemule/MobArena/ArenaListener.java +++ b/src/main/java/com/garbagemule/MobArena/ArenaListener.java @@ -1103,7 +1103,6 @@ public class ArenaListener contents = newContents; } arena.assignClassGiveInv(p, className, contents); - p.getInventory().setContents(contents); Messenger.tell(p, Msg.LOBBY_CLASS_PICKED, TextUtils.camelCase(className)); if (price > 0D) { Messenger.tell(p, Msg.LOBBY_CLASS_PRICE, plugin.economyFormat(price)); diff --git a/src/main/java/com/garbagemule/MobArena/commands/user/PickClassCommand.java b/src/main/java/com/garbagemule/MobArena/commands/user/PickClassCommand.java index 54e5af1..5fc3f79 100644 --- a/src/main/java/com/garbagemule/MobArena/commands/user/PickClassCommand.java +++ b/src/main/java/com/garbagemule/MobArena/commands/user/PickClassCommand.java @@ -101,7 +101,6 @@ public class PickClassCommand implements Command contents = newContents; } arena.assignClassGiveInv(p, lowercase, contents); - p.getInventory().setContents(contents); Messenger.tell(p, Msg.LOBBY_CLASS_PICKED, TextUtils.camelCase(lowercase)); if (price > 0D) { Messenger.tell(p, Msg.LOBBY_CLASS_PRICE, am.getPlugin().economyFormat(price));