Return silently if the modifier is already present

quentin452 · quentin452 · commit 7184f9d9daff · 2025-10-17T05:16:18.000+02:00
diff --git a/src/main/java/fr/iamacat/optimizationsandtweaks/mixins/common/core/MixinModifiableAttributeInstance.java b/src/main/java/fr/iamacat/optimizationsandtweaks/mixins/common/core/MixinModifiableAttributeInstance.java
@@ -57,7 +57,8 @@ public AttributeModifier getModifier(UUID id) {
     public void applyModifier(AttributeModifier modifier) {
         UUID id = modifier.getID();
         if (this.mapByUUID.containsKey(id)) {
-            throw new IllegalArgumentException("Modifier is already applied on this attribute!");
+            // Return silently if the modifier is already present
+            return; 
         }
 
         Set<AttributeModifier> operationSet = (Set<AttributeModifier>) this.mapByOperation

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,8 @@ public AttributeModifier getModifier(UUID id) {`
`57`	`57`	`public void applyModifier(AttributeModifier modifier) {`
`58`	`58`	`UUID id = modifier.getID();`
`59`	`59`	`if (this.mapByUUID.containsKey(id)) {`
`60`		`- throw new IllegalArgumentException("Modifier is already applied on this attribute!");`
	`60`	`+ // Return silently if the modifier is already present`
	`61`	`+ return;`
`61`	`62`	`}`
`62`	`63`
`63`	`64`	`Set<AttributeModifier> operationSet = (Set<AttributeModifier>) this.mapByOperation`