Cleanup

26 files changed, 45 insertions, 75 deletions

diff --git a/src/main/java/com/leonardobishop/quests/Quests.java b/src/main/java/com/leonardobishop/quests/Quests.java
index 286ed571..ad5a1798 100644
--- a/src/main/java/com/leonardobishop/quests/Quests.java
+++ b/src/main/java/com/leonardobishop/quests/Quests.java
@@ -49,7 +49,6 @@ public class Quests extends JavaPlugin {
     private PlaceholderExpansion placeholder;
 
     private boolean brokenConfig = false;
-    private BukkitTask questCompleterTask;
     private BukkitTask questAutosaveTask;
 
     public static Quests get() {
@@ -233,10 +232,8 @@ public class Quests extends JavaPlugin {
         questsConfigLoader.loadConfig();
 
         long autocompleteInterval = 12000;
-        long completerPollInterval = 100;
         if (!isBrokenConfig()) {
             autocompleteInterval = this.getConfig().getLong("options.performance-tweaking.quest-autocomplete-interval", 12000);
-            completerPollInterval = this.getConfig().getLong("options.performance-tweaking.quest-completer-poll-interval", 100);
         }
         if (questAutosaveTask != null) {
             try {
diff --git a/src/main/java/com/leonardobishop/quests/QuestsLogger.java b/src/main/java/com/leonardobishop/quests/QuestsLogger.java
index 3d8c6515..895b03dd 100644
--- a/src/main/java/com/leonardobishop/quests/QuestsLogger.java
+++ b/src/main/java/com/leonardobishop/quests/QuestsLogger.java
@@ -2,7 +2,7 @@ package com.leonardobishop.quests;
 
 public class QuestsLogger {
 
-    private Quests plugin;
+    private final Quests plugin;
     private LoggingLevel serverLoggingLevel;
 
     public QuestsLogger(Quests plugin, LoggingLevel serverLoggingLevel) {
diff --git a/src/main/java/com/leonardobishop/quests/blocktype/Block.java b/src/main/java/com/leonardobishop/quests/blocktype/Block.java
index 4fa71515..4b3b7d6a 100644
--- a/src/main/java/com/leonardobishop/quests/blocktype/Block.java
+++ b/src/main/java/com/leonardobishop/quests/blocktype/Block.java
@@ -2,6 +2,7 @@ package com.leonardobishop.quests.blocktype;
 
 import org.bukkit.Material;
 
+@Deprecated
 public class Block {
 
     private Material material;
diff --git a/src/main/java/com/leonardobishop/quests/commands/CommandQuests.java b/src/main/java/com/leonardobishop/quests/commands/CommandQuests.java
index ef824dcc..6c72a4e8 100644
--- a/src/main/java/com/leonardobishop/quests/commands/CommandQuests.java
+++ b/src/main/java/com/leonardobishop/quests/commands/CommandQuests.java
@@ -23,7 +23,6 @@ import org.jetbrains.annotations.Nullable;
 
 import java.io.File;
 import java.io.IOException;
-import java.net.URI;
 import java.nio.file.*;
 import java.nio.file.attribute.BasicFileAttributes;
 import java.util.*;
diff --git a/src/main/java/com/leonardobishop/quests/events/EventInventory.java b/src/main/java/com/leonardobishop/quests/events/EventInventory.java
index 3a5ffb7a..10f76538 100644
--- a/src/main/java/com/leonardobishop/quests/events/EventInventory.java
+++ b/src/main/java/com/leonardobishop/quests/events/EventInventory.java
@@ -9,18 +9,12 @@ import com.leonardobishop.quests.obj.misc.QMenuCancel;
 import com.leonardobishop.quests.obj.misc.QMenuCategory;
 import com.leonardobishop.quests.obj.misc.QMenuQuest;
 import com.leonardobishop.quests.quests.Quest;
-import org.bukkit.ChatColor;
-import org.bukkit.Material;
-import org.bukkit.block.Block;
-import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
-import org.bukkit.event.block.Action;
 import org.bukkit.event.inventory.ClickType;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.event.inventory.InventoryCloseEvent;
 import org.bukkit.event.inventory.InventoryType;
-import org.bukkit.event.player.PlayerInteractEvent;
 
 import java.util.ArrayList;
 import java.util.HashMap;
@@ -28,12 +22,12 @@ import java.util.UUID;
 
 public class EventInventory implements Listener {
 
-    private static HashMap<UUID, QMenu> tracker = new HashMap<>();
-    private Quests plugin;
+    private static final HashMap<UUID, QMenu> tracker = new HashMap<>();
+    private final Quests plugin;
 
     // ADD PLAYERS TO THE BUFFER BEFORE AN ANTICIPATED MENU CHANGE SO THAT
     // THEY ARE NOT LOST FROM THE TRACKER WHEN CHANGING MENUS
-    private ArrayList<UUID> buffer = new ArrayList<>();
+    private final ArrayList<UUID> buffer = new ArrayList<>();
 
     public EventInventory(Quests plugin) {
         this.plugin = plugin;
diff --git a/src/main/java/com/leonardobishop/quests/events/EventPlayerJoin.java b/src/main/java/com/leonardobishop/quests/events/EventPlayerJoin.java
index b9e1723f..52020ef7 100644
--- a/src/main/java/com/leonardobishop/quests/events/EventPlayerJoin.java
+++ b/src/main/java/com/leonardobishop/quests/events/EventPlayerJoin.java
@@ -12,7 +12,7 @@ import java.util.UUID;
 
 public class EventPlayerJoin implements Listener {
 
-    private Quests plugin;
+    private final Quests plugin;
 
     public EventPlayerJoin(Quests plugin) {
         this.plugin = plugin;
diff --git a/src/main/java/com/leonardobishop/quests/events/EventPlayerLeave.java b/src/main/java/com/leonardobishop/quests/events/EventPlayerLeave.java
index 9992bc22..2253cbec 100644
--- a/src/main/java/com/leonardobishop/quests/events/EventPlayerLeave.java
+++ b/src/main/java/com/leonardobishop/quests/events/EventPlayerLeave.java
@@ -1,7 +1,6 @@
 package com.leonardobishop.quests.events;
 
 import com.leonardobishop.quests.Quests;
-import org.bukkit.Bukkit;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.event.player.PlayerQuitEvent;
diff --git a/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter.java b/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter.java
index 3ffb2113..444d63dc 100644
--- a/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter.java
+++ b/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter.java
@@ -2,9 +2,7 @@ package com.leonardobishop.quests.itemgetter;
 
 import com.leonardobishop.quests.Quests;
 import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.inventory.ItemStack;
-import org.bukkit.plugin.java.JavaPlugin;
 
 public interface ItemGetter {
 
diff --git a/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter_Late_1_8.java b/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter_Late_1_8.java
index 9ecf809e..33389165 100644
--- a/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter_Late_1_8.java
+++ b/src/main/java/com/leonardobishop/quests/itemgetter/ItemGetter_Late_1_8.java
@@ -5,12 +5,10 @@ import org.apache.commons.lang.StringUtils;
 import org.bukkit.ChatColor;
 import org.bukkit.Material;
 import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.enchantments.Enchantment;
 import org.bukkit.inventory.ItemFlag;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
-import org.bukkit.plugin.java.JavaPlugin;
 
 import java.util.ArrayList;
 import java.util.Arrays;
diff --git a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCancel.java b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCancel.java
index 9a4a2c4d..703827fd 100644
--- a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCancel.java
+++ b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCancel.java
@@ -1,11 +1,10 @@
 package com.leonardobishop.quests.obj.misc;
 
-import com.leonardobishop.quests.player.QPlayer;
-import com.leonardobishop.quests.quests.Quest;
 import com.leonardobishop.quests.obj.Items;
 import com.leonardobishop.quests.obj.Options;
+import com.leonardobishop.quests.player.QPlayer;
+import com.leonardobishop.quests.quests.Quest;
 import org.bukkit.Bukkit;
-import org.bukkit.Material;
 import org.bukkit.inventory.Inventory;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
diff --git a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCategory.java b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCategory.java
index 3418831f..24d708e2 100644
--- a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCategory.java
+++ b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuCategory.java
@@ -1,9 +1,9 @@
 package com.leonardobishop.quests.obj.misc;
 
-import com.leonardobishop.quests.player.QPlayer;
-import com.leonardobishop.quests.quests.Category;
 import com.leonardobishop.quests.Quests;
 import com.leonardobishop.quests.obj.Options;
+import com.leonardobishop.quests.player.QPlayer;
+import com.leonardobishop.quests.quests.Category;
 import org.bukkit.Bukkit;
 import org.bukkit.Material;
 import org.bukkit.inventory.Inventory;
diff --git a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuDaily.java b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuDaily.java
index 220f288a..5277d195 100644
--- a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuDaily.java
+++ b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuDaily.java
@@ -1,8 +1,8 @@
 package com.leonardobishop.quests.obj.misc;
 
+import com.leonardobishop.quests.obj.Options;
 import com.leonardobishop.quests.player.QPlayer;
 import com.leonardobishop.quests.quests.Quest;
-import com.leonardobishop.quests.obj.Options;
 import org.bukkit.Bukkit;
 import org.bukkit.inventory.Inventory;
 import org.bukkit.inventory.ItemStack;
diff --git a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuQuest.java b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuQuest.java
index 622e4558..243e08d2 100644
--- a/src/main/java/com/leonardobishop/quests/obj/misc/QMenuQuest.java
+++ b/src/main/java/com/leonardobishop/quests/obj/misc/QMenuQuest.java
@@ -1,11 +1,11 @@
 package com.leonardobishop.quests.obj.misc;
 
-import com.leonardobishop.quests.player.QPlayer;
-import com.leonardobishop.quests.player.questprogressfile.QuestProgress;
-import com.leonardobishop.quests.quests.Quest;
 import com.leonardobishop.quests.Quests;
 import com.leonardobishop.quests.obj.Items;
 import com.leonardobishop.quests.obj.Options;
+import com.leonardobishop.quests.player.QPlayer;
+import com.leonardobishop.quests.player.questprogressfile.QuestProgress;
+import com.leonardobishop.quests.quests.Quest;
 import org.bukkit.Bukkit;
 import org.bukkit.inventory.Inventory;
 import org.bukkit.inventory.ItemStack;
@@ -20,15 +20,16 @@ import java.util.concurrent.TimeUnit;
 public class QMenuQuest implements QMenu {
 
     private final HashMap<Integer, String> slotsToQuestIds = new HashMap<>();
+    private final QMenuCategory superMenu;
+    private final String categoryName;
+    private final int pageSize = 45;
+    private final QPlayer owner;
+
     private int backButtonLocation = -1;
     private int pagePrevLocation = -1;
     private int pageNextLocation = -1;
     private int currentPage = -1;
     private boolean backButtonEnabled = true;
-    private final QMenuCategory superMenu;
-    private final String categoryName;
-    private final int pageSize = 45;
-    private final QPlayer owner;
 
     public QMenuQuest(QPlayer owner, String categoryName, QMenuCategory superMenu) {
         this.owner = owner;
diff --git a/src/main/java/com/leonardobishop/quests/player/QPlayer.java b/src/main/java/com/leonardobishop/quests/player/QPlayer.java
index 0b12b91f..84653040 100644
--- a/src/main/java/com/leonardobishop/quests/player/QPlayer.java
+++ b/src/main/java/com/leonardobishop/quests/player/QPlayer.java
@@ -11,7 +11,10 @@ import com.leonardobishop.quests.quests.Quest;
 import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
 
-import java.util.*;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Map;
+import java.util.UUID;
 
 public class QPlayer {
 
diff --git a/src/main/java/com/leonardobishop/quests/player/questprogressfile/QuestProgress.java b/src/main/java/com/leonardobishop/quests/player/questprogressfile/QuestProgress.java
index b4e4fc54..442b802c 100644
--- a/src/main/java/com/leonardobishop/quests/player/questprogressfile/QuestProgress.java
+++ b/src/main/java/com/leonardobishop/quests/player/questprogressfile/QuestProgress.java
@@ -11,13 +11,14 @@ public class QuestProgress {
 
     private final Quests plugin;
 
-    private Map<String, TaskProgress> taskProgress = new HashMap<>();
-    private String questid;
+    private final Map<String, TaskProgress> taskProgress = new HashMap<>();
+    private final String questid;
+    private final UUID player;
+
     private boolean started;
     private boolean completed;
     private boolean completedBefore;
     private long completionDate;
-    private UUID player;
     private boolean modified;
 
     public QuestProgress(Quests plugin, String questid, boolean completed, boolean completedBefore, long completionDate, UUID player, boolean started) {
diff --git a/src/main/java/com/leonardobishop/quests/player/questprogressfile/TaskProgress.java b/src/main/java/com/leonardobishop/quests/player/questprogressfile/TaskProgress.java
index 3ae38664..fa361a7a 100644
--- a/src/main/java/com/leonardobishop/quests/player/questprogressfile/TaskProgress.java
+++ b/src/main/java/com/leonardobishop/quests/player/questprogressfile/TaskProgress.java
@@ -5,17 +5,18 @@ import java.util.UUID;
 public class TaskProgress {
 
     private final QuestProgress questProgress;
+    private final String taskid;
+    private final UUID player;
 
     private boolean modified;
-    private String taskid;
     private Object progress;
-    private UUID player;
     private boolean completed;
 
     public TaskProgress(QuestProgress questProgress, String taskid, Object progress, UUID player, boolean completed) {
         this.questProgress = questProgress;
         this.taskid = taskid;
         this.progress = progress;
+        this.player = player;
         this.completed = completed;
     }
 
diff --git a/src/main/java/com/leonardobishop/quests/quests/Category.java b/src/main/java/com/leonardobishop/quests/quests/Category.java
index 8ace7e03..0fdedad4 100644
--- a/src/main/java/com/leonardobishop/quests/quests/Category.java
+++ b/src/main/java/com/leonardobishop/quests/quests/Category.java
@@ -8,9 +8,9 @@ import java.util.List;
 
 public class Category {
 
-    private String id;
-    private ItemStack displayItem;
-    private boolean permissionRequired;
+    private final String id;
+    private final ItemStack displayItem;
+    private final boolean permissionRequired;
     private final List<String> registeredQuestIds = new ArrayList<>();
 
     public Category(String id, ItemStack displayItem, boolean permissionRequired) {
@@ -23,26 +23,14 @@ public class Category {
         return id;
     }
 
-    public void setId(String id) {
-        this.id = id;
-    }
-
     public boolean isPermissionRequired() {
         return permissionRequired;
     }
 
-    public void setPermissionRequired(boolean permissionRequired) {
-        this.permissionRequired = permissionRequired;
-    }
-
     public ItemStack getDisplayItem() {
         return displayItem;
     }
 
-    public void setDisplayItem(ItemStack displayItem) {
-        this.displayItem = displayItem;
-    }
-
     public void registerQuestId(String questid) {
         registeredQuestIds.add(questid);
     }
diff --git a/src/main/java/com/leonardobishop/quests/quests/Quest.java b/src/main/java/com/leonardobishop/quests/quests/Quest.java
index 36429637..78bbc6b0 100644
--- a/src/main/java/com/leonardobishop/quests/quests/Quest.java
+++ b/src/main/java/com/leonardobishop/quests/quests/Quest.java
@@ -19,7 +19,7 @@ public class Quest implements Comparable<Quest> {
     private final boolean cooldownEnabled;
     private final int cooldown;
     private final int sortOrder;
-    private boolean permissionRequired;
+    private final boolean permissionRequired;
     private String categoryid;
 
 
@@ -69,10 +69,6 @@ public class Quest implements Comparable<Quest> {
         return permissionRequired;
     }
 
-    public void setPermissionRequired(boolean permissionRequired) {
-        this.permissionRequired = permissionRequired;
-    }
-
     public List<String> getRewardString() {
         return rewardString;
     }
diff --git a/src/main/java/com/leonardobishop/quests/quests/QuestManager.java b/src/main/java/com/leonardobishop/quests/quests/QuestManager.java
index 914eabb4..022c2c89 100644
--- a/src/main/java/com/leonardobishop/quests/quests/QuestManager.java
+++ b/src/main/java/com/leonardobishop/quests/quests/QuestManager.java
@@ -1,9 +1,11 @@
 package com.leonardobishop.quests.quests;
 
 import com.leonardobishop.quests.Quests;
-import org.bukkit.ChatColor;
 
-import java.util.*;
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
 
 public class QuestManager {
 
diff --git a/src/main/java/com/leonardobishop/quests/quests/Task.java b/src/main/java/com/leonardobishop/quests/quests/Task.java
index d986a9bc..ca0f7150 100644
--- a/src/main/java/com/leonardobishop/quests/quests/Task.java
+++ b/src/main/java/com/leonardobishop/quests/quests/Task.java
@@ -6,9 +6,8 @@ import java.util.Map;
 public class Task {
 
     private final Map<String, Object> configValues = new HashMap<>();
-
-    private String id;
-    private String type;
+    private final String id;
+    private final String type;
 
     public Task(String id, String type) {
         this.id = id;
diff --git a/src/main/java/com/leonardobishop/quests/quests/tasktypes/TaskTypeManager.java b/src/main/java/com/leonardobishop/quests/quests/tasktypes/TaskTypeManager.java
index 465ee527..d5196af0 100644
--- a/src/main/java/com/leonardobishop/quests/quests/tasktypes/TaskTypeManager.java
+++ b/src/main/java/com/leonardobishop/quests/quests/tasktypes/TaskTypeManager.java
@@ -9,7 +9,7 @@ import java.util.ArrayList;
 
 public class TaskTypeManager {
 
-    private Quests plugin;
+    private final Quests plugin;
     private boolean allowRegistrations;
 
     public TaskTypeManager(Quests plugin) {
diff --git a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/CommandTaskType.java b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/CommandTaskType.java
index 520b2103..4a1d096c 100644
--- a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/CommandTaskType.java
+++ b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/CommandTaskType.java
@@ -14,7 +14,6 @@ import com.leonardobishop.quests.quests.tasktypes.TaskUtils;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
-import org.bukkit.event.entity.EntityDamageByEntityEvent;
 import org.bukkit.event.player.PlayerCommandPreprocessEvent;
 
 import java.util.ArrayList;
diff --git a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/InventoryTaskType.java b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/InventoryTaskType.java
index 25ef76ec..93a4763f 100644
--- a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/InventoryTaskType.java
+++ b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/InventoryTaskType.java
@@ -15,13 +15,10 @@ import com.leonardobishop.quests.quests.tasktypes.TaskUtils;
 import org.bukkit.Bukkit;
 import org.bukkit.Material;
 import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.configuration.MemorySection;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
-import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.event.inventory.InventoryCloseEvent;
-import org.bukkit.event.inventory.InventoryInteractEvent;
 import org.bukkit.event.player.PlayerPickupItemEvent;
 import org.bukkit.inventory.ItemStack;
 
diff --git a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/MiningCertainTaskType.java b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/MiningCertainTaskType.java
index 8aa3090a..b2670425 100644
--- a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/MiningCertainTaskType.java
+++ b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/MiningCertainTaskType.java
@@ -11,7 +11,6 @@ import com.leonardobishop.quests.quests.Task;
 import com.leonardobishop.quests.quests.tasktypes.ConfigValue;
 import com.leonardobishop.quests.quests.tasktypes.TaskType;
 import com.leonardobishop.quests.quests.tasktypes.TaskUtils;
-import org.bukkit.Bukkit;
 import org.bukkit.Material;
 import org.bukkit.block.Block;
 import org.bukkit.event.EventHandler;
diff --git a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/dependent/CitizensInteractTaskType.java b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/dependent/CitizensInteractTaskType.java
index acb39823..a5e53ea0 100644
--- a/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/dependent/CitizensInteractTaskType.java
+++ b/src/main/java/com/leonardobishop/quests/quests/tasktypes/types/dependent/CitizensInteractTaskType.java
@@ -1,6 +1,5 @@
 package com.leonardobishop.quests.quests.tasktypes.types.dependent;
 
-import com.leonardobishop.quests.Quests;
 import com.leonardobishop.quests.QuestsConfigLoader;
 import com.leonardobishop.quests.api.QuestsAPI;
 import com.leonardobishop.quests.player.QPlayer;
@@ -14,8 +13,6 @@ import com.leonardobishop.quests.quests.tasktypes.TaskType;
 import com.leonardobishop.quests.quests.tasktypes.TaskUtils;
 import net.citizensnpcs.api.event.NPCRightClickEvent;
 import org.bukkit.ChatColor;
-import org.bukkit.Material;
-import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 
diff --git a/src/main/java/com/leonardobishop/quests/updater/Updater.java b/src/main/java/com/leonardobishop/quests/updater/Updater.java
index f0705d1e..6c197b88 100644
--- a/src/main/java/com/leonardobishop/quests/updater/Updater.java
+++ b/src/main/java/com/leonardobishop/quests/updater/Updater.java
@@ -14,10 +14,12 @@ import java.util.concurrent.TimeUnit;
 public class Updater {
  
     private static final int PROJECT_ID = 23696;
-    private String installedVersion;
+
+    private final String installedVersion;
+    private final Quests plugin;
+
     private String returnedVersion;
     private URL api;
-    private Quests plugin;
     private boolean updateReady;
     private long lastCheck;