().configureEach {
+ sourceCompatibility = "21"
+ targetCompatibility = "21"
+
+ options.encoding = "UTF-8"
+}
+
+kotlin {
+ compilerOptions {
+ apiVersion.set(org.jetbrains.kotlin.gradle.dsl.KotlinVersion.KOTLIN_2_2)
+ jvmTarget.set(JvmTarget.JVM_21)
+ }
+}
diff --git a/nms/v1_21R7/src/main/kotlin/xyz/alexcrea/cuanvil/dependency/packet/versions/V1_21R7_PacketManager.kt b/nms/v1_21R7/src/main/kotlin/xyz/alexcrea/cuanvil/dependency/packet/versions/V1_21R7_PacketManager.kt
new file mode 100644
index 0000000..59ae9ce
--- /dev/null
+++ b/nms/v1_21R7/src/main/kotlin/xyz/alexcrea/cuanvil/dependency/packet/versions/V1_21R7_PacketManager.kt
@@ -0,0 +1,33 @@
+package xyz.alexcrea.cuanvil.dependency.packet.versions
+
+import net.minecraft.network.protocol.game.ClientboundPlayerAbilitiesPacket
+import net.minecraft.world.entity.player.Abilities
+import org.bukkit.craftbukkit.entity.CraftPlayer
+import org.bukkit.entity.Player
+import xyz.alexcrea.cuanvil.dependency.packet.PacketManager
+import xyz.alexcrea.cuanvil.dependency.packet.PacketManagerBase
+
+class V1_21R7_PacketManager : PacketManagerBase(), PacketManager {
+ override val canSetInstantBuild: Boolean
+ get() = true
+
+ override fun setInstantBuild(player: Player, instantBuild: Boolean) {
+ val nmsPlayer = (player as CraftPlayer).handle
+ val playerAbilities = nmsPlayer.abilities
+ val sendedAbilities: Abilities
+ if (playerAbilities.instabuild == instantBuild) {
+ sendedAbilities = playerAbilities
+ } else {
+ sendedAbilities = Abilities()
+ sendedAbilities.invulnerable = playerAbilities.invulnerable
+ sendedAbilities.flying = playerAbilities.flying
+ sendedAbilities.mayfly = playerAbilities.mayfly
+ sendedAbilities.instabuild = instantBuild
+ sendedAbilities.mayBuild = playerAbilities.mayBuild
+ sendedAbilities.flyingSpeed = playerAbilities.flyingSpeed
+ sendedAbilities.walkingSpeed = playerAbilities.walkingSpeed
+ }
+ val packet = ClientboundPlayerAbilitiesPacket(sendedAbilities)
+ nmsPlayer.connection.send(packet)
+ }
+}
\ No newline at end of file
diff --git a/settings.gradle.kts b/settings.gradle.kts
index 03fe07e..9de7d8c 100644
--- a/settings.gradle.kts
+++ b/settings.gradle.kts
@@ -1,2 +1,22 @@
rootProject.name = "CustomAnvil"
+// NMS subproject
+include("nms:nms-common")
+findProject(":nms:nms-common")?.name = "nms-common"
+include("nms:nms-paper")
+findProject(":nms:nms-paper")?.name = "nms-paper"
+
+
+val reobfNMS = providers.gradleProperty("subprojects.reobfnms")
+ .get().split(",")
+
+for (nmsPart in reobfNMS) {
+ include("nms:$nmsPart")
+ findProject(":nms:$nmsPart")?.name = nmsPart
+}
+
+// compatibility subprojects
+include(":impl:LegacyEcoEnchant")
+findProject(":impl:LegacyEcoEnchant")?.name = "LegacyEcoEnchant"
+include("impl:ExcellentEnchant5_4")
+findProject(":impl:ExcellentEnchant5_4")?.name = "ExcellentEnchant5_4"
\ No newline at end of file
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/AnvilRecipeBuilder.java b/src/main/java/xyz/alexcrea/cuanvil/api/AnvilRecipeBuilder.java
index 74e8118..4292fa0 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/AnvilRecipeBuilder.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/AnvilRecipeBuilder.java
@@ -14,7 +14,10 @@ public class AnvilRecipeBuilder {
private @NotNull String name;
private boolean exactCount;
- private int xpCostPerCraft;
+ private int levelCostPerCraft;
+ private int linearXpCostPerCraft;
+
+ private boolean removeExactLinearXp;
private @Nullable ItemStack leftItem;
private @Nullable ItemStack rightItem;
@@ -23,7 +26,7 @@ public class AnvilRecipeBuilder {
/**
* Instantiates a new Anvil recipe builder.
* exact count default to true.
- * xp cost per craft default to 1.
+ * xp level and linear cost per craft default to 0.
*
* @param name The recipe name
*/
@@ -31,7 +34,9 @@ public class AnvilRecipeBuilder {
this.name = name;
this.exactCount = true;
- this.xpCostPerCraft = 1;
+ this.levelCostPerCraft = 0;
+ this.linearXpCostPerCraft = 0;
+ this.removeExactLinearXp = false;
this.leftItem = null;
this.rightItem = null;
@@ -60,7 +65,7 @@ public class AnvilRecipeBuilder {
}
/**
- * Get if the recipe is exact count.
+ * Get if the recipe is exact count. (default 0)
*
* Exact count mean the recipe can only be crafted 1 by 1.
* If set to false, then it will craft as much as possible in 1 go and will keep unused material onto the anvil inventory.
@@ -86,12 +91,14 @@ public class AnvilRecipeBuilder {
}
/**
- * Get the xp level cost per craft.
+ * Get the xp level cost per craft. (default 0)
*
* @return The xp level cost per craft
+ * @deprecated use {@link #getLevelCostPerCraft() getLevelCostPerCraft} instead
*/
+ @Deprecated(since = "1.13.0")
public int getXpCostPerCraft() {
- return xpCostPerCraft;
+ return getLevelCostPerCraft();
}
/**
@@ -99,9 +106,78 @@ public class AnvilRecipeBuilder {
*
* @param xpCostPerCraft The xp level cost per craft
* @return This recipe builder instance.
+ * @deprecated use {@link #setLevelCostPerCraft(int) setLevelCostPerCraft} instead
*/
+ @Deprecated(since = "1.13.0")
public AnvilRecipeBuilder setXpCostPerCraft(int xpCostPerCraft) {
- this.xpCostPerCraft = xpCostPerCraft;
+ return setLevelCostPerCraft(xpCostPerCraft);
+ }
+
+ /**
+ * Get the xp level cost per craft. (default 0)
+ *
+ * @return The xp level cost per craft
+ */
+ public int getLevelCostPerCraft() {
+ return levelCostPerCraft;
+ }
+
+ /**
+ * Sets the xp level cost per craft.
+ *
+ * @param levelCostPerCraft The xp level cost per craft
+ * @return This recipe builder instance.
+ */
+ public AnvilRecipeBuilder setLevelCostPerCraft(int levelCostPerCraft) {
+ this.levelCostPerCraft = levelCostPerCraft;
+ return this;
+ }
+
+ /**
+ * Get the linear xp cost (not xp level cost) per craft.
+ *
+ * @return The xp level cost per craft
+ */
+ public int getLinearXpCostPerCraft() {
+ return linearXpCostPerCraft;
+ }
+
+ /**
+ * Sets the linear xp cost (not xp level cost) per craft.
+ *
+ * @param linearXpCostPerCraft The linear xp cost per craft
+ * @return This recipe builder instance.
+ */
+ public AnvilRecipeBuilder setLinearXpCostPerCraft(int linearXpCostPerCraft) {
+ this.linearXpCostPerCraft = linearXpCostPerCraft;
+ return this;
+ }
+
+ /**
+ * Get if the linear xp should get removed by an exact amount.
+ *
+ * If false (default) level cost will be the level that would be reached by a player with this amount of xp.
+ * If true will require the level that has at least the specified level of xp then on click remove only the necessary xp
+ *
+ * linear xp cost are applied after level cost
+ * @return if we should remove the exact amount of linear xp
+ */
+ public boolean isRemoveExactLinearXp() {
+ return removeExactLinearXp;
+ }
+
+ /**
+ * Set if the linear xp should get removed by an exact amount.
+ *
+ * If false (default) level cost will be the level that would be reached by a player with this amount of xp.
+ * If true will require the level that has at least the specified level of xp then on click remove only the necessary xp
+ *
+ * linear xp cost are applied after level cost
+ * @param removeExactLinearXp if we should remove the exact amount of linear xp
+ * @return This recipe builder instance.
+ */
+ public AnvilRecipeBuilder setRemoveExactLinearXp(boolean removeExactLinearXp) {
+ this.removeExactLinearXp = removeExactLinearXp;
return this;
}
@@ -182,12 +258,14 @@ public class AnvilRecipeBuilder {
*/
@Nullable // null if missing argument
public AnvilCustomRecipe build() {
- if(leftItem == null || rightItem == null) return null;
+ if (leftItem == null || resultItem == null) return null;
return new AnvilCustomRecipe(
this.name,
this.exactCount,
- this.xpCostPerCraft,
+ this.levelCostPerCraft,
+ this.linearXpCostPerCraft,
+ this.removeExactLinearXp,
this.leftItem, this.rightItem, this.resultItem
);
}
@@ -198,7 +276,7 @@ public class AnvilRecipeBuilder {
*
* @return True if successful.
*/
- public boolean registerIfAbsent(){
+ public boolean registerIfAbsent() {
return CustomAnvilRecipeApi.addRecipe(this);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/ConflictAPI.java b/src/main/java/xyz/alexcrea/cuanvil/api/ConflictAPI.java
index 28fccd2..fe2715e 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/ConflictAPI.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/ConflictAPI.java
@@ -1,15 +1,17 @@
package xyz.alexcrea.cuanvil.api;
import io.delilaheve.CustomAnvil;
-import org.bukkit.Bukkit;
import org.bukkit.NamespacedKey;
import org.bukkit.configuration.file.FileConfiguration;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.dependency.DependencyManager;
import xyz.alexcrea.cuanvil.group.EnchantConflictGroup;
import xyz.alexcrea.cuanvil.gui.config.global.EnchantConflictGui;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
/**
* Custom Anvil api for conflict registry.
@@ -17,31 +19,49 @@ import java.util.*;
@SuppressWarnings("unused")
public class ConflictAPI {
- private ConflictAPI() {}
+ private ConflictAPI() {
+ }
- private static int saveChangeTask = -1;
- private static int reloadChangeTask = -1;
+ private static Object saveChangeTask = null;
+ private static Object reloadChangeTask = null;
/**
* Write and add a conflict.
* Will not write the conflict if it already exists.
+ * Will not be successful if the conflict is empty.
*
* @param builder The conflict builder to be based on
* @return True if successful.
*/
- public static boolean addConflict(@NotNull ConflictBuilder builder){
- FileConfiguration config = ConfigHolder.CONFLICT_HOLDER.getConfig();
- if(config.contains(builder.getName())) return false;
+ public static boolean addConflict(@NotNull ConflictBuilder builder) {
+ return addConflict(builder, false);
+ }
- if(!writeConflict(builder, false)) return false;
+ /**
+ * Write and add a conflict.
+ * Will not write the conflict if it already exists.
+ * Will not be successful if the conflict is empty.
+ *
+ * @param builder The conflict builder to be based on
+ * @param overrideDeleted If we should write even if the conflict was previously deleted.
+ * @return True if successful.
+ */
+ public static boolean addConflict(@NotNull ConflictBuilder builder, boolean overrideDeleted) {
+ FileConfiguration config = ConfigHolder.CONFLICT_HOLDER.getConfig();
+
+ // Test if conflict can be added
+ if (!overrideDeleted && ConfigHolder.CONFLICT_HOLDER.isDeleted(builder.getName())) return false;
+ if (config.contains(builder.getName())) return false;
+
+ if (!writeConflict(builder, false)) return false;
EnchantConflictGroup conflict = builder.build();
-
// Register conflict
ConfigHolder.CONFLICT_HOLDER.getConflictManager().addConflict(conflict);
// Add conflict to gui
- EnchantConflictGui.INSTANCE.updateValueForGeneric(conflict, true);
+ EnchantConflictGui conflictGui = EnchantConflictGui.getCurrentInstance();
+ if (conflictGui != null) conflictGui.updateValueForGeneric(conflict, true);
return true;
}
@@ -51,10 +71,10 @@ public class ConflictAPI {
*
* You may want to use {@link #addConflict(ConflictBuilder)} instead as it is more performance in most case as this function will reload every conflict.
*
- * @param builder The builder
- * @return True if successful.
+ * @param builder the builder
+ * @return true if was written successfully.
*/
- public static boolean writeConflict(@NotNull ConflictBuilder builder){
+ public static boolean writeConflict(@NotNull ConflictBuilder builder) {
return writeConflict(builder, true);
}
@@ -65,14 +85,14 @@ public class ConflictAPI {
*
* @param builder The builder
* @param updatePlanned If we should plan a global update for conflicts
- * @return True if successful.
+ * @return true if was written successfully.
*/
- public static boolean writeConflict(@NotNull ConflictBuilder builder, boolean updatePlanned){
+ public static boolean writeConflict(@NotNull ConflictBuilder builder, boolean updatePlanned) {
FileConfiguration config = ConfigHolder.CONFLICT_HOLDER.getConfig();
String name = builder.getName();
- if(name.contains(".")) {
- CustomAnvil.instance.getLogger().warning("Conflict " + name +" contain \".\" in its name but should not. this conflict is ignored.");
+ if (name.contains(".")) {
+ CustomAnvil.instance.getLogger().warning("Conflict " + name + " contain \".\" in its name but should not. this conflict is ignored.");
logConflictOrigin(builder);
return false;
}
@@ -81,27 +101,30 @@ public class ConflictAPI {
List enchantments = extractEnchantments(builder);
List excludedGroups = new ArrayList<>(builder.getExcludedGroupNames());
- if(!enchantments.isEmpty()) config.set(basePath + "enchantments", enchantments);
- if(!excludedGroups.isEmpty()) config.set(basePath + "notAffectedGroups", excludedGroups);
- if(builder.getMaxBeforeConflict() > 0) config.set(basePath + "maxEnchantmentBeforeConflict", builder.getMaxBeforeConflict());
+ if (!enchantments.isEmpty()) config.set(basePath + "enchantments", enchantments);
+ if (!excludedGroups.isEmpty()) config.set(basePath + "notAffectedGroups", excludedGroups);
+ if (builder.getMaxBeforeConflict() > 0)
+ config.set(basePath + "maxEnchantmentBeforeConflict", builder.getMaxBeforeConflict());
+ if (!config.isConfigurationSection(name)) return false;
prepareSaveTask();
- if(updatePlanned) prepareUpdateTask();
+ if (updatePlanned) prepareUpdateTask();
return true;
}
/**
* Extract every enchantment names from a builder.
+ *
* @param builder The builder storing the enchantments
* @return Builder's stored enchantment.
*/
@NotNull
- private static List extractEnchantments(@NotNull ConflictBuilder builder){
+ private static List extractEnchantments(@NotNull ConflictBuilder builder) {
List result = new ArrayList<>(builder.getEnchantmentNames());
for (NamespacedKey enchantmentKey : builder.getEnchantmentKeys()) {
- result.add(enchantmentKey.getKey());
+ result.add(enchantmentKey.toString());
}
return result;
@@ -113,16 +136,17 @@ public class ConflictAPI {
* @param conflict The conflict to remove
* @return True if successful.
*/
- public static boolean removeConflict(@NotNull EnchantConflictGroup conflict){
+ public static boolean removeConflict(@NotNull EnchantConflictGroup conflict) {
// Remove from registry
ConfigHolder.CONFLICT_HOLDER.getConflictManager().removeConflict(conflict);
- // Write as null and save to file
- ConfigHolder.CONFLICT_HOLDER.getConfig().set(conflict.getName(), null);
+ // Delete and save to file
+ ConfigHolder.CONFLICT_HOLDER.delete(conflict.getName());
prepareSaveTask();
// Remove from gui
- EnchantConflictGui.INSTANCE.removeGeneric(conflict);
+ EnchantConflictGui conflictGui = EnchantConflictGui.getCurrentInstance();
+ if (conflictGui != null) conflictGui.removeGeneric(conflict);
return true;
}
@@ -131,38 +155,40 @@ public class ConflictAPI {
* Prepare a task to save conflict configuration.
*/
private static void prepareSaveTask() {
- if(saveChangeTask != -1) return;
+ if (saveChangeTask != null) return;
- saveChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ saveChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, () -> {
ConfigHolder.CONFLICT_HOLDER.saveToDisk(true);
- saveChangeTask = -1;
- }, 0L);
+ saveChangeTask = null;
+ });
}
/**
* Prepare a task to reload every conflict.
*/
private static void prepareUpdateTask() {
- if(reloadChangeTask != -1) return;
+ if (reloadChangeTask != null) return;
- reloadChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ reloadChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, () -> {
ConfigHolder.CONFLICT_HOLDER.reload();
- EnchantConflictGui.INSTANCE.reloadValues();
- reloadChangeTask = -1;
- }, 0L);
+ EnchantConflictGui conflictGui = EnchantConflictGui.getCurrentInstance();
+ if (conflictGui != null) conflictGui.reloadValues();
+ reloadChangeTask = null;
+ });
}
- static void logConflictOrigin(@NotNull ConflictBuilder builder){
- CustomAnvil.instance.getLogger().warning("Conflict " + builder.getName() +" came from " + builder.getSourceName() + ".");
+ static void logConflictOrigin(@NotNull ConflictBuilder builder) {
+ CustomAnvil.instance.getLogger().warning("Conflict " + builder.getName() + " came from " + builder.getSourceName() + ".");
}
/**
* Get every registered conflict.
+ *
* @return An immutable collection of conflict.
*/
@NotNull
- public static List getRegisteredConflict(){
+ public static List getRegisteredConflict() {
List mutableList = ConfigHolder.CONFLICT_HOLDER.getConflictManager().getConflictList();
return Collections.unmodifiableList(mutableList);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/ConflictBuilder.java b/src/main/java/xyz/alexcrea/cuanvil/api/ConflictBuilder.java
index 3e63b36..1460766 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/ConflictBuilder.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/ConflictBuilder.java
@@ -10,8 +10,10 @@ import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import xyz.alexcrea.cuanvil.group.*;
import java.util.HashSet;
+import java.util.List;
import java.util.Set;
+//TODO add conflict after level
/**
* A Builder for material conflict.
*/
@@ -35,7 +37,7 @@ public class ConflictBuilder {
* @param maxBeforeConflict Maximum number of conflicting enchantment before conflict is active
* @param source The conflict source
*/
- public ConflictBuilder(@NotNull String name, int maxBeforeConflict, @Nullable Plugin source){
+ public ConflictBuilder(@NotNull String name, int maxBeforeConflict, @Nullable Plugin source) {
this.source = source;
this.name = name;
@@ -53,7 +55,7 @@ public class ConflictBuilder {
* @param name The conflict name
* @param source The conflict source
*/
- public ConflictBuilder(@NotNull String name, @Nullable Plugin source){
+ public ConflictBuilder(@NotNull String name, @Nullable Plugin source) {
this(name, 0, source);
}
@@ -62,7 +64,7 @@ public class ConflictBuilder {
*
* @param name The conflict name
*/
- public ConflictBuilder(@NotNull String name){
+ public ConflictBuilder(@NotNull String name) {
this(name, null);
}
@@ -83,7 +85,7 @@ public class ConflictBuilder {
*/
@NotNull
public String getSourceName() {
- if(source == null) return "an unknown source";
+ if (source == null) return "an unknown source";
return source.getName();
}
@@ -176,7 +178,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder addEnchantment(@NotNull String enchantmentName){
+ public ConflictBuilder addEnchantment(@NotNull String enchantmentName) {
enchantmentNames.add(enchantmentName);
return this;
}
@@ -188,7 +190,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder addEnchantment(@NotNull NamespacedKey enchantmentKey){
+ public ConflictBuilder addEnchantment(@NotNull NamespacedKey enchantmentKey) {
enchantmentKeys.add(enchantmentKey);
return this;
}
@@ -200,7 +202,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder addEnchantment(@NotNull CAEnchantment enchantment){
+ public ConflictBuilder addEnchantment(@NotNull CAEnchantment enchantment) {
addEnchantment(enchantment.getKey());
return this;
}
@@ -212,7 +214,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder removeEnchantment(@NotNull String enchantmentName){
+ public ConflictBuilder removeEnchantment(@NotNull String enchantmentName) {
enchantmentNames.remove(enchantmentName);
return this;
}
@@ -224,7 +226,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder removeEnchantment(@NotNull NamespacedKey enchantmentKey){
+ public ConflictBuilder removeEnchantment(@NotNull NamespacedKey enchantmentKey) {
enchantmentKeys.remove(enchantmentKey);
return removeEnchantment(enchantmentKey.getKey());
}
@@ -236,7 +238,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder removeEnchantment(@NotNull CAEnchantment enchantment){
+ public ConflictBuilder removeEnchantment(@NotNull CAEnchantment enchantment) {
return removeEnchantment(enchantment.getKey());
}
@@ -255,7 +257,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder addExcludedGroup(@NotNull String groupName){
+ public ConflictBuilder addExcludedGroup(@NotNull String groupName) {
excludedGroupNames.add(groupName);
return this;
}
@@ -275,7 +277,7 @@ public class ConflictBuilder {
* @return this conflict builder instance.
*/
@NotNull
- public ConflictBuilder addExcludedGroup(@NotNull AbstractMaterialGroup group){
+ public ConflictBuilder addExcludedGroup(@NotNull AbstractMaterialGroup group) {
return addExcludedGroup(group.getName());
}
@@ -294,7 +296,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder removeExcludedGroup(@NotNull String groupName){
+ public ConflictBuilder removeExcludedGroup(@NotNull String groupName) {
excludedGroupNames.remove(groupName);
return this;
}
@@ -314,7 +316,7 @@ public class ConflictBuilder {
* @return This conflict builder instance.
*/
@NotNull
- public ConflictBuilder removeExcludedGroup(@NotNull AbstractMaterialGroup group){
+ public ConflictBuilder removeExcludedGroup(@NotNull AbstractMaterialGroup group) {
return removeExcludedGroup(group.getName());
}
@@ -327,7 +329,7 @@ public class ConflictBuilder {
public ConflictBuilder copy() {
ConflictBuilder copy = new ConflictBuilder(this.name, this.source);
- setMaxBeforeConflict(this.maxBeforeConflict);
+ copy.setMaxBeforeConflict(this.maxBeforeConflict);
// Set Enchantments
for (NamespacedKey key : this.enchantmentKeys) {
@@ -344,11 +346,13 @@ public class ConflictBuilder {
return copy;
}
+
/**
* Build a new Enchant conflict group by this builder.
+ *
* @return An Enchant conflict group with this builder parameters.
*/
- public EnchantConflictGroup build(){
+ public EnchantConflictGroup build() {
AbstractMaterialGroup materials = extractGroups();
EnchantConflictGroup conflict = new EnchantConflictGroup(getName(), materials, getMaxBeforeConflict());
appendEnchantments(conflict);
@@ -358,10 +362,21 @@ public class ConflictBuilder {
/**
* Register this conflict if not yet registered.
- * Equivalent to {@link ConflictAPI#addConflict(ConflictBuilder)}
+ * Equivalent to {@link ConflictAPI#addConflict(ConflictBuilder, boolean) ConflictAPI.addConflict(this, true)}}
+ *
* @return True if successful.
*/
- public boolean registerIfAbsent(){
+ public boolean registerIfAbsent() {
+ return ConflictAPI.addConflict(this, true);
+ }
+
+ /**
+ * Register this conflict if not yet registered or deleted.
+ * Equivalent to {@link ConflictAPI#addConflict(ConflictBuilder) ConflictAPI.addConflict(this)}
+ *
+ * @return True if successful.
+ */
+ public boolean registerIfNew() {
return ConflictAPI.addConflict(this);
}
@@ -370,15 +385,15 @@ public class ConflictBuilder {
*
* @param conflict The conflict target
*/
- protected void appendEnchantments(@NotNull EnchantConflictGroup conflict){
- for (String enchantmentName : getEnchantmentNames()){
- if(appendEnchantment(conflict, EnchantmentApi.getByName(enchantmentName))){
+ protected void appendEnchantments(@NotNull EnchantConflictGroup conflict) {
+ for (String enchantmentName : getEnchantmentNames()) {
+ if (appendEnchantments(conflict, EnchantmentApi.getListByName(enchantmentName)) == 0) {
CustomAnvil.instance.getLogger().warning("Could not find enchantment " + enchantmentName + " for conflict " + getName());
ConflictAPI.logConflictOrigin(this);
}
}
- for (NamespacedKey enchantmentKey : getEnchantmentKeys()){
- if(!appendEnchantment(conflict, EnchantmentApi.getByKey(enchantmentKey))){
+ for (NamespacedKey enchantmentKey : getEnchantmentKeys()) {
+ if (!appendEnchantment(conflict, EnchantmentApi.getByKey(enchantmentKey))) {
CustomAnvil.instance.getLogger().warning("Could not find enchantment " + enchantmentKey + " for conflict " + getName());
ConflictAPI.logConflictOrigin(this);
}
@@ -392,26 +407,44 @@ public class ConflictBuilder {
* @param enchantment The enchantment
* @return True if successful.
*/
- protected static boolean appendEnchantment(@NotNull EnchantConflictGroup conflict, @Nullable CAEnchantment enchantment){
- if(enchantment == null)
+ protected static boolean appendEnchantment(@NotNull EnchantConflictGroup conflict, @Nullable CAEnchantment enchantment) {
+ if (enchantment == null)
return false;
conflict.addEnchantment(enchantment);
return true;
}
+ /**
+ * Append a list of enchantments.
+ *
+ * @param conflict The conflict target
+ * @param enchantments List of enchantment to add
+ * @return Number of enchantment added
+ */
+ protected static int appendEnchantments(@NotNull EnchantConflictGroup conflict, @NotNull List enchantments) {
+ int numberValid = 0;
+ for (CAEnchantment enchantment : enchantments) {
+ if (appendEnchantment(conflict, enchantment)) {
+ numberValid++;
+ }
+ }
+
+ return numberValid;
+ }
+
/**
* Extract group abstract material group.
*
* @return The abstract material group from the builder.
*/
- protected AbstractMaterialGroup extractGroups(){
+ protected AbstractMaterialGroup extractGroups() {
ItemGroupManager itemGroupManager = ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager();
IncludeGroup group = new IncludeGroup(EnchantConflictManager.DEFAULT_GROUP_NAME);
for (String groupName : getExcludedGroupNames()) {
AbstractMaterialGroup materialGroup = itemGroupManager.get(groupName);
- if(materialGroup == null){
+ if (materialGroup == null) {
CustomAnvil.instance.getLogger().warning("Material group " + groupName + " do not exist but is ask by conflict " + getName());
ConflictAPI.logConflictOrigin(this);
continue;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/CustomAnvilRecipeApi.java b/src/main/java/xyz/alexcrea/cuanvil/api/CustomAnvilRecipeApi.java
index 69bfad2..8f80aa3 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/CustomAnvilRecipeApi.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/CustomAnvilRecipeApi.java
@@ -1,10 +1,10 @@
package xyz.alexcrea.cuanvil.api;
import io.delilaheve.CustomAnvil;
-import org.bukkit.Bukkit;
import org.bukkit.configuration.file.FileConfiguration;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.dependency.DependencyManager;
import xyz.alexcrea.cuanvil.gui.config.global.CustomRecipeConfigGui;
import xyz.alexcrea.cuanvil.recipe.AnvilCustomRecipe;
@@ -19,7 +19,7 @@ public class CustomAnvilRecipeApi {
private CustomAnvilRecipeApi(){}
- private static int saveChangeTask = -1;
+ private static Object saveChangeTask = null;
/**
* Write and add a custom anvil recipe.
@@ -29,10 +29,24 @@ public class CustomAnvilRecipeApi {
* @return True if successful.
*/
public static boolean addRecipe(@NotNull AnvilRecipeBuilder builder){
+ return addRecipe(builder, false);
+ }
+
+ /**
+ * Write and add a custom anvil recipe.
+ * Will not write the recipe if it already exists.
+ *
+ * @param builder The recipe builder to be based on
+ * @param overrideDeleted If we should write even if the recipe was previously deleted.
+ * @return True if successful.
+ */
+ public static boolean addRecipe(@NotNull AnvilRecipeBuilder builder, boolean overrideDeleted){
FileConfiguration config = ConfigHolder.CUSTOM_RECIPE_HOLDER.getConfig();
String name = builder.getName();
+ if(!overrideDeleted && ConfigHolder.CUSTOM_RECIPE_HOLDER.isDeleted(builder.getName())) return false;
if(config.contains(builder.getName())) return false;
+
if(builder.getName().contains(".")) {
CustomAnvil.instance.getLogger().warning("Custom anvil recipe " + name + " contain \".\" in its name but should not. this recipe is ignored.");
return false;
@@ -58,11 +72,13 @@ public class CustomAnvilRecipeApi {
prepareSaveTask();
// Add from gui
- CustomRecipeConfigGui.INSTANCE.updateValueForGeneric(recipe, true);
+ CustomRecipeConfigGui recipeConfigGui = CustomRecipeConfigGui.getCurrentInstance();
+ if(recipeConfigGui != null) recipeConfigGui.updateValueForGeneric(recipe, true);
return true;
}
+ // TODO remove by name and/or by builder (as name is keept) (and maybe create a get by name)
/**
* Remove a custom anvil recipe.
*
@@ -71,14 +87,16 @@ public class CustomAnvilRecipeApi {
*/
public static boolean removeRecipe(@NotNull AnvilCustomRecipe recipe){
// Remove from registry
- ConfigHolder.CUSTOM_RECIPE_HOLDER.getRecipeManager().cleanRemove(recipe);
+ boolean result = ConfigHolder.CUSTOM_RECIPE_HOLDER.getRecipeManager().cleanRemove(recipe);
+ if(!result) return false;
- // Write as null and save to file
- ConfigHolder.CUSTOM_RECIPE_HOLDER.getConfig().set(recipe.getName(), null);
+ // Delete and save to file
+ ConfigHolder.CUSTOM_RECIPE_HOLDER.delete(recipe.getName());
prepareSaveTask();
// Remove from gui
- CustomRecipeConfigGui.INSTANCE.removeGeneric(recipe);
+ CustomRecipeConfigGui recipeConfigGui = CustomRecipeConfigGui.getCurrentInstance();
+ if(recipeConfigGui != null) recipeConfigGui.removeGeneric(recipe);
return true;
}
@@ -87,12 +105,12 @@ public class CustomAnvilRecipeApi {
* Prepare a task to save custom recipe configuration.
*/
private static void prepareSaveTask() {
- if(saveChangeTask != -1) return;
+ if(saveChangeTask != null) return;
- saveChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ saveChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, ()->{
ConfigHolder.CONFLICT_HOLDER.saveToDisk(true);
- saveChangeTask = -1;
- }, 0L);
+ saveChangeTask = null;
+ });
}
/**
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/EnchantmentApi.java b/src/main/java/xyz/alexcrea/cuanvil/api/EnchantmentApi.java
index d2244ad..ac98225 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/EnchantmentApi.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/EnchantmentApi.java
@@ -1,13 +1,14 @@
package xyz.alexcrea.cuanvil.api;
import io.delilaheve.CustomAnvil;
-import org.bukkit.Bukkit;
+import io.delilaheve.util.ConfigOptions;
import org.bukkit.NamespacedKey;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.enchantments.Enchantment;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.dependency.DependencyManager;
import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry;
import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
@@ -18,6 +19,7 @@ import xyz.alexcrea.cuanvil.gui.config.global.EnchantCostConfigGui;
import xyz.alexcrea.cuanvil.gui.config.global.EnchantLimitConfigGui;
import java.util.Collections;
+import java.util.List;
import java.util.Map;
/**
@@ -26,7 +28,7 @@ import java.util.Map;
@SuppressWarnings("unused")
public class EnchantmentApi {
- private static int saveChangeTask = -1;
+ private static Object saveChangeTask = null;
private EnchantmentApi() {}
@@ -104,7 +106,7 @@ public class EnchantmentApi {
* @return True if successful.
*/
public static boolean unregisterEnchantment(@NotNull NamespacedKey key){
- CAEnchantment enchantment = CAEnchantmentRegistry.getInstance().getByKey(key);
+ CAEnchantment enchantment = CAEnchantment.getByKey(key);
return unregisterEnchantment(enchantment);
}
@@ -126,7 +128,7 @@ public class EnchantmentApi {
*/
@Nullable
public static CAEnchantment getByKey(@NotNull NamespacedKey key){
- return CAEnchantmentRegistry.getInstance().getByKey(key);
+ return CAEnchantment.getByKey(key);
}
/**
@@ -134,10 +136,22 @@ public class EnchantmentApi {
*
* @param name The name used to fetch
* @return The custom anvil enchantment of this name. null if not found.
+ * @deprecated use {@link #getListByName(String)}
*/
+ @Deprecated(since = "1.6.3")
@Nullable
public static CAEnchantment getByName(@NotNull String name){
- return CAEnchantmentRegistry.getInstance().getByName(name);
+ return CAEnchantment.getByName(name);
+ }
+
+ /**
+ * Get list of enchantment using the provided name.
+ *
+ * @param name The name used to fetch
+ * @return List of custom anvil enchantments of this name. May be empty if not found.
+ */
+ public static List getListByName(@NotNull String name){
+ return CAEnchantment.getListByName(name);
}
/**
@@ -157,35 +171,49 @@ public class EnchantmentApi {
*/
public static boolean writeDefaultConfig(CAEnchantment enchantment, boolean override){
FileConfiguration config = ConfigHolder.DEFAULT_CONFIG.getConfig();
- if(!override && config.contains(enchantment.getName())) return false;
- writeDefaultConfig(config, enchantment);
-
- prepareSaveTask();
+ if(tryWriteDefaultConfig(config, enchantment, override)){
+ prepareSaveTask();
+ }
return true;
}
+ private static boolean tryWriteDefaultConfig(FileConfiguration defaultConfig, CAEnchantment enchantment, boolean override) {
+ boolean hasChange = false;
- private static void writeDefaultConfig(FileConfiguration defaultConfig, CAEnchantment enchantment) {
- defaultConfig.set("enchant_limits." + enchantment.getKey().getKey(), enchantment.defaultMaxLevel());
+ String levelPath = ConfigOptions.ENCHANT_LIMIT_ROOT + "." + enchantment.getKey();
+ if(override || !defaultConfig.isSet(levelPath)){
+ defaultConfig.set(levelPath, enchantment.defaultMaxLevel());
+ hasChange = true;
+ }
- String basePath = "enchant_values." + enchantment.getKey().getKey();
+ String basePath = ConfigOptions.ENCHANT_VALUES_ROOT + "." + enchantment.getKey();
EnchantmentRarity rarity = enchantment.defaultRarity();
- defaultConfig.set(basePath + ".item", rarity.getItemValue());
- defaultConfig.set(basePath + ".book", rarity.getBookValue());
+ String itemPath = basePath + ".item";
+ String bookPath = basePath + ".book";
+ if(override || !defaultConfig.isSet(itemPath)){
+ defaultConfig.set(itemPath, rarity.getItemValue());
+ hasChange = true;
+ }
+ if(override || !defaultConfig.isSet(bookPath)){
+ defaultConfig.set(bookPath, rarity.getBookValue());
+ hasChange = true;
+ }
+
+ return hasChange;
}
/**
* Prepare a task to save custom recipe configuration.
*/
private static void prepareSaveTask() {
- if(saveChangeTask != -1) return;
+ if(saveChangeTask != null) return;
- saveChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ saveChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, ()->{
ConfigHolder.DEFAULT_CONFIG.saveToDisk(true);
- saveChangeTask = -1;
- }, 0L);
+ saveChangeTask = null;
+ });
}
/**
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/MaterialGroupApi.java b/src/main/java/xyz/alexcrea/cuanvil/api/MaterialGroupApi.java
index b3283aa..cd71c7a 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/MaterialGroupApi.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/MaterialGroupApi.java
@@ -2,12 +2,13 @@ package xyz.alexcrea.cuanvil.api;
import io.delilaheve.CustomAnvil;
import io.delilaheve.util.ConfigOptions;
-import org.bukkit.Bukkit;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import org.bukkit.configuration.file.FileConfiguration;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.dependency.DependencyManager;
import xyz.alexcrea.cuanvil.group.AbstractMaterialGroup;
import xyz.alexcrea.cuanvil.group.ExcludeGroup;
import xyz.alexcrea.cuanvil.group.IncludeGroup;
@@ -22,30 +23,51 @@ import java.util.*;
@SuppressWarnings("unused")
public class MaterialGroupApi {
- private MaterialGroupApi(){}
+ private MaterialGroupApi() {
+ }
- private static int saveChangeTask = -1;
- private static int reloadChangeTask = -1;
+ private static Object saveChangeTask = null;
+ private static Object reloadChangeTask = null;
/**
* Write and add a group.
* Will not write the group if it already exists.
+ * Will not be successful if the group is empty.
*
- * @param group the group to add
+ * @param group The group to add
* @return true if successful.
*/
- public static boolean addMaterialGroup(@NotNull AbstractMaterialGroup group){
+ public static boolean addMaterialGroup(@NotNull AbstractMaterialGroup group) {
+ return addMaterialGroup(group, false);
+ }
+
+ /**
+ * Write and add a group.
+ * Will not write the group if it already exists.
+ * Will not be successful if the group is empty.
+ *
+ * @param group The group to add
+ * @param overrideDeleted If we should write even if the group was previously deleted.
+ * @return true if successful.
+ */
+ public static boolean addMaterialGroup(@NotNull AbstractMaterialGroup group, boolean overrideDeleted) {
ItemGroupManager itemGroupManager = ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager();
- if(itemGroupManager.get(group.getName()) != null) return false;
+
+ // Test if it exists/existed
+ if (!overrideDeleted && ConfigHolder.ITEM_GROUP_HOLDER.isDeleted(group.getName())) return false;
+ if (itemGroupManager.get(group.getName()) != null) return false;
+
+ // Add group
itemGroupManager.getGroupMap().put(group.getName(), group);
- if(!writeMaterialGroup(group, false)) return false;
+ if (!writeMaterialGroup(group, false)) return false;
- if(group instanceof IncludeGroup includeGroup){
- GroupConfigGui.INSTANCE.updateValueForGeneric(includeGroup, true);
+ if (group instanceof IncludeGroup includeGroup) {
+ GroupConfigGui configGui = GroupConfigGui.getCurrentInstance();
+ if (configGui != null) configGui.updateValueForGeneric(includeGroup, true);
}
- if(ConfigOptions.INSTANCE.getVerboseDebugLog()){
+ if (ConfigOptions.INSTANCE.getVerboseDebugLog()) {
CustomAnvil.instance.getLogger().info("Registered group " + group.getName());
}
@@ -58,9 +80,9 @@ public class MaterialGroupApi {
* You may want to use {@link #addMaterialGroup(AbstractMaterialGroup)} instead as it is more performance in most case as this function will reload every conflict.
*
* @param group the group to write
- * @return true if successful.
+ * @return true if was written successfully.
*/
- public static boolean writeMaterialGroup(@NotNull AbstractMaterialGroup group){
+ public static boolean writeMaterialGroup(@NotNull AbstractMaterialGroup group) {
return writeMaterialGroup(group, true);
}
@@ -71,64 +93,82 @@ public class MaterialGroupApi {
*
* @param group the group to write
* @param updatePlanned if we should plan a global update for material groups
- * @return true if successful.
+ * @return true if was written successfully.
*/
- public static boolean writeMaterialGroup(@NotNull AbstractMaterialGroup group, boolean updatePlanned){
+ public static boolean writeMaterialGroup(@NotNull AbstractMaterialGroup group, boolean updatePlanned) {
String name = group.getName();
- if(name.contains(".")) {
- CustomAnvil.instance.getLogger().warning("Group " + name +" contain . in its name but should not. this material group is ignored.");
+ if (name.contains(".")) {
+ CustomAnvil.instance.getLogger().warning("Group " + name + " contain . in its name but should not. this material group is ignored.");
return false;
}
- if(group instanceof IncludeGroup includeGroup){
- writeKnownGroup("include", includeGroup);
- }else if(group instanceof ExcludeGroup excludeGroup){
- writeKnownGroup("exclude", excludeGroup);
- }else{
- writeUnknownGroup(group);
+ boolean changed;
+ if (group instanceof IncludeGroup includeGroup) {
+ changed = writeKnownGroup("include", includeGroup);
+ } else if (group instanceof ExcludeGroup excludeGroup) {
+ throw new UnsupportedOperationException("exclude group is temporarily disable for the time being. sorry");
+ // This code do not do what is intended ? idk why do it exist
+ //changed = writeKnownGroup("exclude", excludeGroup);
+ } else {
+ changed = writeUnknownGroup(group);
}
+ if (!changed) return false;
prepareSaveTask();
- if(updatePlanned) prepareUpdateTask();
+ if (updatePlanned) prepareUpdateTask();
return true;
}
- private static void writeKnownGroup(@NotNull String groupType, @NotNull AbstractMaterialGroup group){
+ private static boolean writeKnownGroup(@NotNull String groupType, @NotNull AbstractMaterialGroup group) {
FileConfiguration config = ConfigHolder.ITEM_GROUP_HOLDER.getConfig();
String basePath = group.getName() + ".";
- Set materialSet = group.getNonGroupInheritedMaterials();
+ Set materialSet = group.getNonGroupInheritedMaterials();
Set groupSet = group.getGroups();
- config.set(basePath + ItemGroupManager.GROUP_TYPE_PATH, groupType);
- if(!materialSet.isEmpty()){
+ boolean empty = true;
+ if (!materialSet.isEmpty()) {
config.set(basePath + ItemGroupManager.MATERIAL_LIST_PATH, materialSetToStringList(materialSet));
+ empty = false;
+ } else {
+ config.set(basePath + ItemGroupManager.MATERIAL_LIST_PATH, null);
}
- if(!groupSet.isEmpty()){
- config.set(basePath + ItemGroupManager.GROUP_LIST_PATH, materialGroupSEtToStringList(groupSet));
+ if (!groupSet.isEmpty()) {
+ config.set(basePath + ItemGroupManager.GROUP_LIST_PATH, materialGroupSetToStringList(groupSet));
+ empty = false;
+ } else {
+ config.set(basePath + ItemGroupManager.GROUP_LIST_PATH, null);
}
+ if (empty) {
+ config.set(basePath + ItemGroupManager.GROUP_TYPE_PATH, null);
+ return false;
+ }
+
+ config.set(basePath + ItemGroupManager.GROUP_TYPE_PATH, groupType);
+ return true;
}
- private static void writeUnknownGroup(@NotNull AbstractMaterialGroup group) {
+ private static boolean writeUnknownGroup(@NotNull AbstractMaterialGroup group) {
FileConfiguration config = ConfigHolder.ITEM_GROUP_HOLDER.getConfig();
String basePath = group.getName() + ".";
- EnumSet materials = group.getMaterials();
+ Set materials = group.getMaterials();
+
+ if (materials.isEmpty()) return false;
config.set(basePath + ItemGroupManager.GROUP_TYPE_PATH, "include");
- if(!materials.isEmpty()){
- config.set(basePath + ItemGroupManager.MATERIAL_LIST_PATH, materialSetToStringList(materials));
- }
+ config.set(basePath + ItemGroupManager.MATERIAL_LIST_PATH, materialSetToStringList(materials));
+ return true;
}
- public static List materialSetToStringList(@NotNull Set materials){
- return materials.stream().map(material -> material.getKey().getKey().toLowerCase()).toList();
+ public static List materialSetToStringList(@NotNull Set materials) {
+ return materials.stream().map(NamespacedKey::toString).toList();
}
- public static List materialGroupSEtToStringList(@NotNull Set groups){
+ public static List materialGroupSetToStringList(@NotNull Set groups) {
return groups.stream().map(AbstractMaterialGroup::getName).toList();
}
@@ -138,19 +178,21 @@ public class MaterialGroupApi {
* For that reason, it is not recommended to use this function.
*
* @param group The recipe to remove
- * @return True if successful.
+ * @return True if the group was present.
*/
- public static boolean removeGroup(@NotNull AbstractMaterialGroup group){
+ public static boolean removeGroup(@NotNull AbstractMaterialGroup group) {
// Remove from registry
- ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager().groupMap.remove(group.getName());
+ AbstractMaterialGroup removed = ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager().groupMap.remove(group.getName());
+ if (removed == null) return false;
- // Write as null and save to file
- ConfigHolder.ITEM_GROUP_HOLDER.getConfig().set(group.getName(), null);
+ // Delete and save to file
+ ConfigHolder.ITEM_GROUP_HOLDER.delete(group.getName());
prepareSaveTask();
// Remove from gui
- if(group instanceof IncludeGroup includeGroup){
- GroupConfigGui.INSTANCE.removeGeneric(includeGroup);
+ if (group instanceof IncludeGroup includeGroup) {
+ GroupConfigGui configGui = GroupConfigGui.getCurrentInstance();
+ if (configGui != null) configGui.removeGeneric(includeGroup);
}
return true;
@@ -160,25 +202,28 @@ public class MaterialGroupApi {
* Prepare a task to reload every conflict.
*/
private static void prepareSaveTask() {
- if(saveChangeTask != -1) return;
+ if (saveChangeTask != null) return;
- saveChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ saveChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, () -> {
ConfigHolder.ITEM_GROUP_HOLDER.saveToDisk(true);
- saveChangeTask = -1;
- }, 0L);
+ saveChangeTask = null;
+ });
}
/**
* Prepare a task to save configuration.
*/
private static void prepareUpdateTask() {
- if(reloadChangeTask != -1) return;
+ if (reloadChangeTask != null) return;
- reloadChangeTask = Bukkit.getScheduler().scheduleSyncDelayedTask(CustomAnvil.instance, ()->{
+ reloadChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, () -> {
ConfigHolder.ITEM_GROUP_HOLDER.reload();
- GroupConfigGui.INSTANCE.reloadValues();
- reloadChangeTask = -1;
- }, 0L);
+
+ GroupConfigGui configGui = GroupConfigGui.getCurrentInstance();
+ if (configGui != null) configGui.reloadValues();
+
+ reloadChangeTask = null;
+ });
}
@@ -189,16 +234,17 @@ public class MaterialGroupApi {
* @return the abstract group of this name. null if not found.
*/
@Nullable
- public static AbstractMaterialGroup getGroup(@NotNull String groupName){
+ public static AbstractMaterialGroup getGroup(@NotNull String groupName) {
return ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager().get(groupName);
}
/**
* Get every registered material groups.
+ *
* @return An immutable map of group name as its key and group as mapped value.
*/
@NotNull
- public static Map getRegisteredGroups(){
+ public static Map getRegisteredGroups() {
return Collections.unmodifiableMap(ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager().getGroupMap());
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/UnitRepairApi.java b/src/main/java/xyz/alexcrea/cuanvil/api/UnitRepairApi.java
new file mode 100644
index 0000000..bc50c16
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/UnitRepairApi.java
@@ -0,0 +1,218 @@
+package xyz.alexcrea.cuanvil.api;
+
+import io.delilaheve.CustomAnvil;
+import kotlin.Triple;
+import org.bukkit.Material;
+import org.bukkit.configuration.ConfigurationSection;
+import org.bukkit.configuration.file.FileConfiguration;
+import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.dependency.DependencyManager;
+import xyz.alexcrea.cuanvil.gui.config.global.UnitRepairConfigGui;
+import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
+import xyz.alexcrea.cuanvil.gui.config.list.UnitRepairElementListGui;
+
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
+/**
+ * Custom Anvil api for unit repair.
+ */
+@SuppressWarnings("unused")
+public class UnitRepairApi {
+
+ private UnitRepairApi(){}
+
+ private static Object saveChangeTask = null;
+
+ /**
+ * Write and add a custom anvil unit repair recipe.
+ * Will not write the recipe if it already exists or was deleted.
+ * Set the value to minecraft default value (0.25 = 25%)
+ *
+ * @param unit The unit material used to repair the bellow item.
+ * @param repairable The item to be repaired.
+ * @return true if successful.
+ */
+ public static boolean addUnitRepair(@NotNull Material unit, @NotNull Material repairable){
+ return addUnitRepair(unit, repairable, 0.25, false);
+ }
+
+ /**
+ * Write and add a custom anvil unit repair recipe.
+ * Will not write the recipe if it already exists or was deleted.
+ *
+ * @param unit The unit material used to repair the bellow item.
+ * @param repairable The item to be repaired.
+ * @param value The amount to be repaired by every unit. (1% = 0.01)
+ * @return true if successful.
+ */
+ public static boolean addUnitRepair(@NotNull Material unit, @NotNull Material repairable, double value){
+ return addUnitRepair(unit, repairable, value, false);
+ }
+
+ /**
+ * Write and add a custom anvil unit repair recipe.
+ * Will not write the recipe if it already exists.
+ *
+ * @param unit The unit material used to repair the bellow item.
+ * @param repairable The item to be repaired.
+ * @param value The amount to be repaired by every unit. (1% = 0.01)
+ * @param overrideDeleted If we should write even if the recipe was previously deleted.
+ * @return true if successful.
+ */
+ public static boolean addUnitRepair(@NotNull Material unit, @NotNull Material repairable, double value, boolean overrideDeleted){
+ FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
+ String path = unit.name().toLowerCase() + "." + repairable.name().toLowerCase();
+
+ if(!overrideDeleted && ConfigHolder.UNIT_REPAIR_HOLDER.isDeleted(path)) return false;
+ if(config.contains(path)) return false;
+
+ // Set unit repair
+ return setUnitRepair(unit, repairable, value);
+ }
+
+ /**
+ * Write and add a custom anvil unit repair recipe.
+ * Do not check if it previously existed or exist.
+ *
+ * @param unit The unit material used to repair the bellow item.
+ * @param repairable The item to be repaired.
+ * @param value The amount to be repaired by every unit. (1% = 0.01)
+ * @return true if successful.
+ */
+ public static boolean setUnitRepair(@NotNull Material unit, @NotNull Material repairable, double value){
+ FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
+
+ String repairableName = repairable.name().toLowerCase();
+ String path = unit.name().toLowerCase() + "." + repairableName;
+
+ // Add to config then prepare save
+ config.set(path, value);
+ prepareSaveTask();
+
+ // Add to gui
+ UnitRepairConfigGui repairConfigGui = UnitRepairConfigGui.getCurrentInstance();
+ if(repairConfigGui != null) {
+ UnitRepairElementListGui elementGui = repairConfigGui.getInstanceOrCreate(unit).getStored();
+
+ if(elementGui != null) elementGui.updateValueForGeneric(repairableName, true);
+ repairConfigGui.updateValueForGeneric(unit, true);
+ }
+
+ return true;
+ }
+
+ /**
+ * Remove a custom anvil unit repair recipe.
+ *
+ * @param unit The unit material used to repair the bellow item.
+ * @param repairable The item used to be repaired.
+ * @return true if successful.
+ */
+ public static boolean removeUnitRepair(@NotNull Material unit, @NotNull Material repairable){
+ // Delete every possible variation and save to file
+ String unitName = unit.name();
+ String repairableName = repairable.name();
+
+ FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
+ config.set(unitName.toLowerCase() + "." + repairableName.toUpperCase(), null);
+ config.set(unitName.toUpperCase() + "." + repairableName.toLowerCase(), null);
+ config.set(unitName.toUpperCase() + "." + repairableName.toUpperCase(), null);
+ config.set(unitName.toLowerCase() + "." + repairableName.toLowerCase(), null);
+
+ // Test if it was the last value of this section
+ boolean lastValue = false;
+ if(config.isConfigurationSection(unitName.toLowerCase())) {
+ ConfigurationSection section = config.getConfigurationSection(unitName.toLowerCase());
+
+ if(section != null && section.getKeys(false).isEmpty()) {
+ lastValue = true;
+ config.set(unitName.toLowerCase(), null);
+ }
+
+ } else if (config.isConfigurationSection(unitName.toUpperCase())) {
+ ConfigurationSection section = config.getConfigurationSection(unitName.toUpperCase());
+ if(section != null && section.getKeys(false).isEmpty()) {
+ lastValue = true;
+ config.set(unitName.toUpperCase(), null);
+ }
+
+ } else lastValue = true;
+
+
+ // We only need to "delete" as the lower case to be counted as deleted
+ ConfigHolder.UNIT_REPAIR_HOLDER.delete(unitName.toLowerCase() + "." + repairableName.toLowerCase());
+ prepareSaveTask();
+
+ // Remove from gui
+ UnitRepairConfigGui repairConfigGui = UnitRepairConfigGui.getCurrentInstance();
+ if(repairConfigGui != null) {
+ UnitRepairElementListGui elementGui = repairConfigGui.getInstanceOrCreate(unit).getStored();
+
+ if(elementGui != null) elementGui.removeGeneric(repairableName);
+ if(lastValue){
+ repairConfigGui.removeGeneric(unit);
+ }
+ }
+
+ return true;
+ }
+
+ /**
+ * Prepare a task to save custom unit repair recipe configuration.
+ */
+ private static void prepareSaveTask() {
+ if(saveChangeTask != null) return;
+
+ saveChangeTask = DependencyManager.scheduler.scheduleGlobally(CustomAnvil.instance, ()->{
+ ConfigHolder.UNIT_REPAIR_HOLDER.saveToDisk(true);
+ saveChangeTask = null;
+ });
+ }
+
+ /**
+ * Get every unit repair recipes.
+ * @return An immutable collection of unit repair recipes.
+ *
+ * Each element of the provided triple represent a part of the recipe
+ *
+ * - First object is the unit material used to repair the bellow item.
+ *
- Second object is the item to be repaired.
+ *
- Last object is the amount to be repaired by every unit. (1% = 0.01)
+ *
+ */
+ @NotNull
+ public static List> getUnitRepairs(){
+ List> mutableList = new ArrayList<>();
+
+ FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
+ for (String unitKey : config.getKeys(false)) {
+ // Test if config section exist
+ if(!config.isConfigurationSection(unitKey)) continue;
+
+ // Test if unit is a material
+ Material unit = Material.getMaterial(unitKey.toUpperCase());
+ if(unit == null) continue;
+
+ // Iterate over reparable items
+ ConfigurationSection section = config.getConfigurationSection(unitKey);
+ for (String repairableKey : section.getKeys(false)) {
+ // Test if value section exist
+ if(!section.isDouble(repairableKey)) continue;
+
+ // Test if repairable is valid a material
+ Material repairable = Material.getMaterial(repairableKey.toUpperCase());
+ if(repairable == null) continue;
+
+ // Add the values
+ mutableList.add(new Triple<>(unit, repairable, section.getDouble(repairableKey)));
+
+ }
+ }
+
+ return Collections.unmodifiableList(mutableList);
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/CAConfigReadyEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/CAConfigReadyEvent.java
index 24691db..67d27a8 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/event/CAConfigReadyEvent.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/CAConfigReadyEvent.java
@@ -3,6 +3,23 @@ package xyz.alexcrea.cuanvil.api.event;
import org.bukkit.event.Event;
import org.bukkit.event.HandlerList;
+/**
+ * Called when the configuration of CustomAnvil is ready.
+ * It is called either on the plugin startup or on the plugin config reload.
+ *
+ * If you want to listen to the first trigger of this event (first configuration load. aka plugin load)
+ * you will need to register the listener on your plugin onEnable or earlier
+ *
+ * This event indicate that can start to register your recipes, item groups and conflicts.
+ * The vanilla and custom enchantments should already have been provided to CustomAnvil.
+ * Configuration can be changed any time after this event is triggered but never before.
+ *
+ * use {@link xyz.alexcrea.cuanvil.api.ConflictAPI ConflictApi},
+ * {@link xyz.alexcrea.cuanvil.gui.config.global.CustomRecipeConfigGui CustomRecipeConfigGui},
+ * {@link xyz.alexcrea.cuanvil.api.MaterialGroupApi MaterialGroupApi}
+ * and {@link xyz.alexcrea.cuanvil.api.UnitRepairApi UnitRepairApi}
+ * to add/remove/edit configurations
+ */
public class CAConfigReadyEvent extends Event {
private static final HandlerList HANDLERS = new HandlerList();
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/CAEnchantRegistryReadyEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/CAEnchantRegistryReadyEvent.java
index 3e2fdf8..3ffe372 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/api/event/CAEnchantRegistryReadyEvent.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/CAEnchantRegistryReadyEvent.java
@@ -3,6 +3,17 @@ package xyz.alexcrea.cuanvil.api.event;
import org.bukkit.event.Event;
import org.bukkit.event.HandlerList;
+/**
+ * Called when custom anvil is ready to accept registration on custom enchantment.
+ *
+ * If you want to listen this event
+ * you will need to register the listener on your plugin onEnable or earlier
+ *
+ * Custom enchantments may be registered later but may cause issue if registered too later
+ * (after configuration loading phase. see {@link CAConfigReadyEvent})
+ *
+ * use {@link xyz.alexcrea.cuanvil.api.EnchantmentApi EnchantmentApi} to register and unregister your custom enchantments
+ */
public class CAEnchantRegistryReadyEvent extends Event {
private static final HandlerList HANDLERS = new HandlerList();
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAClickResultBypassEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAClickResultBypassEvent.java
new file mode 100644
index 0000000..fe5e199
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAClickResultBypassEvent.java
@@ -0,0 +1,63 @@
+package xyz.alexcrea.cuanvil.api.event.listener;
+
+import org.bukkit.event.Cancellable;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.inventory.InventoryClickEvent;
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * Called before custom anvil process the click on the result on the anvil inventory.
+ *
+ * This event is called after checking that the inventory is an anvil inventory and that the click is on the result slot
+ * but before checking if the player has the custom anvil affected permission.
+ *
+ * This event being cancelled will make CustomAnvil abort the click on result process.
+ *
+ * Most of the time you would likely need {@link CAPreAnvilBypassEvent} or {@link CAEarlyPreAnvilBypassEvent}
+ * for this event to be useful.
+ *
+ * There is also {@link CATreatAnvilResult2Event} that may be better for some use case.
+ */
+public class CAClickResultBypassEvent extends Event implements Cancellable {
+
+ private static final HandlerList HANDLERS = new HandlerList();
+
+ public static HandlerList getHandlerList() {
+ return HANDLERS;
+ }
+
+ @Override
+ public @NotNull HandlerList getHandlers() {
+ return HANDLERS;
+ }
+
+ private boolean cancelled = false;
+
+ @Override
+ public boolean isCancelled() {
+ return cancelled;
+ }
+
+ @Override
+ public void setCancelled(boolean cancel) {
+ this.cancelled = cancel;
+ }
+
+ @NotNull
+ private final InventoryClickEvent event;
+
+ /**
+ * Get the bukkit inventory click event causing to this event
+ *
+ * @return The click event causing to this event
+ */
+ @NotNull
+ public InventoryClickEvent getEvent() {
+ return event;
+ }
+
+ public CAClickResultBypassEvent(@NotNull InventoryClickEvent event) {
+ this.event = event;
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAEarlyPreAnvilBypassEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAEarlyPreAnvilBypassEvent.java
new file mode 100644
index 0000000..e92b4cd
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAEarlyPreAnvilBypassEvent.java
@@ -0,0 +1,63 @@
+package xyz.alexcrea.cuanvil.api.event.listener;
+
+import org.bukkit.event.Cancellable;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.inventory.PrepareAnvilEvent;
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * Called before custom anvil process the prepare anvil event.
+ *
+ * This event will always get called when CustomAnvil need to handle
+ *
+ * This event being cancelled will make CustomAnvil abort the anvil process.
+ *
+ * You should also use {@link CAClickResultBypassEvent} if you want to use this event for something useful.
+ *
+ * It is also recommended that you read about {@link CAPreAnvilBypassEvent} and {@link CATreatAnvilResult2Event}
+ * as your use case may be more prone to use theses.
+ */
+public class CAEarlyPreAnvilBypassEvent extends Event implements Cancellable {
+
+ private static final HandlerList HANDLERS = new HandlerList();
+
+ public static HandlerList getHandlerList() {
+ return HANDLERS;
+ }
+
+ @Override
+ public @NotNull HandlerList getHandlers() {
+ return HANDLERS;
+ }
+
+ private boolean cancelled = false;
+
+ @Override
+ public boolean isCancelled() {
+ return cancelled;
+ }
+
+ @Override
+ public void setCancelled(boolean cancel) {
+ this.cancelled = cancel;
+ }
+
+ @NotNull
+ private final PrepareAnvilEvent event;
+
+ /**
+ * Get the bukkit pre anvil event causing this event
+ *
+ * @return The pre anvil event causing to this event
+ */
+ @NotNull
+ public PrepareAnvilEvent getEvent() {
+ return event;
+ }
+
+ public CAEarlyPreAnvilBypassEvent(@NotNull PrepareAnvilEvent event) {
+ this.event = event;
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAPreAnvilBypassEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAPreAnvilBypassEvent.java
new file mode 100644
index 0000000..9103a4b
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CAPreAnvilBypassEvent.java
@@ -0,0 +1,66 @@
+package xyz.alexcrea.cuanvil.api.event.listener;
+
+import org.bukkit.event.Cancellable;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.inventory.PrepareAnvilEvent;
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * Called before custom anvil process the prepare anvil event.
+ *
+ * This event is called after {@link CAEarlyPreAnvilBypassEvent},
+ * after checking that there is at least an item on the left slot
+ * and after checking if any of the 2 item is marked as immutable
+ * but before checking if the player has the custom anvil affected permission.
+ *
+ * This event being cancelled will make CustomAnvil abort the anvil process.
+ *
+ * You should also use {@link CAClickResultBypassEvent} if you want to use this event for something useful.
+ *
+ * It is also recommended that you read about {@link CAEarlyPreAnvilBypassEvent} and {@link CATreatAnvilResult2Event}
+ * as your use case may be more prone to use theses.
+ */
+public class CAPreAnvilBypassEvent extends Event implements Cancellable {
+
+ private static final HandlerList HANDLERS = new HandlerList();
+
+ public static HandlerList getHandlerList() {
+ return HANDLERS;
+ }
+
+ @Override
+ public @NotNull HandlerList getHandlers() {
+ return HANDLERS;
+ }
+
+ private boolean cancelled = false;
+
+ @Override
+ public boolean isCancelled() {
+ return cancelled;
+ }
+
+ @Override
+ public void setCancelled(boolean cancel) {
+ this.cancelled = cancel;
+ }
+
+ @NotNull
+ private final PrepareAnvilEvent event;
+
+ /**
+ * Get the bukkit pre anvil event causing this event
+ *
+ * @return The pre anvil event causing this event
+ */
+ @NotNull
+ public PrepareAnvilEvent getEvent() {
+ return event;
+ }
+
+ public CAPreAnvilBypassEvent(@NotNull PrepareAnvilEvent event) {
+ this.event = event;
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResult2Event.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResult2Event.java
new file mode 100644
index 0000000..30c5380
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResult2Event.java
@@ -0,0 +1,196 @@
+package xyz.alexcrea.cuanvil.api.event.listener;
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.InventoryView;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.anvil.AnvilCost;
+import xyz.alexcrea.cuanvil.anvil.AnvilUseType;
+
+/**
+ * Called after custom anvil processed the click on the result on the anvil inventory.
+ * This event should be used to modify the result of an anvil use.
+ *
+ * You may also want to check {@link CAClickResultBypassEvent},
+ * {@link CAPreAnvilBypassEvent}
+ * and {@link CAEarlyPreAnvilBypassEvent} for your use case
+ *
+ * A null result will cancel this event
+ */
+@SuppressWarnings("unused")
+public class CATreatAnvilResult2Event extends Event {
+
+ private static final HandlerList HANDLERS = new HandlerList();
+
+ public static HandlerList getHandlerList() {
+ return HANDLERS;
+ }
+
+ @Override
+ public @NotNull HandlerList getHandlers() {
+ return HANDLERS;
+ }
+
+ @NotNull
+ private final InventoryView view;
+
+ private final AnvilUseType useType;
+
+ @Nullable
+ private final ItemStack left;
+ @Nullable
+ private final ItemStack right;
+
+ @Nullable
+ private ItemStack result;
+
+ private final AnvilCost cost;
+
+ @ApiStatus.Internal
+ public CATreatAnvilResult2Event(
+ @NotNull InventoryView view,
+ Inventory inv,
+ AnvilUseType useType,
+ @Nullable ItemStack result,
+ AnvilCost cost) {
+ this.view = view;
+ this.useType = useType;
+
+ this.left = inv.getItem(0); // TODO use view here
+ this.right = inv.getItem(1);
+ this.result = result;
+ this.cost = cost;
+ }
+
+ /**
+ * Get the bukkit inventory view.
+ *
+ * Temporarily marked as internal as it will get changed to anvil view on legacy removal
+ * so signature will change
+ *
+ * @return The inventory view of this event.
+ */
+ @ApiStatus.Internal
+ public @NotNull InventoryView getView() {
+ return view;
+ }
+
+
+ /**
+ * Get the type of use source of the result.
+ *
+ * @return The craft use type.
+ */
+ public AnvilUseType getUseType() {
+ return useType;
+ }
+
+ /**
+ * Get the left item of the anvil use
+ *
+ * @return the left item
+ */
+ public @Nullable ItemStack getLeftItem() {
+ return left;
+ }
+
+ /**
+ * Get the right item of the anvil use
+ *
+ * @return the right item
+ */
+ public @Nullable ItemStack getRightItem() {
+ return right;
+ }
+
+ /**
+ * Get the current result
+ *
+ * note that it will not be null unless another listener previously set it to null.
+ *
+ * @return The current result.
+ */
+ public @Nullable ItemStack getResult() {
+ return result;
+ }
+
+ /**
+ * Set the current result
+ *
+ * note that a null result will cancel this anvil use.
+ *
+ * @param result The new result
+ */
+ public void setResult(@Nullable ItemStack result) {
+ this.result = result;
+ }
+
+ /**
+ * Get the level cost displayed on the anvil.
+ *
Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ *
+ * @return The current cost.
+ * @deprecated use #{@link #getCost()} instead
+ */
+ @Deprecated(forRemoval = true, since = "1.17.0")
+ public int getLevelCost() {
+ return cost.asXpCost();
+ }
+
+ /**
+ * Set the level cost displayed on the anvil.
+ * Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ *
+ * @param levelCost The new cost.
+ * @deprecated use #{@link #getCost()} and set value on this instead
+ */
+ @Deprecated(forRemoval = true, since = "1.17.0")
+ public void setLevelCost(int levelCost) {
+ cost.setGeneric(levelCost - cost.getGeneric() - cost.asXpCost());
+ }
+
+ /**
+ * Allow access to the current cost of the event
+ * Note that modifying this object will change the event resulting cost
+ *
+ * Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ * @return the current anvil cost
+ */
+ public AnvilCost getCost() {
+ return cost;
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResultEvent.java b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResultEvent.java
new file mode 100644
index 0000000..80965b5
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/api/event/listener/CATreatAnvilResultEvent.java
@@ -0,0 +1,162 @@
+package xyz.alexcrea.cuanvil.api.event.listener;
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.inventory.PrepareAnvilEvent;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.anvil.AnvilCost;
+import xyz.alexcrea.cuanvil.anvil.AnvilUseType;
+
+/**
+ * Called after custom anvil processed the click on the result on the anvil inventory.
+ * This event should be used to modify the result of an anvil use.
+ *
+ * You may also want to check {@link CAClickResultBypassEvent},
+ * {@link CAPreAnvilBypassEvent}
+ * and {@link CAEarlyPreAnvilBypassEvent} for your use case
+ *
+ * A null result will cancel this pre anvil event
+ *
+ * @deprecated Prepare anvil Event cannot be provided as it can be called on result and therefore not have prepared anvil event
+ * use {@link CATreatAnvilResult2Event} instead
+ */
+@SuppressWarnings("unused")
+@Deprecated(forRemoval = true, since = "1.17.0")
+public class CATreatAnvilResultEvent extends Event {
+
+ private static final HandlerList HANDLERS = new HandlerList();
+
+ public static HandlerList getHandlerList() {
+ return HANDLERS;
+ }
+
+ @Override
+ public @NotNull HandlerList getHandlers() {
+ return HANDLERS;
+ }
+
+ @NotNull
+ private final PrepareAnvilEvent event;
+
+ private final AnvilUseType useType;
+
+ @Nullable
+ private ItemStack result;
+
+ private final AnvilCost cost;
+
+ public CATreatAnvilResultEvent(@NotNull PrepareAnvilEvent event, AnvilUseType useType, @Nullable ItemStack result, AnvilCost cost) {
+ this.event = event;
+ this.useType = useType;
+ this.result = result;
+ this.cost = cost;
+ }
+
+ /**
+ * Get the bukkit inventory click event causing to this event.
+ *
+ * @return The click event causing to this event.
+ */
+ public @NotNull PrepareAnvilEvent getEvent() {
+ return event;
+ }
+
+ /**
+ * Get the type of use source of the result.
+ *
+ * @return The craft use type.
+ */
+ public AnvilUseType getUseType() {
+ return useType;
+ }
+
+ /**
+ * Get the current result
+ *
+ * note that it will not be null unless another listener previously set it to null.
+ *
+ * @return The current result.
+ */
+ public @Nullable ItemStack getResult() {
+ return result;
+ }
+
+ /**
+ * Set the current result
+ *
+ * note that a null result will cancel this anvil use.
+ *
+ * @param result The new result
+ */
+ public void setResult(@Nullable ItemStack result) {
+ this.result = result;
+ }
+
+ /**
+ * Get the level cost displayed on the anvil.
+ *
Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ *
+ * @return The current cost.
+ * @deprecated use #{@link #getCost()} instead
+ */
+ @Deprecated(forRemoval = true, since = "1.17.0")
+ public int getLevelCost() {
+ return cost.asXpCost();
+ }
+
+ /**
+ * Set the level cost displayed on the anvil.
+ * Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ *
+ * @param levelCost The new cost.
+ * @deprecated use #{@link #getCost()} and set value on this instead
+ */
+ @Deprecated(forRemoval = true, since = "1.17.0")
+ public void setLevelCost(int levelCost) {
+ cost.setGeneric(levelCost - cost.getGeneric() - cost.asXpCost());
+ }
+
+ /**
+ * Allow access to the current cost of the event
+ * Note that modifying this object will change the event resulting cost
+ *
+ * Important note:
+ * the final price are re calculated on click for the following use case:
+ *
+ * - Custom craft
+ * - Unit repair
+ * - Lore edit
+ *
+ * This value will be used as final price for:
+ * Item merge
+ * Item rename
+ *
+ * @return the current anvil cost
+ */
+ public AnvilCost getCost() {
+ return cost;
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/config/ConfigHolder.java b/src/main/java/xyz/alexcrea/cuanvil/config/ConfigHolder.java
index 3cfaf03..f6a7e80 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/config/ConfigHolder.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/config/ConfigHolder.java
@@ -4,13 +4,18 @@ import com.google.common.io.Files;
import io.delilaheve.CustomAnvil;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.group.EnchantConflictManager;
import xyz.alexcrea.cuanvil.group.ItemGroupManager;
import xyz.alexcrea.cuanvil.recipe.CustomAnvilRecipeManager;
+import xyz.alexcrea.cuanvil.util.MetricsUtil;
import java.io.File;
import java.io.IOException;
+import java.util.logging.Level;
+@SuppressWarnings("unused")
public abstract class ConfigHolder {
// Available configuration:
@@ -140,7 +145,8 @@ public abstract class ConfigHolder {
Files.copy(base, firstBackup);
sufficientSuccess = true;
} catch (IOException e) {
- e.printStackTrace();
+ CustomAnvil.instance.getLogger().log(Level.WARNING, "Could not copy backup saving config " + base.getName(), e);
+ MetricsUtil.INSTANCE.trackError(e);
}
}
// save last backup
@@ -200,15 +206,128 @@ public abstract class ConfigHolder {
YamlConfiguration configuration = CustomAnvil.instance.reloadResource(
getConfigFileName() + getConfigFileExtension(), hardFail);
if (configuration == null) return false;
+
this.configuration = configuration;
reload();
+
return true;
}
}
+ public abstract static class DeletableResource extends ResourceConfigHolder{
+
+ private static final String DELETED_FOLDER_PATH = "deleted";
+
+ private final @NotNull File parent;
+ private final @NotNull File deletedConfigFile;
+
+ private @Nullable YamlConfiguration deletedListConfig;
+ private DeletableResource(String resourceName) {
+ super(resourceName);
+ this.parent = new File(CustomAnvil.instance.getDataFolder(), DELETED_FOLDER_PATH);
+ this.deletedConfigFile = new File(this.parent, "deleted_" + resourceName + getConfigFileExtension());
+ }
+
+ @Override
+ public boolean reloadFromDisk(boolean hardFail) {
+ if(!super.reloadFromDisk(hardFail)) return false;
+ loadDeletedListFile(hardFail);
+
+ return true;
+ }
+
+ private void loadDeletedListFile(boolean hardFail){
+ this.deletedListConfig = CustomAnvil.instance.reloadResource(this.deletedConfigFile, hardFail);
+
+ }
+
+ /**
+ * Test if the provided element was deleted.
+ * @param objectPath The object path to delete.
+ * @return True if successful.
+ */
+ public boolean isDeleted(String objectPath){
+ if(this.deletedListConfig == null) return false;
+
+ return this.deletedListConfig.getBoolean(objectPath, false);
+ }
+
+ /**
+ * Delete a certain object by its path. do not save the config.
+ * @param objectPath The object path to delete.
+ * @return True if successful.
+ */
+ public boolean delete(String objectPath){
+ return delete(objectPath, false, false);
+ }
+
+ /**
+ * Delete a certain object by its path.
+ * @param objectPath The object path to delete.
+ * @param doSave If we should save the config after deleting.
+ * @param doBackup If we should create a backup.
+ * @return True if successful.
+ */
+ public boolean delete(String objectPath, boolean doSave, boolean doBackup){
+ // Create deleted list if it does not yet exist
+ if(this.deletedListConfig == null){
+ this.parent.mkdirs();
+ try {
+ this.deletedConfigFile.createNewFile();
+ } catch (IOException e) {
+ CustomAnvil.instance.getLogger().log(Level.WARNING, "Could not create " + this.deletedConfigFile.getPath(), e);
+ MetricsUtil.INSTANCE.trackError(e);
+ }
+ loadDeletedListFile(false);
+
+ // Something was wrong somehow
+ if(this.deletedListConfig == null) return false;
+ }
+
+ // Add to the deleted config
+ this.deletedListConfig.set(objectPath, true);
+ this.getConfig().set(objectPath, null);
+
+ // Save the deleted config (may not be the most efficient, but I will handle it later)
+ if(doSave){
+ return saveToDisk(doBackup);
+ }
+
+ return true;
+ }
+
+ @Override
+ public boolean saveToDisk(boolean doBackup) {
+ boolean deletedSaveSuccess = saveDeletedList();
+
+ return super.saveToDisk(doBackup) && deletedSaveSuccess;
+ }
+
+ /**
+ * Save list of deleted elements.
+ * @return true if successful.
+ */
+ public boolean saveDeletedList() {
+ if(this.deletedListConfig == null) return true;
+
+ try {
+ this.deletedListConfig.save(this.deletedConfigFile);
+ } catch (IOException e) {
+ CustomAnvil.instance.getLogger().log(Level.WARNING, "Could not save " + this.deletedConfigFile.getPath(), e);
+ MetricsUtil.INSTANCE.trackError(e);
+ return false;
+ }
+
+ return true;
+ }
+
+
+ }
+
+
// Class for itemGroupsManager config
- public static class ItemGroupConfigHolder extends ResourceConfigHolder {
+ public static class ItemGroupConfigHolder extends DeletableResource {
private static final String FILE_NAME = "item_groups";
ItemGroupManager itemGroupsManager;
@@ -235,7 +354,7 @@ public abstract class ConfigHolder {
}
// Class for enchant conflict config
- public static class ConflictConfigHolder extends ResourceConfigHolder {
+ public static class ConflictConfigHolder extends DeletableResource {
private static final String FILE_NAME = "enchant_conflict";
EnchantConflictManager conflictManager;
@@ -259,10 +378,9 @@ public abstract class ConfigHolder {
}
// Class for unit repair config
- public static class UnitRepairHolder extends ResourceConfigHolder {
+ public static class UnitRepairHolder extends DeletableResource {
private static final String ITEM_GROUP_FILE_NAME = "unit_repair_item";
-
private UnitRepairHolder() {
super(ITEM_GROUP_FILE_NAME);
}
@@ -275,7 +393,7 @@ public abstract class ConfigHolder {
// Class for custom anvil craft
- public static class CustomAnvilCraftHolder extends ResourceConfigHolder {
+ public static class CustomAnvilCraftHolder extends DeletableResource {
private static final String CUSTOM_RECIPE_FILE_NAME = "custom_recipes";
CustomAnvilRecipeManager recipeManager;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/config/WorkPenaltyType.java b/src/main/java/xyz/alexcrea/cuanvil/config/WorkPenaltyType.java
new file mode 100644
index 0000000..d374999
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/config/WorkPenaltyType.java
@@ -0,0 +1,66 @@
+package xyz.alexcrea.cuanvil.config;
+
+import com.google.common.collect.ImmutableMap;
+import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.anvil.AnvilUseType;
+
+import java.util.EnumMap;
+
+public class WorkPenaltyType {
+
+ public record WorkPenaltyPart(
+ boolean penaltyIncrease,
+ boolean penaltyAdditive,
+ boolean exclusivePenaltyIncrease,
+ boolean exclusivePenaltyAdditive
+ ) {
+
+ @Override
+ public boolean equals(Object obj) {
+ if(!(obj instanceof WorkPenaltyPart other)) return false;
+
+ return other.penaltyIncrease == this.penaltyIncrease &&
+ other.penaltyAdditive == this.penaltyAdditive &&
+ other.exclusivePenaltyIncrease == this.exclusivePenaltyIncrease &&
+ other.exclusivePenaltyAdditive == this.exclusivePenaltyAdditive;
+ }
+
+ public WorkPenaltyPart(boolean penaltyIncrease, boolean penaltyAdditive) {
+ this(penaltyIncrease, penaltyAdditive, false, false);
+ }
+ }
+
+ private final EnumMap partMap;
+
+ public WorkPenaltyType(@Nullable EnumMap partMap) {
+ this.partMap = new EnumMap<>(partMap != null ? partMap : new EnumMap<>(AnvilUseType.class));
+ }
+
+ public ImmutableMap getPartMap() {
+ return ImmutableMap.copyOf(partMap);
+ }
+
+ public WorkPenaltyPart getPenaltyInfo(AnvilUseType type) {
+ return partMap.getOrDefault(type, type.getDefaultPenalty());
+ }
+
+ public boolean isPenaltyIncreasing(AnvilUseType type) {
+ return partMap.getOrDefault(type, type.getDefaultPenalty()).penaltyIncrease;
+ }
+
+ public boolean isPenaltyAdditive(AnvilUseType type) {
+ return partMap.getOrDefault(type, type.getDefaultPenalty()).penaltyAdditive;
+ }
+
+ @Override
+ public boolean equals(Object o) {
+ if (this == o) return true;
+ if (!(o instanceof WorkPenaltyType that)) return false;
+
+ for (AnvilUseType type : AnvilUseType.getEntries()) {
+ if(!getPenaltyInfo(type).equals(that.getPenaltyInfo(type))) return false;
+ }
+ return true;
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/AdditionalTestEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/AdditionalTestEnchantment.java
index 832e5af..821838f 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/AdditionalTestEnchantment.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/AdditionalTestEnchantment.java
@@ -1,6 +1,7 @@
package xyz.alexcrea.cuanvil.enchant;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import org.bukkit.inventory.ItemStack;
import org.jetbrains.annotations.NotNull;
@@ -11,24 +12,23 @@ public interface AdditionalTestEnchantment {
/**
* Test if the provided enchantments can be compatible with this enchantment. only non-Custom Anvil conflict.
* @param enchantments Immutable map of validated enchantments for the item.
- * @param itemMat Material of the tested item.
+ * @param itemType Material namespaced key of the tested item.
* @return If there is a conflict with the enchantments.
*/
boolean isEnchantConflict(
@NotNull Map enchantments,
- @NotNull Material itemMat);
-
+ @NotNull NamespacedKey itemType);
/**
* Test if the provided item can be compatible with this enchantment. only non-Custom Anvil conflict.
* @param enchantments Immutable map of validated enchantments for the item.
- * @param itemMat Material of the tested item.
+ * @param itemType Material namespaced key of the tested item.
* @param item Provide a new instance of the used item stack with the partial enchantment applied.
* @return If there is a conflict with the enchantment and the item.
*/
boolean isItemConflict(
@NotNull Map enchantments,
- @NotNull Material itemMat,
+ @NotNull NamespacedKey itemType,
@NotNull ItemStack item);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantment.java
index 0303733..ea657ac 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantment.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantment.java
@@ -12,6 +12,7 @@ import xyz.alexcrea.cuanvil.group.EnchantConflictGroup;
import java.util.Collection;
import java.util.HashMap;
+import java.util.List;
import java.util.Map;
/**
@@ -206,7 +207,6 @@ public interface CAEnchantment {
@NotNull ItemMeta meta,
@NotNull Map enchantments,
@NotNull Collection enchantmentToTest){
-
for (CAEnchantment enchantment : enchantmentToTest) {
if(enchantment.isEnchantmentPresent(item, meta)){
enchantments.put(enchantment, enchantment.getLevel(item, meta));
@@ -226,12 +226,24 @@ public interface CAEnchantment {
}
/**
- * Gets a list of all the unoptimised enchantments.
- * @param name The enchantment name
- * @return List of enchantment.
+ * Gets the enchantment by the provided name.
+ * @param name Name to fetch.
+ * @return Registered enchantment. null if absent.
+ *
+ * @deprecated use {@link #getListByName(String)}
*/
+ @Deprecated(since = "1.6.3")
static @Nullable CAEnchantment getByName(@NotNull String name){
return CAEnchantmentRegistry.getInstance().getByName(name);
}
+ /**
+ * Gets list of enchantment using the provided name.
+ * @param name Name to fetch.
+ * @return List of registered enchantment.
+ */
+ static List getListByName(@NotNull String name){
+ return CAEnchantmentRegistry.getInstance().getListByName(name);
+ }
+
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantmentRegistry.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantmentRegistry.java
index f4d3dec..854ed55 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantmentRegistry.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/CAEnchantmentRegistry.java
@@ -5,10 +5,12 @@ import org.bukkit.NamespacedKey;
import org.bukkit.enchantments.Enchantment;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.enchant.bulk.BukkitEnchantBulkOperation;
import xyz.alexcrea.cuanvil.enchant.bulk.BulkCleanEnchantOperation;
import xyz.alexcrea.cuanvil.enchant.bulk.BulkGetEnchantOperation;
import xyz.alexcrea.cuanvil.enchant.wrapped.CABukkitEnchantment;
+import xyz.alexcrea.cuanvil.util.MetricsUtil;
import java.util.*;
import java.util.logging.Level;
@@ -16,13 +18,16 @@ import java.util.logging.Level;
public class CAEnchantmentRegistry {
private static final CAEnchantmentRegistry instance = new CAEnchantmentRegistry();
+
public static CAEnchantmentRegistry getInstance() {
return instance;
}
// Register enchantment functions
private final HashMap byKeyMap;
- private final HashMap byNameMap;
+ private final HashMap> byNameMap;
+
+ private final SortedSet nameSortedEnchantments;
private final List unoptimisedGetValues;
private final List unoptimisedCleanValues;
@@ -34,6 +39,8 @@ public class CAEnchantmentRegistry {
byKeyMap = new HashMap<>();
byNameMap = new HashMap<>();
+ nameSortedEnchantments = new TreeSet<>(Comparator.comparing(CAEnchantment::getName));
+
unoptimisedGetValues = new ArrayList<>();
unoptimisedCleanValues = new ArrayList<>();
@@ -45,7 +52,7 @@ public class CAEnchantmentRegistry {
* This should only be called on main of custom anvil.
* If called more than one time, chance of thing being broken will be high.
*/
- public void registerBukkit(){
+ public void registerBukkit() {
// Register enchantment
for (Enchantment enchantment : Enchantment.values()) {
register(new CABukkitEnchantment(enchantment));
@@ -55,38 +62,58 @@ public class CAEnchantmentRegistry {
BukkitEnchantBulkOperation bukkitOperation = new BukkitEnchantBulkOperation();
optimisedGetOperators.add(bukkitOperation);
optimisedCleanOperators.add(bukkitOperation);
-
}
+ private static boolean hasWarnedRegistering = false;
+
/**
* Can be used to register new enchantment.
*
* No guarantee that the enchantment will be present on the config gui if registered late.
* (By late I mean after custom anvil startup.)
+ *
* @param enchantment The enchantment to be registered.
* @return If the operation was successful.
*/
- public boolean register(@NotNull CAEnchantment enchantment){
- if(byKeyMap.containsKey(enchantment.getKey())){
+ public boolean register(@NotNull CAEnchantment enchantment) {
+ if (byKeyMap.containsKey(enchantment.getKey())) {
+ if (Objects.equals(enchantment, byKeyMap.get(enchantment.getKey()))) {
+ // We are trying to register the exact same enchantment. so we just skip it.
+ return false;
+ }
+
+ if (ConfigHolder.DEFAULT_CONFIG.getConfig().getBoolean("caution_secret_do_not_log_duplicated_registered_key", false)) {
+ return false;
+ }
+
+ var error = new IllegalStateException("enchantment " + enchantment.getKey() + " was already registered");
CustomAnvil.instance.getLogger().log(Level.WARNING,
- "Duplicate registered enchantment. This should NOT happen.",
- new IllegalStateException(enchantment.getKey()+" enchantment was already registered"));
+ "Duplicate distinct registered enchantment. This should NOT happen any time.\n" +
+ "If you are a custom anvil developer: Maybe custom anvil detected your enchantment as a bukkit enchantment. " +
+ "you should maybe remove enchantment with the same key before registering yours",
+ error);
+ MetricsUtil.INSTANCE.trackError(error);
return false;
}
- if(byNameMap.containsKey(enchantment.getName())){
+
+ if ((!hasWarnedRegistering) && byNameMap.containsKey(enchantment.getName())) {
+ hasWarnedRegistering = true;
+
CustomAnvil.instance.getLogger().log(Level.WARNING,
- "Duplicate registered enchantment name. There will have issue. " +
- "\nI hope this do not happen to you on a production server. If it do, there is probably a plugin trying to register an enchantment with the same name than another one",
- new IllegalStateException(enchantment.getKey()+" enchantment name was already registered"));
+ "Duplicate registered enchantment name. Please check that configuration is using namespace.");
}
byKeyMap.put(enchantment.getKey(), enchantment);
- byNameMap.put(enchantment.getName(), enchantment);
- if(!enchantment.isGetOptimised()){
+ byNameMap.putIfAbsent(enchantment.getName(), new ArrayList<>());
+ byNameMap.get(enchantment.getName()).add(enchantment);
+
+ nameSortedEnchantments.add(enchantment);
+
+ if (!enchantment.isGetOptimised()) {
unoptimisedGetValues.add(enchantment);
}
- if(!enchantment.isCleanOptimised()){
+ if (!enchantment.isCleanOptimised()) {
unoptimisedCleanValues.add(enchantment);
}
@@ -100,14 +127,17 @@ public class CAEnchantmentRegistry {
*
* No guarantee that the enchantment will absent if the config guis if unregistered late.
* (By late I mean after custom anvil startup.)
+ *
* @param enchantment The enchantment to be unregistered.
* @return If the operation was successful.
*/
- public boolean unregister(@Nullable CAEnchantment enchantment){
- if(enchantment == null) return false;
+ public boolean unregister(@Nullable CAEnchantment enchantment) {
+ if (enchantment == null) return false;
byKeyMap.remove(enchantment.getKey());
- byNameMap.remove(enchantment.getName());
+ byNameMap.get(enchantment.getName()).remove(enchantment);
+
+ nameSortedEnchantments.remove(enchantment);
unoptimisedGetValues.remove(enchantment);
unoptimisedCleanValues.remove(enchantment);
@@ -116,26 +146,45 @@ public class CAEnchantmentRegistry {
/**
* Gets the enchantment by the provided key.
+ *
* @param key Key to fetch.
* @return Registered enchantment. null if absent.
*/
@Nullable
- public CAEnchantment getByKey(@NotNull NamespacedKey key){
+ public CAEnchantment getByKey(@NotNull NamespacedKey key) {
return byKeyMap.get(key);
}
/**
* Gets the enchantment by the provided name.
+ *
* @param name Name to fetch.
* @return Registered enchantment. null if absent.
+ * @deprecated use {@link #getListByName(String)}
*/
+ @Deprecated(since = "1.6.3")
@Nullable
- public CAEnchantment getByName(@NotNull String name){
- return byNameMap.get(name);
+ public CAEnchantment getByName(@NotNull String name) {
+ List enchantments = getListByName(name);
+ if (enchantments.isEmpty()) return null;
+
+ return enchantments.get(0);
+ }
+
+ /**
+ * Gets list of enchantment using the provided name.
+ *
+ * @param name Name to fetch.
+ * @return List of registered enchantment.
+ */
+ @NotNull
+ public List getListByName(@NotNull String name) {
+ return byNameMap.getOrDefault(name, Collections.emptyList());
}
/**
* Gets an array of all the registered enchantments.
+ *
* @return Array of enchantments.
*/
@NotNull
@@ -145,14 +194,16 @@ public class CAEnchantmentRegistry {
/**
* Gets a map of all the registered enchantments.
- * @return Map of enchantments.
+ *
+ * @return Immutable map of enchantments.
*/
public Map registeredEnchantments() {
- return byKeyMap;
+ return Collections.unmodifiableMap(byKeyMap);
}
/**
* Gets a list of all the unoptimised get operation enchantments.
+ *
* @return List of unoptimised enchantments.
*/
@NotNull
@@ -162,6 +213,7 @@ public class CAEnchantmentRegistry {
/**
* Gets a list of all the unoptimised clean operation enchantments.
+ *
* @return List of unoptimised enchantments.
*/
@NotNull
@@ -171,7 +223,8 @@ public class CAEnchantmentRegistry {
/**
* Get "clean optimised operation" for get enchantments.
- * @return Get mutable "clean enchantments optimised operation" list.
+ *
+ * @return Mutable "clean enchantments optimised operation" list.
*/
public List getOptimisedCleanOperators() {
return optimisedCleanOperators;
@@ -179,10 +232,19 @@ public class CAEnchantmentRegistry {
/**
* Get "get optimised operation" for get enchantments.
- * @return Get mutable "get enchantments optimised operation" list.
+ *
+ * @return Mutable "get enchantments optimised operation" list.
*/
public List getOptimisedGetOperators() {
return optimisedGetOperators;
}
+ /**
+ * Get custom anvil enchantment sorted by name.
+ *
+ * @return An immutable sorted set of every registered enchantment sorted by name.
+ */
+ public SortedSet getNameSortedEnchantments() {
+ return Collections.unmodifiableSortedSet(nameSortedEnchantments);
+ }
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentRarity.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentRarity.java
index 57ccb72..3718f39 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentRarity.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentRarity.java
@@ -12,12 +12,12 @@ public class EnchantmentRarity {
private final int itemValue;
private final int bookValue;
- public EnchantmentRarity(int itemValue, int bookValue) {
+ private EnchantmentRarity(int itemValue, int bookValue) {
this.itemValue = itemValue;
this.bookValue = bookValue;
}
- public EnchantmentRarity(int itemValue) {
+ private EnchantmentRarity(int itemValue) {
this(itemValue, Math.max(1, itemValue / 2));
}
@@ -29,4 +29,24 @@ public class EnchantmentRarity {
return itemValue;
}
+
+ public static EnchantmentRarity getRarity(int itemValue, int bookValue){
+ int expectedBook = Math.max(1, itemValue / 2);
+ if((expectedBook == bookValue) && (itemValue != 0)) return getRarity(itemValue);
+
+ if(itemValue == 0 && bookValue == 0) return NO_RARITY;
+ return new EnchantmentRarity(itemValue, bookValue);
+ }
+
+ public static EnchantmentRarity getRarity(int itemValue){
+ return switch (itemValue) {
+ case 0 -> NO_RARITY;
+ case 1 -> COMMON;
+ case 2 -> UNCOMMON;
+ case 4 -> RARE;
+ case 8 -> VERY_RARE;
+ default -> new EnchantmentRarity(itemValue);
+ };
+ }
+
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BukkitEnchantBulkOperation.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BukkitEnchantBulkOperation.java
index d34bd49..73e4185 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BukkitEnchantBulkOperation.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BukkitEnchantBulkOperation.java
@@ -1,7 +1,10 @@
package xyz.alexcrea.cuanvil.enchant.bulk;
+import io.delilaheve.CustomAnvil;
+import io.delilaheve.util.ConfigOptions;
import io.delilaheve.util.ItemUtil;
import org.bukkit.Material;
+import org.bukkit.enchantments.Enchantment;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.EnchantmentStorageMeta;
import org.bukkit.inventory.meta.ItemMeta;
@@ -14,22 +17,37 @@ import java.util.Map;
public class BukkitEnchantBulkOperation implements BulkGetEnchantOperation, BulkCleanEnchantOperation {
@Override
- public void bulkGet(@NotNull Map enchantmentList, @NotNull ItemStack item, @NotNull ItemMeta meta) {
- if (ItemUtil.INSTANCE.isEnchantedBook(item)) {
- ((EnchantmentStorageMeta)meta).getStoredEnchants().forEach((enchantment, level) ->
- enchantmentList.put(EnchantmentApi.getByKey(enchantment.getKey()), level)
+ public void bulkGet(@NotNull Map enchantmentMap, @NotNull ItemStack item, @NotNull ItemMeta meta) {
+ boolean isBook = ItemUtil.INSTANCE.isEnchantedBook(item);
+
+ if (isBook) {
+ ((EnchantmentStorageMeta) meta).getStoredEnchants().forEach((enchantment, level) ->
+ addEnchantment(enchantmentMap, enchantment, level)
);
- } else {
+ }
+ if(!isBook || ConfigOptions.INSTANCE.getAddBookEnchantmentAsStoredEnchantment()){
item.getEnchantments().forEach((enchantment, level) ->
- enchantmentList.put(EnchantmentApi.getByKey(enchantment.getKey()), level)
+ addEnchantment(enchantmentMap, enchantment, level)
);
}
}
+ public void addEnchantment(@NotNull Map enchantmentMap, @NotNull Enchantment enchantment, int level) {
+ CAEnchantment enchant = EnchantmentApi.getByKey(enchantment.getKey());
+ if (enchant == null) {
+ CustomAnvil.instance.getLogger().warning("Enchantment of key " + enchantment.getKey() +
+ " somehow not found in CustomAnvil ?");
+ return;
+ }
+
+ enchantmentMap.put(enchant, level);
+ }
+
@Override
public void bulkClear(@NotNull ItemStack item) {
- if (item.getType() != Material.ENCHANTED_BOOK) {
- item.getEnchantments().forEach((enchantment, leve) ->
+ if (item.getType() != Material.ENCHANTED_BOOK || ConfigOptions.INSTANCE.getAddBookEnchantmentAsStoredEnchantment()) {
+
+ item.getEnchantments().forEach((enchantment, level) ->
item.removeEnchantment(enchantment)
);
}
@@ -43,5 +61,6 @@ public class BukkitEnchantBulkOperation implements BulkGetEnchantOperation, Bulk
bookMeta.removeStoredEnchant(enchantment)
);
}
+
}
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BulkGetEnchantOperation.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BulkGetEnchantOperation.java
index a985edd..7c66e8a 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BulkGetEnchantOperation.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/BulkGetEnchantOperation.java
@@ -14,10 +14,10 @@ public interface BulkGetEnchantOperation {
/**
* Bulk get part of the stored enchantment of this item.
- * @param enchantmentList Mutable map of collected enchantment. should b
+ * @param enchantmentMap Mutable map of collected enchantment. should b
* @param item The item to get enchantment from. Should not get edited.
* @param meta The item meta to get enchantment from. Should not get edited.
*/
- void bulkGet(@NotNull Map enchantmentList, @NotNull ItemStack item, @NotNull ItemMeta meta);
+ void bulkGet(@NotNull Map enchantmentMap, @NotNull ItemStack item, @NotNull ItemMeta meta);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/EnchantSquaredBulkOperation.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/EnchantSquaredBulkOperation.java
index 59bd6ec..57ecf60 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/EnchantSquaredBulkOperation.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/EnchantSquaredBulkOperation.java
@@ -5,7 +5,7 @@ import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.dependency.DependencyManager;
-import xyz.alexcrea.cuanvil.dependency.EnchantmentSquaredDependency;
+import xyz.alexcrea.cuanvil.dependency.plugins.EnchantmentSquaredDependency;
import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import java.util.Collections;
@@ -14,10 +14,10 @@ import java.util.Map;
public class EnchantSquaredBulkOperation implements BulkGetEnchantOperation, BulkCleanEnchantOperation {
@Override
- public void bulkGet(@NotNull Map enchantmentList, @NotNull ItemStack item, @NotNull ItemMeta meta) {
+ public void bulkGet(@NotNull Map enchantmentMap, @NotNull ItemStack item, @NotNull ItemMeta meta) {
EnchantmentSquaredDependency enchantmentSquared = DependencyManager.INSTANCE.getEnchantmentSquaredCompatibility();
if(enchantmentSquared != null){
- enchantmentSquared.getEnchantmentsSquared(item, enchantmentList);
+ enchantmentSquared.getEnchantmentsSquared(item, enchantmentMap);
}
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/SuperEnchantBulkOperation.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/SuperEnchantBulkOperation.java
new file mode 100644
index 0000000..8bc729a
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/bulk/SuperEnchantBulkOperation.java
@@ -0,0 +1,47 @@
+package xyz.alexcrea.cuanvil.enchant.bulk;
+
+import com.maddoxh.superEnchants.items.EnchantApplicator;
+import com.maddoxh.superEnchants.items.EnchantReader;
+import io.delilaheve.CustomAnvil;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.inventory.meta.ItemMeta;
+import org.bukkit.plugin.Plugin;
+import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.api.EnchantmentApi;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+
+import java.util.Map;
+
+public class SuperEnchantBulkOperation implements BulkGetEnchantOperation, BulkCleanEnchantOperation {
+
+ private Plugin plugin;
+ public SuperEnchantBulkOperation(Plugin plugin) {
+ this.plugin = plugin;
+ }
+
+ @Override
+ public void bulkGet(@NotNull Map enchantmentMap, @NotNull ItemStack item, @NotNull ItemMeta meta) {
+ EnchantReader.INSTANCE.readEnchants(item).forEach((ench, level) -> {
+ var enchantment = EnchantmentApi.getByKey(NamespacedKey.fromString(ench, plugin));
+ if(enchantment == null) {
+ CustomAnvil.log("Enchantment " + ench + " not found in custom anvil");
+ return;
+ }
+
+ enchantmentMap.put(enchantment, level);
+ }
+ );
+ }
+
+ @Override
+ public void bulkClear(@NotNull ItemStack item) {
+ EnchantApplicator.INSTANCE.clearAllCustomEnchants(item);
+ }
+
+ @Override
+ public void bulkClear(@NotNull ItemStack item, @NotNull ItemMeta meta) {
+ // item meta is not preferred for enchantment squared clear
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CABukkitEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CABukkitEnchantment.java
index 0f4a0e3..0e630ea 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CABukkitEnchantment.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CABukkitEnchantment.java
@@ -1,7 +1,10 @@
package xyz.alexcrea.cuanvil.enchant.wrapped;
+import io.delilaheve.CustomAnvil;
+import io.delilaheve.util.ConfigOptions;
import io.delilaheve.util.ItemUtil;
import org.bukkit.enchantments.Enchantment;
+import org.bukkit.enchantments.EnchantmentTarget;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.EnchantmentStorageMeta;
import org.bukkit.inventory.meta.ItemMeta;
@@ -11,24 +14,30 @@ import xyz.alexcrea.cuanvil.enchant.CAEnchantmentBase;
import xyz.alexcrea.cuanvil.enchant.EnchantmentProperties;
import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.HashMap;
import java.util.Locale;
+import java.util.Map;
+import java.util.Objects;
+import java.util.logging.Level;
/**
* Custom Anvil enchantment implementation for vanilla registered enchantment.
*/
public class CABukkitEnchantment extends CAEnchantmentBase {
- private final @NotNull Enchantment enchantment;
+ public final @NotNull Enchantment bukkit;
- public CABukkitEnchantment(@NotNull Enchantment enchantment, @Nullable EnchantmentRarity rarity){
- super(enchantment.getKey(),
+ public CABukkitEnchantment(@NotNull Enchantment bukkit, @Nullable EnchantmentRarity rarity) {
+ super(bukkit.getKey(),
rarity,
- enchantment.getMaxLevel());
- this.enchantment = enchantment;
+ bukkit.getMaxLevel());
+ this.bukkit = bukkit;
}
- public CABukkitEnchantment(@NotNull Enchantment enchantment){
- this(enchantment, getRarity(enchantment));
+ public CABukkitEnchantment(@NotNull Enchantment bukkit) {
+ this(bukkit, getRarity(bukkit));
}
@Override
@@ -44,33 +53,34 @@ public class CABukkitEnchantment extends CAEnchantmentBase {
@Override
public int getLevel(@NotNull ItemStack item, @NotNull ItemMeta meta) {
if (ItemUtil.INSTANCE.isEnchantedBook(item)) {
- return ((EnchantmentStorageMeta)meta).getStoredEnchantLevel(this.enchantment);
+ return ((EnchantmentStorageMeta) meta).getStoredEnchantLevel(this.bukkit);
} else {
- return meta.getEnchantLevel(this.enchantment);
+ return meta.getEnchantLevel(this.bukkit);
}
}
@Override
public boolean isEnchantmentPresent(@NotNull ItemStack item, @NotNull ItemMeta meta) {
if (ItemUtil.INSTANCE.isEnchantedBook(item)) {
- EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta)meta);
+ EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta) meta);
- return bookMeta.getStoredEnchants().containsKey(this.enchantment);
- }else{
- return item.containsEnchantment(this.enchantment);
+ return bookMeta.getStoredEnchants().containsKey(this.bukkit) ||
+ (ConfigOptions.INSTANCE.getAddBookEnchantmentAsStoredEnchantment() && item.containsEnchantment(this.bukkit));
+ } else {
+ return item.containsEnchantment(this.bukkit);
}
}
@Override
public void addEnchantmentUnsafe(@NotNull ItemStack item, int level) {
if (ItemUtil.INSTANCE.isEnchantedBook(item)) {
- EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta)item.getItemMeta());
+ EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta) item.getItemMeta());
assert bookMeta != null;
- bookMeta.addStoredEnchant(this.enchantment, level, true);
+ bookMeta.addStoredEnchant(this.bukkit, level, true);
item.setItemMeta(bookMeta);
} else {
- item.addUnsafeEnchantment(this.enchantment, level);
+ item.addUnsafeEnchantment(this.bukkit, level);
}
}
@@ -78,28 +88,87 @@ public class CABukkitEnchantment extends CAEnchantmentBase {
@Override
public void removeFrom(@NotNull ItemStack item) {
if (ItemUtil.INSTANCE.isEnchantedBook(item)) {
- EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta)item.getItemMeta());
+ EnchantmentStorageMeta bookMeta = ((EnchantmentStorageMeta) item.getItemMeta());
assert bookMeta != null;
- bookMeta.removeStoredEnchant(this.enchantment);
+ bookMeta.removeStoredEnchant(this.bukkit);
+ bookMeta.removeEnchant(this.bukkit);
item.setItemMeta(bookMeta);
- }else{
- item.removeEnchantment(this.enchantment);
+ } else {
+ item.removeEnchantment(this.bukkit);
}
}
@NotNull
- public static EnchantmentRarity getRarity(Enchantment enchantment){
+ public static EnchantmentRarity getRarity(Enchantment enchantment) {
try {
return EnchantmentProperties.valueOf(enchantment.getKey().getKey().toUpperCase(Locale.ENGLISH)).getRarity();
} catch (IllegalArgumentException ignored) {
- return EnchantmentRarity.COMMON;
+ return findRarity(enchantment);
}
}
@NotNull
protected Enchantment getEnchant() {
- return this.enchantment;
+ return this.bukkit;
}
+
+ private static Method getAnvilCostMethod;
+
+ static {
+ Class clazz = Enchantment.class;
+ try {
+ getAnvilCostMethod = clazz.getDeclaredMethod("getAnvilCost");
+ getAnvilCostMethod.setAccessible(true);
+
+ CustomAnvil.Companion.log("Detected getAnvilCost method");
+ } catch (NoSuchMethodException e) {
+ getAnvilCostMethod = null;
+ }
+
+ }
+
+ private static final Map targetToGroup = new HashMap<>();
+ static {
+ targetToGroup.put(EnchantmentTarget.ARMOR, "armors");
+ targetToGroup.put(EnchantmentTarget.ARMOR_HEAD, "helmets");
+ targetToGroup.put(EnchantmentTarget.ARMOR_TORSO, "chestplate");
+ targetToGroup.put(EnchantmentTarget.ARMOR_LEGS, "leggings");
+ targetToGroup.put(EnchantmentTarget.ARMOR_FEET, "boots");
+ targetToGroup.put(EnchantmentTarget.BOW, "bow");
+ targetToGroup.put(EnchantmentTarget.BREAKABLE, "can_unbreak");
+ targetToGroup.put(EnchantmentTarget.CROSSBOW, "crossbow");
+ targetToGroup.put(EnchantmentTarget.FISHING_ROD, "fishing_rod");
+ targetToGroup.put(EnchantmentTarget.TOOL, "tools");
+ targetToGroup.put(EnchantmentTarget.TRIDENT, "trident");
+ targetToGroup.put(EnchantmentTarget.VANISHABLE, "can_vanish");
+ targetToGroup.put(EnchantmentTarget.WEAPON, "swords");
+ targetToGroup.put(EnchantmentTarget.WEARABLE, "wearable");
+ }
+
+ private static EnchantmentRarity findRarity(Enchantment enchantment) {
+ if (getAnvilCostMethod == null) return EnchantmentRarity.COMMON;
+
+ try {
+ int itemCost = (int) getAnvilCostMethod.invoke(enchantment);
+
+ return EnchantmentRarity.getRarity(itemCost);
+ } catch (IllegalAccessException | InvocationTargetException e) {
+ CustomAnvil.instance.getLogger().log(Level.SEVERE, "could not find cost for enchantment " + enchantment.getKey(), e);
+
+ return EnchantmentRarity.COMMON;
+ }
+
+ }
+
+ @Override
+ public boolean equals(Object obj) {
+ if (!(obj instanceof CABukkitEnchantment other)) {
+ return false;
+ }
+
+ return Objects.equals(this.bukkit, other.getEnchant());
+ }
+
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEPreV5Enchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEPreV5Enchantment.java
new file mode 100644
index 0000000..783798d
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEPreV5Enchantment.java
@@ -0,0 +1,61 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import su.nightexpress.excellentenchants.api.enchantment.CustomEnchantment;
+import su.nightexpress.excellentenchants.api.enchantment.Definition;
+import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+public class CAEEPreV5Enchantment extends CABukkitEnchantment implements AdditionalTestEnchantment {
+
+ @NotNull CustomEnchantment eeenchantment;
+ @NotNull Definition definition;
+
+ public CAEEPreV5Enchantment(@NotNull CustomEnchantment enchantment) {
+ super(enchantment.getBukkitEnchantment(), getRarity(enchantment.getBukkitEnchantment()));
+ this.eeenchantment = enchantment;
+ try {
+ this.definition = (Definition) getDefinition.invoke(enchantment);
+ } catch (IllegalAccessException | InvocationTargetException e) {
+ throw new RuntimeException(e);
+ }
+
+ }
+
+ private final static Method getDefinition;
+ static {
+ try {
+ getDefinition = CustomEnchantment.class.getMethod("getDefinition");
+ } catch (NoSuchMethodException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ if (!definition.hasConflicts()) return false;
+
+ Set conflicts = definition.getConflicts();
+
+ for (CAEnchantment caEnchantment : enchantments.keySet()) {
+ if (conflicts.contains(caEnchantment.getName())) return true;
+ }
+
+ return false;
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType, @NotNull ItemStack item) {
+ if (Material.ENCHANTED_BOOK.getKey().equals(itemType)) return false;
+
+ return !definition.getSupportedItems().is(item);
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5Enchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5Enchantment.java
new file mode 100644
index 0000000..2d8f945
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5Enchantment.java
@@ -0,0 +1,128 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import su.nightexpress.excellentenchants.api.enchantment.CustomEnchantment;
+import su.nightexpress.excellentenchants.api.item.ItemSet;
+import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+public class CAEEV5Enchantment extends CABukkitEnchantment implements AdditionalTestEnchantment {
+
+ @NotNull CustomEnchantment eeenchantment;
+ @NotNull Object definition;
+
+ public CAEEV5Enchantment(@NotNull CustomEnchantment enchantment) {
+ super(enchantment.getBukkitEnchantment(), EnchantmentRarity.getRarity(getAnvilCost(enchantment)));
+ this.eeenchantment = enchantment;
+ this.definition = getDefinition(enchantment);
+
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ if (!hasConflicts()) return false;
+
+ Set conflicts = getExclusiveSet();
+
+ for (CAEnchantment caEnchantment : enchantments.keySet()) {
+ if (conflicts.contains(caEnchantment.getName())) return true;
+ if (conflicts.contains(caEnchantment.getKey().toString())) return true;
+ }
+
+ return false;
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType, @NotNull ItemStack item) {
+ if (Material.ENCHANTED_BOOK.getKey().equals(itemType)) return false;
+
+ String key = itemType.getKey();
+ ItemSet primary = eeenchantment.getPrimaryItems();
+ if (primary.getMaterials().contains(key)) return false;
+
+ ItemSet supported = eeenchantment.getSupportedItems();
+ if (supported.getMaterials().contains(key)) return false;
+
+ return true;
+ }
+
+
+ private static final Method getDefinitonMethod;
+
+ private static final Method getAnvilCostMethod;
+ private static final Method hasConflictsMethod;
+ private static final Method getExclusiveSetMethod;
+ static {
+ var enchClazz = CustomEnchantment.class;
+ try {
+ getDefinitonMethod = enchClazz.getDeclaredMethod("getDefinition");
+ } catch (NoSuchMethodException e) {
+ throw new RuntimeException(e);
+ }
+
+ Class definitionClazz;
+ try {
+ definitionClazz = Class.forName("su.nightexpress.excellentenchants.api.EnchantDefinition");
+ } catch (ClassNotFoundException e) {
+ try {
+ definitionClazz = Class.forName("su.nightexpress.excellentenchants.api.wrapper.EnchantDefinition");
+ } catch (ClassNotFoundException ex) {
+ throw new RuntimeException(ex);
+ }
+ }
+
+ // Now definition methods
+ try {
+ getAnvilCostMethod = definitionClazz.getDeclaredMethod("getAnvilCost");
+ hasConflictsMethod = definitionClazz.getDeclaredMethod("hasConflicts");
+ getExclusiveSetMethod = definitionClazz.getDeclaredMethod("getExclusiveSet");
+ } catch (NoSuchMethodException e) {
+ throw new RuntimeException(e);
+ }
+
+ }
+
+ private static Object getDefinition(CustomEnchantment enchantment) {
+ try {
+ return getDefinitonMethod.invoke(enchantment);
+ } catch (Exception e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+ private static int getAnvilCost(CustomEnchantment enchantment) {
+ try {
+ return (int) getAnvilCostMethod.invoke(getDefinition(enchantment));
+ } catch (IllegalAccessException | InvocationTargetException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+ private boolean hasConflicts() {
+ try {
+ return (boolean) hasConflictsMethod.invoke(definition);
+ } catch (IllegalAccessException | InvocationTargetException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+
+ private Set getExclusiveSet() {
+ try {
+ return (Set) getExclusiveSetMethod.invoke(definition);
+ } catch (IllegalAccessException | InvocationTargetException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5_4Enchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5_4Enchantment.java
new file mode 100644
index 0000000..7fb8627
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEEV5_4Enchantment.java
@@ -0,0 +1,29 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import org.bukkit.NamespacedKey;
+import org.jetbrains.annotations.NotNull;
+import su.nightexpress.excellentenchants.api.enchantment.CustomEnchantment;
+import xyz.alexcrea.cuanvil.dependency.plugins.ExcellentEnchant5_4EnchantSettings;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+
+import java.util.Map;
+
+public class CAEEV5_4Enchantment extends CAEEV5Enchantment {
+
+ public CAEEV5_4Enchantment(@NotNull CustomEnchantment enchantment) {
+ super(enchantment);
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemMat) {
+ if(super.isEnchantConflict(enchantments, itemMat)) return true;
+
+ var limit = ExcellentEnchant5_4EnchantSettings.anvilLimit();
+ var count = enchantments.keySet().stream()
+ .filter(key -> key instanceof CAEEV5_4Enchantment)
+ .count();
+
+ return count > limit;
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEcoEnchant.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEcoEnchant.java
index 0d85ffd..32d1346 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEcoEnchant.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEcoEnchant.java
@@ -2,13 +2,16 @@ package xyz.alexcrea.cuanvil.enchant.wrapped;
import com.willfp.ecoenchants.enchant.EcoEnchant;
import com.willfp.ecoenchants.target.EnchantmentTarget;
+import com.willfp.ecoenchants.type.EnchantmentType;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import org.bukkit.inventory.ItemStack;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+import java.util.HashMap;
import java.util.Map;
public class CAEcoEnchant extends CABukkitEnchantment implements AdditionalTestEnchantment {
@@ -21,33 +24,52 @@ public class CAEcoEnchant extends CABukkitEnchantment implements AdditionalTestE
}
@Override
- public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull Material itemMat) {
- if(!enchantments.isEmpty()) {
- if (this.ecoEnchant.getConflictsWithEverything()) {
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ if (enchantments.isEmpty()) return false;
+
+ // Check if there is only self
+ if (enchantments.size() == 1 && this.equals(enchantments.keySet().stream().findFirst().get()))
+ return false;
+
+ if (this.ecoEnchant.getConflictsWithEverything()) {
+ return true;
+ }
+
+ HashMap typeAmountMap = new HashMap<>();
+
+ for (CAEnchantment other : enchantments.keySet()) {
+ if (other instanceof CABukkitEnchantment otherVanilla
+ && this.ecoEnchant.conflictsWith(otherVanilla.getEnchant())) {
return true;
}
- for (CAEnchantment other : enchantments.keySet()) {
- if(other instanceof CABukkitEnchantment otherVanilla
- && this.ecoEnchant.conflictsWith(otherVanilla.getEnchant())){
+ if (other instanceof CAEcoEnchant ecoOther) {
+ EnchantmentType type = ecoOther.ecoEnchant.getType();
+ typeAmountMap.putIfAbsent(type, 0);
+
+ int amount = typeAmountMap.get(type) + 1;
+ if (amount > type.getLimit()) {
return true;
}
+
+ typeAmountMap.put(type, amount);
}
}
+
return false;
}
@Override
public boolean isItemConflict(@NotNull Map enchantments,
- @NotNull Material itemMat,
+ @NotNull NamespacedKey itemType,
@NotNull ItemStack item) {
- if(Material.ENCHANTED_BOOK.equals(itemMat)){
+ if (Material.ENCHANTED_BOOK.getKey().equals(itemType)) {
return false;
}
for (EnchantmentTarget target : this.ecoEnchant.getTargets()) {
- if(target.matches(item)){
+ if (target.matches(item)) {
return false;
}
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEnchantSquaredEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEnchantSquaredEnchantment.java
index 56b4bc7..8f1058e 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEnchantSquaredEnchantment.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAEnchantSquaredEnchantment.java
@@ -3,7 +3,6 @@ package xyz.alexcrea.cuanvil.enchant.wrapped;
import me.athlaeos.enchantssquared.enchantments.CustomEnchant;
import me.athlaeos.enchantssquared.managers.CustomEnchantManager;
import org.bukkit.entity.HumanEntity;
-import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
@@ -17,15 +16,20 @@ import java.util.Objects;
public class CAEnchantSquaredEnchantment extends CAEnchantmentBase {
public final @NotNull CustomEnchant enchant;
+
public CAEnchantSquaredEnchantment(@NotNull CustomEnchant enchant) {
super(Objects.requireNonNull(
- Objects.requireNonNull(DependencyManager.INSTANCE.getEnchantmentSquaredCompatibility()).getKeyFromEnchant(enchant)),
+ Objects.requireNonNull(DependencyManager.INSTANCE.getEnchantmentSquaredCompatibility()).getKeyFromEnchant(enchant)),
EnchantmentRarity.COMMON,
enchant.getMaxLevel());
this.enchant = enchant;
}
+ public @NotNull CustomEnchant getEnchant() {
+ return enchant;
+ }
+
@Override
public boolean isGetOptimised() {
return true;
@@ -62,4 +66,14 @@ public class CAEnchantSquaredEnchantment extends CAEnchantmentBase {
CustomEnchantManager.getInstance().removeEnchant(item, this.enchant.getType());
}
+
+ @Override
+ public boolean equals(Object obj) {
+ if (!(obj instanceof CAEnchantSquaredEnchantment other)) {
+ return false;
+ }
+
+ return this.enchant.equals(other.getEnchant());
+ }
+
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAIncompatibleAllEnchant.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAIncompatibleAllEnchant.java
new file mode 100644
index 0000000..552ecd4
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CAIncompatibleAllEnchant.java
@@ -0,0 +1,36 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.enchantments.Enchantment;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.enchant.*;
+
+import java.util.Map;
+
+/**
+ * Represent an enchantment incompatible with every other enchantments
+ */
+public class CAIncompatibleAllEnchant extends CABukkitEnchantment implements AdditionalTestEnchantment {
+
+ public CAIncompatibleAllEnchant(@NotNull Enchantment enchantment, @Nullable EnchantmentRarity rarity) {
+ super(enchantment, rarity);
+ }
+
+ public CAIncompatibleAllEnchant(@NotNull Enchantment enchantment) {
+ super(enchantment);
+ }
+
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ return !enchantments.isEmpty() && !(enchantments.size() == 1 && enchantments.containsKey(this));
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType, @NotNull ItemStack item) {
+ return false;
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEEEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEEEnchantment.java
new file mode 100644
index 0000000..74068d4
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEEEnchantment.java
@@ -0,0 +1,44 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import su.nightexpress.excellentenchants.api.enchantment.EnchantmentData;
+import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+
+import java.util.Map;
+import java.util.Set;
+
+public class CALegacyEEEnchantment extends CABukkitEnchantment implements AdditionalTestEnchantment {
+
+ @NotNull EnchantmentData eeenchantment;
+
+ public CALegacyEEEnchantment(@NotNull EnchantmentData enchantment) {
+ super(enchantment.getEnchantment(), EnchantmentRarity.getRarity(enchantment.getAnvilCost()));
+ this.eeenchantment = enchantment;
+
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ if (!eeenchantment.hasConflicts()) return false;
+
+ Set conflicts = eeenchantment.getConflicts();
+
+ for (CAEnchantment caEnchantment : enchantments.keySet()) {
+ if (conflicts.contains(caEnchantment.getName())) return true;
+ }
+
+ return false;
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType, @NotNull ItemStack item) {
+ if (Material.ENCHANTED_BOOK.getKey().equals(itemType)) return false;
+
+ return !eeenchantment.getSupportedItems().is(item);
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEcoEnchant.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEcoEnchant.java
new file mode 100644
index 0000000..cb24def
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CALegacyEcoEnchant.java
@@ -0,0 +1,68 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import com.willfp.ecoenchants.enchantments.EcoEnchant;
+import com.willfp.ecoenchants.enchantments.meta.EnchantmentTarget;
+import com.willfp.ecoenchants.enchantments.meta.EnchantmentType;
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.enchantments.Enchantment;
+import org.bukkit.inventory.ItemStack;
+import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+import xyz.alexcrea.cuanvil.util.MaterialUtil;
+
+import java.util.Map;
+
+public class CALegacyEcoEnchant extends CABukkitEnchantment implements AdditionalTestEnchantment {
+
+ private final @NotNull EcoEnchant ecoEnchant;
+
+ public CALegacyEcoEnchant(@NotNull EcoEnchant ecoEnchant, @NotNull Enchantment enchantment) {
+ super(enchantment, EnchantmentRarity.COMMON);
+ this.ecoEnchant = ecoEnchant;
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ if (enchantments.isEmpty()) return false;
+
+ EnchantmentType type = this.ecoEnchant.getType();
+ boolean isSingular = type.isSingular();
+
+ for (CAEnchantment other : enchantments.keySet()) {
+ if (other instanceof CABukkitEnchantment otherVanilla
+ && this.ecoEnchant.conflictsWith(otherVanilla.getEnchant())) {
+ return true;
+ }
+
+ if (isSingular &&
+ other != this &&
+ (other instanceof CALegacyEcoEnchant otherEco) &&
+ type.equals(otherEco.ecoEnchant.getType())) {
+ return true;
+ }
+ }
+
+ return false;
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments,
+ @NotNull NamespacedKey itemType,
+ @NotNull ItemStack item) {
+ if (Material.ENCHANTED_BOOK.getKey().equals(itemType)) {
+ return false;
+ }
+
+ var mat = MaterialUtil.INSTANCE.getMatFromKey(itemType);
+ for (EnchantmentTarget target : this.ecoEnchant.getTargets()) {
+ if (target.getMaterials().contains(mat)) {
+ return false;
+ }
+ }
+
+ return true;
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CASuperEnchantEnchantment.java b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CASuperEnchantEnchantment.java
new file mode 100644
index 0000000..6039dc8
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/enchant/wrapped/CASuperEnchantEnchantment.java
@@ -0,0 +1,76 @@
+package xyz.alexcrea.cuanvil.enchant.wrapped;
+
+import com.maddoxh.superEnchants.enchants.CustomEnchant;
+import com.maddoxh.superEnchants.enchants.EnchantManager;
+import com.maddoxh.superEnchants.items.EnchantApplicator;
+import com.maddoxh.superEnchants.items.EnchantReader;
+import com.maddoxh.superEnchants.util.ConflictChecker;
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.inventory.meta.ItemMeta;
+import org.bukkit.plugin.Plugin;
+import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.enchant.AdditionalTestEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantmentBase;
+import xyz.alexcrea.cuanvil.enchant.EnchantmentRarity;
+
+import java.util.HashMap;
+import java.util.Map;
+
+public class CASuperEnchantEnchantment extends CAEnchantmentBase implements AdditionalTestEnchantment {
+
+ private @NotNull CustomEnchant enchant;
+ private @NotNull EnchantManager enchantManager;
+
+ public CASuperEnchantEnchantment(@NotNull CustomEnchant enchant, @NotNull Plugin plugin, @NotNull EnchantManager enchantManager) {
+ super(NamespacedKey.fromString(enchant.getId(), plugin), EnchantmentRarity.COMMON, enchant.getMaxLevel());
+
+ this.enchant = enchant;
+ this.enchantManager = enchantManager;
+ }
+
+ @Override
+ public int getLevel(@NotNull ItemStack item, @NotNull ItemMeta meta) {
+ return EnchantReader.INSTANCE.getEnchantLevel(item, enchant.getId());
+ }
+
+ @Override
+ public boolean isEnchantmentPresent(@NotNull ItemStack item, @NotNull ItemMeta meta) {
+ return EnchantReader.INSTANCE.hasEnchant(item, enchant.getId());
+ }
+
+ @Override
+ public void addEnchantmentUnsafe(@NotNull ItemStack item, int level) {
+ EnchantApplicator.INSTANCE.applyEnchant(item, enchant.getId(), level);
+ }
+
+ @Override
+ public void removeFrom(@NotNull ItemStack item) {
+ EnchantApplicator.INSTANCE.removeEnchant(item, enchant.getId());
+ }
+
+ @Override
+ public boolean isEnchantConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType) {
+ var idMap = new HashMap();
+
+ enchantments.forEach((enchant, level) -> {
+ if(!(enchant instanceof CASuperEnchantEnchantment superEnch)) return;
+ idMap.put(superEnch.enchant.getId(), level);
+ });
+
+ return ConflictChecker.INSTANCE.hasConflict(
+ idMap,
+ enchant.getId(),
+ enchantManager
+ ) != null;
+ }
+
+ @Override
+ public boolean isItemConflict(@NotNull Map enchantments, @NotNull NamespacedKey itemType, @NotNull ItemStack item) {
+ if(Material.ENCHANTED_BOOK.equals(item.getType())) return false;
+
+ return !enchant.canApplyTo(item.getType());
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/MainConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/MainConfigGui.java
index b6c1062..cc4fddc 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/MainConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/MainConfigGui.java
@@ -8,9 +8,10 @@ import io.delilaheve.CustomAnvil;
import org.bukkit.Material;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
-import xyz.alexcrea.cuanvil.dependency.protocolib.PacketManager;
+import xyz.alexcrea.cuanvil.dependency.packet.PacketManager;
import xyz.alexcrea.cuanvil.gui.config.global.*;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
+import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
import java.util.Collections;
@@ -23,13 +24,13 @@ public class MainConfigGui extends ChestGui {
}
private MainConfigGui() {
- super(3, "\u00A78Anvil Config", CustomAnvil.instance);
+ super(3, "§8Anvil Config", CustomAnvil.instance);
}
public void init(PacketManager packetManager) {
Pattern pattern = new Pattern(
- "000000000",
- "012304567",
+ GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ "012345678",
"Q00000000"
);
PatternPane pane = new PatternPane(0, 0, 9, 3, pattern);
@@ -42,8 +43,8 @@ public class MainConfigGui extends ChestGui {
ItemMeta basicConfigMeta = basicConfigItemstack.getItemMeta();
assert basicConfigMeta != null;
- basicConfigMeta.setDisplayName("\u00A7aBasic Config Menu");
- basicConfigMeta.setLore(Collections.singletonList("\u00A77Click here to open basic config menu"));
+ basicConfigMeta.setDisplayName("§aBasic Config Menu");
+ basicConfigMeta.setLore(Collections.singletonList("§7Click here to open basic config menu"));
basicConfigItemstack.setItemMeta(basicConfigMeta);
GuiItem basicConfigItem = GuiGlobalItems.goToGuiItem(basicConfigItemstack, new BasicConfigGui(packetManager));
@@ -54,80 +55,92 @@ public class MainConfigGui extends ChestGui {
ItemMeta enchantLimitMeta = enchantLimitItemstack.getItemMeta();
assert enchantLimitMeta != null;
- enchantLimitMeta.setDisplayName("\u00A7aEnchantment Level Limit");
- enchantLimitMeta.setLore(Collections.singletonList("\u00A77Click here to open enchantment level limit menu"));
+ enchantLimitMeta.setDisplayName("§aEnchantment Level Limit");
+ enchantLimitMeta.setLore(Collections.singletonList("§7Click here to open enchantment level limit menu"));
enchantLimitItemstack.setItemMeta(enchantLimitMeta);
GuiItem enchantLimitItem = GuiGlobalItems.goToGuiItem(enchantLimitItemstack, new EnchantLimitConfigGui());
pane.bindItem('2', enchantLimitItem);
+ // enchant level limit item
+ ItemStack enchantMergeLimitItemstack = new ItemStack(Material.ENCHANTED_BOOK);
+ ItemMeta enchantMergeLimitMeta = enchantMergeLimitItemstack.getItemMeta();
+ assert enchantMergeLimitMeta != null;
+
+ enchantMergeLimitMeta.setDisplayName("§aEnchantment Merge Limit");
+ enchantMergeLimitMeta.setLore(Collections.singletonList("§7Click here to open enchantment merge limit menu"));
+ enchantMergeLimitItemstack.setItemMeta(enchantMergeLimitMeta);
+
+ GuiItem enchantMergeLimitItem = GuiGlobalItems.goToGuiItem(enchantMergeLimitItemstack, new EnchantMergeLimitConfigGui());
+ pane.bindItem('3', enchantMergeLimitItem);
+
// enchant cost item
ItemStack enchantCostItemstack = new ItemStack(Material.EXPERIENCE_BOTTLE);
ItemMeta enchantCostMeta = enchantCostItemstack.getItemMeta();
assert enchantCostMeta != null;
- enchantCostMeta.setDisplayName("\u00A7aEnchantment Cost");
- enchantCostMeta.setLore(Collections.singletonList("\u00A77Click here to open enchantment costs menu"));
+ enchantCostMeta.setDisplayName("§aEnchantment Cost");
+ enchantCostMeta.setLore(Collections.singletonList("§7Click here to open enchantment costs menu"));
enchantCostItemstack.setItemMeta(enchantCostMeta);
GuiItem enchantCostItem = GuiGlobalItems.goToGuiItem(enchantCostItemstack, new EnchantCostConfigGui());
- pane.bindItem('3', enchantCostItem);
+ pane.bindItem('4', enchantCostItem);
// Enchantment Conflicts item
ItemStack enchantConflictItemstack = new ItemStack(Material.OAK_FENCE);
ItemMeta enchantConflictMeta = enchantConflictItemstack.getItemMeta();
assert enchantConflictMeta != null;
- enchantConflictMeta.setDisplayName("\u00A7aEnchantment Conflict");
- enchantConflictMeta.setLore(Collections.singletonList("\u00A77Click here to open enchantment conflict menu"));
+ enchantConflictMeta.setDisplayName("§aEnchantment Conflict");
+ enchantConflictMeta.setLore(Collections.singletonList("§7Click here to open enchantment conflict menu"));
enchantConflictItemstack.setItemMeta(enchantConflictMeta);
- GuiItem enchantConflictItem = GuiGlobalItems.goToGuiItem(enchantConflictItemstack, EnchantConflictGui.INSTANCE);
- pane.bindItem('4', enchantConflictItem);
+ GuiItem enchantConflictItem = GuiGlobalItems.goToGuiItem(enchantConflictItemstack, EnchantConflictGui.getInstance());
+ pane.bindItem('5', enchantConflictItem);
// Group config items
ItemStack groupItemstack = new ItemStack(Material.CHEST);
ItemMeta groupMeta = groupItemstack.getItemMeta();
assert groupMeta != null;
- groupMeta.setDisplayName("\u00A7aGroups");
- groupMeta.setLore(Collections.singletonList("\u00A77Click here to open material group menu"));
+ groupMeta.setDisplayName("§aItem Groups");
+ groupMeta.setLore(Collections.singletonList("§7Click here to open item group menu"));
groupItemstack.setItemMeta(groupMeta);
- GuiItem groupConfigItem = GuiGlobalItems.goToGuiItem(groupItemstack, GroupConfigGui.INSTANCE);
+ GuiItem groupConfigItem = GuiGlobalItems.goToGuiItem(groupItemstack, GroupConfigGui.getInstance());
- pane.bindItem('5', groupConfigItem);
+ pane.bindItem('6', groupConfigItem);
// Unit repair item
ItemStack unirRepairItemstack = new ItemStack(Material.DIAMOND);
ItemMeta unitRepairMeta = unirRepairItemstack.getItemMeta();
assert unitRepairMeta != null;
- unitRepairMeta.setDisplayName("\u00A7aUnit Repair");
- unitRepairMeta.setLore(Collections.singletonList("\u00A77Click here to open anvil unit repair menu"));
+ unitRepairMeta.setDisplayName("§aUnit Repair");
+ unitRepairMeta.setLore(Collections.singletonList("§7Click here to open anvil unit repair menu"));
unirRepairItemstack.setItemMeta(unitRepairMeta);
- GuiItem unitRepairItem = GuiGlobalItems.goToGuiItem(unirRepairItemstack, UnitRepairConfigGui.INSTANCE);
- pane.bindItem('6', unitRepairItem);
+ GuiItem unitRepairItem = GuiGlobalItems.goToGuiItem(unirRepairItemstack, UnitRepairConfigGui.getInstance());
+ pane.bindItem('7', unitRepairItem);
// Custom recipe item
ItemStack customRecipeItemstack = new ItemStack(Material.CRAFTING_TABLE);
ItemMeta customRecipeMeta = customRecipeItemstack.getItemMeta();
assert customRecipeMeta != null;
- customRecipeMeta.setDisplayName("\u00A7aCustom recipes");
- customRecipeMeta.setLore(Collections.singletonList("\u00A77Click here to open anvil custom recipe menu"));
+ customRecipeMeta.setDisplayName("§aCustom recipes");
+ customRecipeMeta.setLore(Collections.singletonList("§7Click here to open anvil custom recipe menu"));
customRecipeItemstack.setItemMeta(customRecipeMeta);
- GuiItem customRecipeItem = GuiGlobalItems.goToGuiItem(customRecipeItemstack, CustomRecipeConfigGui.INSTANCE);
- pane.bindItem('7', customRecipeItem);
+ GuiItem customRecipeItem = GuiGlobalItems.goToGuiItem(customRecipeItemstack, CustomRecipeConfigGui.getInstance());
+ pane.bindItem('8', customRecipeItem);
// quit item
ItemStack quitItemstack = new ItemStack(Material.BARRIER);
ItemMeta quitMeta = quitItemstack.getItemMeta();
assert quitMeta != null;
- quitMeta.setDisplayName("\u00A7cQuit");
+ quitMeta.setDisplayName("§cQuit");
quitItemstack.setItemMeta(quitMeta);
GuiItem quitItem = new GuiItem(quitItemstack, event -> {
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectGroupContainer.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectGroupContainer.java
index c8668d0..49f8b3b 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectGroupContainer.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectGroupContainer.java
@@ -19,12 +19,12 @@ public interface SelectGroupContainer {
static List getGroupLore(SelectGroupContainer container, String containerType, String groupAction){
// Prepare group lore
ArrayList groupLore = new ArrayList<>();
- groupLore.add("\u00A77Allow you to select a list of \u00A73Groups \u00A77that this " + containerType + " should " + groupAction);
+ groupLore.add("§7Allow you to select a list of §3Groups §7that this " + containerType + " should " + groupAction);
Set grouos = container.getSelectedGroups();
if (grouos.isEmpty()) {
- groupLore.add("\u00A77There is no "+groupAction+"d group for this "+containerType+".");
+ groupLore.add("§7There is no "+groupAction+"d group for this "+containerType+".");
} else {
- groupLore.add("\u00A77List of "+groupAction+"d groups for this "+containerType+":");
+ groupLore.add("§7List of "+groupAction+"d groups for this "+containerType+":");
Iterator groupIterator = grouos.iterator();
boolean greaterThanMax = grouos.size() > 5;
@@ -32,11 +32,11 @@ public interface SelectGroupContainer {
for (int i = 0; i < maxindex; i++) {
// format string like "- Melee Weapons"
String formattedName = CasedStringUtil.snakeToUpperSpacedCase(groupIterator.next().getName());
- groupLore.add("\u00A77- \u00A73" + formattedName);
+ groupLore.add("§7- §3" + formattedName);
}
if (greaterThanMax) {
- groupLore.add("\u00A77And " + (grouos.size() - 4) + " more...");
+ groupLore.add("§7And " + (grouos.size() - 4) + " more...");
}
}
return groupLore;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectMaterialContainer.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectMaterialContainer.java
index 397030e..3756341 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectMaterialContainer.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/SelectMaterialContainer.java
@@ -1,39 +1,40 @@
package xyz.alexcrea.cuanvil.gui.config;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import xyz.alexcrea.cuanvil.util.CasedStringUtil;
import java.util.*;
public interface SelectMaterialContainer {
- EnumSet getSelectedMaterials();
+ Set getSelectedMaterials();
- boolean setSelectedMaterials(EnumSet materials);
+ boolean setSelectedMaterials(Set materials);
- EnumSet illegalMaterials();
+ Set illegalMaterials();
static List getMaterialLore(SelectMaterialContainer container, String containerType, String action){
// Prepare material lore
ArrayList groupLore = new ArrayList<>();
- groupLore.add("\u00A77Allow you to select a list of \u00A7ematerials \u00A77that this " + containerType + " should " + action);
- Set materialSet = container.getSelectedMaterials();
+ groupLore.add("§7Allow you to select a list of §ematerials §7that this " + containerType + " should " + action);
+ Set materialSet = container.getSelectedMaterials();
if (materialSet.isEmpty()) {
- groupLore.add("\u00A77There is no "+action+"d material for this "+containerType+".");
+ groupLore.add("§7There is no "+action+"d material for this "+containerType+".");
} else {
- groupLore.add("\u00A77List of "+action+"d materials for this "+containerType+":");
- Iterator materialIterator = materialSet.iterator();
+ groupLore.add("§7List of "+action+"d materials for this "+containerType+":");
+ Iterator materialIterator = materialSet.iterator();
boolean greaterThanMax = materialSet.size() > 5;
int maxindex = (greaterThanMax ? 4 : materialSet.size());
for (int i = 0; i < maxindex; i++) {
// format string like "- Stone Sword"
- String formattedName = CasedStringUtil.snakeToUpperSpacedCase(materialIterator.next().name().toLowerCase());
- groupLore.add("\u00A77- \u00A7e" + formattedName);
+ String formattedName = CasedStringUtil.snakeToUpperSpacedCase(materialIterator.next().getKey().toLowerCase());
+ groupLore.add("§7- §e" + formattedName);
}
if (greaterThanMax) {
- groupLore.add("\u00A77And " + (materialSet.size() - 4) + " more...");
+ groupLore.add("§7And " + (materialSet.size() - 4) + " more...");
}
}
return groupLore;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/ConfirmActionGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/ConfirmActionGui.java
index d306ff1..5839663 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/ConfirmActionGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/ConfirmActionGui.java
@@ -11,6 +11,7 @@ import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
+import xyz.alexcrea.cuanvil.util.MetricsUtil;
import java.util.Arrays;
import java.util.function.Supplier;
@@ -41,11 +42,12 @@ public class ConfirmActionGui extends AbstractAskGui {
success = onConfirm.get();
} catch (Exception e) {
CustomAnvil.instance.getLogger().log(Level.WARNING, "Could not process confirmation supplier.", e);
+ MetricsUtil.INSTANCE.trackError(e);
success = false;
}
if (!success) {
- event.getWhoClicked().sendMessage("\u00A7cAction could not be completed. ");
+ event.getWhoClicked().sendMessage("§cAction could not be completed. ");
}
backOnConfirm.show(player);
@@ -55,7 +57,7 @@ public class ConfirmActionGui extends AbstractAskGui {
ItemStack infoItem = new ItemStack(Material.PAPER);
ItemMeta infoMeta = infoItem.getItemMeta();
- infoMeta.setDisplayName("\u00A7eAre you sure ?");
+ infoMeta.setDisplayName("§eAre you sure ?");
if(actionDescription != null){
infoMeta.setLore(Arrays.asList(actionDescription.split("\n")));
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/SelectItemTypeGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/SelectItemTypeGui.java
index eb15243..66411bd 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/SelectItemTypeGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/ask/SelectItemTypeGui.java
@@ -12,6 +12,7 @@ import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
+import xyz.alexcrea.cuanvil.util.MaterialUtil;
import java.util.Arrays;
import java.util.concurrent.atomic.AtomicReference;
@@ -52,7 +53,7 @@ public class SelectItemTypeGui extends AbstractAskGui {
event.setCancelled(true);
ItemStack cursor = event.getWhoClicked().getItemOnCursor();
- if(cursor.getType().isAir()) return;
+ if(MaterialUtil.INSTANCE.isAir(cursor)) return;
ItemStack finalItem;
if(materialOnly){
@@ -71,7 +72,7 @@ public class SelectItemTypeGui extends AbstractAskGui {
this.pane.bindItem('V', selectGuiItem.get());
// Temporary leave item
- GuiItem temporaryLeave = GuiGlobalItems.temporaryCloseGuiToSelectItem(Material.YELLOW_TERRACOTTA, this);
+ GuiItem temporaryLeave = GuiGlobalItems.temporaryCloseGuiToSelectItem(Material.YELLOW_STAINED_GLASS_PANE, this);
this.pane.bindItem('s', temporaryLeave);
@@ -80,7 +81,7 @@ public class SelectItemTypeGui extends AbstractAskGui {
private ItemStack setDisplayMeta(ItemStack item, String actionDescription){
ItemMeta meta = item.getItemMeta();
- meta.setDisplayName("\u00A7ePlace an item here");
+ meta.setDisplayName("§ePlace an item here");
meta.setLore(Arrays.asList(actionDescription.split("\n")));
item.setItemMeta(meta);
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/AbstractEnchantConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/AbstractEnchantConfigGui.java
index 369924b..6bd7ea3 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/AbstractEnchantConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/AbstractEnchantConfigGui.java
@@ -4,6 +4,7 @@ import com.github.stefvanschie.inventoryframework.gui.GuiItem;
import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
import org.bukkit.event.inventory.InventoryClickEvent;
import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
import xyz.alexcrea.cuanvil.gui.config.list.SettingGuiListConfigGui;
import xyz.alexcrea.cuanvil.gui.config.settings.SettingGui;
@@ -19,7 +20,7 @@ import java.util.function.Consumer;
*
* @param Type of the factory of the type of setting the gui should edit.
*/
-public abstract class AbstractEnchantConfigGui extends SettingGuiListConfigGui implements ValueUpdatableGui {
+public abstract class AbstractEnchantConfigGui extends SettingGuiListConfigGui{
/**
* Constructor for a gui displaying available enchantment to edit a enchantment setting.
@@ -37,21 +38,61 @@ public abstract class AbstractEnchantConfigGui getEveryDisplayableInstanceOfGeneric() {
- return GuiSharedConstant.SORTED_ENCHANTMENT_LIST;
+ return CAEnchantmentRegistry.getInstance().getNameSortedEnchantments();
}
@Override
protected Pattern getBackgroundPattern(){
return new Pattern(
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ GuiSharedConstant.UPPER_FILLER_FULL_PLANE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
"B11L1R111"
);
}
+ @Override
+ public void updateValueForGeneric(CAEnchantment generic, boolean shouldUpdate) {
+ updateValueForGeneric(generic, shouldUpdate, true);
+ }
+
+ public void updateValueForGeneric(CAEnchantment generic, boolean shouldUpdate, boolean prepareSorting) {
+ if(!prepareSorting) {
+ super.updateValueForGeneric(generic, shouldUpdate);
+ return;
+ }
+
+ if(!this.factoryMap.containsKey(generic)){
+ // We need to sort elements again
+ super.updateValueForGeneric(generic, false);
+
+ // Clear page then refill all of them
+ this.firstPage.clear();
+ this.pages.clear();
+ this.pages.add(this.firstPage);
+
+ for (CAEnchantment enchantment : getEveryDisplayableInstanceOfGeneric()) {
+ GuiItem item = this.guiItemMap.get(enchantment);
+
+ if(item == null) {
+ updateValueForGeneric(enchantment, false, false);
+ }else {
+ addToPage(item);
+ }
+
+ }
+
+ if(shouldUpdate) update();
+
+ }else{
+ super.updateValueForGeneric(generic, shouldUpdate);
+
+ }
+
+ }
+
// Unused methods
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/BasicConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/BasicConfigGui.java
index 4d900dc..51936c7 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/BasicConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/BasicConfigGui.java
@@ -14,11 +14,13 @@ import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
-import xyz.alexcrea.cuanvil.dependency.protocolib.PacketManager;
+import xyz.alexcrea.cuanvil.dependency.MinecraftVersionUtil;
+import xyz.alexcrea.cuanvil.dependency.packet.PacketManager;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
import xyz.alexcrea.cuanvil.gui.config.settings.BoolSettingsGui;
import xyz.alexcrea.cuanvil.gui.config.settings.IntSettingsGui;
+import xyz.alexcrea.cuanvil.gui.config.settings.WorkPenaltyTypeSettingGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
@@ -44,7 +46,7 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
* Constructor of this Global gui for basic settings.
*/
public BasicConfigGui(PacketManager packetManager) {
- super(4, "\u00A78Basic Config", CustomAnvil.instance);
+ super(4, "§8Basic Config", CustomAnvil.instance);
if(INSTANCE == null) INSTANCE = this;
this.packetManager = packetManager;
@@ -59,8 +61,8 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
private void init() {
Pattern pattern = new Pattern(
GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- "0L0T0I0S0",
- "0C0R0U0r0",
+ "LT0IWS0cp",
+ "CR0U0r0hP",
"B00000000"
);
pane = new PatternPane(0, 0, 9, 4, pattern);
@@ -73,9 +75,9 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
updateGuiValues();
}
- private BoolSettingsGui.BoolSettingFactory capAnvilCostFactory; // L character
+ private BoolSettingsGui.BoolSettingFactory capAnvilCost; // L character
private GuiItem noCapRepairItem;
- private IntSettingsGui.IntSettingFactory maxAnvilCostFactory; // C character
+ private IntSettingsGui.IntSettingFactory maxAnvilCost; // C character
private GuiItem noMaxCostItem;
private BoolSettingsGui.BoolSettingFactory removeAnvilCostLimit; // R character
@@ -86,38 +88,46 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
private IntSettingsGui.IntSettingFactory itemRenameCost; // r character
private IntSettingsGui.IntSettingFactory sacrificeIllegalEnchantCost; // S character
+ private BoolSettingsGui.BoolSettingFactory allowColorCode; // c character
+ private BoolSettingsGui.BoolSettingFactory allowHexColor; // h character
+
+ private BoolSettingsGui.BoolSettingFactory permissionNeededForColor; // p character
+ private GuiItem noPermissionNeededItem;
+ private IntSettingsGui.IntSettingFactory useOfColorCost; // P character
+ private GuiItem noColorCostItem;
+
/**
* Prepare basic gui displayed items factory and static items..
*/
protected void prepareValues() {
// cap anvil cost
- this.capAnvilCostFactory = BoolSettingsGui.boolFactory("\u00A78Cap Anvil Cost ?", this,
+ this.capAnvilCost = new BoolSettingsGui.BoolSettingFactory("§8Cap Anvil Cost ?", this,
ConfigHolder.DEFAULT_CONFIG,
ConfigOptions.CAP_ANVIL_COST, ConfigOptions.DEFAULT_CAP_ANVIL_COST,
- "\u00A77All anvil cost will be capped to \u00A7aMax Anvil Cost\u00A77 if enabled.",
- "\u00A77In other words:",
- "\u00A77For any anvil cost greater than \u00A7aMax Anvil Cost\u00A77, Cost will be set to \u00A7aMax Anvil Cost\u00A77.");
+ "§7All anvil cost will be capped to §aMax Anvil Cost§7 if enabled.",
+ "§7In other words:",
+ "§7For any anvil cost greater than §aMax Anvil Cost§7, Cost will be set to §aMax Anvil Cost§7.");
// cap anvil cost not needed
ItemStack item = new ItemStack(Material.BARRIER);
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7cCap Anvil Cost ?");
- meta.setLore(Collections.singletonList("\u00A77This config only work if \u00A7cLimit Repair Cost\u00A77 is disabled."));
+ meta.setDisplayName("§cCap Anvil Cost ?");
+ meta.setLore(Collections.singletonList("§7This config only work if §cLimit Repair Cost§7 is disabled."));
item.setItemMeta(meta);
this.noCapRepairItem = new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
// repair cost item
IntRange range = ConfigOptions.MAX_ANVIL_COST_RANGE;
- this.maxAnvilCostFactory = IntSettingsGui.intFactory("\u00A78Max Anvil Cost", this,
+ this.maxAnvilCost = new IntSettingsGui.IntSettingFactory("§8Max Anvil Cost", this,
ConfigOptions.MAX_ANVIL_COST, ConfigHolder.DEFAULT_CONFIG,
Arrays.asList(
- "\u00A77Max cost the Anvil can get to.",
- "\u00A77Valid values include \u00A7e0 \u00A77to \u00A7e1000\u00A77.",
- "\u00A77Cost will be displayed as \u00A7cToo Expensive\u00A77:",
- "\u00A77- If Cost is above \u00A7e39",
- "\u00A77- And \u00A7eReplace Too Expensive\u00A77 is disabled"
+ "§7Max cost the Anvil can get to.",
+ "§7Valid values include §e0 §7to §e1000§7.",
+ "§7Cost will be displayed as §cToo Expensive§7:",
+ "§7- If Cost is above §e39",
+ "§7- And §eReplace Too Expensive§7 is disabled"
),
range.getFirst(), range.getLast(),
ConfigOptions.DEFAULT_MAX_ANVIL_COST,
@@ -127,45 +137,49 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7cMax Anvil Cost");
- meta.setLore(Collections.singletonList("\u00A77This config only work if \u00A7cLimit Repair Cost\u00A77 is disabled."));
+ meta.setDisplayName("§cMax Anvil Cost");
+ meta.setLore(Collections.singletonList("§7This config only work if §cLimit Repair Cost§7 is disabled."));
item.setItemMeta(meta);
this.noMaxCostItem = new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
// remove repair limit item
- this.removeAnvilCostLimit = BoolSettingsGui.boolFactory("\u00A78Remove Anvil Cost Limit ?", this,
+ this.removeAnvilCostLimit = new BoolSettingsGui.BoolSettingFactory("§8Remove Anvil Cost Limit ?", this,
ConfigHolder.DEFAULT_CONFIG,
ConfigOptions.REMOVE_ANVIL_COST_LIMIT, ConfigOptions.DEFAULT_REMOVE_ANVIL_COST_LIMIT,
- "\u00A77Whether the anvil's cost limit should be removed entirely.",
- "\u00A77The anvil will still visually display \u00A7cToo Expensive\u00A77 if \u00A7eReplace Too Expensive\u00A77 is disabled.",
- "\u00A77However, the action will be completable if xp requirement is meet.");
+ "§7Whether the anvil's cost limit should be removed entirely.",
+ "§7The anvil will still visually display §cToo Expensive§7 if §eReplace Too Expensive§7 is disabled.",
+ "§7However, the action will be completable if xp requirement is meet.");
// replace too expensive item
- this.replaceTooExpensive = BoolSettingsGui.boolFactory("\u00A78Replace Too Expensive ?", this,
+ this.replaceTooExpensive = new BoolSettingsGui.BoolSettingFactory("§8Replace Too Expensive ?", this,
ConfigHolder.DEFAULT_CONFIG,
ConfigOptions.REPLACE_TOO_EXPENSIVE, ConfigOptions.DEFAULT_REPLACE_TOO_EXPENSIVE,
getReplaceToExpensiveLore());
+ // ------------
+ // Cost config
+ // ------------
+
// item repair cost
range = ConfigOptions.REPAIR_COST_RANGE;
- this.itemRepairCost = IntSettingsGui.intFactory("\u00A78Item Repair Cost", this,
+ this.itemRepairCost = new IntSettingsGui.IntSettingFactory("§8Item Repair Cost", this,
ConfigOptions.ITEM_REPAIR_COST, ConfigHolder.DEFAULT_CONFIG,
Arrays.asList(
- "\u00A77XP Level amount added to the anvil when the item",
- "\u00A77is repaired by another item of the same type."
+ "§7XP Level amount added to the anvil when the item",
+ "§7is repaired by another item of the same type."
),
range.getFirst(), range.getLast(),
ConfigOptions.DEFAULT_ITEM_REPAIR_COST,
1, 5, 10, 50, 100);
// unit repair cost
- this.unitRepairCost = IntSettingsGui.intFactory("\u00A78Unit Repair Cost", this,
+ this.unitRepairCost = new IntSettingsGui.IntSettingFactory("§8Unit Repair Cost", this,
ConfigOptions.UNIT_REPAIR_COST, ConfigHolder.DEFAULT_CONFIG,
Arrays.asList(
- "\u00A77XP Level amount added to the anvil when the item is repaired by an \u00A7eunit\u00A77.",
- "\u00A77For example: a Diamond on a Diamond Sword.",
- "\u00A77What's considered unit for what can be edited on the unit repair configuration."
+ "§7XP Level amount added to the anvil when the item is repaired by an §eunit§7.",
+ "§7For example: a Diamond on a Diamond Sword.",
+ "§7What's considered unit for what can be edited on the unit repair configuration."
),
range.getFirst(), range.getLast(),
ConfigOptions.DEFAULT_UNIT_REPAIR_COST,
@@ -173,10 +187,10 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
// item rename cost
range = ConfigOptions.ITEM_RENAME_COST_RANGE;
- this.itemRenameCost = IntSettingsGui.intFactory("\u00A78Rename Cost", this,
+ this.itemRenameCost = new IntSettingsGui.IntSettingFactory("§8Rename Cost", this,
ConfigOptions.ITEM_RENAME_COST, ConfigHolder.DEFAULT_CONFIG,
Arrays.asList(
- "\u00A77XP Level amount added to the anvil when the item is renamed."
+ "§7XP Level amount added to the anvil when the item is renamed."
),
range.getFirst(), range.getLast(),
ConfigOptions.DEFAULT_ITEM_RENAME_COST,
@@ -184,29 +198,94 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
// sacrifice illegal enchant cost
range = ConfigOptions.SACRIFICE_ILLEGAL_COST_RANGE;
- this.sacrificeIllegalEnchantCost = IntSettingsGui.intFactory("\u00A78Sacrifice Illegal Enchant Cost", this,
+ this.sacrificeIllegalEnchantCost = new IntSettingsGui.IntSettingFactory("§8Sacrifice Illegal Enchant Cost", this,
ConfigOptions.SACRIFICE_ILLEGAL_COST, ConfigHolder.DEFAULT_CONFIG,
Arrays.asList(
- "\u00A77XP Level amount added to the anvil when a sacrifice enchantment",
- "\u00A77conflict With one of the left item enchantment"
+ "§7XP Level amount added to the anvil when a sacrifice enchantment",
+ "§7conflict With one of the left item enchantment"
),
range.getFirst(), range.getLast(),
ConfigOptions.DEFAULT_SACRIFICE_ILLEGAL_COST,
1, 5, 10, 50, 100);
+ // -------------
+ // Color config
+ // -------------
+
+ // Allow us of color code
+ this.allowColorCode = new BoolSettingsGui.BoolSettingFactory("§8Allow Use Of Color Code ?", this,
+ ConfigHolder.DEFAULT_CONFIG,
+ ConfigOptions.ALLOW_COLOR_CODE, ConfigOptions.DEFAULT_ALLOW_COLOR_CODE,
+ "§7Whether players can use color code.",
+ "§7Color code a formatted like §a&a§7 and is used in the rename field of the anvil.",
+ "§7Player may need permission to use color code if §ePlayer need permission to use color§7 is enabled.");
+
+ // Allow us of hexadecimal color
+ this.allowHexColor = new BoolSettingsGui.BoolSettingFactory("§8Allow Use Of Hexadecimal Color ?", this,
+ ConfigHolder.DEFAULT_CONFIG,
+ ConfigOptions.ALLOW_HEXADECIMAL_COLOR, ConfigOptions.DEFAULT_ALLOW_HEXADECIMAL_COLOR,
+ "§7Whether players can use hexadecimal color.",
+ "§7Color code a formatted like §2#012345 §7and is used in the rename field of the anvil.",
+ "§7Player may need permission to use color code if §ePermission Needed For Color§7 is enabled.");
+
+ // Permission needed for color
+ this.permissionNeededForColor = new BoolSettingsGui.BoolSettingFactory("§8Need Permission To Use Color ?", this,
+ ConfigHolder.DEFAULT_CONFIG,
+ ConfigOptions.PERMISSION_NEEDED_FOR_COLOR, ConfigOptions.DEFAULT_PERMISSION_NEEDED_FOR_COLOR,
+ "§7Whether players should have permission to be able to use colors.",
+ "§7Give player §eca.color.code§7 Permission to allow use of color code.",
+ "§7Give player §eca.color.hex§7 Permission to allow use of hexadecimal color.");
+
+ // Permission needed for color not necessary
+ item = new ItemStack(Material.BARRIER);
+ meta = item.getItemMeta();
+ assert meta != null;
+
+ meta.setDisplayName("§cNeed Permission To Use Color ?");
+ meta.setLore(Arrays.asList("§7This config can do something only if one of the following config is enabled:",
+ "§7- §aAllow Use Of Color Code",
+ "§7- §aAllow Use Of Hexadecimal Color"));
+ item.setItemMeta(meta);
+ this.noPermissionNeededItem = new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
+
+ // Cost of using color
+ range = ConfigOptions.USE_OF_COLOR_COST_RANGE;
+ this.useOfColorCost = new IntSettingsGui.IntSettingFactory("§8Cost Of Using Color", this,
+ ConfigOptions.USE_OF_COLOR_COST, ConfigHolder.DEFAULT_CONFIG,
+ Arrays.asList(
+ "§7XP level cost when using color code or hexadecimal color using the anvil.",
+ "§7conflict With one of the left item enchantment"
+ ),
+ range.getFirst(), range.getLast(),
+ ConfigOptions.DEFAULT_USE_OF_COLOR_COST,
+ 1, 5, 10, 50, 100);
+
+ // Permission needed for color not necessary
+ item = new ItemStack(Material.BARRIER);
+ meta = item.getItemMeta();
+ assert meta != null;
+
+ meta.setDisplayName("§cCost Of Using Color");
+ meta.setLore(Arrays.asList("§7This config can do something only if one of the following config is enabled:",
+ "§7- §aAllow Use Of Color Code",
+ "§7- §aAllow Use Of Hexadecimal Color"));
+ item.setItemMeta(meta);
+ this.noColorCostItem = new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
+
}
@NotNull
private String[] getReplaceToExpensiveLore() {
ArrayList lore = new ArrayList<>();
- lore.add("\u00A77Whenever anvil cost is above \u00A7e39\u00A77 should display the true price and not \u00A7cToo Expensive\u00A77.");
- lore.add("\u00A77However, when bypassing \u00A7cToo Expensive\u00A77, anvil price will be displayed as \u00A7aGreen\u00A77.");
- lore.add("\u00A77Even if cost is displayed as \u00A7aGreen\u00A77:");
- lore.add("\u00A77If the player do not have the required xp level, the action will not be completable.");
+ lore.add("§7Whenever anvil cost is above §e39§7 should display the true price and not §cToo Expensive§7.");
+ lore.add("§7However, when bypassing §cToo Expensive§7, anvil price will be displayed as §aGreen§7.");
+ lore.add("§7Even if cost is displayed as §aGreen§7:");
+ lore.add("§7If the player do not have the required xp level, the action will not be completable.");
- if(!this.packetManager.isProtocoLibInstalled()){
+ if(!this.packetManager.getCanSetInstantBuild()){
lore.add("");
- lore.add("\u00A74/!\\\u00A7cCaution/!\\ \u00A7cYou need ProtocoLib installed for this to work.");
+ lore.add("§4/!\\§cCaution§4/!\\ §cYou need ProtocoLib installed and working or a paper server.");
+ lore.add("§cCurrently ProtocoLib is not detected.");
}
String[] loreAsArray = new String[lore.size()];
@@ -219,8 +298,8 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
GuiItem capAnvilCostItem;
GuiItem maxAnvilCostItem;
if (!this.removeAnvilCostLimit.getConfiguredValue()) {
- capAnvilCostItem = this.capAnvilCostFactory.getItem("Cap Anvil Cost");
- maxAnvilCostItem = this.maxAnvilCostFactory.getItem(Material.EXPERIENCE_BOTTLE, "Max Anvil Cost");
+ capAnvilCostItem = this.capAnvilCost.getItem("Cap Anvil Cost");
+ maxAnvilCostItem = this.maxAnvilCost.getItem(Material.EXPERIENCE_BOTTLE, "Max Anvil Cost");
} else {
capAnvilCostItem = this.noCapRepairItem;
maxAnvilCostItem = this.noMaxCostItem;
@@ -247,13 +326,40 @@ public class BasicConfigGui extends ChestGui implements ValueUpdatableGui {
pane.bindItem('U', unitRepairCostItem);
// item rename cost
- GuiItem itemRenameCost = this.itemRenameCost.getItem(Material.NAME_TAG);
- pane.bindItem('r', itemRenameCost);
+ GuiItem itemRenameCostItem = this.itemRenameCost.getItem(Material.NAME_TAG);
+ pane.bindItem('r', itemRenameCostItem);
// sacrifice illegal enchant cost
GuiItem illegalCostItem = this.sacrificeIllegalEnchantCost.getItem(Material.ENCHANTED_BOOK);
pane.bindItem('S', illegalCostItem);
+ // work penalty type
+ GuiItem workPenaltyType = WorkPenaltyTypeSettingGui.getDisplayItem(this, Material.DAMAGED_ANVIL, "§aWork Penalty Type");
+ pane.bindItem('W', workPenaltyType);
+
+ // allow color code
+ GuiItem allowColorCodeItem = this.allowColorCode.getItem();
+ pane.bindItem('c', allowColorCodeItem);
+
+ // allow hex color
+ GuiItem allowHexColorItem = this.allowHexColor.getItem();
+ pane.bindItem('h', allowHexColorItem);
+
+ // True if player could place color
+ if(ConfigOptions.INSTANCE.getRenameColorPossible()){
+ // use permission for color
+ GuiItem permissionNeededItem = this.permissionNeededForColor.getItem();
+ pane.bindItem('p', permissionNeededItem);
+
+ // using color cost
+ GuiItem useColorCostItem = this.useOfColorCost.getItem(Material.EXPERIENCE_BOTTLE, "Use color");
+ pane.bindItem('P', useColorCostItem);
+ }else{
+ pane.bindItem('p', this.noPermissionNeededItem);
+ pane.bindItem('P', this.noColorCostItem);
+ }
+
+
update();
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/CustomRecipeConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/CustomRecipeConfigGui.java
index 797a3ab..e21ad75 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/CustomRecipeConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/CustomRecipeConfigGui.java
@@ -5,63 +5,82 @@ import org.bukkit.Material;
import org.bukkit.inventory.ItemFlag;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
import xyz.alexcrea.cuanvil.gui.config.list.elements.CustomRecipeSubSettingGui;
+import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
import xyz.alexcrea.cuanvil.recipe.AnvilCustomRecipe;
import xyz.alexcrea.cuanvil.util.CasedStringUtil;
-import java.util.Arrays;
+import java.util.ArrayList;
import java.util.Collection;
-public class CustomRecipeConfigGui extends MappedGuiListConfigGui {
+public class CustomRecipeConfigGui extends MappedGuiListConfigGui> {
+ private static CustomRecipeConfigGui INSTANCE = new CustomRecipeConfigGui();
- public final static CustomRecipeConfigGui INSTANCE = new CustomRecipeConfigGui();
+ @Nullable
+ public static CustomRecipeConfigGui getCurrentInstance() {
+ return INSTANCE;
+ }
- static {
- INSTANCE.init();
+ @NotNull
+ public static CustomRecipeConfigGui getInstance() {
+ if (INSTANCE == null) INSTANCE = new CustomRecipeConfigGui();
+
+ return INSTANCE;
}
private CustomRecipeConfigGui() {
super("Custom Recipe Config");
+ init();
}
@Override
protected ItemStack createItemForGeneric(AnvilCustomRecipe recipe) {
// Get base item to display
ItemStack craftResultItem = recipe.getResultItem();
- ItemStack displaydItem;
- if(craftResultItem == null){
- displaydItem = new ItemStack(Material.BARRIER);
- }else{
- displaydItem = craftResultItem.clone();
+ ItemStack displayedItem;
+ if (craftResultItem == null) {
+ displayedItem = new ItemStack(Material.BARRIER);
+ } else {
+ displayedItem = craftResultItem.clone();
}
// edit displayed item
- ItemMeta meta = displaydItem.getItemMeta();
+ ItemMeta meta = displayedItem.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(recipe.toString()) + " \u00A7fCustom recipe");
+ meta.setDisplayName("§e" + CasedStringUtil.snakeToUpperSpacedCase(recipe.toString()) + " §fCustom recipe");
meta.addItemFlags(ItemFlag.values());
+ meta.setLore(getRecipeLore(recipe));
+
+ displayedItem.setItemMeta(meta);
+ return displayedItem;
+ }
+
+ private static @NotNull ArrayList getRecipeLore(AnvilCustomRecipe recipe) {
boolean shouldWork = recipe.validate();
- meta.setLore(Arrays.asList(
- "\u00A77Should work: \u00A7"+(shouldWork ? "aYes" : "cNo"),
- "\u00A77Exact count: \u00A7"+(recipe.getExactCount() ? "aYes" : "cNo"),
- "\u00A77Recipe Xp Cost: \u00A7e"+recipe.getXpCostPerCraft()
-
- ));
-
- displaydItem.setItemMeta(meta);
- return displaydItem;
+ ArrayList lore = new ArrayList<>();
+ lore.add("§7Is valid: §" + (shouldWork ? "aYes" : "cNo"));
+ lore.add("§7Exact count: §" + (recipe.getExactCount() ? "aYes" : "cNo"));
+ lore.add("§7Recipe Level Cost: §e" + recipe.getLevelCostPerCraft());
+ lore.add("§7Recipe Linear Xp Cost: §e" + recipe.getXpCostPerCraft());
+ if (recipe.getXpCostPerCraft() != 0) {
+ lore.add("§7Exact Linear xp remove: §" + (recipe.getRemoveExactLinearXp() ? "aYes" : "cNo"));
+ }
+ return lore;
}
@Override
- protected CustomRecipeSubSettingGui newInstanceOfGui(AnvilCustomRecipe generic, GuiItem item) {
- return new CustomRecipeSubSettingGui(this, generic, item);
+ protected LazyElement newInstanceOfGui(AnvilCustomRecipe generic, GuiItem item) {
+ return new LazyElement<>(item, () -> new CustomRecipeSubSettingGui(this, generic));
}
@Override
@@ -75,7 +94,11 @@ public class CustomRecipeConfigGui extends MappedGuiListConfigGui {
+public class EnchantConflictGui extends MappedGuiListConfigGui> {
- public static final EnchantConflictGui INSTANCE = new EnchantConflictGui();
+ private static EnchantConflictGui INSTANCE;
- static {
- INSTANCE.init();
+ @Nullable
+ public static EnchantConflictGui getCurrentInstance(){
+ return INSTANCE;
}
+ @NotNull
+ public static EnchantConflictGui getInstance(){
+ if(INSTANCE == null) INSTANCE = new EnchantConflictGui();
+
+ return INSTANCE;
+ }
+
+
private EnchantConflictGui() {
super( "Conflict Config");
+
+ init();
}
@Override
@@ -61,11 +75,11 @@ public class EnchantConflictGui extends MappedGuiListConfigGui newInstanceOfGui(EnchantConflictGroup conflict, GuiItem item) {
+ return new LazyElement<>(item, () -> new EnchantConflictSubSettingGui(this, conflict));
}
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantCostConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantCostConfigGui.java
index b45a7b3..a614536 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantCostConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantCostConfigGui.java
@@ -36,7 +36,7 @@ public class EnchantCostConfigGui extends AbstractEnchantConfigGui lore = new ArrayList<>();
- lore.add("\u00A77Item Cost: \u00A7e" + itemCost);
- lore.add("\u00A77Book Cost: \u00A7e" + bookCost);
+ lore.add("§7Item Cost: §e" + itemCost);
+ lore.add("§7Book Cost: §e" + bookCost);
List displayLore = factory.getDisplayLore();
if(!displayLore.isEmpty()){
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantLimitConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantLimitConfigGui.java
index 9529495..e9edbeb 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantLimitConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantLimitConfigGui.java
@@ -1,6 +1,7 @@
package xyz.alexcrea.cuanvil.gui.config.global;
import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import io.delilaheve.util.ConfigOptions;
import org.bukkit.Material;
import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
@@ -16,7 +17,7 @@ import java.util.Locale;
*/
public class EnchantLimitConfigGui extends AbstractEnchantConfigGui {
- private static final String SECTION_NAME = "enchant_limits";
+ private static final String SECTION_NAME = ConfigOptions.ENCHANT_LIMIT_ROOT;
private static EnchantLimitConfigGui INSTANCE = null;
@@ -29,7 +30,7 @@ public class EnchantLimitConfigGui extends AbstractEnchantConfigGui "Default (" + defaultValue + ")";
+ case RESET -> String.valueOf(defaultValue);
+ default -> "Default";
+ };
+
+ }
+ else return super.valueDisplayName(type, value);
+ }
+ };
}
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantMergeLimitConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantMergeLimitConfigGui.java
new file mode 100644
index 0000000..0d391e7
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantMergeLimitConfigGui.java
@@ -0,0 +1,67 @@
+package xyz.alexcrea.cuanvil.gui.config.global;
+
+import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import io.delilaheve.util.ConfigOptions;
+import org.bukkit.Material;
+import org.jetbrains.annotations.Nullable;
+import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
+import xyz.alexcrea.cuanvil.gui.config.settings.IntSettingsGui;
+import xyz.alexcrea.cuanvil.util.CasedStringUtil;
+
+import java.util.Arrays;
+import java.util.Locale;
+
+public class EnchantMergeLimitConfigGui extends AbstractEnchantConfigGui {
+
+ private static final String SECTION_NAME = "disable-merge-over";
+
+ private static EnchantMergeLimitConfigGui INSTANCE = null;
+
+ @Nullable
+ public static EnchantMergeLimitConfigGui getInstance() {
+ return INSTANCE;
+ }
+
+ /**
+ * Constructor of this Global gui for enchantment level limit settings.
+ */
+ public EnchantMergeLimitConfigGui() {
+ super("§8Enchantment Maximum Merge Level");
+ if(INSTANCE == null) INSTANCE = this;
+
+ init();
+ }
+
+ @Override
+ public IntSettingsGui.IntSettingFactory createFactory(CAEnchantment enchant) {
+ String key = enchant.getKey().toString().toLowerCase(Locale.ROOT);
+ String prettyKey = CasedStringUtil.snakeToUpperSpacedCase(key.replace(":", "_"));
+
+ return new IntSettingsGui.IntSettingFactory(prettyKey + " Merge Limit", this,
+ SECTION_NAME + '.' + key, ConfigHolder.DEFAULT_CONFIG,
+ Arrays.asList(
+ "§7Maximum merge level for for " + prettyKey,
+ "",
+ "§7For example, if set to §e2§7, §alvl1 §7+ §alvl1 §7of will give a §alvl2",
+ "§7But §alvl2 §7+ §alvl2 §7will not give a §clv3§7.",
+ "§7Will still not merge above max enchantment level",
+ "§e-1 §7(default) will set the merge limit to enchantment's maximum level"
+ ),
+ -1, 255, -1,
+ 1, 5, 10, 50, 100){
+
+ @Override
+ public int getConfiguredValue() {
+ return ConfigOptions.INSTANCE.maxBeforeMergeDisabled(enchant);
+ }
+ };
+ }
+
+ @Override
+ public GuiItem itemFromFactory(CAEnchantment enchantment, IntSettingsGui.IntSettingFactory inventoryFactory) {
+ return inventoryFactory.getItem(
+ Material.ENCHANTED_BOOK,
+ inventoryFactory.getTitle());
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/GroupConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/GroupConfigGui.java
index d5fb430..8e20751 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/GroupConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/GroupConfigGui.java
@@ -5,6 +5,8 @@ import org.bukkit.configuration.ConfigurationSection;
import org.bukkit.inventory.ItemFlag;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.group.AbstractMaterialGroup;
import xyz.alexcrea.cuanvil.group.GroupType;
@@ -13,21 +15,32 @@ import xyz.alexcrea.cuanvil.group.ItemGroupManager;
import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
import xyz.alexcrea.cuanvil.gui.config.list.elements.GroupConfigSubSettingGui;
import xyz.alexcrea.cuanvil.util.CasedStringUtil;
+import xyz.alexcrea.cuanvil.util.LazyValue;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
-public class GroupConfigGui extends MappedGuiListConfigGui {
+public class GroupConfigGui extends MappedGuiListConfigGui> {
- public final static GroupConfigGui INSTANCE = new GroupConfigGui();
+ private static GroupConfigGui INSTANCE;
- static {
- INSTANCE.init();
+ @Nullable
+ public static GroupConfigGui getCurrentInstance(){
+ return INSTANCE;
+ }
+
+ @NotNull
+ public static GroupConfigGui getInstance(){
+ if(INSTANCE == null) INSTANCE = new GroupConfigGui();
+
+ return INSTANCE;
}
public GroupConfigGui() {
super("Group Config");
+
+ init();
}
@Override
@@ -37,12 +50,12 @@ public class GroupConfigGui extends MappedGuiListConfigGui newInstanceOfGui(IncludeGroup group, GuiItem item) {
+ return new LazyElement<>(item, () -> new GroupConfigSubSettingGui(this, group));
}
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/UnitRepairConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/UnitRepairConfigGui.java
index 30c5d2e..0e366ae 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/UnitRepairConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/global/UnitRepairConfigGui.java
@@ -6,6 +6,8 @@ import org.bukkit.Material;
import org.bukkit.configuration.ConfigurationSection;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.gui.config.ask.SelectItemTypeGui;
import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
@@ -16,23 +18,36 @@ import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
-public class UnitRepairConfigGui extends MappedGuiListConfigGui {
+public class UnitRepairConfigGui extends
+ MappedGuiListConfigGui> {
- public static final UnitRepairConfigGui INSTANCE = new UnitRepairConfigGui();
+ private static UnitRepairConfigGui INSTANCE;
- static {
- INSTANCE.init();
+ @Nullable
+ public static UnitRepairConfigGui getCurrentInstance(){
+ return INSTANCE;
+ }
+
+ @NotNull
+ public static UnitRepairConfigGui getInstance(){
+ if(INSTANCE == null) INSTANCE = new UnitRepairConfigGui();
+
+ return INSTANCE;
}
private UnitRepairConfigGui() {
super("Unit Repair Config");
+
+ init();
}
@Override
- protected UnitRepairElementListGui newInstanceOfGui(Material material, GuiItem item) {
- UnitRepairElementListGui element = new UnitRepairElementListGui(material, this, item);
- element.init();
- return element;
+ protected LazyElement newInstanceOfGui(Material material, GuiItem item) {
+ return new LazyElement<>(item, () -> {
+ UnitRepairElementListGui element = new UnitRepairElementListGui(material, this);
+ element.init();
+ return element;
+ });
}
@Override
@@ -50,10 +65,10 @@ public class UnitRepairConfigGui extends MappedGuiListConfigGui {
Material type = itemStack.getType();
@@ -103,13 +118,25 @@ public class UnitRepairConfigGui extends MappedGuiListConfigGui getInstanceOrCreate(Material mat){
+ LazyElement element = this.elementGuiMap.get(mat);
+ if(element == null){
+ updateValueForGeneric(mat, false);
+
+ element = this.elementGuiMap.get(mat);
+ }
+
+ return element;
+ }
+
@Override // Not used in this implementation.
protected String genericDisplayedName() {
return "this function Should not be used.";
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/ElementListConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/ElementListConfigGui.java
index f11cdf5..f3cc0b4 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/ElementListConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/ElementListConfigGui.java
@@ -16,7 +16,6 @@ import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
-import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
@@ -31,24 +30,29 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
protected PatternPane backgroundPane;
- protected ElementListConfigGui(@NotNull String title) {
+ public static final int LIST_FILLER_START_X = 1;
+ public static final int LIST_FILLER_START_Y = 1;
+ public static final int LIST_FILLER_LENGTH = 7;
+ public static final int LIST_FILLER_HEIGHT = 4;
+
+ protected ElementListConfigGui(@NotNull String title, Gui parent) {
super(6, title, CustomAnvil.instance);
this.namePrefix = title;
// Back item panel
Pattern pattern = getBackgroundPattern();
this.backgroundPane = new PatternPane(0, 0, 9, 6, Pane.Priority.LOW, pattern);
- GuiGlobalItems.addBackItem(this.backgroundPane, MainConfigGui.getInstance());
+ GuiGlobalItems.addBackItem(this.backgroundPane, parent);
}
protected Pattern getBackgroundPattern(){
return new Pattern(
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ GuiSharedConstant.UPPER_FILLER_FULL_PLANE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
"B11L1R11C"
);
}
@@ -79,7 +83,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
protected void prepareStaticValues(){
// Left item creation for consumer & bind
- this.goLeftItem = new GuiItem(new ItemStack(Material.RED_TERRACOTTA), event -> {
+ this.goLeftItem = new GuiItem(new ItemStack(Material.RED_STAINED_GLASS_PANE), event -> {
HumanEntity viewer = event.getWhoClicked();
UUID playerUUID = viewer.getUniqueId();
int page = this.pageMap.getOrDefault(playerUUID, 0);
@@ -94,7 +98,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
}, CustomAnvil.instance);
// Right item creation for consumer & bind
- this.goRightItem = new GuiItem(new ItemStack(Material.GREEN_TERRACOTTA), event -> {
+ this.goRightItem = new GuiItem(new ItemStack(Material.LIME_STAINED_GLASS_PANE), event -> {
HumanEntity viewer = event.getWhoClicked();
UUID playerUUID = viewer.getUniqueId();
int page = pageMap.getOrDefault(playerUUID, 0);
@@ -118,8 +122,8 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
this.pages.clear();
this.pages.add(this.firstPage);
- for (T conflict : getEveryDisplayableInstanceOfGeneric()) {
- updateValueForGeneric(conflict, false);
+ for (T generic : getEveryDisplayableInstanceOfGeneric()) {
+ updateValueForGeneric(generic, false);
}
update();
@@ -128,7 +132,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
protected abstract GuiItem prepareCreateNewItem();
protected OutlinePane createEmptyPage() {
- OutlinePane page = new OutlinePane(0, 0, 9, 5);
+ OutlinePane page = new OutlinePane(LIST_FILLER_START_X, LIST_FILLER_START_Y, LIST_FILLER_LENGTH, LIST_FILLER_HEIGHT);
page.align(OutlinePane.Alignment.BEGIN);
page.setOrientation(Orientable.Orientation.HORIZONTAL);
@@ -146,7 +150,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
protected void addToPage(GuiItem guiItem) {
// Get first available page or create one
OutlinePane page = this.pages.get(this.pages.size() - 1);
- if (page.getItems().size() >= 5 * 9) {
+ if (page.getItems().size() >= LIST_FILLER_LENGTH * LIST_FILLER_HEIGHT) {
page = createEmptyPage();
this.pages.add(page);
}
@@ -199,7 +203,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
ItemStack leftItem = this.goLeftItem.getItem();
ItemMeta leftMeta = leftItem.getItemMeta();
- leftMeta.setDisplayName("\u00A7eReturn to page " + (page));
+ leftMeta.setDisplayName("§eReturn to page " + (page));
leftItem.setItemMeta(leftMeta);
this.goLeftItem.setItem(leftItem);
@@ -216,7 +220,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
ItemStack rightItem = this.goRightItem.getItem();
ItemMeta rightMeta = rightItem.getItemMeta();
- rightMeta.setDisplayName("\u00A7eGo to page " + (page + 2));
+ rightMeta.setDisplayName("§eGo to page " + (page + 2));
rightItem.setItemMeta(rightMeta);
this.goRightItem.setItem(rightItem);
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedElementListConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedElementListConfigGui.java
index f2793a8..31ab718 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedElementListConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedElementListConfigGui.java
@@ -7,6 +7,7 @@ import org.bukkit.entity.HumanEntity;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import java.util.Arrays;
@@ -17,7 +18,7 @@ public abstract class MappedElementListConfigGui< T, S > extends ElementListConf
protected final HashMap elementGuiMap;
protected MappedElementListConfigGui(@NotNull String title) {
- super(title);
+ super(title, MainConfigGui.getInstance());
this.elementGuiMap = new HashMap<>();
}
@@ -29,11 +30,11 @@ public abstract class MappedElementListConfigGui< T, S > extends ElementListConf
ItemMeta createMeta = createItem.getItemMeta();
assert createMeta != null;
- createMeta.setDisplayName("\u00A7aCreate new "+genericDisplayedName());
+ createMeta.setDisplayName("§aCreate new "+genericDisplayedName());
createMeta.setLore(Arrays.asList(
- "\u00A77Create a new "+genericDisplayedName()+".",
- "\u00A77You will be asked to name the "+genericDisplayedName()+" in chat.",
- "\u00A77Then, you should edit the "+genericDisplayedName()+" config as you need"
+ "§7Create a new "+genericDisplayedName()+".",
+ "§7You will be asked to name the "+genericDisplayedName()+" in chat.",
+ "§7Then, you should edit the "+genericDisplayedName()+" config as you need"
));
createItem.setItemMeta(createMeta);
@@ -50,8 +51,8 @@ public abstract class MappedElementListConfigGui< T, S > extends ElementListConf
}
player.closeInventory();
- player.sendMessage("\u00A7eWrite the "+genericDisplayedName()+" name you want to create in the chat.\n" +
- "\u00A7eOr write \u00A7ccancel \u00A7eto go back to "+genericDisplayedName()+" config menu");
+ player.sendMessage("§eWrite the "+genericDisplayedName()+" name you want to create in the chat.\n" +
+ "§eOr write §ccancel §eto go back to "+genericDisplayedName()+" config menu");
CustomAnvil.Companion.getChatListener().setListenedCallback(player, prepareCreateItemConsumer(player));
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedGuiListConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedGuiListConfigGui.java
index 9c73760..3aa18e0 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedGuiListConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedGuiListConfigGui.java
@@ -3,15 +3,19 @@ package xyz.alexcrea.cuanvil.gui.config.list;
import com.github.stefvanschie.inventoryframework.gui.GuiItem;
import io.delilaheve.CustomAnvil;
import org.bukkit.entity.HumanEntity;
+import org.bukkit.event.inventory.InventoryClickEvent;
import org.jetbrains.annotations.NotNull;
import xyz.alexcrea.cuanvil.gui.config.list.elements.ElementMappedToListGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
+import xyz.alexcrea.cuanvil.util.LazyValue;
import java.util.Locale;
import java.util.concurrent.atomic.AtomicReference;
import java.util.function.Consumer;
+import java.util.function.Supplier;
-public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGui> extends MappedElementListConfigGui< T, S > {
+public abstract class MappedGuiListConfigGui< T, S extends MappedGuiListConfigGui.LazyElement>
+ extends MappedElementListConfigGui< T, S > {
protected MappedGuiListConfigGui(@NotNull String title) {
super(title);
@@ -20,7 +24,10 @@ public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGu
@Override
public void reloadValues() {
- this.elementGuiMap.forEach((conflict, gui) -> gui.cleanAndBeUnusable());
+ this.elementGuiMap.forEach((conflict, element) -> {
+ ElementMappedToListGui gui = element.getStored();
+ if(gui != null) gui.cleanAndBeUnusable();
+ });
this.elementGuiMap.clear();
super.reloadValues();
@@ -30,23 +37,24 @@ public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGu
protected S newElementRequested(T generic, GuiItem newItem) {
S element = newInstanceOfGui(generic, newItem);
- newItem.setAction(GuiGlobalActions.openGuiAction(element.getMappedGui()));
+ newItem.setAction(element.openAction());
return element;
}
@Override
protected GuiItem findItemFromElement(T generic, S element) {
- return element.getParentItemForThisGui();
+ return element.getParentItem();
}
@Override
protected void updateElement(T generic, S element) {
- element.updateLocal();
+ ElementMappedToListGui gui = element.getStored();
+ if(gui != null) gui.updateLocal();
}
@Override
protected GuiItem findGuiItemForRemoval(T generic, S element) {
- return element.getParentItemForThisGui();
+ return element.getParentItem();
}
@Override
@@ -74,7 +82,7 @@ public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGu
// Not the most efficient on large number of conflict, but it should not run often.
for (T generic : getEveryDisplayableInstanceOfGeneric()) {
if (generic.toString().equalsIgnoreCase(message)) {
- player.sendMessage("\u00A7cPlease enter a "+genericDisplayedName()+" name that do not already exist...");
+ player.sendMessage("§cPlease enter a "+genericDisplayedName()+" name that do not already exist...");
// wait next message.
CustomAnvil.Companion.getChatListener().setListenedCallback(player, selfRef.get());
return;
@@ -90,7 +98,7 @@ public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGu
updateValueForGeneric(generic, true);
// show the new conflict config to the player
- this.elementGuiMap.get(generic).getMappedGui().show(player);
+ this.elementGuiMap.get(generic).get().getMappedGui().show(player);
update();
};
@@ -105,4 +113,28 @@ public abstract class MappedGuiListConfigGui< T, S extends ElementMappedToListGu
protected abstract T createAndSaveNewEmptyGeneric(String name);
+ public static class LazyElement extends LazyValue {
+
+ private final GuiItem parentItem;
+ private final LazyValue> lazyOpenConsumer;
+ public LazyElement(GuiItem parentItem, Supplier valueSupplier) {
+ super(valueSupplier);
+ this.parentItem = parentItem;
+
+ this.lazyOpenConsumer = new LazyValue<>(() ->
+ GuiGlobalActions.openGuiAction(this.get().getMappedGui()))
+ ;
+ }
+
+ public GuiItem getParentItem() {
+ return parentItem;
+ }
+
+ @NotNull
+ public Consumer openAction(){
+ return event -> lazyOpenConsumer.get().accept(event);
+ }
+
+ }
+
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/SettingGuiListConfigGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/SettingGuiListConfigGui.java
index 808b40d..f0e7cb0 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/SettingGuiListConfigGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/SettingGuiListConfigGui.java
@@ -1,12 +1,14 @@
package xyz.alexcrea.cuanvil.gui.config.list;
import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
import io.delilaheve.CustomAnvil;
import org.bukkit.Material;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
import xyz.alexcrea.cuanvil.gui.config.settings.SettingGui;
import java.util.HashMap;
@@ -17,12 +19,16 @@ public abstract class SettingGuiListConfigGui< T, S extends SettingGui.SettingGu
protected HashMap guiItemMap;
protected HashMap factoryMap;
- protected SettingGuiListConfigGui(@NotNull String title) {
- super(title);
+ protected SettingGuiListConfigGui(@NotNull String title, Gui parent) {
+ super(title, parent);
this.guiItemMap = new HashMap<>();
this.factoryMap = new HashMap<>();
}
+ protected SettingGuiListConfigGui(@NotNull String title) {
+ this(title, MainConfigGui.getInstance());
+ }
+
@Override
protected GuiItem prepareCreateNewItem() {
ItemStack createItem = new ItemStack(Material.PAPER);
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/UnitRepairElementListGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/UnitRepairElementListGui.java
index 2d47c05..35f8ebb 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/UnitRepairElementListGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/UnitRepairElementListGui.java
@@ -26,17 +26,14 @@ import java.util.function.Consumer;
public class UnitRepairElementListGui extends SettingGuiListConfigGui implements ElementMappedToListGui {
- private final GuiItem parentItem;
private final Material parentMaterial;
private final UnitRepairConfigGui parentGui;
private final String materialName;
private boolean shouldWork = true;
public UnitRepairElementListGui(@NotNull Material parentMaterial,
- @NotNull UnitRepairConfigGui parentGui,
- @NotNull GuiItem parentItem) {
- super("\u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.name().toLowerCase()) + " \u00A7rUnit repair");
- this.parentItem = parentItem;
+ @NotNull UnitRepairConfigGui parentGui) {
+ super("§e" + CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.name().toLowerCase()) + " §rUnit repair");
this.parentMaterial = parentMaterial;
this.parentGui = parentGui;
this.materialName = CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.name().toLowerCase());
@@ -48,8 +45,8 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui getCreateItemLore() {
return Arrays.asList(
- "\u00A77Select a new item to be repairable.",
- "\u00A77You will be asked the material to use."
+ "§7Select a new item to be repairable.",
+ "§7You will be asked the material to use."
);
}
@@ -64,26 +61,26 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui {
ItemMeta meta = itemStack.getItemMeta();
Material type = itemStack.getType();
if(!(meta instanceof Damageable) || (type.getMaxDurability() <= 0)) {
- player.sendMessage("\u00A7cThis item can't be damaged, so it can't be repaired.");
+ player.sendMessage("§cThis item can't be damaged, so it can't be repaired.");
return;
}
if(type == this.parentMaterial){
- player.sendMessage("\u00A7cItem can't repair something of the same type.");
+ player.sendMessage("§cItem can't repair something of the same type.");
return;
}
String materialName = type.name().toLowerCase();
// Add new material
- ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().set(parentMaterial.name().toLowerCase()+"."+materialName,0.25);
+ ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().set(parentMaterial.name().toLowerCase() + "." + materialName,0.25);
if (GuiSharedConstant.TEMPORARY_DO_SAVE_TO_DISK_EVERY_CHANGE) {
ConfigHolder.UNIT_REPAIR_HOLDER.saveToDisk(GuiSharedConstant.TEMPORARY_DO_BACKUP_EVERY_SAVE);
@@ -105,21 +102,21 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui {
event.setCancelled(true);
EnchantSelectSettingGui enchantGui = new EnchantSelectSettingGui(
- "\u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(enchantConflict.toString()) + "\u00A75",
+ "§e" + CasedStringUtil.snakeToUpperSpacedCase(enchantConflict.toString()) + "§5",
this, this);
enchantGui.show(event.getWhoClicked());
}, CustomAnvil.instance);
@@ -90,17 +89,17 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
this.groupSettingItem = new GuiItem(new ItemStack(Material.PAPER), event -> {
event.setCancelled(true);
GroupSelectSettingGui enchantGui = new GroupSelectSettingGui(
- "\u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(this.enchantConflict.toString()) + " \u00A73Groups",
+ "§e" + CasedStringUtil.snakeToUpperSpacedCase(this.enchantConflict.toString()) + " §3Groups",
this, this, 0);
enchantGui.show(event.getWhoClicked());
}, CustomAnvil.instance);
- this.minBeforeActiveSettingFactory = IntSettingsGui.intFactory(
- "\u00A78Minimum enchantment count",
+ this.minBeforeActiveSettingFactory = new IntSettingsGui.IntSettingFactory(
+ "§8Minimum enchantment count",
this, this.enchantConflict + ".maxEnchantmentBeforeConflict", ConfigHolder.CONFLICT_HOLDER,
Arrays.asList(
- "\u00A77Minimum enchantment count set to X mean only X enchantment can be put",
- "\u00A77on an item before the conflict is active."
+ "§7Minimum enchantment count set to X mean only X enchantment can be put",
+ "§7on an item before the conflict is active."
),
0, 255, 0, 1
);
@@ -127,7 +126,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
cleanAndBeUnusable();
// Update config file storage
- ConfigHolder.CONFLICT_HOLDER.getConfig().set(this.enchantConflict.toString(), null);
+ ConfigHolder.CONFLICT_HOLDER.delete(this.enchantConflict.toString());
// Save
boolean success = true;
@@ -138,8 +137,8 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
return success;
};
- return new ConfirmActionGui("\u00A7cDelete \u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(this.enchantConflict.toString()) + "\u00A7c?",
- "\u00A77Confirm that you want to delete this conflict.",
+ return new ConfirmActionGui("§cDelete §e" + CasedStringUtil.snakeToUpperSpacedCase(this.enchantConflict.toString()) + "§c?",
+ "§7Confirm that you want to delete this conflict.",
this, this.parent, deleteSupplier
);
}
@@ -160,12 +159,12 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
// Prepare enchantment lore
ArrayList enchantLore = new ArrayList<>();
- enchantLore.add("\u00A77Allow you to select a list of \u00A75Enchantments \u00A77that this conflict should include");
+ enchantLore.add("§7Allow you to select a list of §5Enchantments §7that this conflict should include");
Set enchants = getSelectedEnchantments();
if (enchants.isEmpty()) {
- enchantLore.add("\u00A77There is no included enchantment for this conflict.");
+ enchantLore.add("§7There is no included enchantment for this conflict.");
} else {
- enchantLore.add("\u00A77List of included enchantment for this conflict:");
+ enchantLore.add("§7List of included enchantment for this conflict:");
Iterator enchantIterator = enchants.iterator();
boolean greaterThanMax = enchants.size() > 5;
@@ -173,10 +172,10 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
for (int i = 0; i < maxindex; i++) {
// format string like "- Fire Protection"
String formattedName = CasedStringUtil.snakeToUpperSpacedCase(enchantIterator.next().getKey().getKey());
- enchantLore.add("\u00A77- \u00A75" + formattedName);
+ enchantLore.add("§7- §5" + formattedName);
}
if (greaterThanMax) {
- enchantLore.add("\u00A77And " + (enchants.size() - 4) + " more...");
+ enchantLore.add("§7And " + (enchants.size() - 4) + " more...");
}
}
@@ -189,7 +188,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
ItemMeta enchantMeta = enchantItem.getItemMeta();
assert enchantMeta != null;
- enchantMeta.setDisplayName("\u00A7aSelect included \u00A75Enchantments \u00A7aSettings");
+ enchantMeta.setDisplayName("§aSelect included §5Enchantments §aSettings");
enchantMeta.setLore(enchantLore);
enchantItem.setItemMeta(enchantMeta);
@@ -201,7 +200,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
ItemMeta groupMeta = groupItem.getItemMeta();
assert groupMeta != null;
- groupMeta.setDisplayName("\u00A7aSelect Excluded \u00A73Groups \u00A7aSettings");
+ groupMeta.setDisplayName("§aSelect Excluded §3Groups §aSettings");
groupMeta.setLore(groupLore);
groupItem.setItemMeta(groupMeta);
@@ -258,7 +257,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
String[] enchantKeys = new String[enchantments.size()];
int index = 0;
for (CAEnchantment enchantment : enchantments) {
- enchantKeys[index++] = enchantment.getKey().getKey();
+ enchantKeys[index++] = enchantment.getKey().toString();
}
ConfigHolder.CONFLICT_HOLDER.getConfig().set(enchantConflict + ".enchantments", enchantKeys);
@@ -266,6 +265,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
updateGuiValues();
} catch (Exception e) {
CustomAnvil.instance.getLogger().log(Level.WARNING, "An error occurred while updating enchants for " + this.enchantConflict, e);
+ MetricsUtil.INSTANCE.trackError(e);
}
// Save file configuration to disk
@@ -310,6 +310,7 @@ public class EnchantConflictSubSettingGui extends MappedToListSubSettingGui impl
updateGuiValues();
} catch (Exception e) {
CustomAnvil.instance.getLogger().log(Level.WARNING, "An error occurred while updating group for " + this.enchantConflict, e);
+ MetricsUtil.INSTANCE.trackError(e);
}
// Save file configuration to disk
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/GroupConfigSubSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/GroupConfigSubSettingGui.java
index 29d412b..3d05674 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/GroupConfigSubSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/GroupConfigSubSettingGui.java
@@ -5,6 +5,7 @@ import com.github.stefvanschie.inventoryframework.pane.PatternPane;
import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
import io.delilaheve.CustomAnvil;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import org.bukkit.entity.HumanEntity;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.ItemFlag;
@@ -37,10 +38,9 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
public GroupConfigSubSettingGui(
@NotNull GroupConfigGui parent,
- @NotNull IncludeGroup group,
- @NotNull GuiItem item) {
- super(item, 3,
- CasedStringUtil.snakeToUpperSpacedCase(group.getName()) + " Config");
+ @NotNull IncludeGroup group) {
+ super(3,
+ "§e" + CasedStringUtil.snakeToUpperSpacedCase(group.getName()) + " §rConfig");
this.parent = parent;
this.group = group;
@@ -65,27 +65,38 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
ItemStack deleteItem = new ItemStack(Material.RED_TERRACOTTA);
ItemMeta deleteMeta = deleteItem.getItemMeta();
- deleteMeta.setDisplayName("\u00A74DELETE GROUP");
- deleteMeta.setLore(Collections.singletonList("\u00A7cCaution with this button !"));
+ deleteMeta.setDisplayName("§4DELETE GROUP");
+ deleteMeta.setLore(Collections.singletonList("§cCaution with this button !"));
deleteItem.setItemMeta(deleteMeta);
this.pane.bindItem('D', new GuiItem(deleteItem, openGuiAndCheckAction(), CustomAnvil.instance));
// Displayed item will be updated later
- this.materialSelection = new GuiItem(new ItemStack(Material.DIAMOND_SWORD), (event) -> {
- event.setCancelled(true);
+ String materialSelectionName = "§e" + CasedStringUtil.snakeToUpperSpacedCase(group.getName()) + " §rMaterials";
+ ItemStack selectItem = new ItemStack(Material.DIAMOND_SWORD);
+ ItemMeta selectItemMeta = selectItem.getItemMeta();
+ selectItemMeta.setDisplayName(materialSelectionName);
+ selectItem.setItemMeta(selectItemMeta);
+ this.materialSelection = new GuiItem(selectItem, (event) -> {
+ event.setCancelled(true);
MaterialSelectSettingGui selectGui = new MaterialSelectSettingGui(this,
- CasedStringUtil.snakeToUpperSpacedCase(group.getName()) + " Materials"
+ materialSelectionName
, this);
selectGui.show(event.getWhoClicked());
}, CustomAnvil.instance);
- this.groupSelection = new GuiItem(new ItemStack(Material.CHEST), (event) -> {
+ String selectGroupName = "§e" + CasedStringUtil.snakeToUpperSpacedCase(this.group.getName()) + " §rGroups";
+ ItemStack selectGroup = new ItemStack(Material.CHEST);
+ ItemMeta selectGroupMeta = selectGroup.getItemMeta();
+ selectGroupMeta.setDisplayName(selectGroupName);
+
+ selectGroup.setItemMeta(selectGroupMeta);
+ this.groupSelection = new GuiItem(selectGroup, (event) -> {
event.setCancelled(true);
GroupSelectSettingGui enchantGui = new GroupSelectSettingGui(
- CasedStringUtil.snakeToUpperSpacedCase(this.group.getName()) + " Groups",
+ selectGroupName,
this, this, 0);
enchantGui.show(event.getWhoClicked());
}, CustomAnvil.instance);
@@ -129,7 +140,7 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
cleanAndBeUnusable();
// Update config file storage
- ConfigHolder.CUSTOM_RECIPE_HOLDER.getConfig().set(this.group.getName(), null);
+ ConfigHolder.CUSTOM_RECIPE_HOLDER.delete(this.group.getName());
// Save
boolean success = true;
@@ -140,8 +151,8 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
return success;
};
- return new ConfirmActionGui("\u00A7cDelete \u00A7e" + CasedStringUtil.snakeToUpperSpacedCase(this.group.toString()) + "\u00A7c?",
- "\u00A77Confirm that you want to delete this group.",
+ return new ConfirmActionGui("§cDelete §e" + CasedStringUtil.snakeToUpperSpacedCase(this.group.toString()) + "§c?",
+ "§7Confirm that you want to delete this group.",
this, this.parent, deleteSupplier
);
}
@@ -151,8 +162,8 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
if(usedLoc.isEmpty()){
return false;
}
- StringBuilder stb = new StringBuilder("\u00A7cCan't delete group " +this.group.getName()+
- "\n\u00A7eUsed by:");
+ StringBuilder stb = new StringBuilder("§cCan't delete group " +this.group.getName()+
+ "\n§eUsed by:");
int maxIndex = usedLoc.size();
int nbMore = 0;
if(maxIndex > 10){
@@ -160,10 +171,10 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
maxIndex = 9;
}
for (int i = 0; i < maxIndex; i++) {
- stb.append("\n\u00A7r-\u00A7e ").append(usedLoc.get(i));
+ stb.append("\n§r-§e ").append(usedLoc.get(i));
}
if(nbMore > 0){
- stb.append("\u00A7cAnd ").append(nbMore).append(" More...");
+ stb.append("§cAnd ").append(nbMore).append(" More...");
}
player.sendMessage(stb.toString());
@@ -214,7 +225,7 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
ItemStack matSelectItem = this.materialSelection.getItem();
ItemMeta matSelectMeta = matSelectItem.getItemMeta();
- matSelectMeta.setDisplayName("\u00A7aSelect included \u00A7eMaterials \u00A7aSettings");
+ matSelectMeta.setDisplayName("§aSelect included §eMaterials §aSettings");
matSelectMeta.setLore(matLore);
matSelectMeta.addItemFlags(ItemFlag.values());
@@ -226,7 +237,7 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
ItemStack groupSelectItem = this.groupSelection.getItem();
ItemMeta groupSelectMeta = groupSelectItem.getItemMeta();
- groupSelectMeta.setDisplayName("\u00A7aSelect included \u00A73Groups \u00A7aSettings");
+ groupSelectMeta.setDisplayName("§aSelect included §3Groups §aSettings");
groupSelectMeta.setLore(groupLore);
groupSelectItem.setItemMeta(groupSelectMeta);
@@ -311,23 +322,23 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
// ----------------------------
// End of SelectGroupContainer related methods
// ----------------------------
- // SelectGroupContainer related methods
+ // SelectMaterialContainer related methods
// ----------------------------
@Override
- public EnumSet getSelectedMaterials() {
+ public Set getSelectedMaterials() {
return this.group.getNonGroupInheritedMaterials();
}
@Override
- public boolean setSelectedMaterials(EnumSet materials) {
+ public boolean setSelectedMaterials(Set materials) {
this.group.setNonGroupInheritedMaterials(materials);
// Write to file configuration
String[] groupNames = new String[materials.size()];
int index = 0;
- for (Material otherGroup : materials) {
- groupNames[index++] = otherGroup.name().toLowerCase();
+ for (NamespacedKey otherGroup : materials) {
+ groupNames[index++] = otherGroup.getKey().toLowerCase();
}
ConfigHolder.ITEM_GROUP_HOLDER.getConfig().set(this.group.getName()+"."+ItemGroupManager.MATERIAL_LIST_PATH, groupNames);
@@ -343,12 +354,12 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
}
@Override
- public EnumSet illegalMaterials() {
- return EnumSet.of(Material.AIR);
+ public Set illegalMaterials() {
+ return Set.of(Material.AIR.getKey());
}
// ----------------------------
- // End of SelectGroupContainer related methods
+ // End of SelectMaterialContainer related methods
// ----------------------------
private void updateDirectReferencingGroups(AbstractMaterialGroup referenceTo){
@@ -371,7 +382,7 @@ public class GroupConfigSubSettingGui extends MappedToListSubSettingGui implemen
for (AbstractMaterialGroup otherGroup : everyStoredGroups) {
if(otherGroup.getGroups().contains(testGroup)){
otherGroup.updateMaterials();
- toUpdate.add(otherGroup);
+ updateFuture.add(otherGroup);
}
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/MappedToListSubSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/MappedToListSubSettingGui.java
index 020e6ed..1d86781 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/MappedToListSubSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/list/elements/MappedToListSubSettingGui.java
@@ -9,18 +9,10 @@ import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
public abstract class MappedToListSubSettingGui extends ChestGui implements ValueUpdatableGui, ElementMappedToListGui {
- private final GuiItem item;
protected MappedToListSubSettingGui(
- GuiItem item,
int rows,
@NotNull String title) {
super(rows, title, CustomAnvil.instance);
- this.item = item;
- }
-
- @Override
- public GuiItem getParentItemForThisGui() {
- return item;
}
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/AbstractSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/AbstractSettingGui.java
index c9ce78e..b07e7c1 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/AbstractSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/AbstractSettingGui.java
@@ -17,7 +17,7 @@ import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
*/
public abstract class AbstractSettingGui extends ChestGui implements SettingGui {
- protected static final String CLICK_LORE = "\u00A77Click Here to change the value";
+ public static final String CLICK_LORE = "§7Click Here to change the value";
private PatternPane pane;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/BoolSettingsGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/BoolSettingsGui.java
index bb058c6..1d5e73d 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/BoolSettingsGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/BoolSettingsGui.java
@@ -66,9 +66,9 @@ public class BoolSettingsGui extends AbstractSettingGui {
// Prepare default Value text
String defaultValueLore;
if(holder.defaultVal){
- defaultValueLore = "\u00A7aYes \u00A77Is the default value";
+ defaultValueLore = "§aYes §7Is the default value";
}else{
- defaultValueLore = "\u00A7cNo \u00A77Is the default value";
+ defaultValueLore = "§cNo §7Is the default value";
}
// Create reset to default item
@@ -76,7 +76,7 @@ public class BoolSettingsGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7eReset to default value");
+ meta.setDisplayName("§eReset to default value");
meta.setLore(Collections.singletonList(defaultValueLore));
item.setItemMeta(meta);
returnToDefault = new GuiItem(item, event -> {
@@ -97,10 +97,10 @@ public class BoolSettingsGui extends AbstractSettingGui {
String displayedName;
Material displayedMat;
if (now) {
- displayedName = "\u00A7aYes";
+ displayedName = "§aYes";
displayedMat = Material.GREEN_TERRACOTTA;
} else {
- displayedName = "\u00A7cNo";
+ displayedName = "§cNo";
displayedMat = Material.RED_TERRACOTTA;
}
@@ -161,28 +161,6 @@ public class BoolSettingsGui extends AbstractSettingGui {
return now != before;
}
- /**
- * Create a bool setting factory from setting's parameters.
- *
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param config Configuration holder of this setting.
- * @param configPath Configuration path of this setting.
- * @param defaultVal Default value if not found on the config.
- * @param displayLore Gui display item lore.
- * @return A factory for a boolean setting gui.
- */
- public static BoolSettingFactory boolFactory(@NotNull String title, @NotNull ValueUpdatableGui parent,
- @NotNull ConfigHolder config,
- @NotNull String configPath, boolean defaultVal,
- String... displayLore) {
- return new BoolSettingFactory(
- title, parent,
- config,
- configPath, defaultVal,
- displayLore);
- }
-
/**
* A factory for a boolean setting gui that hold setting's information.
*/
@@ -206,7 +184,7 @@ public class BoolSettingsGui extends AbstractSettingGui {
* @param defaultVal Default value if not found on the config.
* @param displayLore Gui display item lore.
*/
- protected BoolSettingFactory(
+ public BoolSettingFactory(
@NotNull String title, @NotNull ValueUpdatableGui parent,
@NotNull ConfigHolder config, @NotNull String configPath,
boolean defaultVal, String... displayLore) {
@@ -254,14 +232,14 @@ public class BoolSettingsGui extends AbstractSettingGui {
boolean value = getConfiguredValue();
Material itemMat;
- StringBuilder itemName = new StringBuilder("\u00A7e");
+ StringBuilder itemName = new StringBuilder("§e");
String finalValue;
if (value) {
itemMat = Material.GREEN_TERRACOTTA;
- finalValue = "\u00A7aYes";
+ finalValue = "§aYes";
} else {
itemMat = Material.RED_TERRACOTTA;
- finalValue = "\u00A7cNo";
+ finalValue = "§cNo";
}
itemName.append(name);
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/DoubleSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/DoubleSettingGui.java
index 5c2692f..c0dcff6 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/DoubleSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/DoubleSettingGui.java
@@ -68,9 +68,9 @@ public class DoubleSettingGui extends AbstractSettingGui {
ItemMeta meta = DELETE_ITEM_STACK.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7cDisable item being repaired ?");
- meta.setLore(Arrays.asList("\u00A77Confirm disabling unit repair for this item..",
- "\u00A74Cation: This action can't be canceled."));
+ meta.setDisplayName("§cDisable item being repaired ?");
+ meta.setLore(Arrays.asList("§7Confirm disabling unit repair for this item..",
+ "§4Cation: This action can't be canceled."));
DELETE_ITEM_STACK.setItemMeta(meta);
}
@@ -121,8 +121,8 @@ public class DoubleSettingGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7eReset to default value");
- meta.setLore(Collections.singletonList("\u00A77Default value is \u00A7e" + displayValue(holder.defaultVal)));
+ meta.setDisplayName("§eReset to default value");
+ meta.setLore(Collections.singletonList("§7Default value is §e" + displayValue(holder.defaultVal)));
item.setItemMeta(meta);
returnToDefault = new GuiItem(item, event -> {
event.setCancelled(true);
@@ -144,7 +144,7 @@ public class DoubleSettingGui extends AbstractSettingGui {
if (now.compareTo(holder.min) > 0) {
BigDecimal planned = holder.min.max(now.subtract(step));
- minusItem = getSetValueItem(Material.RED_TERRACOTTA, planned, "\u00A7c-");
+ minusItem = getSetValueItem(Material.RED_TERRACOTTA, planned, "§c-");
} else {
minusItem = GuiGlobalItems.backgroundItem(Material.BARRIER);
}
@@ -155,7 +155,7 @@ public class DoubleSettingGui extends AbstractSettingGui {
if (now.compareTo(holder.max) < 0) {
BigDecimal planned = holder.max.min(now.add(step));
- plusItem = getSetValueItem(Material.GREEN_TERRACOTTA, planned, "\u00A7a+");
+ plusItem = getSetValueItem(Material.GREEN_TERRACOTTA, planned, "§a+");
} else {
plusItem = GuiGlobalItems.backgroundItem(Material.BARRIER);
}
@@ -166,7 +166,7 @@ public class DoubleSettingGui extends AbstractSettingGui {
ItemMeta resultMeta = resultPaper.getItemMeta();
assert resultMeta != null;
- resultMeta.setDisplayName("\u00A7fValue: \u00A7e" + displayValue(now));
+ resultMeta.setDisplayName("§fValue: §e" + displayValue(now));
resultPaper.setItemMeta(resultMeta);
GuiItem resultItem = new GuiItem(resultPaper, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
@@ -197,8 +197,8 @@ public class DoubleSettingGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7e" + displayValue(now) + " \u00A7f-> \u00A7e" + displayValue(planned)
- + " \u00A7r(" + numberPrefix + (displayValue(planned.subtract(now).abs()) + "\u00A7r)"));
+ meta.setDisplayName("§e" + displayValue(now) + " §f-> §e" + displayValue(planned)
+ + " §r(" + numberPrefix + (displayValue(planned.subtract(now).abs()) + "§r)"));
meta.setLore(setLoreItem);
item.setItemMeta(meta);
@@ -271,21 +271,21 @@ public class DoubleSettingGui extends AbstractSettingGui {
// Get material properties
Material stepMat;
- StringBuilder stepName = new StringBuilder("\u00A7");
+ StringBuilder stepName = new StringBuilder("§");
List stepLore;
Consumer clickEvent;
if (stepValue.compareTo(step) == 0) {
stepMat = Material.GREEN_STAINED_GLASS_PANE;
stepName.append('a');
- stepLore = Collections.singletonList("\u00A77Value is changing by " + displayValue(stepValue));
+ stepLore = Collections.singletonList("§7Value is changing by " + displayValue(stepValue));
clickEvent = GuiGlobalActions.stayInPlace;
} else {
stepMat = Material.RED_STAINED_GLASS_PANE;
stepName.append('c');
- stepLore = Collections.singletonList("\u00A77Click here to change the value by " + displayValue(stepValue));
+ stepLore = Collections.singletonList("§7Click here to change the value by " + displayValue(stepValue));
clickEvent = updateStepValue(stepValue);
}
- stepName.append("Step of \u00A7e").append(displayValue(stepValue));
+ stepName.append("Step of §e").append(displayValue(stepValue));
// Create item stack then gui item
ItemStack item = new ItemStack(stepMat);
@@ -316,7 +316,11 @@ public class DoubleSettingGui extends AbstractSettingGui {
@Override
public boolean onSave() {
if(isNull()){
- this.holder.config.getConfig().set(this.holder.configPath, null);
+ if(this.holder.config instanceof ConfigHolder.DeletableResource deletableResource){
+ deletableResource.delete(this.holder.configPath);
+ }else{
+ this.holder.config.getConfig().set(this.holder.configPath, null);
+ }
}else{
this.holder.config.getConfig().set(this.holder.configPath, now.doubleValue());
}
@@ -348,42 +352,6 @@ public class DoubleSettingGui extends AbstractSettingGui {
return value.toString();
}
- /**
- * Create a double setting factory from setting's parameters.
- *
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param config Configuration holder of this setting.
- * @param configPath Configuration path of this setting.
- * @param displayLore Gui display item lore.
- * @param scale The scale of the decimal.
- * @param asPercentage If we should display the value as a %.
- * @param nullOnZero Set the value as null (deleting it) when equal to 0
- * @param min Minimum value of this setting.
- * @param max Maximum value of this setting.
- * @param defaultVal Default value if not found on the config.
- * @param steps List of step the value can increment/decrement.
- * List's size should be between 1 (included) and 5 (included).
- * it is visually preferable to have an odd number of step.
- * If step only contain 1 value, no step item should be displayed.
- * @return A factory for a double setting gui.
- */
- @NotNull
- public static DoubleSettingFactory doubleFactory(@NotNull String title, @NotNull ValueUpdatableGui parent,
- @NotNull ConfigHolder config,
- @NotNull String configPath,
- @Nullable List displayLore,
- int scale, boolean asPercentage, boolean nullOnZero,
- double min, double max, double defaultVal, double... steps) {
- return new DoubleSettingFactory(
- title, parent,
- config,
- configPath,
- displayLore,
- scale, asPercentage, nullOnZero,
- min, max, defaultVal, steps);
- }
-
/**
* A factory for a double setting gui that hold setting's information.
*/
@@ -423,7 +391,7 @@ public class DoubleSettingGui extends AbstractSettingGui {
* it is visually preferable to have an odd number of step.
* If step only contain 1 value, no step item should be displayed.
*/
- protected DoubleSettingFactory(
+ public DoubleSettingFactory(
@NotNull String title, @NotNull ValueUpdatableGui parent,
@NotNull ConfigHolder config,
@NotNull String configPath,
@@ -483,10 +451,10 @@ public class DoubleSettingGui extends AbstractSettingGui {
public GuiItem getItem(Material itemMat, String name){
// Get item properties
BigDecimal value = getConfiguredValue();
- StringBuilder itemName = new StringBuilder("\u00A7a").append(name);
+ StringBuilder itemName = new StringBuilder("§a").append(name);
return GuiGlobalItems.createGuiItemFromProperties(this, itemMat, itemName,
- "\u00A7e" + displayValue(value, this.asPercentage),
+ "§e" + displayValue(value, this.asPercentage),
this.displayLore, true);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantCostSettingsGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantCostSettingsGui.java
index f347588..3bd923b 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantCostSettingsGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantCostSettingsGui.java
@@ -5,6 +5,7 @@ import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
import com.github.stefvanschie.inventoryframework.pane.PatternPane;
import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
import io.delilaheve.CustomAnvil;
+import io.delilaheve.util.ConfigOptions;
import org.bukkit.Material;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.ItemFlag;
@@ -13,6 +14,7 @@ import org.bukkit.inventory.meta.ItemMeta;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
@@ -83,10 +85,10 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
ItemMeta bookMeta = bookItemstack.getItemMeta();
assert bookMeta != null;
- bookMeta.setDisplayName("\u00A7aCost of an Enchantment by Book");
+ bookMeta.setDisplayName("§aCost of an Enchantment by Book");
bookMeta.setLore(Arrays.asList(
- "\u00A77Cost per result item level of an sacrifice enchantment",
- "\u00A77Only apply if sacrificed item \u00A7cis \u00A77a book"));
+ "§7Cost per result item level of an sacrifice enchantment",
+ "§7Only apply if sacrificed item §cis §7a book"));
bookItemstack.setItemMeta(bookMeta);
// sword display
@@ -95,10 +97,10 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
assert swordMeta != null;
swordMeta.addItemFlags(ItemFlag.values());
- swordMeta.setDisplayName("\u00A7aCost of an Enchantment by Item");
+ swordMeta.setDisplayName("§aCost of an Enchantment by Item");
swordMeta.setLore(Arrays.asList(
- "\u00A77Cost per result item level of an sacrifice enchantment",
- "\u00A77Only apply if sacrificed item \u00A7cis not \u00A77a book"));
+ "§7Cost per result item level of an sacrifice enchantment",
+ "§7Only apply if sacrificed item §cis not §7a book"));
swordItemstack.setItemMeta(swordMeta);
pane.bindItem('1', GuiGlobalItems.backgroundItem(Material.BLACK_STAINED_GLASS_PANE));
@@ -115,10 +117,10 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
// assume holder is an instance of EnchantCostSettingFactory
EnchantCostSettingFactory holder = (EnchantCostSettingFactory) this.holder;
- meta.setDisplayName("\u00A7eReset to default value");
+ meta.setDisplayName("§eReset to default value");
meta.setLore(Arrays.asList(
- "\u00A77Default item value is: \u00A7e" + holder.defaultVal,
- "\u00A77Default book value is: \u00A7e" + holder.defaultBookVal));
+ "§7Default item value is: §e" + holder.defaultVal,
+ "§7Default book value is: §e" + holder.defaultBookVal));
item.setItemMeta(meta);
returnToDefault = new GuiItem(item, event -> {
event.setCancelled(true);
@@ -147,7 +149,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7e" + nowBook + " \u00A7f-> \u00A7e" + planned + " \u00A7r(\u00A7c-" + (nowBook - planned) + "\u00A7r)");
+ meta.setDisplayName("§e" + nowBook + " §f-> §e" + planned + " §r(§c-" + (nowBook - planned) + "§r)");
meta.setLore(Collections.singletonList(AbstractSettingGui.CLICK_LORE));
item.setItemMeta(meta);
@@ -165,7 +167,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7e" + nowBook + " \u00A7f-> \u00A7e" + planned + " \u00A7r(\u00A7a+" + (planned - nowBook) + "\u00A7r)");
+ meta.setDisplayName("§e" + nowBook + " §f-> §e" + planned + " §r(§a+" + (planned - nowBook) + "§r)");
meta.setLore(Collections.singletonList(AbstractSettingGui.CLICK_LORE));
item.setItemMeta(meta);
@@ -180,7 +182,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
ItemMeta nowMeta = nowPaper.getItemMeta();
assert nowMeta != null;
- nowMeta.setDisplayName("\u00A7fValue: \u00A7e" + nowBook);
+ nowMeta.setDisplayName("§fValue: §e" + nowBook);
if(!holder.displayLore.isEmpty()){
nowMeta.setLore(holder.displayLore);
}
@@ -237,73 +239,45 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
return super.hadChange() || nowBook != beforeBook;
}
- /**
- * Create an int setting factory from setting's parameters.
- *
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param config Configuration holder of this setting.
- * @param configPath Configuration path of this setting.
- * @param displayLore Gui display item lore.
- * @param min Minimum value of this setting.
- * @param max Maximum value of this setting.
- * @param defaultItemVal Default item value if not found on the config.
- * @param defaultBookVal Default book value if not found on the config.
- * @param steps List of step the value can increment/decrement.
- * List's size should be between 1 (included) and 3 (included).
- * it is visually preferable to have an odd number of step.
- * If step only contain 1 value, no step item should be displayed.
- * @return A factory for an enchant cost setting gui.
- */
- public static EnchantCostSettingFactory enchantCostFactory(
- @NotNull String title, @NotNull ValueUpdatableGui parent,
- @NotNull ConfigHolder config, @NotNull String configPath,
- @Nullable List displayLore,
- int min, int max, int defaultItemVal, int defaultBookVal,
- int... steps) {
- return new EnchantCostSettingFactory(
- title, parent,
- configPath, config,
- displayLore,
- min, max, defaultItemVal, defaultBookVal, steps);
- }
-
/**
* A factory for an enchantment cost setting gui that hold setting's information.
*/
public static class EnchantCostSettingFactory extends IntSettingsGui.IntSettingFactory {
int defaultBookVal;
+ @NotNull CAEnchantment enchantment;
/**
* Constructor for an enchantment cost setting gui factory.
*
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param configPath Configuration path of this setting.
- * @param config Configuration holder of this setting.
- * @param displayLore Gui display item lore.
- * @param min Minimum value of this setting.
- * @param max Maximum value of this setting.
- * @param defaultItemVal Default item value if not found on the config.
- * @param defaultBookVal Default book value if not found on the config.
- * @param steps List of step the value can increment/decrement.
- * List's size should be between 1 (included) and 3 (included).
- * it is visually preferable to have an odd number of step.
- * If step only contain 1 value, no step item should be displayed.
+ * @param title The title of the gui.
+ * @param parent Parent gui to go back when completed.
+ * @param configPath Configuration path of this setting.
+ * @param config Configuration holder of this setting.
+ * @param displayLore Gui display item lore.
+ * @param min Minimum value of this setting.
+ * @param max Maximum value of this setting.
+ * @param enchantment Enchantment to change the cost to
+ * @param steps List of step the value can increment/decrement.
+ * List's size should be between 1 (included) and 3 (included).
+ * it is visually preferable to have an odd number of step.
+ * If step only contain 1 value, no step item should be displayed.
*/
- protected EnchantCostSettingFactory(
+ public EnchantCostSettingFactory(
@NotNull String title, ValueUpdatableGui parent,
@NotNull String configPath, @NotNull ConfigHolder config,
@Nullable List displayLore,
- int min, int max, int defaultItemVal, int defaultBookVal,
- int... steps) {
+ @NotNull CAEnchantment enchantment,
+ int min, int max, int... steps) {
super(title, parent,
configPath, config,
displayLore,
- min, max, defaultItemVal, steps);
- this.defaultBookVal = defaultBookVal;
+ min, max, enchantment.defaultRarity().getItemValue(),
+ steps);
+
+ this.defaultBookVal = enchantment.defaultRarity().getBookValue();
+ this.enchantment = enchantment;
}
/**
@@ -311,14 +285,14 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
*/
@Override
public int getConfiguredValue() {
- return this.config.getConfig().getInt(this.configPath + ITEM_PATH, this.defaultVal);
+ return ConfigOptions.INSTANCE.enchantmentValue(enchantment, false);
}
/**
* @return The configured value for the enchant setting book value.
*/
public int getConfiguredBookValue() {
- return this.config.getConfig().getInt(this.configPath + BOOK_PATH, this.defaultBookVal);
+ return ConfigOptions.INSTANCE.enchantmentValue(enchantment, true);
}
@Override
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantSelectSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantSelectSettingGui.java
index 308d51f..176da55 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantSelectSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantSelectSettingGui.java
@@ -15,6 +15,7 @@ import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
+import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
import xyz.alexcrea.cuanvil.gui.config.SelectEnchantmentContainer;
import xyz.alexcrea.cuanvil.gui.config.list.SettingGuiListConfigGui;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
@@ -35,7 +36,7 @@ public class EnchantSelectSettingGui extends SettingGuiListConfigGui(enchantContainer.getSelectedEnchantments());
@@ -52,11 +53,11 @@ public class EnchantSelectSettingGui extends SettingGuiListConfigGui getEveryDisplayableInstanceOfGeneric() {
Stream toDisplayStream;
if(this.displayUnselected){
- toDisplayStream = CAEnchantmentRegistry.getInstance().values().stream();
+ toDisplayStream = CAEnchantmentRegistry.getInstance().getNameSortedEnchantments().stream();
}else{
- toDisplayStream = this.selectedEnchant.stream();
+ toDisplayStream = this.selectedEnchant.stream().sorted(Comparator.comparing(CAEnchantment::getName));
}
Set illegalEnchantments = this.enchantContainer.illegalEnchantments();
@@ -107,9 +108,9 @@ public class EnchantSelectSettingGui extends SettingGuiListConfigGui TRUE_LORE = Collections.singletonList("\u00A77Value: \u00A7aSelected");
- private static final List FALSE_LORE = Collections.singletonList("\u00A77Value: \u00A7cNot Selected");
+ private static final List TRUE_LORE = Collections.singletonList("§7Value: §aSelected");
+ private static final List FALSE_LORE = Collections.singletonList("§7Value: §cNot Selected");
public void setEnchantItemMeta(ItemStack item, String name, boolean isIn) {
ItemMeta meta = item.getItemMeta();
@@ -138,7 +139,7 @@ public class EnchantSelectSettingGui extends SettingGuiListConfigGui & EnumSettingGui.ConfigurableEnum> extends AbstractSettingGui {
+
+ private final EnumSettingFactory holder;
+ private final T before;
+ private T now;
+
+ /**
+ * Create an item setting config gui.
+ *
+ * @param holder Configuration factory of this setting.
+ * @param now The defined value of this setting.
+ */
+ protected EnumSettingGui(EnumSettingFactory holder, T now) {
+ super(3, holder.getTitle(), holder.parent);
+ this.holder = holder;
+ this.before = now;
+ this.now = now;
+
+ prepareStaticItems();
+ updateValueDisplay();
+ }
+
+ @Override
+ public Pattern getGuiPattern() {
+ return new Pattern(
+ GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ "D000v0000",
+ "B0000000S"
+ );
+ }
+
+
+ public void prepareStaticItems(){
+ prepareReturnToDefault();
+ }
+
+
+ protected GuiItem returnToDefault;
+
+ /**
+ * Prepare "return to default value" gui item.
+ */
+ protected void prepareReturnToDefault() {
+ ItemStack item = new ItemStack(Material.COMMAND_BLOCK);
+ ItemMeta meta = item.getItemMeta();
+ assert meta != null;
+
+ meta.setDisplayName("§eReset to default value");
+ meta.setLore(Collections.singletonList("§7Default value is §e" + holder.getDefault().configurationGuiName()));
+ item.setItemMeta(meta);
+ returnToDefault = new GuiItem(item, event -> {
+ event.setCancelled(true);
+ now = holder.getDefault();
+ updateValueDisplay();
+ update();
+ }, CustomAnvil.instance);
+ }
+
+ /**
+ * Update item using the setting value to match the new value
+ */
+ protected void updateValueDisplay() {
+ PatternPane pane = getPane();
+
+ // Get displayed value for this config.
+ ItemStack displayedItem = now.configurationGuiItem();
+
+ GuiItem resultItem = new GuiItem(displayedItem, selectNext(), CustomAnvil.instance);
+ pane.bindItem('v', resultItem);
+
+ // reset to default
+ GuiItem returnToDefault;
+ if (now != holder.getDefault()) {
+ returnToDefault = this.returnToDefault;
+ } else {
+ returnToDefault = GuiGlobalItems.backgroundItem();
+ }
+ pane.bindItem('D', returnToDefault);
+
+ }
+
+ /**
+ * @return A consumer to update the current setting's value.
+ */
+ protected Consumer selectNext() {
+ return event -> {
+ event.setCancelled(true);
+
+ this.now = this.holder.next(this.now);
+
+ updateValueDisplay();
+ update();
+ };
+
+ }
+
+ @Override
+ public boolean onSave() {
+ holder.config.getConfig().set(holder.configPath, this.now.configName());
+
+ if (GuiSharedConstant.TEMPORARY_DO_SAVE_TO_DISK_EVERY_CHANGE) {
+ return holder.config.saveToDisk(GuiSharedConstant.TEMPORARY_DO_BACKUP_EVERY_SAVE);
+ }
+ return true;
+ }
+
+ @Override
+ public boolean hadChange() {
+ return !now.equals(before);
+ }
+
+
+ /**
+ * A factory for an enum setting gui that hold setting's information.
+ */
+ public abstract static class EnumSettingFactory & ConfigurableEnum> extends SettingGuiFactory {
+ @NotNull
+ String title;
+ @NotNull
+ ValueUpdatableGui parent;
+
+ /**
+ * Constructor for an enum settings gui factory
+ *
+ * @param title The title of the gui.
+ * @param parent Parent gui to go back when completed.
+ * @param configPath Configuration path of this setting.
+ * @param config Configuration holder of this setting.
+ */
+ protected EnumSettingFactory(
+ @NotNull String title, @NotNull ValueUpdatableGui parent,
+ @NotNull String configPath, @NotNull ConfigHolder config) {
+ super(configPath, config);
+ this.title = title;
+ this.parent = parent;
+
+ }
+ /**
+ * @return Get setting's gui title.
+ */
+ @NotNull
+ public String getTitle() {
+ return title;
+ }
+
+ /**
+ * @return The configured value for the associated setting.
+ */
+ @NotNull
+ public abstract T getConfiguredValue();
+
+ @NotNull
+ public abstract List getDisplayLore(T value);
+
+ /**
+ * @return Next value for a given enum
+ */
+ @NotNull
+ public T next(@NotNull T now){
+ Class clazz = now.getDeclaringClass();
+ T[] values = clazz.getEnumConstants();
+
+ int index = now.ordinal();
+ if(index == values.length - 1)
+ return values[0];
+
+ return values[index + 1];
+ }
+
+ /**
+ * Get default value value
+ * @return default value
+ */
+ @NotNull
+ public abstract T getDefault();
+
+ @Override
+ public Gui create() {
+ // Get value or default
+ T now = getConfiguredValue();
+
+ // create new gui
+ return new EnumSettingGui<>(this, now);
+ }
+
+ /**
+ * Create a new enum setting GuiItem.
+ * This item will create and open an enum setting GUI from the factory.
+ *
+ * @param name Name of the display.
+ * @return A formatted GuiItem that will create and open a GUI for the enum setting.
+ */
+ public GuiItem getItem(@NotNull Material material, @NotNull String name) {
+ T value = getConfiguredValue();
+
+ ItemStack item = new ItemStack(material);
+ ItemMeta meta = item.getItemMeta();
+ assert meta != null;
+
+ meta.setDisplayName(name);
+ meta.setLore(getDisplayLore(value));
+ meta.addItemFlags(ItemFlag.values());
+
+ item.setItemMeta(meta);
+
+ return GuiGlobalItems.openSettingGuiItem(item, this);
+ }
+ }
+
+ public interface ConfigurableEnum {
+
+ String configName();
+
+ ItemStack configurationGuiItem();
+ String configurationGuiName();
+
+
+ }
+
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/GroupSelectSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/GroupSelectSettingGui.java
index 9808a8b..69986a0 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/GroupSelectSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/GroupSelectSettingGui.java
@@ -17,6 +17,7 @@ import xyz.alexcrea.cuanvil.config.ConfigHolder;
import xyz.alexcrea.cuanvil.group.AbstractMaterialGroup;
import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
import xyz.alexcrea.cuanvil.gui.config.SelectGroupContainer;
+import xyz.alexcrea.cuanvil.gui.config.list.ElementListConfigGui;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
import xyz.alexcrea.cuanvil.util.CasedStringUtil;
@@ -50,18 +51,22 @@ public class GroupSelectSettingGui extends AbstractSettingGui {
@Override
protected Pattern getGuiPattern() {
return new Pattern(
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ GuiSharedConstant.UPPER_FILLER_FULL_PLANE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
"B1111111S"
);
}
protected void initGroups() {
// Add enchantment gui item
- OutlinePane filledEnchant = new OutlinePane(0, 0, 9, 5);
+ OutlinePane filledEnchant = new OutlinePane(
+ ElementListConfigGui.LIST_FILLER_START_X,
+ ElementListConfigGui.LIST_FILLER_START_Y,
+ ElementListConfigGui.LIST_FILLER_LENGTH,
+ ElementListConfigGui.LIST_FILLER_HEIGHT);
filledEnchant.setPriority(Pane.Priority.HIGH);
filledEnchant.align(OutlinePane.Alignment.BEGIN);
filledEnchant.setOrientation(Orientable.Orientation.HORIZONTAL);
@@ -91,8 +96,8 @@ public class GroupSelectSettingGui extends AbstractSettingGui {
return guiItem;
}
- private static final List TRUE_LORE = Collections.singletonList("\u00A77Value: \u00A7aSelected");
- private static final List FALSE_LORE = Collections.singletonList("\u00A77Value: \u00A7cNot Selected");
+ private static final List TRUE_LORE = Collections.singletonList("§7Value: §aSelected");
+ private static final List FALSE_LORE = Collections.singletonList("§7Value: §cNot Selected");
public void setGroupItemMeta(ItemStack item, String name, boolean isIn) {
ItemMeta meta = item.getItemMeta();
@@ -105,7 +110,7 @@ public class GroupSelectSettingGui extends AbstractSettingGui {
assert meta != null;
}
- meta.setDisplayName("\u00A7" + (isIn ? 'a' : 'c') + CasedStringUtil.snakeToUpperSpacedCase(name));
+ meta.setDisplayName("§" + (isIn ? 'a' : 'c') + CasedStringUtil.snakeToUpperSpacedCase(name));
if (isIn) {
meta.addEnchant(Enchantment.DAMAGE_UNDEAD, 1, true);
meta.setLore(TRUE_LORE);
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/IntSettingsGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/IntSettingsGui.java
index 389546b..73121a6 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/IntSettingsGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/IntSettingsGui.java
@@ -71,8 +71,9 @@ public class IntSettingsGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7eReset to default value");
- meta.setLore(Collections.singletonList("\u00A77Default value is \u00A7e" + holder.defaultVal));
+ meta.setDisplayName("§eReset to default value");
+ meta.setLore(Collections.singletonList("§7Default value is §e" +
+ holder.valueDisplayName(ValueDisplayType.RESET, holder.defaultVal)));
item.setItemMeta(meta);
returnToDefault = new GuiItem(item, event -> {
event.setCancelled(true);
@@ -86,41 +87,23 @@ public class IntSettingsGui extends AbstractSettingGui {
* Update item using the setting value to match the new value.
*/
protected void updateValueDisplay() {
-
PatternPane pane = getPane();
// minus item
GuiItem minusItem;
if (now > holder.min) {
int planned = Math.max(holder.min, now - step);
- ItemStack item = new ItemStack(Material.RED_TERRACOTTA);
- ItemMeta meta = item.getItemMeta();
- assert meta != null;
-
- meta.setDisplayName("\u00A7e" + now + " \u00A7f-> \u00A7e" + planned + " \u00A7r(\u00A7c-" + (now - planned) + "\u00A7r)");
- meta.setLore(Collections.singletonList(AbstractSettingGui.CLICK_LORE));
- item.setItemMeta(meta);
-
- minusItem = new GuiItem(item, updateNowConsumer(planned), CustomAnvil.instance);
+ minusItem = valueEditItem(Material.RED_TERRACOTTA, ValueDisplayType.REMOVE, planned);
} else {
minusItem = GuiGlobalItems.backgroundItem(Material.BARRIER);
}
pane.bindItem('-', minusItem);
//plus item
- // may do a function to generalise ?
GuiItem plusItem;
if (now < holder.max) {
int planned = Math.min(holder.max, now + step);
- ItemStack item = new ItemStack(Material.GREEN_TERRACOTTA);
- ItemMeta meta = item.getItemMeta();
- assert meta != null;
-
- meta.setDisplayName("\u00A7e" + now + " \u00A7f-> \u00A7e" + planned + " \u00A7r(\u00A7a+" + (planned - now) + "\u00A7r)");
- meta.setLore(Collections.singletonList(AbstractSettingGui.CLICK_LORE));
- item.setItemMeta(meta);
-
- plusItem = new GuiItem(item, updateNowConsumer(planned), CustomAnvil.instance);
+ plusItem = valueEditItem(Material.GREEN_TERRACOTTA, ValueDisplayType.ADD, planned);
} else {
plusItem = GuiGlobalItems.backgroundItem(Material.BARRIER);
}
@@ -131,7 +114,7 @@ public class IntSettingsGui extends AbstractSettingGui {
ItemMeta resultMeta = resultPaper.getItemMeta();
assert resultMeta != null;
- resultMeta.setDisplayName("\u00A7fValue: \u00A7e" + now);
+ resultMeta.setDisplayName("§fValue: §e" + holder.valueDisplayName(ValueDisplayType.CURRENT, now));
resultMeta.setLore(holder.displayLore);
resultPaper.setItemMeta(resultMeta);
@@ -149,7 +132,21 @@ public class IntSettingsGui extends AbstractSettingGui {
}
pane.bindItem('D', returnToDefault);
+ }
+ private GuiItem valueEditItem(Material mat, ValueDisplayType type, int planned) {
+ ItemStack item = new ItemStack(mat);
+ ItemMeta meta = item.getItemMeta();
+ assert meta != null;
+
+ var nowDisplay = holder.valueDisplayName(type, now);
+ var plannedDisplay = holder.valueDisplayName(type, planned);
+ var deltaDisplay = holder.deltaDisplay(type, now, planned);
+ meta.setDisplayName("§e" + nowDisplay + " §f-> §e" + plannedDisplay + " §r(§c" + deltaDisplay + "§r)");
+
+ meta.setLore(Collections.singletonList(AbstractSettingGui.CLICK_LORE));
+ item.setItemMeta(meta);
+ return new GuiItem(item, updateNowConsumer(planned), CustomAnvil.instance);
}
/**
@@ -218,21 +215,21 @@ public class IntSettingsGui extends AbstractSettingGui {
// Get material properties
Material stepMat;
- StringBuilder stepName = new StringBuilder("\u00A7");
+ StringBuilder stepName = new StringBuilder("§");
List stepLore;
Consumer clickEvent;
if (stepValue == step) {
stepMat = Material.GREEN_STAINED_GLASS_PANE;
stepName.append('a');
- stepLore = Collections.singletonList("\u00A77Value is changing by " + stepValue);
+ stepLore = Collections.singletonList("§7Value is changing by " + stepValue);
clickEvent = GuiGlobalActions.stayInPlace;
} else {
stepMat = Material.RED_STAINED_GLASS_PANE;
stepName.append('c');
- stepLore = Collections.singletonList("\u00A77Click here to change the value by " + stepValue);
+ stepLore = Collections.singletonList("§7Click here to change the value by " + stepValue);
clickEvent = updateStepValue(stepValue);
}
- stepName.append("Step of: \u00A7e").append(stepValue);
+ stepName.append("Step of: §e").append(stepValue);
// Create item stack then gui item
ItemStack item = new ItemStack(stepMat);
@@ -275,38 +272,11 @@ public class IntSettingsGui extends AbstractSettingGui {
return now != before;
}
- /**
- * Create an int setting factory from setting's parameters.
- *
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param configPath Configuration path of this setting.
- * @param config Configuration holder of this setting.
- * @param displayLore Gui display item lore.
- * @param min Minimum value of this setting.
- * @param max Maximum value of this setting.
- * @param defaultVal Default value if not found on the config.
- * @param steps List of step the value can increment/decrement.
- * List's size should be between 1 (included) and 5 (included).
- * it is visually preferable to have an odd number of step.
- * If step only contain 1 value, no step item should be displayed.
- * @return A factory for an int setting gui.
- */
- public static IntSettingFactory intFactory(@NotNull String title, ValueUpdatableGui parent,
- String configPath, ConfigHolder config,
- @Nullable List displayLore,
- int min, int max, int defaultVal, int... steps) {
- return new IntSettingFactory(
- title, parent,
- configPath, config,
- displayLore,
- min, max, defaultVal, steps);
- }
-
/**
* A factory for an int setting gui that hold setting's information.
*/
public static class IntSettingFactory extends SettingGuiFactory {
+
@NotNull
String title;
@NotNull
@@ -335,7 +305,7 @@ public class IntSettingsGui extends AbstractSettingGui {
* it is visually preferable to have an odd number of step.
* If step only contain 1 value, no step item should be displayed.
*/
- protected IntSettingFactory(
+ public IntSettingFactory(
@NotNull String title, @NotNull ValueUpdatableGui parent,
@NotNull String configPath, @NotNull ConfigHolder config,
@Nullable List displayLore,
@@ -393,10 +363,10 @@ public class IntSettingsGui extends AbstractSettingGui {
) {
// Get item properties
int value = getConfiguredValue();
- StringBuilder itemName = new StringBuilder("\u00A7a").append(name);
+ StringBuilder itemName = new StringBuilder("§a").append(name);
return GuiGlobalItems.createGuiItemFromProperties(this, itemMat, itemName,
- "\u00A7e" + value,
+ "§e" + value,
this.displayLore, true);
}
@@ -416,6 +386,23 @@ public class IntSettingsGui extends AbstractSettingGui {
return getItem(itemMat, CasedStringUtil.detectToUpperSpacedCase(configPath));
}
+ protected String valueDisplayName(ValueDisplayType type, int value) {
+ return String.valueOf(value);
+ }
+
+ protected String deltaDisplay(ValueDisplayType type, int now, int planned) {
+ var delta = planned - now;
+ if(delta < 0) return "§c" + delta;
+ else return "§a+" + delta;
+ }
+
+ }
+
+ public enum ValueDisplayType {
+ ADD,
+ CURRENT,
+ REMOVE,
+ RESET,
}
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/ItemSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/ItemSettingGui.java
index e49daa7..3df2af8 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/ItemSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/ItemSettingGui.java
@@ -62,7 +62,7 @@ public class ItemSettingGui extends AbstractSettingGui {
public void prepareStaticItems(){
prepareReturnToDefault();
- GuiItem temporaryLeave = GuiGlobalItems.temporaryCloseGuiToSelectItem(Material.YELLOW_TERRACOTTA, this);
+ GuiItem temporaryLeave = GuiGlobalItems.temporaryCloseGuiToSelectItem(Material.YELLOW_STAINED_GLASS_PANE, this);
getPane().bindItem('s', temporaryLeave);
}
@@ -77,8 +77,8 @@ public class ItemSettingGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7eReset to default value");
- meta.setLore(Collections.singletonList("\u00A77Default value is \u00A7e" + holder.defaultVal));
+ meta.setDisplayName("§eReset to default value");
+ meta.setLore(Collections.singletonList("§7Default value is §e" + holder.defaultVal));
item.setItemMeta(meta);
returnToDefault = new GuiItem(item, event -> {
event.setCancelled(true);
@@ -88,7 +88,7 @@ public class ItemSettingGui extends AbstractSettingGui {
}, CustomAnvil.instance);
}
- protected final static List CLICK_LORE = Collections.singletonList("\u00A77Click Here with an item to change the value");
+ protected final static List CLICK_LORE = Collections.singletonList("§7Click Here with an item to change the value");
/**
* Update item using the setting value to match the new value
@@ -105,7 +105,7 @@ public class ItemSettingGui extends AbstractSettingGui {
ItemMeta valueMeta = displayedItem.getItemMeta();
assert valueMeta != null;
- valueMeta.setDisplayName("\u00A74NO ITEM SET");
+ valueMeta.setDisplayName("§4NO ITEM SET");
valueMeta.setLore(CLICK_LORE);
displayedItem.setItemMeta(valueMeta);
@@ -163,27 +163,6 @@ public class ItemSettingGui extends AbstractSettingGui {
return !now.equals(before);
}
- /**
- * Create aa item setting factory from setting's parameters.
- *
- * @param title The title of the gui.
- * @param parent Parent gui to go back when completed.
- * @param configPath Configuration path of this setting.
- * @param config Configuration holder of this setting.
- * @param defaultVal Default value if not found on the config.
- * @param displayLore Gui display item lore.
- * @return A factory for an item setting gui.
- */
- public static ItemSettingGui.ItemSettingFactory itemFactory(@NotNull String title, @NotNull ValueUpdatableGui parent,
- @NotNull String configPath, @NotNull ConfigHolder config,
- @Nullable ItemStack defaultVal,
- String... displayLore) {
- return new ItemSettingGui.ItemSettingFactory(
- title, parent,
- configPath, config,
- defaultVal, displayLore);
- }
-
/**
* A factory for an item setting gui that hold setting's information.
*/
@@ -207,7 +186,7 @@ public class ItemSettingGui extends AbstractSettingGui {
* @param defaultVal Default value if not found on the config.
* @param displayLore Gui display item lore.
*/
- protected ItemSettingFactory(
+ public ItemSettingFactory(
@NotNull String title, @NotNull ValueUpdatableGui parent,
@NotNull String configPath, @NotNull ConfigHolder config,
@Nullable ItemStack defaultVal,
@@ -266,7 +245,7 @@ public class ItemSettingGui extends AbstractSettingGui {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7a" + name);
+ meta.setDisplayName("§a" + name);
meta.setLore(getDisplayLore());
meta.addItemFlags(ItemFlag.values());
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/MaterialSelectSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/MaterialSelectSettingGui.java
index 7998548..fc519ff 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/MaterialSelectSettingGui.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/MaterialSelectSettingGui.java
@@ -5,6 +5,7 @@ import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
import io.delilaheve.CustomAnvil;
import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
import org.bukkit.entity.HumanEntity;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.ItemFlag;
@@ -18,18 +19,19 @@ import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
import xyz.alexcrea.cuanvil.util.CasedStringUtil;
+import xyz.alexcrea.cuanvil.util.MaterialUtil;
import java.util.*;
import java.util.function.Consumer;
-public class MaterialSelectSettingGui extends MappedElementListConfigGui {
+public class MaterialSelectSettingGui extends MappedElementListConfigGui {
private final SelectMaterialContainer selector;
private final Gui backGui;
private boolean instantRemove;
- private final List defaultMaterials;
- private final EnumSet illegalMaterials;
+ private final List defaultMaterials;
+ private final Set illegalMaterials;
private final int defaultMaterialHash;
private int nowMaterialHash;
@@ -57,11 +59,11 @@ public class MaterialSelectSettingGui extends MappedElementListConfigGui result = EnumSet.noneOf(Material.class);
- result.addAll(this.elementGuiMap.keySet());
+ Set result = new HashSet<>(this.elementGuiMap.keySet());
if(!this.selector.setSelectedMaterials(result)){
- player.sendMessage("\u00A7cSomething went wrong while saving the change of value.");
+ player.sendMessage("§cSomething went wrong while saving the change of value.");
}
// Return to parent
@@ -185,8 +186,8 @@ public class MaterialSelectSettingGui extends MappedElementListConfigGui getEveryDisplayableInstanceOfGeneric() {
+ protected Collection getEveryDisplayableInstanceOfGeneric() {
return this.defaultMaterials;
}
@Override
- protected void updateElement(Material material, GuiItem element) {
+ protected void updateElement(NamespacedKey material, GuiItem element) {
// Nothing happen here I think
}
@Override
- protected GuiItem newElementRequested(Material material, GuiItem newItem) {
+ protected GuiItem newElementRequested(NamespacedKey material, GuiItem newItem) {
newItem.setAction(event -> {
if(this.instantRemove){
removeMaterial(material);
}else {
- String materialName = CasedStringUtil.snakeToUpperSpacedCase(material.name().toLowerCase());
+ String materialName = CasedStringUtil.snakeToUpperSpacedCase(material.getKey().toLowerCase());
// Create and show confirm remove gui.
ConfirmActionGui confirmGui = new ConfirmActionGui(
"Remove " + materialName,
- "\u00A77Confirm Remove " + materialName.toLowerCase() + " from this list.",
+ "§7Confirm Remove " + materialName.toLowerCase() + " from this list.",
this, this,
() -> {
removeMaterial(material);
@@ -250,7 +251,7 @@ public class MaterialSelectSettingGui extends MappedElementListConfigGui materialList){
+ private static int hashFromMaterialList(List materialList){
int defaultMaterialHash = 0;
- for (Material material : materialList) {
+ for (NamespacedKey material : materialList) {
defaultMaterialHash ^= material.hashCode();
}
return defaultMaterialHash;
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/WorkPenaltyTypeSettingGui.java b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/WorkPenaltyTypeSettingGui.java
new file mode 100644
index 0000000..4345aa1
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/WorkPenaltyTypeSettingGui.java
@@ -0,0 +1,252 @@
+package xyz.alexcrea.cuanvil.gui.config.settings;
+
+import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.pane.PatternPane;
+import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
+import io.delilaheve.CustomAnvil;
+import io.delilaheve.util.ConfigOptions;
+import org.bukkit.Material;
+import org.bukkit.configuration.file.FileConfiguration;
+import org.bukkit.entity.HumanEntity;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.inventory.meta.ItemMeta;
+import org.jetbrains.annotations.NotNull;
+import xyz.alexcrea.cuanvil.anvil.AnvilUseType;
+import xyz.alexcrea.cuanvil.config.ConfigHolder;
+import xyz.alexcrea.cuanvil.config.WorkPenaltyType;
+import xyz.alexcrea.cuanvil.gui.config.global.BasicConfigGui;
+import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
+
+import java.util.ArrayList;
+import java.util.EnumMap;
+import java.util.List;
+import java.util.Map;
+
+public class WorkPenaltyTypeSettingGui extends AbstractSettingGui {
+
+ private static final String INCREASING_EXPLANATION = "§eIncreasing§7: will penalty be increased (in item)";
+ private static final String ADDING_EXPLANATION = "§eAdditive§7: will penalty be added to the cost";
+
+ private static final String SHARED_EXPLANATION = "§eShared§7: Vanilla, shared penalty. it will be kept from before the plugin installation.";
+ private static final String EXCLUSIVE_EXPLANATION = "§eExclusive§7: Custom, per anvil use type penalty. it will be lost after plugin uninstallation";
+
+ private final @NotNull WorkPenaltyType currentType;
+ private final @NotNull Map items;
+
+ public WorkPenaltyTypeSettingGui(@NotNull BasicConfigGui parent) {
+ super(4, "§8Work Penalty Type", parent);
+
+ this.currentType = ConfigOptions.INSTANCE.getWorkPenaltyType();
+ this.items = new EnumMap<>(this.currentType.getPartMap());
+
+ for (AnvilUseType type : useTypes.keySet()) {
+ updateGuiForType(type);
+ }
+ }
+
+ public static GuiItem getDisplayItem(@NotNull BasicConfigGui parent,
+ @NotNull Material itemMat,
+ @NotNull String name) {
+ List displayLore = new ArrayList<>();
+
+ displayLore.add("§7Work penalty increase the price for every anvil use.");
+ displayLore.add("§7This config allow you to choose the comportment of work penalty.");
+ displayLore.add(INCREASING_EXPLANATION);
+ displayLore.add(ADDING_EXPLANATION);
+ displayLore.add("");
+ displayLore.add("§7About shared/exclusive penalty:");
+ displayLore.add(SHARED_EXPLANATION);
+ displayLore.add(EXCLUSIVE_EXPLANATION);
+
+ ItemStack item = new ItemStack(itemMat);
+
+ ItemMeta meta = item.getItemMeta();
+ meta.setDisplayName(name);
+ meta.setLore(displayLore);
+
+ item.setItemMeta(meta);
+
+ return new GuiItem(item, (event) -> {
+ event.setCancelled(true);
+ HumanEntity player = event.getWhoClicked();
+
+ // Do not allow to open inventory if player do not have edit configuration permission
+ if (!player.hasPermission(CustomAnvil.editConfigPermission)) {
+ player.closeInventory();
+ player.sendMessage(GuiGlobalActions.NO_EDIT_PERM);
+ return;
+ }
+ new WorkPenaltyTypeSettingGui(parent).show(player);
+ }, CustomAnvil.instance);
+ }
+
+ private static final Map useTypes =
+ Map.of(
+ AnvilUseType.RENAME_ONLY, "a1z9Z",
+ AnvilUseType.MERGE, "b2y8Y",
+ AnvilUseType.UNIT_REPAIR, "c3x7X",
+ AnvilUseType.CUSTOM_CRAFT, "d4w6W"
+ );
+
+ @Override
+ protected Pattern getGuiPattern() {
+ return new Pattern( // Yeah that a mess
+ "00a1z9Z00",
+ "00b2y8Y00",
+ "00c3x7X00",
+ "B004w600S"
+ );
+ }
+
+ public void updateGuiForType(AnvilUseType type) {
+ PatternPane pane = getPane();
+
+ String typeVals = useTypes.get(type);
+
+ char increment = typeVals.charAt(0);
+ char additive = typeVals.charAt(1);
+ char display = typeVals.charAt(2);
+ char exclusiveIncrement = typeVals.charAt(3);
+ char exclusiveAdditive = typeVals.charAt(4);
+
+ WorkPenaltyType.WorkPenaltyPart part = items.get(type);
+ String increasingStr = (part.penaltyIncrease() ? "§a" : "§c") + "Increasing";
+ String additiveStr = (part.penaltyAdditive() ? "§a" : "§c") + "Additive";
+ String exclusiveIncreasingStr = (part.exclusivePenaltyIncrease() ? "§a" : "§c") + "Increasing";
+ String exclusiveAdditiveStr = (part.exclusivePenaltyAdditive() ? "§a" : "§c") + "Additive";
+
+ // Display item
+ ItemStack displayItem = new ItemStack(type.getDisplayMat());
+
+ ArrayList displayLore = new ArrayList<>();
+ displayLore.add("§eShared§7: " + additiveStr + " §7| " + increasingStr);
+ displayLore.add("§eExclusive§7: " + exclusiveAdditiveStr + " §7| " + exclusiveIncreasingStr);
+
+ ItemMeta meta = displayItem.getItemMeta();
+ meta.setDisplayName("§e" + type.getDisplayName());
+ meta.setLore(displayLore);
+ displayItem.setItemMeta(meta);
+
+ pane.bindItem(display, new GuiItem(displayItem, (event) -> {
+ event.setCancelled(true);
+ }));
+
+ // Can probably put this in a function but this works so
+ // "Increment" item
+ ItemStack incrementItem = new ItemStack(part.penaltyIncrease() ? Material.GREEN_TERRACOTTA : Material.RED_TERRACOTTA);
+
+ meta = incrementItem.getItemMeta();
+ meta.setDisplayName(increasingStr);
+ meta.setLore(List.of(INCREASING_EXPLANATION));
+ meta.setLore(List.of(SHARED_EXPLANATION));
+ incrementItem.setItemMeta(meta);
+
+ pane.bindItem(increment, new GuiItem(incrementItem, (event) -> {
+ event.setCancelled(true);
+
+ WorkPenaltyType.WorkPenaltyPart newPart = new WorkPenaltyType.WorkPenaltyPart(
+ !part.penaltyIncrease(), part.penaltyAdditive(),
+ part.exclusivePenaltyIncrease(), part.exclusivePenaltyAdditive());
+ items.replace(type, newPart);
+ updateGuiForType(type);
+ update();
+ }));
+
+ // "Additive" item
+ ItemStack additiveItem = new ItemStack(part.penaltyAdditive() ? Material.GREEN_TERRACOTTA : Material.RED_TERRACOTTA);
+
+ meta = additiveItem.getItemMeta();
+ meta.setDisplayName(additiveStr);
+ meta.setLore(List.of(ADDING_EXPLANATION));
+ meta.setLore(List.of(SHARED_EXPLANATION));
+ additiveItem.setItemMeta(meta);
+
+ pane.bindItem(additive, new GuiItem(additiveItem, (event) -> {
+ event.setCancelled(true);
+
+ WorkPenaltyType.WorkPenaltyPart newPart = new WorkPenaltyType.WorkPenaltyPart(
+ part.penaltyIncrease(), !part.penaltyAdditive(),
+ part.exclusivePenaltyIncrease(), part.exclusivePenaltyAdditive());
+ items.replace(type, newPart);
+ updateGuiForType(type);
+ update();
+ }));
+
+ // exclusive "Increment" item
+ ItemStack exclusiveIncrementItem = new ItemStack(part.exclusivePenaltyIncrease() ? Material.GREEN_TERRACOTTA : Material.RED_TERRACOTTA);
+
+ meta = exclusiveIncrementItem.getItemMeta();
+ meta.setDisplayName(exclusiveIncreasingStr);
+ meta.setLore(List.of(INCREASING_EXPLANATION));
+ meta.setLore(List.of(EXCLUSIVE_EXPLANATION));
+ exclusiveIncrementItem.setItemMeta(meta);
+
+ pane.bindItem(exclusiveIncrement, new GuiItem(exclusiveIncrementItem, (event) -> {
+ event.setCancelled(true);
+
+ WorkPenaltyType.WorkPenaltyPart newPart = new WorkPenaltyType.WorkPenaltyPart(
+ part.penaltyIncrease(), part.penaltyAdditive(),
+ !part.exclusivePenaltyIncrease(), part.exclusivePenaltyAdditive());
+ items.replace(type, newPart);
+ updateGuiForType(type);
+ update();
+ }));
+
+ // exclusive "Additive" item
+ ItemStack exclusiveAdditiveItem = new ItemStack(part.exclusivePenaltyAdditive() ? Material.GREEN_TERRACOTTA : Material.RED_TERRACOTTA);
+
+ meta = exclusiveAdditiveItem.getItemMeta();
+ meta.setDisplayName(exclusiveAdditiveStr);
+ meta.setLore(List.of(ADDING_EXPLANATION));
+ meta.setLore(List.of(EXCLUSIVE_EXPLANATION));
+ exclusiveAdditiveItem.setItemMeta(meta);
+
+ pane.bindItem(exclusiveAdditive, new GuiItem(exclusiveAdditiveItem, (event) -> {
+ event.setCancelled(true);
+
+ WorkPenaltyType.WorkPenaltyPart newPart = new WorkPenaltyType.WorkPenaltyPart(
+ part.penaltyIncrease(), part.penaltyAdditive(),
+ part.exclusivePenaltyIncrease(), !part.exclusivePenaltyAdditive());
+ items.replace(type, newPart);
+ updateGuiForType(type);
+ update();
+ }));
+ }
+
+ @Override
+ public boolean onSave() {
+ return saveWorkPenalty(items);
+ }
+
+ public static boolean saveWorkPenalty(Map partEnum) {
+ ConfigHolder configHolder = ConfigHolder.DEFAULT_CONFIG;
+ FileConfiguration config = configHolder.getConfig();
+
+ partEnum.forEach((key, value) -> {
+ String partPath = key.getPath();
+
+ if (key.getDefaultPenalty().equals(value)) {
+ config.set(partPath, null);
+ return;
+ }
+
+ config.set(partPath + '.' + ConfigOptions.WORK_PENALTY_INCREASE, value.penaltyIncrease());
+ config.set(partPath + '.' + ConfigOptions.WORK_PENALTY_ADDITIVE, value.penaltyAdditive());
+ config.set(partPath + '.' + ConfigOptions.EXCLUSIVE_WORK_PENALTY_INCREASE, value.exclusivePenaltyIncrease());
+ config.set(partPath + '.' + ConfigOptions.EXCLUSIVE_WORK_PENALTY_ADDITIVE, value.exclusivePenaltyAdditive());
+ });
+
+ return configHolder.saveToDisk(true);
+ }
+
+ @Override
+ public boolean hadChange() {
+ for (AnvilUseType type : items.keySet()) {
+ if (!currentType.getPenaltyInfo(type).equals(items.get(type))) {
+ return true;
+ }
+ }
+
+ return false;
+ }
+}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalActions.java b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalActions.java
index d18d170..694fa14 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalActions.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalActions.java
@@ -133,7 +133,7 @@ public class GuiGlobalActions {
// Save setting
if (!setting.onSave()) {
- player.sendMessage("\u00A7cSomething went wrong while saving the change of value.");
+ player.sendMessage("§cSomething went wrong while saving the change of value.");
}
// Update gui for those who have it open.
goal.updateGuiValues();
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalItems.java b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalItems.java
index 2eb42e3..79f1462 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalItems.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalItems.java
@@ -30,7 +30,7 @@ public class GuiGlobalItems {
ItemMeta meta = BACK_ITEM.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7cBack");
+ meta.setDisplayName("§cBack");
BACK_ITEM.setItemMeta(meta);
}
@@ -82,7 +82,7 @@ public class GuiGlobalItems {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7c");
+ meta.setDisplayName("§c");
item.setItemMeta(meta);
return new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
}
@@ -139,7 +139,7 @@ public class GuiGlobalItems {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7aSave");
+ meta.setDisplayName("§aSave");
item.setItemMeta(meta);
return new GuiItem(item,
GuiGlobalActions.saveSettingAction(setting, goal),
@@ -154,7 +154,7 @@ public class GuiGlobalItems {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A77No change. can't save.");
+ meta.setDisplayName("§7No change. can't save.");
item.setItemMeta(meta);
NO_CHANGE_ITEM = new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
}
@@ -185,7 +185,7 @@ public class GuiGlobalItems {
}
// Prefix of the one line lore that will be added to setting's item.
- public static final String SETTING_ITEM_LORE_PREFIX = "\u00A77value: ";
+ public static final String SETTING_ITEM_LORE_PREFIX = "§7value: ";
/**
* Create an arbitrary GuiItem from a unique setting and item's property.
@@ -247,8 +247,8 @@ public class GuiGlobalItems {
ItemMeta meta = item.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7eTemporary close this menu");
- meta.setLore(Collections.singletonList("\u00A77Allow you to chose other item then return here."));
+ meta.setDisplayName("§eTemporary close this menu");
+ meta.setLore(Collections.singletonList("§7Allow you to chose other item then return here."));
item.setItemMeta(meta);
return new GuiItem(item, event -> {
@@ -263,7 +263,7 @@ public class GuiGlobalItems {
});
- player.sendMessage("\u00A7eWrite something in chat to return to the item config menu.");
+ player.sendMessage("§eWrite something in chat to return to the item config menu.");
player.closeInventory();
}, CustomAnvil.instance);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiSharedConstant.java b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiSharedConstant.java
index bcab7fa..11f3657 100644
--- a/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiSharedConstant.java
+++ b/src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiSharedConstant.java
@@ -7,25 +7,21 @@ import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
import org.bukkit.Material;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
-import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
-import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry;
import xyz.alexcrea.cuanvil.gui.config.MainConfigGui;
-import java.util.*;
+import java.util.Arrays;
+import java.util.Collections;
public class GuiSharedConstant {
- public static final List SORTED_ENCHANTMENT_LIST;
-
- static {
- SORTED_ENCHANTMENT_LIST = new ArrayList<>(CAEnchantmentRegistry.getInstance().values());
- SORTED_ENCHANTMENT_LIST.sort(Comparator.comparing(ench -> ench.getKey().getKey()));
- }
+ private GuiSharedConstant(){}
public static final Material SECONDARY_BACKGROUND_MATERIAL = Material.BLACK_STAINED_GLASS_PANE;
public static final GuiItem SECONDARY_BACKGROUND_ITEM = GuiGlobalItems.backgroundItem(GuiSharedConstant.SECONDARY_BACKGROUND_MATERIAL);
+ public static final String UPPER_FILLER_FULL_PLANE = "111111111";
public static final String EMPTY_GUI_FULL_LINE = "000000000";
+ public static final String EMPTY_FILLER_FULL_LINE = "100000001";
// Temporary values, until I get something better.
public static final boolean TEMPORARY_DO_SAVE_TO_DISK_EVERY_CHANGE = true;
@@ -35,11 +31,11 @@ public class GuiSharedConstant {
static {
Pattern pattern = new Pattern(
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
- GuiSharedConstant.EMPTY_GUI_FULL_LINE,
+ GuiSharedConstant.UPPER_FILLER_FULL_PLANE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
+ GuiSharedConstant.EMPTY_FILLER_FULL_LINE,
"B11111111"
);
BACK_TO_MAIN_MENU_BIG_LIST_DISPLAY_BACKGROUND_PANE = new PatternPane(0, 0, 9, 6, Pane.Priority.LOW, pattern);
@@ -60,25 +56,25 @@ public class GuiSharedConstant {
ItemMeta meta = CANCEL_ITEM.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7cCancel");
- meta.setLore(Collections.singletonList("\u00A77Cancel current action and return to previous menu."));
+ meta.setDisplayName("§cCancel");
+ meta.setLore(Collections.singletonList("§7Cancel current action and return to previous menu."));
CANCEL_ITEM.setItemMeta(meta);
CONFIRM_ITEM = new ItemStack(Material.GREEN_TERRACOTTA);
meta = CONFIRM_ITEM.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7aConfirm");
- meta.setLore(Collections.singletonList("\u00A77Confirm current action."));
+ meta.setDisplayName("§aConfirm");
+ meta.setLore(Collections.singletonList("§7Confirm current action."));
CONFIRM_ITEM.setItemMeta(meta);
CONFIRM_PERMANENT_ITEM = new ItemStack(Material.GREEN_TERRACOTTA);
meta = CONFIRM_PERMANENT_ITEM.getItemMeta();
assert meta != null;
- meta.setDisplayName("\u00A7aConfirm");
- meta.setLore(Arrays.asList("\u00A77Confirm current action.",
- "\u00A74Cation: This action can't be canceled."));
+ meta.setDisplayName("§aConfirm");
+ meta.setLore(Arrays.asList("§7Confirm current action.",
+ "§4Cation: This action can't be canceled."));
CONFIRM_PERMANENT_ITEM.setItemMeta(meta);
}
diff --git a/src/main/java/xyz/alexcrea/cuanvil/update/ModrinthUpdateChecker.java b/src/main/java/xyz/alexcrea/cuanvil/update/ModrinthUpdateChecker.java
new file mode 100644
index 0000000..489c636
--- /dev/null
+++ b/src/main/java/xyz/alexcrea/cuanvil/update/ModrinthUpdateChecker.java
@@ -0,0 +1,214 @@
+/*
+ * MIT License
+ *
+ * Copyright (c) 2025 Clickism
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
+ * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+package xyz.alexcrea.cuanvil.update;
+
+import com.google.gson.*;
+import org.jetbrains.annotations.Nullable;
+
+import java.net.URI;
+import java.net.http.HttpClient;
+import java.net.http.HttpRequest;
+import java.net.http.HttpResponse;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.function.Consumer;
+import java.util.function.Function;
+
+/**
+ * Utility class to check for newer versions of a project hosted on Modrinth.
+ */
+public class ModrinthUpdateChecker {
+
+ private static final String API_URL = "https://api.modrinth.com/v2/project/{id}/version";
+
+ private final String projectId;
+ private final String loader;
+ @Nullable
+ private final String minecraftVersion;
+
+ @Nullable
+ private Boolean featured = null;
+
+ @Nullable
+ public Consumer onError = null;
+ @Nullable
+ public Function getRawVersion = ModrinthUpdateChecker::getRawVersion;
+
+ /**
+ * Create a new update checker for the given project.
+ * This will check the latest version for the given loader and any minecraft version.
+ *
+ * @param projectId the project ID
+ * @param loader the loader
+ */
+ public ModrinthUpdateChecker(String projectId, String loader) {
+ this(projectId, loader, null);
+ }
+
+ /**
+ * Create a new update checker for the given project.
+ * This will check the latest version for the given loader and minecraft version.
+ *
+ * @param projectId the project ID
+ * @param loader the loader
+ * @param minecraftVersion the minecraft version, or null for any version
+ */
+ public ModrinthUpdateChecker(String projectId, String loader, @Nullable String minecraftVersion) {
+ this.projectId = projectId;
+ this.loader = loader;
+ this.minecraftVersion = minecraftVersion;
+ }
+
+ /**
+ * Check the latest version of the project for the given loader and minecraft version
+ * and call the consumer with it.
+ *
+ * @param consumer the consumer
+ */
+ public void checkVersion(Consumer