MySQL Functional

Fixed problems that were not known before testing like my really bad consistency. I blame being tired but that's why we test code ig.

Tested and works.

Referencing PR #2

Author: Peashooter101

src/main/java/simplexity/simpleprefixes/config/Config.java

@@ -42,10 +42,10 @@ public static void loadConfig() {
         headerLore = config.getStringList("header.lore");
         headerCount = config.getString("header.count", "1");
 
-        sqlUser = config.getString("sql.user");
-        sqlPass = config.getString("sql.pass");
-        sqlDbName = config.getString("sql.dbName");
-        sqlIp = config.getString("sql.ip");
+        sqlUser = config.getString("mysql.user");
+        sqlPass = config.getString("mysql.pass");
+        sqlDbName = config.getString("mysql.name");
+        sqlIp = config.getString("mysql.ip");
     }
 
     public static void generateBaseHeaderItem() {

src/main/java/simplexity/simpleprefixes/util/saving/MySQL.java

@@ -17,7 +17,7 @@ public class MySQL extends SaveHandler {
     private final String CREATE_TABLE = """
             CREATE TABLE IF NOT EXISTS player_prefixes (
                 id VARCHAR(36) PRIMARY KEY,
-                prefix_id VARCHAR(256) NOT NULL
+                prefix_id VARCHAR(255) NOT NULL
             );""";
     private final String INSERT_ROW = "REPLACE INTO player_prefixes (id, prefix_id) VALUES (?, ?);";
     private final String REMOVE_ROW = "DELETE FROM player_prefixes WHERE id = ?;";
@@ -48,8 +48,8 @@ public void init() {
 
     @Override
     public String getPrefixId(OfflinePlayer p) {
-        String prefixId = null;
         if (!isEnabled()) return null;
+        String prefixId = null;
         try (PreparedStatement statement = connection.prepareStatement(SELECT_ROW)) {
             statement.setString(1, p.getUniqueId().toString());
             try (ResultSet result = statement.executeQuery()) {
@@ -65,9 +65,11 @@ public String getPrefixId(OfflinePlayer p) {
 
     @Override
     public void setPrefixId(OfflinePlayer p, String id) {
+        if (!isEnabled()) return;
         if (id == null) {
             try (PreparedStatement statement = connection.prepareStatement(REMOVE_ROW)) {
                 statement.setString(1, p.getUniqueId().toString());
+                statement.execute();
             } catch (SQLException e) {
                 e.printStackTrace();
             }
@@ -76,6 +78,7 @@ public void setPrefixId(OfflinePlayer p, String id) {
         try (PreparedStatement statement = connection.prepareStatement(INSERT_ROW)) {
             statement.setString(1, p.getUniqueId().toString());
             statement.setString(2, id);
+            statement.execute();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/main/resources/config.yml

@@ -7,8 +7,14 @@
 # Don't touch unless asked to
 debug-level: 0
 
-# Saving Types: PDC, File
+# Saving Types: PDC, File, MySQL
 saving-type: "file"
+mysql:
+  ip: "localhost:3306"
+  name: prefixes
+  user: username1
+  pass: badpassword!
+
 default-prefix: "<white>[<gray>Player</gray>]</white> "
 prefix-menu-name: "<bold>Prefix Menu</bold>"