Small code changes that fixes minor bugs
pull/10/head
JaTiTV 2 months ago
parent
commit
326dfae0e9
  1. 2
      CommandGUI V2/pom.xml
  2. 2
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/GUIListener.java
  3. 10
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/ItemChange.java
  4. 13
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/UseItem_Listener/Events.java
  5. 12
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/cmdManagement/Commands.java
  6. 2
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/gui/OpenGUI.java
  7. 6
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/Placeholder.java
  8. 4
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/UseItem.java
  9. 4
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/database/SelectDatabase.java
  10. 4
      CommandGUI V2/src/main/java/de/jatitv/commandguiv2/api/CGuiAPI.java

2
CommandGUI V2/pom.xml

@ -6,7 +6,7 @@ @@ -6,7 +6,7 @@
<groupId>net.t2code</groupId>
<artifactId>CommandGUI_V2</artifactId>
<version>2.7.1</version>
<version>2.7.2</version>
<packaging>jar</packaging>
<name>CommandGUI</name>

2
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/GUIListener.java

@ -261,7 +261,7 @@ public class GUIListener implements Listener { @@ -261,7 +261,7 @@ public class GUIListener implements Listener {
}
private static void toggleUseItem(Player player, Gui gui) {
if (Events.useItemHashMap.get(player)) {
if (Events.useItemHashMap.get(player.getName())) {
Commands.itemOff(player);
} else {
Commands.itemOn(player);

10
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/ItemChange.java

@ -4,6 +4,7 @@ import de.jatitv.commandguiv2.Spigot.Listener.UseItem_Listener.Events; @@ -4,6 +4,7 @@ import de.jatitv.commandguiv2.Spigot.Listener.UseItem_Listener.Events;
import de.jatitv.commandguiv2.Spigot.Main;
import de.jatitv.commandguiv2.Spigot.system.UseItem;
import de.jatitv.commandguiv2.Spigot.config.config.SelectConfig;
import de.jatitv.commandguiv2.Spigot.system.database.SelectDatabase;
import net.t2code.lib.Spigot.Lib.messages.send;
import org.bukkit.entity.Player;
import org.bukkit.scheduler.BukkitRunnable;
@ -14,10 +15,10 @@ public class ItemChange { @@ -14,10 +15,10 @@ public class ItemChange {
if (SelectConfig.UseItem_InventorySlotEnforce) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
if (Events.useItemSlotHashMap.get(player) == null) {
if (Events.useItemSlotHashMap.get(player.getName()) == null) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
slot = Events.useItemSlotHashMap.get(player);
slot = Events.useItemSlotHashMap.get(player.getName());
}
}
@ -27,7 +28,7 @@ public class ItemChange { @@ -27,7 +28,7 @@ public class ItemChange {
UseItem.removeItem(player);
if (UseItem.protection(player, false)) return;
if (SelectConfig.UseItem_GiveOnlyOnFirstJoin) {
if (!SelectConfig.UseItem_AllowToggle || Events.useItemHashMap.get(player)) {
if (!SelectConfig.UseItem_AllowToggle || Events.useItemHashMap.get(player.getName())) {
if (!player.hasPlayedBefore()) {
new BukkitRunnable() {
@Override
@ -53,8 +54,7 @@ public class ItemChange { @@ -53,8 +54,7 @@ public class ItemChange {
new BukkitRunnable() {
@Override
public void run() {
if (player == null) return;
if (!SelectConfig.UseItem_AllowToggle || Events.useItemHashMap.get(player)) {
if (!SelectConfig.UseItem_AllowToggle || Events.useItemHashMap.get(player.getName())) {
if (SelectConfig.UseItem_InventorySlotEnforce || player.getInventory().getItem(slot - 1) == null) {
send.debug(Main.plugin, "Give: " + player.getName());
UseItem.giveUseItem(player);

13
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/Listener/UseItem_Listener/Events.java

@ -34,16 +34,13 @@ import java.util.Iterator; @@ -34,16 +34,13 @@ import java.util.Iterator;
public class Events implements Listener {
public static HashMap<Player, Boolean> useItemHashMap = new HashMap<>();
public static HashMap<Player, Integer> useItemSlotHashMap = new HashMap<>();
public static HashMap<String, Boolean> useItemHashMap = new HashMap<>();
public static HashMap<String, Integer> useItemSlotHashMap = new HashMap<>();
@EventHandler
public void onJoin(PlayerJoinEvent e) {
if (SelectDatabase.selectItemStatus(e.getPlayer())) {
useItemHashMap.put(e.getPlayer(), true);
} else useItemHashMap.put(e.getPlayer(), false);
useItemSlotHashMap.put(e.getPlayer(), SelectDatabase.selectSlot(e.getPlayer()));
useItemHashMap.put(e.getPlayer().getName(), SelectDatabase.selectItemStatus(e.getPlayer()));
useItemSlotHashMap.put(e.getPlayer().getName(), SelectDatabase.selectSlot(e.getPlayer()));
if (CGuiAPI.JoinDisable) {
send.debug(Main.plugin, "JoinDisable");
@ -62,7 +59,7 @@ public class Events implements Listener { @@ -62,7 +59,7 @@ public class Events implements Listener {
@EventHandler
public void onJoin(PlayerQuitEvent e) {
useItemHashMap.remove(e.getPlayer());
useItemHashMap.remove(e.getPlayer().getName());
}
@EventHandler(priority = EventPriority.HIGHEST)

12
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/cmdManagement/Commands.java

@ -124,10 +124,10 @@ public class Commands { @@ -124,10 +124,10 @@ public class Commands {
if (SelectConfig.UseItem_InventorySlotEnforce) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
if (Events.useItemSlotHashMap.get(player) == null) {
if (Events.useItemSlotHashMap.get(player.getName()) == null) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
slot = Events.useItemSlotHashMap.get(player);
slot = Events.useItemSlotHashMap.get(player.getName());
}
}
send.debug(Main.plugin, String.valueOf(slot));
@ -175,8 +175,8 @@ public class Commands { @@ -175,8 +175,8 @@ public class Commands {
return;
}
if (Events.useItemHashMap.get(player)) {
if (Objects.equals(Events.useItemSlotHashMap.get(player), setSlot)) {
if (Events.useItemHashMap.get(player.getName())) {
if (Objects.equals(Events.useItemSlotHashMap.get(player.getName()), setSlot)) {
player.sendMessage(SelectMessages.ItemSlotAlreadySet.replace("[slot]", setSlot.toString()));
return;
}
@ -189,9 +189,9 @@ public class Commands { @@ -189,9 +189,9 @@ public class Commands {
UseItem.removeItem(player);
SelectDatabase.setSlot(player, setSlot);
Events.useItemSlotHashMap.replace(player, setSlot);
Events.useItemSlotHashMap.replace(player.getName(), setSlot);
if (Events.useItemHashMap.get(player)) {
if (Events.useItemHashMap.get(player.getName())) {
UseItem.giveUseItem(player);
}

2
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/gui/OpenGUI.java

@ -95,7 +95,7 @@ public class OpenGUI { @@ -95,7 +95,7 @@ public class OpenGUI {
toggleOff(function, slot.slot, player, inventory);
}
} else if (function.toggleUseItem) {
if (Events.useItemHashMap.get(player)) {
if (Events.useItemHashMap.get(player.getName())) {
toggleOn(function, slot.slot, player, inventory);
} else {
toggleOff(function, slot.slot, player, inventory);

6
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/Placeholder.java

@ -38,17 +38,17 @@ public class Placeholder extends PlaceholderExpansion { @@ -38,17 +38,17 @@ public class Placeholder extends PlaceholderExpansion {
@Override
public String onPlaceholderRequest(Player p, String params) {
if (params.contains("useitem_boolean")) {
return Events.useItemHashMap.get(p).toString();
return Events.useItemHashMap.get(p.getName()).toString();
}
if (params.contains("useitem")) {
if (Events.useItemHashMap.get(p)) {
if (Events.useItemHashMap.get(p.getName())) {
return SelectConfig.placeholderTrue;
} else return SelectConfig.placeholderFalse;
}
if (params.contains("slot")){
return Events.useItemSlotHashMap.get(p).toString();
return Events.useItemSlotHashMap.get(p.getName()).toString();
}
return null;

4
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/UseItem.java

@ -29,10 +29,10 @@ public class UseItem { @@ -29,10 +29,10 @@ public class UseItem {
if (SelectConfig.UseItem_InventorySlotEnforce) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
if (Events.useItemSlotHashMap.get(player) == null) {
if (Events.useItemSlotHashMap.get(player.getName()) == null) {
slot = SelectConfig.UseItem_InventorySlot;
} else {
slot = Events.useItemSlotHashMap.get(player);
slot = Events.useItemSlotHashMap.get(player.getName());
}
}
if (SelectConfig.UseItem_InventorySlot_FreeSlot) {

4
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/Spigot/system/database/SelectDatabase.java

@ -49,7 +49,7 @@ public class SelectDatabase { @@ -49,7 +49,7 @@ public class SelectDatabase {
}
public static void setItemStatusTrue(Player player) {
Events.useItemHashMap.replace(player, true);
Events.useItemHashMap.replace(player.getName(), true);
switch (Storage) {
case MYSQL:
MySQL.query("INSERT INTO `gui-item` (`UUID`, `Name`) VALUES ('" + player.getUniqueId() + "', '" + player.getName()
@ -67,7 +67,7 @@ public class SelectDatabase { @@ -67,7 +67,7 @@ public class SelectDatabase {
}
public static void setItemStatusFalse(Player player) {
Events.useItemHashMap.replace(player, false);
Events.useItemHashMap.replace(player.getName(), false);
switch (Storage) {
case MYSQL:
MySQL.query("INSERT INTO `gui-item` (`UUID`, `Name`, `Status`) VALUES ('" + player.getUniqueId() + "', '" + player.getName()

4
CommandGUI V2/src/main/java/de/jatitv/commandguiv2/api/CGuiAPI.java

@ -44,11 +44,11 @@ public class CGuiAPI { @@ -44,11 +44,11 @@ public class CGuiAPI {
}
public static Boolean selectPlayerItemEnable(Player player) {
return Events.useItemHashMap.get(player);
return Events.useItemHashMap.get(player.getName());
}
public static Integer selectPlayerItemSlot(Player player) {
return Events.useItemSlotHashMap.get(player);
return Events.useItemSlotHashMap.get(player.getName());
}
public static void setPlayerItemEnable(Player player, Boolean value) {

Loading…
Cancel
Save