Помогите Как исправить баг в эвенте InventoryClickEvent?

extThrower · 6 июн 2023

Идея плагина была запретить игрокам, всеми возможными способами класть в сундук кость, но после проверки плагина заметил, что когда я перемещаю кость из хотбара клавишами 0-9, кость перемещается в сундук, а обратно вернуть кость можно только сломав сундук. Не могу сделать так, чтобы кость не перемещалась из инвентаря в сундук клавишами 0-9.

Pigeon · 6 июн 2023

Код:

@EventHandler
public void onInvClick2(InventoryClickEvent e) {
      Player player = (Player) e.getWhoClicked();
      Inventory opened_inv = (Inventory) e.getView().getTopInventory();
       ItemStack item = e.getCurrentItem();
      if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) {
          if (opened_inv.getType() == InventoryType.CHEST) {
              if (item != null && item.getType() == Material.BONE) {
                  e.setCancelled(true);
                  player.sendMessage("yourmessage");
               }
           }
       }
   }

Дело в том, что getInventory() возвращает верхний инвентарь (сундук), а не инвентарь игрока. Нужна проверка, что ты кликаешь в нужном инвентаре.
P.S. По крайней мере так написано в доках:

P.P.S. Код не проверял, не знаю, будет ли работать

Можно ещё через https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/inventory/InventoryMoveItemEvent.html попробовать.

SlenderMix · 6 июн 2023

Pigeon сказал(а):
Код:

@EventHandler public void onInvClick(InventoryClickEvent e) { Player player = (Player) e.getWhoClicked(); if (e.getClickedInventory() == player.getInventory()) { // Inventory player_inv = (Inventory) e.getView().getBottomInventory(); Inventory opened_inv = (Inventory) e.getView().getTopInventory(); ItemStack item = e.getCurrentItem(); if (item != null && item.getType() == Material.BONE) { if (opened_inv.getType() == InventoryType.CHEST) { if (item.getType() == Material.BONE) { e.setCancelled(true); player.sendMessage("yourmessage"); } } } } }

Дело в том, что getInventory() возвращает верхний инвентарь (сундук), а не инвентарь игрока. Нужна проверка, что ты кликаешь в нужном инвентаре.
P.S. По крайней мере так написано в доках:
Посмотреть вложение 65102
P.P.S. Код не проверял, хз будет ли работать

Можно ещё через https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/inventory/InventoryMoveItemEvent.html попробовать.

Он возвращает любой инвентарь, а не только сундука. Это у него указано что брать кость из сундук нельзя.

Pigeon · 6 июн 2023

SlenderMix сказал(а):
Он возвращает любой инвентарь, а не только сундука. Это у него указано что брать кость из сундук нельзя.

Насколько я понял, getInventory() возвращает верхний инвентарь, с которым взаимодействует игрок.

P.S. Он не писал, что нельзя будет из сундука доставать кость)

extThrower сказал(а):
запретить всеми возможными способами класть в сундук кость

Кстати я тут заметил, что, по-моему, не сделал проверку на то, открыт ли просто инвентарь или открыт сундук.

SlenderMix · 6 июн 2023

Код:

@EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        if (event.getClickedInventory() != null && event.getClickedInventory().getType() == org.bukkit.event.inventory.InventoryType.CHEST) {
            // Проверяем, если перемещается кость
            ItemStack item = event.getCurrentItem();
            if (item != null && item.getType() == Material.BONE) {
                boolean isShiftClick = event.isShiftClick();
                boolean isNumberKey = event.getHotbarButton() >= 0 && event.getHotbarButton() <= 8;
                boolean isDragMode = event.getClick().isKeyboardClick() && event.getClick().isCreativeAction();

                if (isShiftClick || isNumberKey || isDragMode) {
                    event.setCancelled(true); // Отменяем событие перемещения
                    event.getWhoClicked().sendMessage("Перемещение кости в сундук запрещено!");
                }
            }
        }
    }

Проверяй

Pigeon · 6 июн 2023

SlenderMix сказал(а):
if (isShiftClick || isNumberKey || isDragMode) { event.setCancelled(true); // Отменяем событие перемещения event.getWhoClicked().sendMessage("Перемещение кости в сундук запрещено!"); }

Такие нажатия не определятся через метод getCurrentItem() ?

Pigeon сказал(а):
Насколько я понял, getInventory() возвращает верхний инвентарь, с которым взаимодействует игрок.

Это неверно, да?

extThrower · 7 июн 2023

SlenderMix сказал(а):

Код:

@EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        if (event.getClickedInventory() != null && event.getClickedInventory().getType() == org.bukkit.event.inventory.InventoryType.CHEST) {
            // Проверяем, если перемещается кость
            ItemStack item = event.getCurrentItem();
            if (item != null && item.getType() == Material.BONE) {
                boolean isShiftClick = event.isShiftClick();
                boolean isNumberKey = event.getHotbarButton() >= 0 && event.getHotbarButton() <= 8;
                boolean isDragMode = event.getClick().isKeyboardClick() && event.getClick().isCreativeAction();

                if (isShiftClick || isNumberKey || isDragMode) {
                    event.setCancelled(true); // Отменяем событие перемещения
                    event.getWhoClicked().sendMessage("Перемещение кости в сундук запрещено!");
                }
            }
        }
    }

Проверяй

Не работает, получается перемещать всеми способами кроме того, где нужно зажать shift

extThrower · 7 июн 2023

Pigeon сказал(а):
Код:

@EventHandler public void onInvClick2(InventoryClickEvent e) { Player player = (Player) e.getWhoClicked(); Inventory opened_inv = (Inventory) e.getView().getTopInventory(); ItemStack item = e.getCurrentItem(); if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) { if (opened_inv.getType() == InventoryType.CHEST) { if (item != null && item.getType() == Material.BONE) { e.setCancelled(true); player.sendMessage("yourmessage"); } } } }

Дело в том, что getInventory() возвращает верхний инвентарь (сундук), а не инвентарь игрока. Нужна проверка, что ты кликаешь в нужном инвентаре.
P.S. По крайней мере так написано в доках:
Посмотреть вложение 65102
P.P.S. Код не проверял, не знаю, будет ли работать

Можно ещё через https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/inventory/InventoryMoveItemEvent.html попробовать.

Все работает, но как запретить перемещать предмет клавишами 0-9?

Pigeon · 7 июн 2023

extThrower сказал(а):
Все работает, но как запретить перемещать предмет клавишами 0-9?

А, всё-таки эти события он тот ивент не обрабатывает.

Тогда как SlenderMix писал попробуй добавить в мой код:

Код:

    @EventHandler
   public void onInvClick2(InventoryClickEvent e) {
      Player player = (Player) e.getWhoClicked();
      Inventory opened_inv = (Inventory) e.getView().getTopInventory();
       ItemStack item = e.getCurrentItem();
      if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) {
          if (opened_inv.getType() == InventoryType.CHEST) {
              if (item != null && item.getType() == Material.BONE) {
                  boolean isShiftClick = e.isShiftClick();
                  boolean isNumberKey = e.getHotbarButton() >= 0 && e.getHotbarButton() <= 8;
                  boolean isDragMode = e.getClick().isKeyboardClick() && e.getClick().isCreativeAction();
                  if (isShiftClick || isNumberKey || isDragMode) {
                      e.setCancelled(true);
                      e.getWhoClicked().sendMessage("yourmessage");
                  }
                  e.setCancelled(true);
                  player.sendMessage("yourmessage");
               }
           }
       }
   }

Либо вот так попробуй:

Код:

    @EventHandler
   public void onInvClick2(InventoryClickEvent e) {
      Player player = (Player) e.getWhoClicked();
      Inventory opened_inv = (Inventory) e.getView().getTopInventory();
       ItemStack item = e.getCurrentItem();
      if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) {
          if (opened_inv.getType() == InventoryType.CHEST) {
              if (item != null && item.getType() == Material.BONE) {
                  if (e.getClick().isCreativeAction() || e.getClick().isKeyboardClick() || e.getClick().isLeftClick() || e.getClick().isRightClick() || e.getClick().isShiftClick()) {
                      e.setCancelled(true);
                      e.getWhoClicked().sendMessage("yourmessage");
                  }
               }
           }
       }
   }

extThrower · 7 июн 2023

Pigeon сказал(а):

А, всё-таки эти события он тот ивент не обрабатывает.

Тогда как SlenderMix писал попробуй добавить в мой код:

Код:

    @EventHandler
   public void onInvClick2(InventoryClickEvent e) {
      Player player = (Player) e.getWhoClicked();
      Inventory opened_inv = (Inventory) e.getView().getTopInventory();
       ItemStack item = e.getCurrentItem();
      if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) {
          if (opened_inv.getType() == InventoryType.CHEST) {
              if (item != null && item.getType() == Material.BONE) {
                  boolean isShiftClick = e.isShiftClick();
                  boolean isNumberKey = e.getHotbarButton() >= 0 && e.getHotbarButton() <= 8;
                  boolean isDragMode = e.getClick().isKeyboardClick() && e.getClick().isCreativeAction();
                  if (isShiftClick || isNumberKey || isDragMode) {
                      e.setCancelled(true);
                      e.getWhoClicked().sendMessage("yourmessage");
                  }
                  e.setCancelled(true);
                  player.sendMessage("yourmessage");
               }
           }
       }
   }

Либо вот так попробуй:

Код:

    @EventHandler
   public void onInvClick2(InventoryClickEvent e) {
      Player player = (Player) e.getWhoClicked();
      Inventory opened_inv = (Inventory) e.getView().getTopInventory();
       ItemStack item = e.getCurrentItem();
      if (opened_inv != null && e.getClickedInventory() != null && e.getClickedInventory() == player.getInventory()) {
          if (opened_inv.getType() == InventoryType.CHEST) {
              if (item != null && item.getType() == Material.BONE) {
                  if (e.getClick().isCreativeAction() || e.getClick().isKeyboardClick() || e.getClick().isLeftClick() || e.getClick().isRightClick() || e.getClick().isShiftClick()) {
                      e.setCancelled(true);
                      e.getWhoClicked().sendMessage("yourmessage");
                  }
               }
           }
       }
   }

Всёравно получается перемещать клавишами 0-9. Два кода попробовал

Pigeon · 7 июн 2023

Баг понял, происходит из-за того, что мы обрабатываем предмет, на который кликнули (хотбар клики тоже).
Но если мы с помощью клавиш 1-9 перемещаем предмет в сундук, то мы на него не кликаем.
Кстати, если кликнуть на предмет в инвентаре клавишами хотбара (навестить и нажать 1-9), то ивент будет отменён как надо.
Как вариант использовать ивент https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/inventory/InventoryMoveItemEvent.html или через getAction() как-нибудь проверять. Как придумаю что-нибудь - обновлю сообщение.

0bsid1 · 7 июн 2023

Потому что в getCurrentItem стоит тот итем, который в слоте куда будет переноситься итем.

Код:

ItemStack item = event.getClick() == ClickType.NUMBER_KEY ? event.getWhoClicked().getInventory().getItem(event.getHotbarButton()) : event.getCurrentItem();

ФорумБыстрые ссылки

СкачатьБыстрые ссылки

Помогите Как исправить баг в эвенте InventoryClickEvent?

extThrower

Активный участник

Pigeon

Активный участник

SlenderMix

Активный участник

Pigeon

Активный участник

SlenderMix

Активный участник

Pigeon

Активный участник

extThrower

Активный участник

extThrower

Активный участник

Pigeon

Активный участник

extThrower

Активный участник

Pigeon

Активный участник

0bsid1

Старожил