I think it's better to avoid nullable variables.
//Copyright 2020 commandblock2 distributed under AGPL-3.0-or-later
GuiInventory = Java.type("net.minecraft.client.gui.inventory.GuiInventory");
Potion = Java.type('net.minecraft.potion.Potion');
var originalIndex = -1;
module = {
name: "AutoGapple",
description: "Eat gapple when your health is low",
author: "commandblock2 && Lolmc",
category: "combat",
values: [
health = value.createFloat("Health", 10, 1, 20),
itemSwitchDelay = value.createInteger("SwitchDelayms", 100, 0, 1000)
],
onDisable: function() {
originalIndex = -1;
},
onUpdate: function () {
if (mc.thePlayer.getHealth() <= health.get() && !mc.thePlayer.isPotionActive(Potion.regeneration)) {
var gAppleIndex = InventoryUtils.findItem(9, 36, Items.golden_apple);
if ((gAppleIndex - 9 | 35 - gAppleIndex) >= 0 && InventoryUtils.hasSpaceHotbar()) {
if (!(mc.currentScreen instanceof GuiInventory))
mc.getNetHandler().addToSendQueue(new C16PacketClientStatus(C16PacketClientStatus.EnumState.OPEN_INVENTORY_ACHIEVEMENT));
mc.playerController.windowClick(0, gAppleIndex, 0, 1, mc.thePlayer);//Shift + left mouse click
if (!(mc.currentScreen instanceof GuiInventory))
mc.getNetHandler().addToSendQueue(new C0DPacketCloseWindow());
return;
} else if ((gAppleIndex - 36 | 44 - gAppleIndex) >= 0 && gAppleIndex != mc.thePlayer.inventory.currentItem) {
if (!~originalIndex)
originalIndex = mc.thePlayer.inventory.currentItem;
mc.thePlayer.inventory.currentItem = gAppleIndex - 36;
}
if (!~gAppleIndex)// fix block when no gapple
mc.gameSettings.keyBindUseItem.pressed = true;
} else if (!~originalIndex) {
mc.gameSettings.keyBindUseItem.pressed = false;
mc.thePlayer.inventory.currentItem = originalIndex;
originalIndex = -1;
}
}
}
script.import("Core.lib")
