From b81a73bdf2b014564e68a6c1802fb2cad7b16cbb Mon Sep 17 00:00:00 2001 From: Fuzzlemann Date: Sun, 20 Aug 2017 15:53:59 +0200 Subject: [PATCH 1/5] Adds a WhereParser --- .../plan/data/server/ServerInfoManager.java | 2 +- .../djrapitops/plan/database/sql/Insert.java | 2 - .../djrapitops/plan/database/sql/Select.java | 16 +------ .../djrapitops/plan/database/sql/Update.java | 30 ++++++++++++ .../plan/database/sql/WhereParser.java | 47 +++++++++++++++++++ .../djrapitops/plan/database/DBUtilsTest.java | 13 ----- 6 files changed, 79 insertions(+), 31 deletions(-) create mode 100644 Plan/src/main/java/com/djrapitops/plan/database/sql/Update.java create mode 100644 Plan/src/main/java/com/djrapitops/plan/database/sql/WhereParser.java diff --git a/Plan/src/main/java/com/djrapitops/plan/data/server/ServerInfoManager.java b/Plan/src/main/java/com/djrapitops/plan/data/server/ServerInfoManager.java index 12d94f308..e656c41b9 100644 --- a/Plan/src/main/java/com/djrapitops/plan/data/server/ServerInfoManager.java +++ b/Plan/src/main/java/com/djrapitops/plan/data/server/ServerInfoManager.java @@ -56,7 +56,7 @@ public class ServerInfoManager { String webAddress = address[0]; int port = Integer.parseInt(address[1]); String name = Settings.SERVER_NAME.toString(); - serverTable.saveCurrentServerInfo(new ServerInfo(-1, serverUUID, name, webAddress, port)); + //TODO serverTable.saveCurrentServerInfo(new ServerInfo(-1, serverUUID, name, webAddress, port)); } public UUID generateNewUUID(Server server) { diff --git a/Plan/src/main/java/com/djrapitops/plan/database/sql/Insert.java b/Plan/src/main/java/com/djrapitops/plan/database/sql/Insert.java index 45423aa17..9b2d1d38e 100644 --- a/Plan/src/main/java/com/djrapitops/plan/database/sql/Insert.java +++ b/Plan/src/main/java/com/djrapitops/plan/database/sql/Insert.java @@ -2,8 +2,6 @@ package main.java.com.djrapitops.plan.database.sql; public class Insert extends SqlParser { - private int conditions = 0; - public Insert(String table) { super("INSERT INTO " + table); addSpace(); diff --git a/Plan/src/main/java/com/djrapitops/plan/database/sql/Select.java b/Plan/src/main/java/com/djrapitops/plan/database/sql/Select.java index 44290f9f7..229c329b0 100644 --- a/Plan/src/main/java/com/djrapitops/plan/database/sql/Select.java +++ b/Plan/src/main/java/com/djrapitops/plan/database/sql/Select.java @@ -2,9 +2,7 @@ package main.java.com.djrapitops.plan.database.sql; import main.java.com.djrapitops.plan.Log; -public class Select extends SqlParser { - - private int conditions = 0; +public class Select extends WhereParser { public Select(String start) { super(start); @@ -24,16 +22,4 @@ public class Select extends SqlParser { Log.debug(parser.toString()); return parser; } - - public Select where(String... conditions) { - append(" WHERE "); - for (String condition : conditions) { - if (this.conditions > 0) { - append(" AND "); - } - append("(").append(condition).append(")"); - this.conditions++; - } - return this; - } } diff --git a/Plan/src/main/java/com/djrapitops/plan/database/sql/Update.java b/Plan/src/main/java/com/djrapitops/plan/database/sql/Update.java new file mode 100644 index 000000000..e726a7a9c --- /dev/null +++ b/Plan/src/main/java/com/djrapitops/plan/database/sql/Update.java @@ -0,0 +1,30 @@ +/* + * Licence is provided in the jar as license.yml also here: + * https://github.com/Rsl1122/Plan-PlayerAnalytics/blob/master/Plan/src/main/resources/license.yml + */ +package main.java.com.djrapitops.plan.database.sql; + +/** + * @author Fuzzlemann + */ +public class Update extends WhereParser { + + public Update(String table) { + super("UPDATE " + table + " SET"); + addSpace(); + } + + public static Update values(String table, String... values) { + Update parser = new Update(table); + + int size = values.length; + for (int i = 0; i < size; i++) { + if (size > 1 && i > 0) { + parser.append(", "); + } + parser.append(values[i] + "=?"); + } + + return parser; + } +} diff --git a/Plan/src/main/java/com/djrapitops/plan/database/sql/WhereParser.java b/Plan/src/main/java/com/djrapitops/plan/database/sql/WhereParser.java new file mode 100644 index 000000000..2462d9e1f --- /dev/null +++ b/Plan/src/main/java/com/djrapitops/plan/database/sql/WhereParser.java @@ -0,0 +1,47 @@ +/* + * Licence is provided in the jar as license.yml also here: + * https://github.com/Rsl1122/Plan-PlayerAnalytics/blob/master/Plan/src/main/resources/license.yml + */ +package main.java.com.djrapitops.plan.database.sql; + +/** + * @author Fuzzlemann + */ +public abstract class WhereParser extends SqlParser { + + public WhereParser() { + super(); + } + + public WhereParser(String start) { + super(start); + } + + private int conditions = 0; + + public WhereParser where(String... conditions) { + return and(conditions); + } + + public WhereParser and(String... conditions) { + return whereOperator("AND", conditions); + } + + public WhereParser or(String... conditions) { + return whereOperator("OR", conditions); + } + + private WhereParser whereOperator(String operator, String... conditions) { + append(" WHERE "); + for (String condition : conditions) { + if (this.conditions > 0) { + addSpace().append(operator).addSpace(); + } + + append("(").append(condition).append(")"); + this.conditions++; + } + + return this; + } +} diff --git a/Plan/test/test/java/main/java/com/djrapitops/plan/database/DBUtilsTest.java b/Plan/test/test/java/main/java/com/djrapitops/plan/database/DBUtilsTest.java index 2632a19e7..6b5af5a2c 100644 --- a/Plan/test/test/java/main/java/com/djrapitops/plan/database/DBUtilsTest.java +++ b/Plan/test/test/java/main/java/com/djrapitops/plan/database/DBUtilsTest.java @@ -7,8 +7,6 @@ package test.java.main.java.com.djrapitops.plan.database; import main.java.com.djrapitops.plan.database.Container; import main.java.com.djrapitops.plan.database.DBUtils; -import org.junit.After; -import org.junit.Before; import org.junit.Test; import java.util.ArrayList; @@ -23,17 +21,6 @@ import static org.junit.Assert.assertEquals; */ public class DBUtilsTest { - public DBUtilsTest() { - } - - @Before - public void setUp() { - } - - @After - public void tearDown() { - } - @Test public void testSplitIntoBatches() { List list = new ArrayList<>(); From 4dac747fedc03264bc82451e889b0174b6459ce7 Mon Sep 17 00:00:00 2001 From: Fuzzlemann Date: Sun, 20 Aug 2017 16:46:19 +0200 Subject: [PATCH 2/5] Optimize some .js and .html files Update German language --- Plan/src/main/resources/html/network.html | 10 +-- Plan/src/main/resources/html/player.html | 18 ++-- Plan/src/main/resources/html/server.html | 82 ++++++++++--------- Plan/src/main/resources/js/activityPie.js | 2 +- .../src/main/resources/js/pluginsTabExpand.js | 4 +- Plan/src/main/resources/js/punchCard.js | 2 +- Plan/src/main/resources/js/resourceGraph.js | 2 +- .../src/main/resources/js/sessionTabExpand.js | 4 +- Plan/src/main/resources/js/tpsGraph.js | 7 +- Plan/src/main/resources/js/worldGraph.js | 2 +- Plan/src/main/resources/js/worldMap.js | 2 +- Plan/src/main/resources/js/worldPie.js | 46 +++++------ Plan/src/main/resources/locale/locale_DE.txt | 4 +- 13 files changed, 96 insertions(+), 89 deletions(-) diff --git a/Plan/src/main/resources/html/network.html b/Plan/src/main/resources/html/network.html index b00173a0d..3b3e69796 100644 --- a/Plan/src/main/resources/html/network.html +++ b/Plan/src/main/resources/html/network.html @@ -22,7 +22,7 @@
-

Players Online

+

Players Online

-

Information

+

Information

${playersOnline}/${playersMax} Players Online

@@ -45,15 +45,15 @@ \ No newline at end of file diff --git a/Plan/src/main/resources/html/player.html b/Plan/src/main/resources/html/player.html index ba0a6743c..b62e887af 100644 --- a/Plan/src/main/resources/html/player.html +++ b/Plan/src/main/resources/html/player.html @@ -72,7 +72,7 @@