Use setUseItemInHand instead of setCancelled

which prevent more bug happening

Author: sandtechnology

src/main/java/org/maxgamer/quickshop/listener/PlayerListener.java

@@ -141,8 +141,9 @@ private void postControlPanel(PlayerInteractEvent e) {
                 MsgUtil.sendControlPanelInfo(p, Objects.requireNonNull(plugin.getShopManager().getShop(block.getLocation())));
                 this.playClickSound(e.getPlayer());
                 Objects.requireNonNull(plugin.getShopManager().getShop(block.getLocation())).setSignText();
+                //Prevent use item by ancient
                 if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
-                    e.setCancelled(true);
+                    e.setUseItemInHand(Event.Result.DENY);
                 }
             }
         }
@@ -188,7 +189,8 @@ private void postTrade(PlayerInteractEvent e) {
             if (!InteractUtil.check(InteractUtil.Action.TRADE, p.isSneaking())) {
                 return;
             }
-            e.setCancelled(true);
+            //Prevent use item by ancient
+            e.setUseItemInHand(Event.Result.DENY);
             shop.onClick();
             this.playClickSound(e.getPlayer());
             // Text menu
@@ -282,8 +284,9 @@ else if (e.useInteractedBlock() == Event.Result.ALLOW
             final SimpleInfo info = new SimpleInfo(b.getLocation(), ShopAction.CREATE, e.getItem(), last, false);
             plugin.getShopManager().getActions().put(p.getUniqueId(), info);
             plugin.text().of(p, "how-much-to-trade-for", MsgUtil.getTranslateText(Objects.requireNonNull(e.getItem())), Integer.toString(plugin.isAllowStack() && QuickShop.getPermissionManager().hasPermission(p, "quickshop.create.stacks") ? item.getAmount() : 1)).send();
+            //Prevent use item by ancient
             if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
-                e.setCancelled(true);
+                e.setUseItemInHand(Event.Result.DENY);
             }
         }
     }