Remove unnecessary replaceAll

GeorgH93 · Dec 19, 2023 · 6f6106f · 6f6106f
1 parent ad22323
commit 6f6106f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/MySQL.java b/Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/MySQL.java
@@ -38,7 +38,7 @@ public MySQL(@NotNull Minepacks plugin, @Nullable ConnectionProvider connectionP
 	protected void updateQueriesForDialect()
 	{
 		queryDeleteOldBackpacks = "DELETE FROM {TableBackpacks} WHERE {FieldBPLastUpdate} + INTERVAL {VarMaxAge} day < NOW()";
-		queryUpdateBp = queryUpdateBp.replaceAll("\\{NOW}", "NOW()");
+		queryUpdateBp = queryUpdateBp.replace("{NOW}", "NOW()");
 	}
 
 	@Override

diff --git a/Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/SQL.java b/Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/SQL.java
@@ -32,6 +32,7 @@
 import org.jetbrains.annotations.NotNull;
 
 import java.sql.*;
+import java.util.logging.Level;
 
 public abstract class SQL extends Database
 {
@@ -163,10 +164,10 @@ protected void setTableAndFieldNames()
 	protected String replacePlaceholders(@Language("SQL") String query)
 	{
 		return query.replaceAll("(\\{\\w+})", "`$1`").replaceAll("`(\\{\\w+})`_(\\w+)", "`$1_$2`").replaceAll("fk_`(\\{\\w+})`_`(\\{\\w+})`_`(\\{\\w+})`", "`fk_$1_$2_$3`") // Fix name formatting
-				.replaceAll("\\{TablePlayers}", tablePlayers).replaceAll("\\{FieldName}", fieldPlayerName).replaceAll("\\{FieldUUID}", fieldPlayerUUID).replaceAll("\\{FieldPlayerID}", fieldPlayerID) // Players
-				.replaceAll("\\{TableBackpacks}", tableBackpacks).replaceAll("\\{FieldBPOwner}", fieldBpOwner).replaceAll("\\{FieldBPITS}", fieldBpIts) // Backpacks
-				.replaceAll("\\{FieldBPVersion}", fieldBpVersion).replaceAll("\\{FieldBPLastUpdate}", fieldBpLastUpdate) // Backpacks
-				.replaceAll("\\{TableCooldowns}", tableCooldowns).replaceAll("\\{FieldCDPlayer}", fieldCdPlayer).replaceAll("\\{FieldCDTime}", fieldCdTime); // Cooldowns
+				.replace("{TablePlayers}", tablePlayers).replace("{FieldName}", fieldPlayerName).replace("{FieldUUID}", fieldPlayerUUID).replace("{FieldPlayerID}", fieldPlayerID) // Players
+				.replace("{TableBackpacks}", tableBackpacks).replace("{FieldBPOwner}", fieldBpOwner).replace("{FieldBPITS}", fieldBpIts) // Backpacks
+				.replace("{FieldBPVersion}", fieldBpVersion).replace("{FieldBPLastUpdate}", fieldBpLastUpdate) // Backpacks
+				.replace("{TableCooldowns}", tableCooldowns).replace("{FieldCDPlayer}", fieldCdPlayer).replace("{FieldCDTime}", fieldCdTime); // Cooldowns
 	}
 
 	protected void runStatementAsync(final String query, final Object... args)
@@ -186,8 +187,7 @@ protected void runStatement(final String query, final Object... args)
 		}
 		catch(SQLException e)
 		{
-			plugin.getLogger().severe("Query: " + query);
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, e, () -> "Query: " + query);
 		}
 	}