2.3.6 Bugfix

Bugfix:
Upper/lower case of player names in config will be ignored from now on to avoid errors.

.gitea/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,37 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: 
+- "bug report"
+
+---
+
+**Description of issue:**
+
+
+---
+
+**Plugin Version (using`/opsecurity info`):** 
+
+**Server Type (Spigot/Paperspigot/etc):** 
+
+**Server Version (using `/ver`):** 
+
+**Relevant plugins (Delete if this isn't needed):**
+
+---
+
+**ERROR (DELETE IF YOU HAVE NO ERROR):**
+```
+######################
+## REPLACE WITH ERROR ##
+######################
+```
+
+**CONFIG SECTION (DELETE IF NOT RELEVANT):**
+```
+#######################################
+## REPLACE WITH RELEVANT CONFIG SECTION  ##
+#######################################
+```

.gitea/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,21 @@
+---
+name: Feature request
+about: Suggest an idea/improvement
+title: ''
+labels: 
+- " feature request"
+
+---
+
+**Description of feature request:**
+
+
+---
+
+**Plugin Version (using`/opsecurity info`):** 
+
+**Server Type (Spigot/Paperspigot/etc):** 
+
+**Server Version (using `/ver`):** 
+
+**Relevant plugins (Delete if this isn't needed):**

.gitea/ISSUE_TEMPLATE/question.md

@@ -0,0 +1,22 @@
+---
+name: Question
+about: Ask a question or get advice
+title: ''
+labels: 
+- question
+
+---
+
+**Share your question here:**
+
+---
+
+**Plugin Version (using`/opsecurity info`):** 
+
+**Server Type (Spigot/Paperspigot/etc):** 
+
+**Server Version (using `/ver`):** 
+
+**Relevant plugins (Delete if this isn't needed):**
+
+---

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>de.jatitv</groupId>
     <artifactId>OPSecurity</artifactId>
-    <version>2.3.5</version>
+    <version>2.3.6_Snapshot_3</version>
     <packaging>jar</packaging>
 
     <name>OPSecurity</name>

src/main/java/de/jatitv/opsecurity/cmdManagement/CmdExecuter.java

@@ -1,6 +1,5 @@
 // This claas was created by JaTiTV
 
-
 package de.jatitv.opsecurity.cmdManagement;
 
 import de.jatitv.opsecurity.config.config.SelectConfig;
@@ -16,7 +15,6 @@ import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 
-
 public class CmdExecuter implements CommandExecutor, TabCompleter {
 
     @Override

src/main/java/de/jatitv/opsecurity/cmdManagement/Commands.java

@@ -38,8 +38,8 @@ public class Commands {
     public static void reload(CommandSender sender) {
         if (sender instanceof Player) {
             Player player = (Player) sender;
-            if (Main.opHashMap.containsKey(player.getName())) {
+            if (Main.opHashMap.containsKey(player.getName().toLowerCase())) {
-                if (Main.opHashMap.get(player.getName()).UUID.equals(player.getUniqueId().toString().replace("-", ""))) {
+                if (Main.opHashMap.get(player.getName().toLowerCase()).UUID.equals(player.getUniqueId().toString().replace("-", ""))) {
                     send.player(player, SelectMessages.ReloadStart);
                     Bukkit.getConsoleSender().sendMessage(Main.Prefix + "§8-------------------------------");
                     Bukkit.getConsoleSender().sendMessage(Main.Prefix + "§6Plugin reload...");

src/main/java/de/jatitv/opsecurity/config/config/SelectConfig.java

@@ -105,7 +105,7 @@ public class SelectConfig {
             PlayerObject player = new PlayerObject(
                     yamlConfiguration.getString("OP_Whitelist.Whitelist." + key + ".UUID").replace("-",""));
             opWhitelist.add(player);
-            Main.opHashMap.put(key, player);
+            Main.opHashMap.put(key.toLowerCase(), player);
         }
 
         no_OP_Player_deop = yamlConfiguration.getBoolean("OP_Whitelist.noOpPlayerDeop.Enable");
@@ -121,7 +121,7 @@ public class SelectConfig {
             PlayerObject player = new PlayerObject(
                     yamlConfiguration.getString("Permission_Whitelist.Whitelist." + key + ".UUID").replace("-",""));
             permWhitelist.add(player);
-            Main.permissionHashMap.put(key, player);
+            Main.permissionHashMap.put(key.toLowerCase(), player);
         }
 
         PlayerWhithPermission_kick = yamlConfiguration.getBoolean("Permission_Whitelist.PlayerWhithPermission_kick");

src/main/java/de/jatitv/opsecurity/listener/Check.java

@@ -136,16 +136,16 @@ public class Check {
     }
 
     private static Boolean opWhitelist(Player player) {
-        if (Main.opHashMap.containsKey(player.getName())) {
+        if (Main.opHashMap.containsKey(player.getName().toLowerCase())) {
-            if (Main.opHashMap.get(player.getName()).UUID.equals(player.getUniqueId().toString().replace("-", ""))) {
+            if (Main.opHashMap.get(player.getName().toLowerCase()).UUID.equals(player.getUniqueId().toString().replace("-", ""))) {
                 return true;
             } else return false;
         } else return false;
     }
 
     private static Boolean permWhitelist(Player player) {
-        if (Main.permissionHashMap.containsKey(player.getName())) {
+        if (Main.permissionHashMap.containsKey(player.getName().toLowerCase())) {
-            if (Main.permissionHashMap.get(player.getName()).UUID.equals(player.getUniqueId().toString().replace("-",""))) {
+            if (Main.permissionHashMap.get(player.getName().toLowerCase()).UUID.equals(player.getUniqueId().toString().replace("-", ""))) {
                 return true;
             } else return false;
         } else return false;

src/main/java/de/jatitv/opsecurity/listener/OPCommand.java

@@ -24,27 +24,25 @@ public class OPCommand implements Listener {
         if (command.charAt(0) == '/') command = command.replaceFirst("/", "");
         String arg = command.replace("op ", "");
 
-
-        String targetUUID = null;
-
         if (SelectConfig.PlayerMustBeOnlineToOp) {
-            if (Main.opHashMap.containsKey(arg)) {
+            if (Main.opHashMap.containsKey(arg.toLowerCase())) {
                 Player target = Bukkit.getPlayer(arg);
                 if (target == null) {
                     return 1;
                 }
-                if (Main.opHashMap.get(target.getName()).UUID.equals(target.getUniqueId().toString().replace("-", ""))) {
+                if (Main.opHashMap.get(target.getName().toLowerCase()).UUID.equals(target.getUniqueId().toString().replace("-", ""))) {
                     return 0;
                 } else return 2;
             } else return 2;
         } else {
-            if (Main.opHashMap.containsKey(arg)) {
+            if (Main.opHashMap.containsKey(arg.toLowerCase())) {
+                String targetUUID = null;
                 try {
                     targetUUID = NameHistory.getPlayerUUID(arg);
                 } catch (IOException e) {
                     e.printStackTrace();
                 }
-                if (Main.opHashMap.get(arg).UUID.equals(targetUUID)) {
+                if (Main.opHashMap.get(arg.toLowerCase()).UUID.equals(targetUUID)) {
                     return 0;
                 } else return 2;
             } else return 2;