/f vault deny perm fix

src/main/java/com/massivecraft/factions/cmd/CmdCreate.java

@@ -85,8 +85,11 @@ public class CmdCreate extends FCommand {
         // join event cannot be cancelled or you'll have an empty faction
 
         // finish setting up the FPlayer
-        fme.setRole(Role.ADMIN);
         fme.setFaction(faction);
+        // We should consider adding the role just AFTER joining the faction.
+        // That way we don't have to mess up deleting more stuff.
+        // And prevent the user from being returned to NORMAL after deleting his old faction.
+        fme.setRole(Role.ADMIN);
 
         for (FPlayer follower : FPlayers.getInstance().getOnlinePlayers()) {
             follower.msg(TL.COMMAND_CREATE_CREATED, fme.describeTo(follower, true), faction.getTag(follower));

src/main/java/com/massivecraft/factions/cmd/CmdVault.java

@@ -49,6 +49,7 @@ public class CmdVault extends FCommand {
        Access access = fme.getFaction().getAccess(fme, PermissableAction.VAULT);
         if (access.equals(Access.DENY)) {
             fme.msg(TL.GENERIC_NOPERMISSION, "vault");
+            return;
         }
 
         if (fme.isInVault()){