From 4e5fe81af9fac69a7bbd2548052c14aeab9e03f8 Mon Sep 17 00:00:00 2001
From: KingSimon <19822231+KingOfSquares@users.noreply.github.com>
Date: Sat, 11 Jun 2022 18:34:49 +0200
Subject: [PATCH 1/3] promote decorationIfAbsent to StyleSetter and public API

---
 .../text/AbstractComponentBuilder.java        | 11 +++++++++
 .../net/kyori/adventure/text/Component.java   | 20 ++++++++++++++++
 .../adventure/text/ComponentBuilder.java      | 13 ++++++++++
 .../kyori/adventure/text/format/Style.java    | 24 +++++++++++++++++++
 .../adventure/text/format/StyleImpl.java      | 16 +++++++++++--
 .../adventure/text/format/StyleSetter.java    | 10 ++++++++
 6 files changed, 92 insertions(+), 2 deletions(-)

diff --git a/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java b/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
index 93f78a058..c73f48062 100644
--- a/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
+++ b/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
@@ -243,6 +243,17 @@ private void prepareChildren() {
     return (B) this;
   }
 
+  @Override
+  @SuppressWarnings("unchecked")
+  public @NotNull B decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state) {
+    requireNonNull(state, "state");
+    final TextDecoration.@NotNull State thisState = this.buildStyle().decoration(decoration);
+    if (thisState == TextDecoration.State.NOT_SET) {
+      this.styleBuilder().decoration(decoration, state);
+    }
+    return (B) this;
+  }
+
   @Override
   @SuppressWarnings("unchecked")
   public @NotNull B clickEvent(final @Nullable ClickEvent event) {
diff --git a/api/src/main/java/net/kyori/adventure/text/Component.java b/api/src/main/java/net/kyori/adventure/text/Component.java
index 1f8419863..9761cccc8 100644
--- a/api/src/main/java/net/kyori/adventure/text/Component.java
+++ b/api/src/main/java/net/kyori/adventure/text/Component.java
@@ -2014,6 +2014,26 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
     return this.style(this.style().decorations(decorations));
   }
 
+  /**
+   * Sets the state of a decoration on this component to {@code state} if the current state of
+   * the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return a component
+   * @since 4.12.0
+   */
+  @Override
+  default @NotNull Component decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state) {
+    requireNonNull(state, "state");
+    //Not delegating this method prevents object creation if decoration is absent
+    final TextDecoration.@NotNull State thisState = this.decoration(decoration);
+    if (thisState == TextDecoration.State.NOT_SET) {
+      return this.style(this.style().decoration(decoration, state));
+    }
+    return this;
+  }
+
   /**
    * Gets the click event of this component.
    *
diff --git a/api/src/main/java/net/kyori/adventure/text/ComponentBuilder.java b/api/src/main/java/net/kyori/adventure/text/ComponentBuilder.java
index c32396098..04047e784 100644
--- a/api/src/main/java/net/kyori/adventure/text/ComponentBuilder.java
+++ b/api/src/main/java/net/kyori/adventure/text/ComponentBuilder.java
@@ -308,6 +308,19 @@ public interface ComponentBuilder<C extends BuildableComponent<C, B>, B extends
   @Override
   @NotNull B decoration(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
 
+  /**
+   * Sets the state of a decoration on this component to {@code state} if the current state of
+   * the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return this builder
+   * @since 4.12.0
+   */
+  @Contract("_, _ -> this")
+  @Override
+  @NotNull B decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state);
+
   /**
    * Sets the click event of this component.
    *
diff --git a/api/src/main/java/net/kyori/adventure/text/format/Style.java b/api/src/main/java/net/kyori/adventure/text/format/Style.java
index 34163f6b9..024d04bd5 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/Style.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/Style.java
@@ -364,6 +364,18 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
   @Override
   @NotNull Style decorations(final @NotNull Map<TextDecoration, TextDecoration.State> decorations);
 
+  /**
+   * Sets the state of a decoration on this style to {@code state} if the current state of
+   * the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return a style
+   * @since 4.12.0
+   */
+  @Override
+  @NotNull Style decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state);
+
   /**
    * Gets the click event.
    *
@@ -761,6 +773,18 @@ interface Builder extends AbstractBuilder<Style>, Buildable.Builder<Style>, Muta
     @Contract("_, _ -> this")
     @NotNull Builder decoration(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
 
+    /**
+     * Sets the state of a decoration on this style to {@code state} if the current state of the decoration is {@link TextDecoration.State#NOT_SET}.
+     *
+     * @param decoration the decoration
+     * @param state the state
+     * @return this builder
+     * @since 4.12.0
+     */
+    @Override
+    @Contract("_, _ -> this")
+    @NotNull Builder decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state);
+
     /**
      * Sets the click event.
      *
diff --git a/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java b/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
index 6fc5c4041..b4e617638 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
@@ -122,6 +122,19 @@ final class StyleImpl implements Style {
     return new StyleImpl(this.font, this.color, DecorationMap.merge(decorations, this.decorations), this.clickEvent, this.hoverEvent, this.insertion);
   }
 
+  @Override
+  public @NotNull Style decorationIfAbsent(final @NotNull TextDecoration decoration, final @NotNull TextDecoration.State state) {
+    requireNonNull(state, "state");
+    final TextDecoration.@Nullable State thisState = this.decorations.get(decoration);
+    if (thisState == TextDecoration.State.NOT_SET) {
+      return new StyleImpl(this.font, this.color, this.decorations.with(decoration, state), this.clickEvent, this.hoverEvent, this.insertion);
+    }
+    if (thisState != null) {
+      return this;
+    }
+    throw new IllegalArgumentException(String.format("unknown decoration '%s'", decoration));
+  }
+
   @Override
   public @Nullable ClickEvent clickEvent() {
     return this.clickEvent;
@@ -280,8 +293,7 @@ static final class BuilderImpl implements Builder {
       return this;
     }
 
-    // todo(kashike): promote to public api?
-    @NotNull Builder decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state) {
+    public @NotNull Builder decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state) {
       requireNonNull(state, "state");
       final TextDecoration.@Nullable State thisState = this.decorations.get(decoration);
       if (thisState == TextDecoration.State.NOT_SET) {
diff --git a/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java b/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
index 85e4d66a4..7dfd1a522 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
@@ -149,6 +149,16 @@ public interface StyleSetter<T extends StyleSetter<?>> {
     return this.decorations(decorations.stream().collect(Collectors.toMap(Function.identity(), decoration -> TextDecoration.State.byBoolean(flag))));
   }
 
+  /**
+   * Sets the state of a decoration to {@code state} if the current state of the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return an object ({@code T})
+   * @since 4.12.0
+   */
+  @NotNull T decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
+
   /**
    * Sets the click event.
    *

From 50195bbd363262dde6e687f4f6fcbe588ad5374c Mon Sep 17 00:00:00 2001
From: KingSimon <19822231+KingOfSquares@users.noreply.github.com>
Date: Sat, 11 Jun 2022 22:37:27 +0200
Subject: [PATCH 2/3] Add tests for decorationIfAbsent

---
 .../net/kyori/adventure/text/Component.java   |  2 +-
 .../text/ComponentDecorationTest.java         | 53 +++++++++++++++++++
 .../adventure/text/format/StyleTest.java      | 17 ++++++
 3 files changed, 71 insertions(+), 1 deletion(-)
 create mode 100644 api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java

diff --git a/api/src/main/java/net/kyori/adventure/text/Component.java b/api/src/main/java/net/kyori/adventure/text/Component.java
index 9761cccc8..a534d400d 100644
--- a/api/src/main/java/net/kyori/adventure/text/Component.java
+++ b/api/src/main/java/net/kyori/adventure/text/Component.java
@@ -2026,7 +2026,7 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
   @Override
   default @NotNull Component decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state) {
     requireNonNull(state, "state");
-    //Not delegating this method prevents object creation if decoration is absent
+    //Not delegating this method prevents object creation if decoration is NOT absent
     final TextDecoration.@NotNull State thisState = this.decoration(decoration);
     if (thisState == TextDecoration.State.NOT_SET) {
       return this.style(this.style().decoration(decoration, state));
diff --git a/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java b/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java
new file mode 100644
index 000000000..94136822a
--- /dev/null
+++ b/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java
@@ -0,0 +1,53 @@
+/*
+ * This file is part of adventure, licensed under the MIT License.
+ *
+ * Copyright (c) 2017-2022 KyoriPowered
+ *
+ * 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 net.kyori.adventure.text;
+
+import com.google.common.collect.ImmutableSet;
+import net.kyori.adventure.text.format.Style;
+import net.kyori.adventure.text.format.TextDecoration;
+import org.junit.jupiter.api.Test;
+
+import static net.kyori.adventure.text.TextAssertions.assertDecorations;
+
+public class ComponentDecorationTest {
+
+  @Test
+  void testDecorationIfAbsent() {
+    final Style s0 = Style.style(TextDecoration.BOLD);
+    final Component c0 = Component.text("tuba time", s0)
+      .decorationIfAbsent(TextDecoration.BOLD, TextDecoration.State.FALSE)
+      .decorationIfAbsent(TextDecoration.ITALIC, TextDecoration.State.FALSE);
+    assertDecorations(c0.style(), ImmutableSet.of(TextDecoration.BOLD), ImmutableSet.of(TextDecoration.ITALIC));
+  }
+
+  @Test
+  void testDecorationIfAbsentWithBuilder() {
+    final Style s0 = Style.style(TextDecoration.BOLD);
+    final Component c0 = Component.text().style(s0).content("tuba time")
+      .decorationIfAbsent(TextDecoration.BOLD, TextDecoration.State.FALSE)
+      .decorationIfAbsent(TextDecoration.ITALIC, TextDecoration.State.FALSE)
+      .build();
+    assertDecorations(c0.style(), ImmutableSet.of(TextDecoration.BOLD), ImmutableSet.of(TextDecoration.ITALIC));
+  }
+}
diff --git a/api/src/test/java/net/kyori/adventure/text/format/StyleTest.java b/api/src/test/java/net/kyori/adventure/text/format/StyleTest.java
index 8213f0b1c..8df6d2b5e 100644
--- a/api/src/test/java/net/kyori/adventure/text/format/StyleTest.java
+++ b/api/src/test/java/net/kyori/adventure/text/format/StyleTest.java
@@ -154,6 +154,23 @@ void testOf_decorations() {
     assertDecorations(s0, ImmutableSet.of(TextDecoration.BOLD, TextDecoration.ITALIC), ImmutableSet.of());
   }
 
+  @Test
+  void testDecorationIfAbsent() {
+    final Style s0 = Style.style(TextDecoration.BOLD)
+      .decorationIfAbsent(TextDecoration.BOLD, TextDecoration.State.FALSE)
+      .decorationIfAbsent(TextDecoration.ITALIC, TextDecoration.State.FALSE);
+    assertDecorations(s0, ImmutableSet.of(TextDecoration.BOLD), ImmutableSet.of(TextDecoration.ITALIC));
+  }
+
+  @Test
+  void testDecorationIfAbsentWithBuilder() {
+    final Style s0 = Style.style().decoration(TextDecoration.BOLD, TextDecoration.State.TRUE)
+      .decorationIfAbsent(TextDecoration.BOLD, TextDecoration.State.FALSE)
+      .decorationIfAbsent(TextDecoration.ITALIC, TextDecoration.State.FALSE)
+      .build();
+    assertDecorations(s0, ImmutableSet.of(TextDecoration.BOLD), ImmutableSet.of(TextDecoration.ITALIC));
+  }
+
   @Test
   void testOf_colorAndDecorations() {
     final Style s0 = Style.style(NamedTextColor.GREEN, TextDecoration.BOLD, TextDecoration.ITALIC);

From 20c55ef5d4c9115de7d8229d2d5d4f97ef188c01 Mon Sep 17 00:00:00 2001
From: KingSimon <19822231+KingOfSquares@users.noreply.github.com>
Date: Mon, 20 Jun 2022 21:02:48 +0200
Subject: [PATCH 3/3] Cleanup decorationIfAbsent code after review

---
 .../text/AbstractComponentBuilder.java        |  6 +--
 .../net/kyori/adventure/text/Component.java   | 40 +++++++++----------
 .../kyori/adventure/text/format/Style.java    | 24 +++++------
 .../adventure/text/format/StyleImpl.java      | 33 +++++++--------
 .../adventure/text/format/StyleSetter.java    | 20 +++++-----
 .../text/ComponentDecorationTest.java         |  2 +-
 6 files changed, 61 insertions(+), 64 deletions(-)

diff --git a/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java b/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
index c73f48062..a6add27c0 100644
--- a/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
+++ b/api/src/main/java/net/kyori/adventure/text/AbstractComponentBuilder.java
@@ -246,11 +246,7 @@ private void prepareChildren() {
   @Override
   @SuppressWarnings("unchecked")
   public @NotNull B decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state) {
-    requireNonNull(state, "state");
-    final TextDecoration.@NotNull State thisState = this.buildStyle().decoration(decoration);
-    if (thisState == TextDecoration.State.NOT_SET) {
-      this.styleBuilder().decoration(decoration, state);
-    }
+    this.styleBuilder().decorationIfAbsent(decoration, state);
     return (B) this;
   }
 
diff --git a/api/src/main/java/net/kyori/adventure/text/Component.java b/api/src/main/java/net/kyori/adventure/text/Component.java
index a534d400d..4efca8834 100644
--- a/api/src/main/java/net/kyori/adventure/text/Component.java
+++ b/api/src/main/java/net/kyori/adventure/text/Component.java
@@ -1988,6 +1988,26 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
     return this.style(this.style().decoration(decoration, state));
   }
 
+  /**
+   * Sets the state of a decoration on this component to {@code state} if the current state of
+   * the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return a component
+   * @since 4.12.0
+   */
+  @Override
+  default @NotNull Component decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state) {
+    requireNonNull(state, "state");
+    // Not delegating this method prevents object creation if decoration is NOT absent
+    final TextDecoration.@NotNull State oldState = this.decoration(decoration);
+    if (oldState == TextDecoration.State.NOT_SET) {
+      return this.style(this.style().decoration(decoration, state));
+    }
+    return this;
+  }
+
   /**
    * Gets a set of decorations this component has.
    *
@@ -2014,26 +2034,6 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
     return this.style(this.style().decorations(decorations));
   }
 
-  /**
-   * Sets the state of a decoration on this component to {@code state} if the current state of
-   * the decoration is {@link TextDecoration.State#NOT_SET}.
-   *
-   * @param decoration the decoration
-   * @param state the state
-   * @return a component
-   * @since 4.12.0
-   */
-  @Override
-  default @NotNull Component decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state) {
-    requireNonNull(state, "state");
-    //Not delegating this method prevents object creation if decoration is NOT absent
-    final TextDecoration.@NotNull State thisState = this.decoration(decoration);
-    if (thisState == TextDecoration.State.NOT_SET) {
-      return this.style(this.style().decoration(decoration, state));
-    }
-    return this;
-  }
-
   /**
    * Gets the click event of this component.
    *
diff --git a/api/src/main/java/net/kyori/adventure/text/format/Style.java b/api/src/main/java/net/kyori/adventure/text/format/Style.java
index 024d04bd5..55015acc4 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/Style.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/Style.java
@@ -341,6 +341,18 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
   @Override
   @NotNull Style decoration(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
 
+  /**
+   * Sets the state of a decoration on this style to {@code state} if the current state of
+   * the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return a style
+   * @since 4.12.0
+   */
+  @Override
+  @NotNull Style decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state);
+
   /**
    * Gets a map of decorations this style has.
    *
@@ -364,18 +376,6 @@ default boolean hasDecoration(final @NotNull TextDecoration decoration) {
   @Override
   @NotNull Style decorations(final @NotNull Map<TextDecoration, TextDecoration.State> decorations);
 
-  /**
-   * Sets the state of a decoration on this style to {@code state} if the current state of
-   * the decoration is {@link TextDecoration.State#NOT_SET}.
-   *
-   * @param decoration the decoration
-   * @param state the state
-   * @return a style
-   * @since 4.12.0
-   */
-  @Override
-  @NotNull Style decorationIfAbsent(final @NotNull TextDecoration decoration, @NotNull final TextDecoration.State state);
-
   /**
    * Gets the click event.
    *
diff --git a/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java b/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
index b4e617638..8be858c1e 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/StyleImpl.java
@@ -112,29 +112,29 @@ final class StyleImpl implements Style {
     return new StyleImpl(this.font, this.color, this.decorations.with(decoration, state), this.clickEvent, this.hoverEvent, this.insertion);
   }
 
-  @Override
-  public @NotNull Map<TextDecoration, TextDecoration.State> decorations() {
-    return this.decorations;
-  }
-
-  @Override
-  public @NotNull Style decorations(final @NotNull Map<TextDecoration, TextDecoration.State> decorations) {
-    return new StyleImpl(this.font, this.color, DecorationMap.merge(decorations, this.decorations), this.clickEvent, this.hoverEvent, this.insertion);
-  }
-
   @Override
   public @NotNull Style decorationIfAbsent(final @NotNull TextDecoration decoration, final @NotNull TextDecoration.State state) {
     requireNonNull(state, "state");
-    final TextDecoration.@Nullable State thisState = this.decorations.get(decoration);
-    if (thisState == TextDecoration.State.NOT_SET) {
+    final TextDecoration.@Nullable State oldState = this.decorations.get(decoration);
+    if (oldState == TextDecoration.State.NOT_SET) {
       return new StyleImpl(this.font, this.color, this.decorations.with(decoration, state), this.clickEvent, this.hoverEvent, this.insertion);
     }
-    if (thisState != null) {
+    if (oldState != null) {
       return this;
     }
     throw new IllegalArgumentException(String.format("unknown decoration '%s'", decoration));
   }
 
+  @Override
+  public @NotNull Map<TextDecoration, TextDecoration.State> decorations() {
+    return this.decorations;
+  }
+
+  @Override
+  public @NotNull Style decorations(final @NotNull Map<TextDecoration, TextDecoration.State> decorations) {
+    return new StyleImpl(this.font, this.color, DecorationMap.merge(decorations, this.decorations), this.clickEvent, this.hoverEvent, this.insertion);
+  }
+
   @Override
   public @Nullable ClickEvent clickEvent() {
     return this.clickEvent;
@@ -293,13 +293,14 @@ static final class BuilderImpl implements Builder {
       return this;
     }
 
+    @Override
     public @NotNull Builder decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state) {
       requireNonNull(state, "state");
-      final TextDecoration.@Nullable State thisState = this.decorations.get(decoration);
-      if (thisState == TextDecoration.State.NOT_SET) {
+      final TextDecoration.@Nullable State oldState = this.decorations.get(decoration);
+      if (oldState == TextDecoration.State.NOT_SET) {
         this.decorations.put(decoration, state);
       }
-      if (thisState != null) {
+      if (oldState != null) {
         return this;
       }
       throw new IllegalArgumentException(String.format("unknown decoration '%s'", decoration));
diff --git a/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java b/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
index 7dfd1a522..a0779caa6 100644
--- a/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
+++ b/api/src/main/java/net/kyori/adventure/text/format/StyleSetter.java
@@ -125,6 +125,16 @@ public interface StyleSetter<T extends StyleSetter<?>> {
    */
   @NotNull T decoration(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
 
+  /**
+   * Sets the state of a decoration to {@code state} if the current state of the decoration is {@link TextDecoration.State#NOT_SET}.
+   *
+   * @param decoration the decoration
+   * @param state the state
+   * @return an object ({@code T})
+   * @since 4.12.0
+   */
+  @NotNull T decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
+
   /**
    * Sets decorations using the specified {@code decorations} map.
    *
@@ -149,16 +159,6 @@ public interface StyleSetter<T extends StyleSetter<?>> {
     return this.decorations(decorations.stream().collect(Collectors.toMap(Function.identity(), decoration -> TextDecoration.State.byBoolean(flag))));
   }
 
-  /**
-   * Sets the state of a decoration to {@code state} if the current state of the decoration is {@link TextDecoration.State#NOT_SET}.
-   *
-   * @param decoration the decoration
-   * @param state the state
-   * @return an object ({@code T})
-   * @since 4.12.0
-   */
-  @NotNull T decorationIfAbsent(final @NotNull TextDecoration decoration, final TextDecoration.@NotNull State state);
-
   /**
    * Sets the click event.
    *
diff --git a/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java b/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java
index 94136822a..ac04f0835 100644
--- a/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java
+++ b/api/src/test/java/net/kyori/adventure/text/ComponentDecorationTest.java
@@ -30,7 +30,7 @@
 
 import static net.kyori.adventure.text.TextAssertions.assertDecorations;
 
-public class ComponentDecorationTest {
+class ComponentDecorationTest {
 
   @Test
   void testDecorationIfAbsent() {