Clean up task type manager

Mark some TaskType methods as final as they aren't really meant to be
overridden

1 files changed, 14 insertions, 6 deletions

diff --git a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/tasktype/BukkitTaskTypeManager.java b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/tasktype/BukkitTaskTypeManager.java
index d87caace..3067bed0 100644
--- a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/tasktype/BukkitTaskTypeManager.java
+++ b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/tasktype/BukkitTaskTypeManager.java
@@ -9,6 +9,7 @@ import org.bukkit.ChatColor;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.NotNull;
 
+import java.util.Objects;
 import java.util.Set;
 import java.util.UUID;
 
@@ -21,22 +22,28 @@ public final class BukkitTaskTypeManager extends TaskTypeManager {
     private final BukkitQuestsPlugin plugin;
 
     /**
-     * Constructs a new BukkitTaskTypeManager.
+     * Constructs a new BukkitTaskTypeManager with exclusions.
      *
      * @param plugin the Bukkit plugin instance
+     * @param exclusions the set of task type exclusions
      */
-    public BukkitTaskTypeManager(final @NotNull BukkitQuestsPlugin plugin) {
+    public BukkitTaskTypeManager(final @NotNull BukkitQuestsPlugin plugin, final @NotNull Set<String> exclusions) {
+        super(exclusions);
+        Objects.requireNonNull(plugin, "plugin cannot be null");
+
         this.plugin = plugin;
     }
 
     /**
-     * Constructs a new BukkitTaskTypeManager with exclusions.
+     * Constructs a new BukkitTaskTypeManager.
      *
      * @param plugin the Bukkit plugin instance
-     * @param exclusions the set of task type exclusions
      */
-    public BukkitTaskTypeManager(final @NotNull BukkitQuestsPlugin plugin, final @NotNull Set<String> exclusions) {
-        super(exclusions);
+    @SuppressWarnings("unused")
+    public BukkitTaskTypeManager(final @NotNull BukkitQuestsPlugin plugin) {
+        super();
+        Objects.requireNonNull(plugin, "plugin cannot be null");
+
         this.plugin = plugin;
     }
 
@@ -71,6 +78,7 @@ public final class BukkitTaskTypeManager extends TaskTypeManager {
      * @param taskId the task ID
      * @param associatedPlayer the UUID of the associated player
      */
+    @Override
     public void sendDebug(final @NotNull String message, final @NotNull String taskType, final @NotNull String questId, final @NotNull String taskId, final @NotNull UUID associatedPlayer) {
         String chatHeader = null;