Fix a bug, that when double clicking on a setting, that it will count as three clicks instead of two

Took 5 minutes

Author: DerFrZocker

custom-ore-generator/src/main/java/de/derfrzocker/custom/ore/generator/factory/gui/BiomeGui.java

@@ -34,6 +34,7 @@
 import de.derfrzocker.spigot.utils.message.MessageValue;
 import org.apache.commons.lang.Validate;
 import org.bukkit.block.Biome;
+import org.bukkit.event.inventory.ClickType;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.plugin.Plugin;
@@ -91,6 +92,10 @@ private ItemStack getItemStack(@NotNull final Biome biome) {
     }
 
     private void handleNormalClick(@NotNull final Biome biome, @NotNull final InventoryClickEvent event) {
+        if (event.getClick() != ClickType.LEFT) {
+            return;
+        }
+
         final OreConfigBuilder oreConfigBuilder = oreConfigFactory.getOreConfigBuilder();
 
         if (oreConfigBuilder.containsBiome(biome)) {

custom-ore-generator/src/main/java/de/derfrzocker/custom/ore/generator/factory/gui/OreSettingGui.java

@@ -35,6 +35,7 @@
 import de.derfrzocker.spigot.utils.gui.BasicGui;
 import de.derfrzocker.spigot.utils.message.MessageUtil;
 import de.derfrzocker.spigot.utils.message.MessageValue;
+import org.bukkit.event.inventory.ClickType;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.plugin.Plugin;
@@ -109,6 +110,10 @@ private SettingConsumer(final double value) {
 
         @Override
         public void accept(@NotNull final InventoryClickEvent event) {
+            if (event.getClick() != ClickType.LEFT) {
+                return;
+            }
+
             final double current = oreSettingContainer.getValue(oreSetting).orElse(0d);
             final double newValue = Double.parseDouble(String.format(Locale.ENGLISH, "%1.2f", current + value));