Merge branch 'beta' into cumcum

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt
#	src/main/java/at/hannibal2/skyhanni/config/features/AshfangConfig.java

98 files changed, 2856 insertions, 866 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarApi.kt b/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarApi.kt
index d17a3dc6b..dfc69dee8 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarApi.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarApi.kt
@@ -21,6 +21,8 @@ class BazaarApi {
         var inBazaarInventory = false
         private var currentSearchedItem = ""
 
+        var currentlyOpenedProduct: NEUInternalName? = null
+
         fun getBazaarDataByName(name: String): BazaarData? = NEUItems.getInternalNameOrNull(name)?.getBazaarData()
 
         fun NEUInternalName.getBazaarData() = if (isBazaarItem()) {
@@ -45,6 +47,21 @@ class BazaarApi {
     @SubscribeEvent
     fun onInventoryOpen(event: InventoryFullyOpenedEvent) {
         inBazaarInventory = checkIfInBazaar(event)
+        if (inBazaarInventory) {
+            val openedProduct = getOpenedProduct(event.inventoryItems) ?: return
+            currentlyOpenedProduct = openedProduct
+            BazaarOpenedProductEvent(openedProduct, event).postAndCatch()
+        }
+    }
+
+    private fun getOpenedProduct(inventoryItems: Map<Int, ItemStack>): NEUInternalName? {
+        val buyInstantly = inventoryItems[10] ?: return null
+
+        if (buyInstantly.displayName != "§aBuy Instantly") return null
+        val bazaarItem = inventoryItems[13] ?: return null
+
+        val itemName = bazaarItem.displayName
+        return NEUItems.getInternalNameOrNull(itemName)
     }
 
     @SubscribeEvent
@@ -118,5 +135,6 @@ class BazaarApi {
     @SubscribeEvent
     fun onInventoryClose(event: InventoryCloseEvent) {
         inBazaarInventory = false
+        currentlyOpenedProduct = null
     }
 }
\ No newline at end of file
diff --git a/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarBestSellMethod.kt b/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarBestSellMethod.kt
index d6145fb63..b42a331fe 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarBestSellMethod.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/bazaar/BazaarBestSellMethod.kt
@@ -1,16 +1,18 @@
 package at.hannibal2.skyhanni.features.bazaar
 
 import at.hannibal2.skyhanni.SkyHanniMod
+import at.hannibal2.skyhanni.events.BazaarOpenedProductEvent
 import at.hannibal2.skyhanni.events.InventoryCloseEvent
 import at.hannibal2.skyhanni.features.bazaar.BazaarApi.Companion.getBazaarData
+import at.hannibal2.skyhanni.utils.InventoryUtils
 import at.hannibal2.skyhanni.utils.ItemUtils.getInternalName
 import at.hannibal2.skyhanni.utils.ItemUtils.getNameWithEnchantment
 import at.hannibal2.skyhanni.utils.LorenzUtils
-import at.hannibal2.skyhanni.utils.NEUItems
+import at.hannibal2.skyhanni.utils.NEUInternalName
 import at.hannibal2.skyhanni.utils.NumberUtil
 import at.hannibal2.skyhanni.utils.RenderUtils.renderString
-import net.minecraft.client.gui.inventory.GuiChest
-import net.minecraft.inventory.ContainerChest
+import io.github.moulberry.notenoughupdates.events.SlotClickEvent
+import net.minecraft.item.ItemStack
 import net.minecraftforge.client.event.GuiScreenEvent
 import net.minecraftforge.fml.common.eventhandler.EventPriority
 import net.minecraftforge.fml.common.eventhandler.SubscribeEvent
@@ -18,40 +20,36 @@ import net.minecraftforge.fml.common.eventhandler.SubscribeEvent
 class BazaarBestSellMethod {
     private var display = ""
 
+    // Working with the last clicked item manually because
+    // the open inventory event happen while the recent clicked item in the inventory is not in the inventory or in the cursor slot
+    private var lastClickedItem: ItemStack? = null
+    private var nextCloseWillResetItem = false
+
     @SubscribeEvent
     fun onInventoryClose(event: InventoryCloseEvent) {
         display = ""
+        if (lastClickedItem != null) {
+            if (nextCloseWillResetItem) {
+                lastClickedItem = null
+            }
+            nextCloseWillResetItem = !nextCloseWillResetItem
+        }
     }
 
     @SubscribeEvent
-    fun onGuiDraw(event: GuiScreenEvent.DrawScreenEvent.Post) {
+    fun onBazaarOpenedProduct(event: BazaarOpenedProductEvent) {
         if (!isEnabled()) return
-        display = getNewText(event)
+        display = updateDisplay(event.openedProduct)
     }
 
-    private fun getNewText(event: GuiScreenEvent.DrawScreenEvent.Post): String {
+    private fun updateDisplay(internalName: NEUInternalName): String {
         try {
-            if (event.gui !is GuiChest) return ""
-            val chest = (event.gui as GuiChest).inventorySlots as ContainerChest
-
-            val inv = chest.lowerChestInventory ?: return ""
-
-            val buyInstantly = inv.getStackInSlot(10)
-            if (buyInstantly == null || buyInstantly.displayName != "§aBuy Instantly") return ""
-            val bazaarItem = inv.getStackInSlot(13) ?: return ""
-
-            val internalName = NEUItems.getInternalNameOrNull(bazaarItem.displayName) ?: return ""
-
-            var having = 0
-            for (slot in chest.inventorySlots) {
-                if (slot == null) continue
-                if (slot.slotNumber == slot.slotIndex) continue
-