Fix PlaceholderAPI integration. v1.0.1 Update!

Author: imDMK

doublejump-api/build.gradle.kts

@@ -1,5 +1,5 @@
 group = "com.github.imdmk"
-version = "1.0.0"
+version = "1.0.1"
 
 repositories {
     maven("https://hub.spigotmc.org/nexus/content/repositories/snapshots/")

doublejump-plugin/build.gradle.kts

@@ -5,7 +5,7 @@ plugins {
 }
 
 group = "com.github.imdmk"
-version = "1.0.0"
+version = "1.0.1"
 
 repositories {
     maven("https://hub.spigotmc.org/nexus/content/repositories/snapshots/") // SpigotMC

doublejump-plugin/src/main/java/com/github/imdmk/doublejump/jump/feature/item/JumpItemService.java

@@ -19,7 +19,7 @@
  */
 public class JumpItemService {
 
-    @Inject private JumpConfig jumpConfig;
+    @Inject private JumpConfig config;
 
     /**
      * Checks whether the provided item matches the configured jump item.
@@ -84,8 +84,8 @@ public boolean isJumpItem(@Nullable ItemStack compare) {
      *
      * @return the jump {@link ItemStack}
      */
-    public JumpItem getJumpItem() {
-        return this.jumpConfig.item.item;
+    public @NotNull JumpItem getJumpItem() {
+        return this.config.item.item;
     }
 
     /**
@@ -94,7 +94,7 @@ public JumpItem getJumpItem() {
      * @return {@code true} if enabled, {@code false} otherwise
      */
     public boolean isEnabled() {
-        return this.jumpConfig.item.enabled;
+        return this.config.item.enabled;
     }
 
     /**
@@ -104,6 +104,6 @@ public boolean isEnabled() {
      * @return {@code true} if enabled and matches the given usage mode
      */
     public boolean isEnabled(@NotNull ItemUsage usage) {
-        return this.isEnabled() && this.jumpConfig.item.usageMode.equals(usage);
+        return this.isEnabled() && this.config.item.usageMode.equals(usage);
     }
 }

doublejump-plugin/src/main/java/com/github/imdmk/doublejump/jump/feature/placeholder/JumpAllowedPlaceholder.java

@@ -11,7 +11,7 @@
 
 public class JumpAllowedPlaceholder extends PluginPlaceholder {
 
-    private static final String PLACEHOLDER_IDENTIFIER = "doublejump_allowed";
+    private static final String PLACEHOLDER_IDENTIFIER = "doublejump-allowed";
 
     @Inject private JumpConfig config;
     @Inject private JumpPlayerCache cache;

doublejump-plugin/src/main/java/com/github/imdmk/doublejump/jump/feature/placeholder/JumpCooldownPlaceholder.java

@@ -19,7 +19,7 @@
  */
 public class JumpCooldownPlaceholder extends PluginPlaceholder {
 
-    private static final String PLACEHOLDER_IDENTIFIER = "doublejump_cooldown";
+    private static final String PLACEHOLDER_IDENTIFIER = "doublejump-cooldown";
 
     @Inject private JumpConfig config;
     @Inject private JumpPlayerCache cache;

doublejump-plugin/src/main/java/com/github/imdmk/doublejump/jump/feature/placeholder/JumpPlaceholderConfig.java

@@ -11,6 +11,9 @@ public class JumpPlaceholderConfig extends OkaeriConfig {
     })
     public boolean enabled = true;
 
+    @Comment("# Text to display when the player's doublejump mode is deactivated.")
+    public String jumpDisabledText = "doublejump disabled";
+
     @Comment({
             "# Text to display when the player's cooldown has expired or is not set.",
             "# Example: 'ready', 'now', '✓'"

doublejump-plugin/src/main/java/com/github/imdmk/doublejump/util/DurationUtil.java

@@ -23,6 +23,7 @@ public final class DurationUtil {
             .withUnit("y", ChronoUnit.YEARS);
 
     private static final long MILLIS_PER_TICK = 50L;
+    private static final long ONE_SECOND_IN_MILLIS = 1000L;
 
     private DurationUtil() {
         throw new UnsupportedOperationException("This is a utility class and cannot be instantiated.");
@@ -36,7 +37,7 @@ private DurationUtil() {
      * @return a formatted string or {@code "<1s"}
      */
     public static @NotNull String format(@NotNull Duration duration) {
-        if (!isValid(duration)) {
+        if (duration.toMillis() < ONE_SECOND_IN_MILLIS || !isValid(duration)) {
             return "<1s";
         }