Fix: Truncate UUID to fit varchar(16) column when handling duplicate player names

pom.xml

@@ -208,7 +208,7 @@
         <dependency>
             <groupId>me.clip</groupId>
             <artifactId>placeholderapi</artifactId>
-            <version>2.11.3</version>
+            <version>2.11.7</version>
             <scope>provided</scope>
         </dependency>
         <dependency>

src/main/java/net/sacredlabyrinth/phaed/simpleclans/listeners/SCPlayerListener.java

@@ -174,26 +174,14 @@ private void registerChatListener() {
     }
 
     private void updatePlayerName(@NotNull final Player player) {
+        // Update in-memory ClanPlayer if exists
         final ClanPlayer cp = plugin.getClanManager().getAnyClanPlayer(player.getUniqueId());
-
-        ClanPlayer duplicate = null;
-        for (ClanPlayer other : plugin.getClanManager().getAllClanPlayers()) {
-            if (other.getName().equals(player.getName()) && !other.getUniqueId().equals(player.getUniqueId())) {
-                duplicate = other;
-                break;
-            }
-        }
-
-        if (duplicate != null) {
-            plugin.getLogger().warning(String.format("Found duplicate for %s, UUIDs: %s, %s", player.getName(),
-                    player.getUniqueId(), duplicate.getUniqueId()));
-            duplicate.setName(duplicate.getUniqueId().toString());
-            plugin.getStorageManager().updatePlayerName(duplicate);
-        }
         if (cp != null) {
             cp.setName(player.getName());
-            plugin.getStorageManager().updatePlayerName(cp);
         }
+
+        // Synchronize player data in database asynchronously
+        plugin.getStorageManager().syncPlayerDataAsync(player);
     }
 
-}
+}

src/main/java/net/sacredlabyrinth/phaed/simpleclans/storage/DBCore.java

@@ -42,8 +42,9 @@ default boolean checkConnection() {
      * @return the result set or null if the query failed
      */
     default @Nullable ResultSet select(String query) {
+        Connection connection = getConnection();
         try {
-            return getConnection().createStatement().executeQuery(query);
+            return connection.createStatement().executeQuery(query);
         } catch (SQLException ex) {
             log.log(Level.SEVERE, String.format("Error executing query: %s", query), ex);
         }
@@ -56,8 +57,8 @@ default boolean checkConnection() {
      * @return true if the statement was executed
      */
     default boolean execute(String query) {
-        try {
-            getConnection().createStatement().execute(query);
+        try (java.sql.Statement statement = getConnection().createStatement()) {
+            statement.execute(query);
             return true;
         } catch (SQLException ex) {
             log.log(Level.SEVERE, String.format("Error executing query: %s", query), ex);
@@ -71,8 +72,7 @@ default boolean execute(String query) {
      * @return true if the table exists
      */
     default boolean existsTable(String table) {
-        try {
-            ResultSet tables = getConnection().getMetaData().getTables(null, null, table, null);
+        try (ResultSet tables = getConnection().getMetaData().getTables(null, null, table, null)) {
             return tables.next();
         } catch (SQLException ex) {
             log.log(Level.SEVERE, String.format("Error checking if table %s exists", table), ex);
@@ -88,8 +88,7 @@ default boolean existsTable(String table) {
      * @return true if the column exists
      */
     default boolean existsColumn(String table, String column) {
-        try {
-            ResultSet col = getConnection().getMetaData().getColumns(null, null, table, column);
+        try (ResultSet col = getConnection().getMetaData().getColumns(null, null, table, column)) {
             return col.next();
         } catch (Exception ex) {
             log.log(Level.SEVERE, String.format("Error checking if column %s exists in table %s", column, table), ex);
@@ -100,9 +99,10 @@ default boolean existsColumn(String table, String column) {
     default void executeUpdate(String query) {
         final Exception exception = new Exception(); // Stores a reference to the caller's stack trace for async tasks
         Runnable executeUpdate = () -> {
-            if (getConnection() != null) {
-                try {
-                    getConnection().createStatement().executeUpdate(query);
+            Connection connection = getConnection();
+            if (connection != null) {
+                try (java.sql.Statement statement = connection.createStatement()) {
+                    statement.executeUpdate(query);
                 } catch (SQLException ex) {
                     log.log(Level.SEVERE, String.format("Error executing query: %s", query), ex);
                     if (!Bukkit.isPrimaryThread()) {