Fix NPE in Slot#handle

cjcameron92 · Sep 13, 2017 · d58bbc2 · d58bbc2
1 parent 91979a8
commit d58bbc2
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -759,7 +759,7 @@ Then, you can add dependencies for each helper module.
     <dependency>
         <groupId>me.lucko</groupId>
         <artifactId>helper</artifactId>
-        <version>2.1.2</version>
+        <version>2.1.3</version>
         <scope>provided</scope>
     </dependency>
 </dependencies>
@@ -768,7 +768,7 @@ Then, you can add dependencies for each helper module.
 #### Gradle
 ```gradle
 dependencies {
-    compile ("me.lucko:helper:2.1.2")
+    compile ("me.lucko:helper:2.1.3")
 }
 ```
 

diff --git a/helper/pom.xml b/helper/pom.xml
@@ -12,7 +12,7 @@
 
     <artifactId>helper</artifactId>
     <packaging>jar</packaging>
-    <version>2.1.2</version>
+    <version>2.1.3</version>
 
     <name>helper</name>
     <description>A utility to reduce boilerplate code in Bukkit plugins.</description>

diff --git a/helper/src/main/java/me/lucko/helper/menu/Slot.java b/helper/src/main/java/me/lucko/helper/menu/Slot.java
@@ -179,6 +179,9 @@ public Slot clearBindings(ClickType type) {
 
     public void handle(@Nonnull InventoryClickEvent event) {
         Set<Consumer<InventoryClickEvent>> handlers = this.handlers.get(event.getClick());
+        if (handlers == null) {
+            return;
+        }
         for (Consumer<InventoryClickEvent> handler : handlers) {
             try (MCTiming t = Timings.ofStart("helper-gui: " + getClass().getSimpleName() + " : " + Delegate.resolve(handler).getClass().getName())) {
                 handler.accept(event);