Fixed general bug with negative hotbar values

IUDevman · Dec 7, 2020 · 59ec692 · 59ec692
1 parent 82d951e
commit 59ec692
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 6 deletions.
diff --git a/src/main/java/com/gamesense/client/module/modules/combat/AutoTrap.java b/src/main/java/com/gamesense/client/module/modules/combat/AutoTrap.java
@@ -137,6 +137,7 @@ public void onUpdate(){
             firstRun = false;
             if (findObsidianSlot() == -1){
                 noObby = true;
+                disable();
             }
         }
         else {
@@ -251,7 +252,7 @@ private boolean placeBlock(BlockPos pos, int range){
 
         int obsidianSlot = findObsidianSlot();
 
-        if (mc.player.inventory.currentItem != obsidianSlot){
+        if (mc.player.inventory.currentItem != obsidianSlot && obsidianSlot != -1){
             mc.player.inventory.currentItem = obsidianSlot;
         }
 

diff --git a/src/main/java/com/gamesense/client/module/modules/combat/AutoWeb.java b/src/main/java/com/gamesense/client/module/modules/combat/AutoWeb.java
@@ -231,7 +231,7 @@ private boolean placeBlock(BlockPos pos, int range){
 
 		int webbSlot = findWebSlot();
 
-		if (mc.player.inventory.currentItem != webbSlot){
+		if (mc.player.inventory.currentItem != webbSlot && webbSlot != -1){
 			mc.player.inventory.currentItem = webbSlot;
 		}
 

diff --git a/src/main/java/com/gamesense/client/module/modules/combat/HoleFill.java b/src/main/java/com/gamesense/client/module/modules/combat/HoleFill.java
@@ -97,7 +97,14 @@ public void onUpdate(){
 
 			if (autoSwitch.getValue()) {
 				int oldHand = mc.player.inventory.currentItem;
-				mc.player.inventory.currentItem = findRightBlock(oldHand);
+				int newHand = findRightBlock(oldHand);
+
+				if (newHand != -1) {
+					mc.player.inventory.currentItem = findRightBlock(oldHand);
+				}
+				else {
+					return;
+				}
 			}
 
 			BlockPos placePos = holePos.stream().sorted(Comparator.comparing(blockPos -> blockPos.getDistance((int) mc.player.posX, (int) mc.player.posY, (int) mc.player.posZ))).findFirst().orElse(null);

diff --git a/src/main/java/com/gamesense/client/module/modules/combat/SelfTrap.java b/src/main/java/com/gamesense/client/module/modules/combat/SelfTrap.java
@@ -147,6 +147,7 @@ public void onUpdate(){
             firstRun = false;
             if (findObsidianSlot() == -1){
                 noObby = true;
+                disable();
             }
         }
         else {
@@ -297,7 +298,7 @@ private boolean placeBlock(BlockPos pos){
 
         int obsidianSlot = findObsidianSlot();
 
-        if (mc.player.inventory.currentItem != obsidianSlot){
+        if (mc.player.inventory.currentItem != obsidianSlot && obsidianSlot != -1){
             mc.player.inventory.currentItem = obsidianSlot;
         }
 

diff --git a/src/main/java/com/gamesense/client/module/modules/combat/SelfWeb.java b/src/main/java/com/gamesense/client/module/modules/combat/SelfWeb.java
@@ -134,6 +134,7 @@ public void onUpdate(){
             firstRun = false;
             if (findWebSlot() == -1){
                 noWeb = true;
+                disable();
             }
         }
         else {
@@ -236,7 +237,7 @@ private boolean placeBlock(BlockPos pos){
 
         int webSlot = findWebSlot();
 
-        if (mc.player.inventory.currentItem != webSlot){
+        if (mc.player.inventory.currentItem != webSlot && webSlot != -1){
             mc.player.inventory.currentItem = webSlot;
         }
 

diff --git a/src/main/java/com/gamesense/client/module/modules/combat/Surround.java b/src/main/java/com/gamesense/client/module/modules/combat/Surround.java
@@ -145,6 +145,7 @@ public void onUpdate(){
             firstRun = false;
             if (findObsidianSlot() == -1){
                 noObby = true;
+                disable();
             }
         }
         else {
@@ -293,7 +294,7 @@ private boolean placeBlock(BlockPos pos){
 
         int obsidianSlot = findObsidianSlot();
 
-        if (mc.player.inventory.currentItem != obsidianSlot){
+        if (mc.player.inventory.currentItem != obsidianSlot && obsidianSlot != -1){
 
             mc.player.inventory.currentItem = obsidianSlot;
         }