Allow negative amplifiers in potion effects (#2478)

SpongePowered · Nov 18, 2023 · 3141631 · 3141631
1 parent f578181
commit 3141631
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/main/java/org/spongepowered/api/effect/potion/PotionEffect.java b/src/main/java/org/spongepowered/api/effect/potion/PotionEffect.java
@@ -61,7 +61,7 @@ static Builder builder() {
      * @param amplifier The zero-indexed amplifier
      * @param duration The duration in ticks
      * @return The potion effect
-     * @throws IllegalArgumentException If the amplifier is negative or the duration is not positive
+     * @throws IllegalArgumentException If the duration is not positive
      */
     static PotionEffect of(final PotionEffectType type, final int amplifier, final Ticks duration) throws IllegalArgumentException {
         return PotionEffect.builder().potionType(type).amplifier(amplifier).duration(duration).build();
@@ -76,7 +76,7 @@ static PotionEffect of(final PotionEffectType type, final int amplifier, final T
      * @param amplifier The amplifier
      * @param duration The duration in ticks
      * @return The potion effect
-     * @throws IllegalArgumentException If the amplifier is negative or the duration is not positive
+     * @throws IllegalArgumentException If the duration is not positive
      */
     static PotionEffect of(final Supplier<? extends PotionEffectType> type, final int amplifier, final Ticks duration) throws IllegalArgumentException {
         return PotionEffect.builder().potionType(type).amplifier(amplifier).duration(duration).build();
@@ -172,15 +172,14 @@ default Builder potionType(final Supplier<? extends PotionEffectType> potionEffe
         /**
          * Sets the amplifier power of the potion effect.
          *
-         * <p>Amplifiers must be greater than or equal to zero. See
+         * <p>See
          * {@link #amplifier()} for an explanation of what the amplifier means.
          * </p>
          *
          * @param amplifier The amplifier power
          * @return This builder, for chaining
-         * @throws IllegalArgumentException If the amplifier is less than zero
          */
-        Builder amplifier(int amplifier) throws IllegalArgumentException;
+        Builder amplifier(int amplifier);
 
         /**
          * Sets whether the potion effect is ambient.