From 85de278dc98a68e7671e7b199fa07f275467623f Mon Sep 17 00:00:00 2001
From: Svenja Reissaus <svenjareissaus@itproaser.com>
Date: Thu, 2 Aug 2018 13:57:57 -0300
Subject: [PATCH] Fixed permission check for /f vault

---
 src/main/java/com/massivecraft/factions/cmd/CmdChest.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/massivecraft/factions/cmd/CmdChest.java b/src/main/java/com/massivecraft/factions/cmd/CmdChest.java
index 9d207729..c8ad67b3 100644
--- a/src/main/java/com/massivecraft/factions/cmd/CmdChest.java
+++ b/src/main/java/com/massivecraft/factions/cmd/CmdChest.java
@@ -30,12 +30,12 @@ public class CmdChest extends FCommand {
             fme.sendMessage("This command is disabled!");
             return;
         }
-        Access access = fme.getFaction().getAccess(fme, PermissableAction.CHEST);
-        if (access.equals(Access.DENY)) {
+        // This permission check is way too explicit but it's clean
+        Access access = myFaction.getAccess(fme, PermissableAction.CHEST);
+        if (access != Access.ALLOW && fme.getRole() != Role.ADMIN) {
             fme.msg(TL.GENERIC_NOPERMISSION, "chest");
+            return;
         }
-        //debug Bukkit.broadcastMessage(fme.getFaction().getUpgrade("Chest") + "");
-
         me.openInventory(fme.getFaction().getChest());