Make controller and listener classes final

Refactor multiple controller and listener classes to be declared as `final` to prevent subclassing and enhance encapsulation.

Author: NonSwag

src/main/java/net/thenextlvl/tweaks/TweaksPlugin.java

@@ -131,7 +131,7 @@
 import static org.bukkit.ServerLinks.Type.WEBSITE;
 
 @NullMarked
-public class TweaksPlugin extends JavaPlugin {
+public final class TweaksPlugin extends JavaPlugin {
     private final Metrics metrics = new Metrics(this, 19651);
     private final PluginMessenger messenger = new PluginMessenger(this);
     private final PluginVersionChecker versionChecker = new PluginVersionChecker(this);

src/main/java/net/thenextlvl/tweaks/controller/BackController.java

@@ -13,7 +13,7 @@
 import java.util.concurrent.LinkedBlockingDeque;
 
 @NullMarked
-public class BackController {
+public final class BackController {
     private final Map<UUID, BlockingDeque<Location>> positions = new ConcurrentHashMap<>();
     private final Map<UUID, Location> positionLock = new ConcurrentHashMap<>();
     private final TweaksPlugin plugin;

src/main/java/net/thenextlvl/tweaks/controller/DataController.java

@@ -19,7 +19,7 @@
 import java.util.Set;
 
 @NullMarked
-public class DataController {
+public final class DataController {
     private static final String TPA_TOGGLED = "tpa_toggled";
     private static final String MSG_TOGGLED = "msg_toggled";
     private final Connection connection;
@@ -294,7 +294,7 @@ private Set<NamedLocation> parseNamedLocations(ResultSet resultSet) throws SQLEx
         return new Location(world, x, y, z, yaw, pitch);
     }
 
-    protected void createHomesTable() throws SQLException {
+    private void createHomesTable() throws SQLException {
         executeUpdate("""
                 CREATE TABLE IF NOT EXISTS homes (
                   uuid TEXT NOT NULL,
@@ -332,7 +332,7 @@ CREATE TABLE IF NOT EXISTS warps (
     }
 
     @SuppressWarnings("SqlSourceToSinkFlow")
-    protected <T> @Nullable T executeQuery(String query, ThrowingFunction<ResultSet, T> mapper, @Nullable Object... parameters) throws SQLException {
+    private <T> @Nullable T executeQuery(String query, ThrowingFunction<ResultSet, T> mapper, @Nullable Object... parameters) throws SQLException {
         try (var preparedStatement = connection.prepareStatement(query)) {
             for (var i = 0; i < parameters.length; i++)
                 preparedStatement.setObject(i + 1, parameters[i]);
@@ -343,7 +343,7 @@ CREATE TABLE IF NOT EXISTS warps (
     }
 
     @SuppressWarnings("SqlSourceToSinkFlow")
-    protected int executeUpdate(String query, @Nullable Object... parameters) throws SQLException {
+    private int executeUpdate(String query, @Nullable Object... parameters) throws SQLException {
         try (var preparedStatement = connection.prepareStatement(query)) {
             for (var i = 0; i < parameters.length; i++)
                 preparedStatement.setObject(i + 1, parameters[i]);

src/main/java/net/thenextlvl/tweaks/controller/HomeController.java

@@ -12,7 +12,7 @@
 import java.util.concurrent.CompletableFuture;
 
 @NullMarked
-public class HomeController {
+public final class HomeController {
     private final TweaksPlugin plugin;
 
     public HomeController(TweaksPlugin plugin) {

src/main/java/net/thenextlvl/tweaks/controller/MSGController.java

@@ -8,7 +8,7 @@
 import java.util.WeakHashMap;
 
 @NullMarked
-public class MSGController {
+public final class MSGController {
     private final Map<CommandSender, CommandSender> conversations = new WeakHashMap<>();
 
     public @Nullable CommandSender getConversation(CommandSender sender) {

src/main/java/net/thenextlvl/tweaks/controller/ServiceController.java

@@ -17,7 +17,7 @@
 import java.util.function.Function;
 
 @NullMarked
-public class ServiceController {
+public final class ServiceController {
     private final @Nullable BankController banks;
     private final @Nullable ChatController chat;
     private final @Nullable EconomyController economy;

src/main/java/net/thenextlvl/tweaks/controller/TPAController.java

@@ -14,7 +14,7 @@
 import java.util.WeakHashMap;
 
 @NullMarked
-public class TPAController {
+public final class TPAController {
     private final Map<Player, Set<Request>> requests = new WeakHashMap<>();
     private final TweaksPlugin plugin;
 

src/main/java/net/thenextlvl/tweaks/controller/TeleportController.java

@@ -14,7 +14,7 @@
 import java.util.concurrent.ConcurrentHashMap;
 
 @NullMarked
-public class TeleportController {
+public final class TeleportController {
     private final Map<UUID, Location> teleports = new ConcurrentHashMap<>();
     private final TweaksPlugin plugin;
 

src/main/java/net/thenextlvl/tweaks/controller/WarpController.java

@@ -10,7 +10,7 @@
 import java.util.concurrent.CompletableFuture;
 
 @NullMarked
-public class WarpController {
+public final class WarpController {
     private final TweaksPlugin plugin;
 
     public WarpController(TweaksPlugin plugin) {

src/main/java/net/thenextlvl/tweaks/gui/HomeGUI.java

@@ -11,7 +11,7 @@
 import static org.bukkit.event.player.PlayerTeleportEvent.TeleportCause.PLUGIN;
 
 @NullMarked
-public class HomeGUI extends NamedLocationGUI {
+public final class HomeGUI extends NamedLocationGUI {
     public HomeGUI(TweaksPlugin plugin, Player owner, Collection<NamedLocation> elements) {
         super(plugin, plugin.config().guis.homes, owner, plugin.bundle().component("gui.title.homes", owner), elements);
     }