Fixed #45

Was a simple oversight with a simple fix, thanks for reporting the bug!
CleverNucleus · Jan 27, 2022 · 4bd33ff · 4bd33ff
1 parent 2d7ad09
commit 4bd33ff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -7,7 +7,7 @@ org.gradle.jvmargs=-Xmx1G
 	loader_version=0.12.10
 
 # Mod Properties
-	mod_version = 3.1.1
+	mod_version = 3.1.2
 	maven_group = com.github.clevernucleus
 	archives_base_name = playerex
 

diff --git a/src/main/java/com/github/clevernucleus/playerex/client/gui/AttributesPageLayer.java b/src/main/java/com/github/clevernucleus/playerex/client/gui/AttributesPageLayer.java
@@ -143,15 +143,17 @@ public void drawBackground(MatrixStack matrices, float delta, int mouseX, int mo
 
 			DataAttributesAPI.ifPresent(player, attribute, (Object)null, value -> {
 				if(BUTTON_KEYS.contains(key)) {
+					double max = ((IEntityAttribute)attribute.get()).maxValue();
+
 					if(key.equals(lvl)) {
-						button.active = (value < ((IEntityAttribute)attribute.get()).maxValue()) && (player.experienceLevel >= ExAPI.getConfig().requiredXp(player));
+						button.active = value < max && player.experienceLevel >= ExAPI.getConfig().requiredXp(player);
 					} else {
 						double modifierValue = this.playerData.get(attribute.get());
 
 						if(this.canRefund()) {
 							button.active = modifierValue >= 1.0D;
 						} else {
-							button.active = this.playerData.skillPoints() >= 1;
+							button.active = modifierValue < max && this.playerData.skillPoints() >= 1;
 						}
 
 						button.alt = this.canRefund();