Require empty hand (#14)

* add config option

* add logic

I also realized I could reuse the 'block' variable again to clean the code up a bit

* add to README

* Use .isEmpty()

* use .getInventory().getItemInMainHand()

Author: UplandJacob

README.md

@@ -46,6 +46,7 @@ players:
   enabled: true
 blocks:
   enabled: true
+  require-empty-hand: false # a player's main hand must be empty to sit
   right-click: true # allow sitting with right click
   left-click: false # allow sitting with left click
   blocks:

src/main/java/de/varilx/sit/listener/BlockSitListener.java

@@ -29,16 +29,16 @@ public BlockSitListener(VSit plugin) {
 
     @EventHandler(priority = EventPriority.LOWEST)
     public void onPlayerInteract(PlayerInteractEvent event) {
-        if (event.getClickedBlock() == null) return;
+        Block block = event.getClickedBlock();
+        if (block == null) return;
         VaxConfiguration configuration = BaseAPI.get().getConfiguration();
         if (!configuration.getBoolean("blocks.enabled")) return;
         if (!configuration.getBoolean("enabled")) return;
-        
-        Block block = event.getClickedBlock();
         if (configuration.getStringList("blocks.blocked-worlds").contains(block.getWorld().getName())) return;
         Player player = event.getPlayer();
         if (player.isSneaking()) return;
-        
+        if (configuration.getBoolean("blocks.require-empty-hand") && !player.getInventory().getItemInMainHand().isEmpty()) return;
+
         for (String blockStr : configuration.getStringList("blocks.blocks")) {
             if (block.getType().name().toLowerCase().contains(blockStr.toLowerCase())) {
                 if (!configuration.getBoolean("blocks.right-click") && event.getAction() == Action.RIGHT_CLICK_BLOCK) return;

src/main/resources/config.yml

@@ -8,6 +8,7 @@ players:
 
 blocks:
   enabled: true
+  require-empty-hand: false
   right-click: true
   left-click: false
   blocked-worlds: