Merge pull request #46 from omegalink12/patch-1

Fix isUserAllowed to actually return false
Darkside138 · Jan 26, 2017 · 87c8d5c · 87c8d5c
2 parents aefef0c + 8dce0d4
commit 87c8d5c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/java/net/dirtydeeds/discordsoundboard/service/SoundPlayerImpl.java b/src/main/java/net/dirtydeeds/discordsoundboard/service/SoundPlayerImpl.java
@@ -339,10 +339,12 @@ public List<net.dirtydeeds.discordsoundboard.beans.User> getUsers() {
     public boolean isUserAllowed(String username) {
         if (allowedUsers == null) {
             return true;
-        } else if (!allowedUsers.isEmpty() && allowedUsers.contains(username)){
+        } else if (allowedUsers.isEmpty()){
             return true;
-        } else {
+        } else if (allowedUsers.contains(username)){
             return true;
+        } else {
+            return false;
         }
     }