Fixed some flaws

AntiCope · Jan 1, 2024 · 2c65a67 · 2c65a67
1 parent de0272e
commit 2c65a67
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 17 deletions.
diff --git a/build.gradle b/build.gradle
@@ -74,7 +74,6 @@ jar {
 tasks.withType(Jar) {
 	duplicatesStrategy = DuplicatesStrategy.EXCLUDE
 }
-def targetJavaVersion = 17
 tasks.withType(JavaCompile).configureEach {
 	// ensure that the encoding is set to UTF-8, no matter what the system default is
 	// this fixes some edge cases with special characters not displaying correctly
@@ -90,15 +89,3 @@ tasks.withType(JavaCompile).configureEach {
 		 it.options.release = targetVersion
 	}
 }
-
-java {
-	def javaVersion = JavaVersion.toVersion(targetJavaVersion)
-	if (JavaVersion.current() < javaVersion) {
-		toolchain.languageVersion = JavaLanguageVersion.of(targetJavaVersion)
-	}
-	archivesBaseName = project.archives_base_name
-	// Loom will automatically attach sourcesJar to a RemapSourcesJar task and to the "build" task
-	// if it is present.
-	// If you remove this line, sources will not be generated.
-	withSourcesJar()
-}
diff --git a/src/main/java/anticope/rejects/modules/AutoEnchant.java b/src/main/java/anticope/rejects/modules/AutoEnchant.java
@@ -55,7 +55,7 @@ public class AutoEnchant extends meteordevelopment.meteorclient.systems.modules.
     );
 
     public AutoEnchant() {
-        super(MeteorRejectsAddon.CATEGORY, "auto-enchant ", "Automatically enchanting items.");
+        super(MeteorRejectsAddon.CATEGORY, "auto-enchant", "Automatically enchanting items.");
     }
 
     @EventHandler
@@ -77,7 +77,7 @@ private void autoEnchant() {
                 info("Enchanting table is closed.");
                 break;
             }
-            if (handler.getLapisCount() < 3 && !fillLapisItem()) {
+            if (handler.getLapisCount() < level.get() && !fillLapisItem()) {
                 info("Lapis lazuli is not found.");
                 break;
             }
@@ -94,7 +94,12 @@ private void autoEnchant() {
                 mc.execute(() -> InvUtils.drop().slotId(0));
             }
 
-            // Why sleep here? I don't know either.
+            /*
+            Although the description here indicates that the tick is the unit,
+            the actual delay is not the tick unit,
+            but it does not affect the normal operation in the game.
+            Perhaps we can ignore it
+            */
             try {
                 Thread.sleep(delay.get());
             } catch (InterruptedException e) {
@@ -112,7 +117,7 @@ private boolean fillCanEnchantItem() {
 
     private boolean fillLapisItem() {
         FindItemResult res = InvUtils.find(Items.LAPIS_LAZULI);
-        if (res.slot() == -1) return false;
+        if (!res.found()) return false;
         InvUtils.shiftClick().slot(res.slot());
         return true;
     }