diff --git a/src/main/java/com/bbn/hadder/Rethink.java b/src/main/java/com/bbn/hadder/Rethink.java index 6b2b137..2a14fac 100644 --- a/src/main/java/com/bbn/hadder/Rethink.java +++ b/src/main/java/com/bbn/hadder/Rethink.java @@ -18,6 +18,7 @@ package com.bbn.hadder; import com.bbn.hadder.core.Config; import com.rethinkdb.RethinkDB; +import com.rethinkdb.gen.exc.ReqlNonExistenceError; import com.rethinkdb.gen.exc.ReqlOpFailedError; import com.rethinkdb.net.Connection; import org.json.JSONArray; @@ -125,10 +126,10 @@ public class Rethink { this.insert("server", r .hashMap("id", id) .with("prefix", "h.") - .with("message_id", "") - .with("role_id", "") + .with("message_id", null) + .with("role_id", null) .with("invite_detect", false) - .with("starboard", "") + .with("starboard", null) .with("neededstars", "4") ); } @@ -143,7 +144,7 @@ public class Rethink { // TODO public boolean hasStarboardChannel(String guild_id) { - return !this.getByID("server", guild_id, "starboard").equals(""); + return this.getByID("server", guild_id, "starboard") != null; } // TODO public void insertStarboardMessage(String message_id, String guild_id, String starboard_message_id) { @@ -159,7 +160,12 @@ public class Rethink { } // TODO public boolean hasStarboardMessage(String message_id) { - return this.getByID("stars", message_id, "guild") != null; + try { + this.getByID("stars", message_id, "guild"); + return true; + } catch (ReqlNonExistenceError e) { + return false; + } } diff --git a/src/main/java/com/bbn/hadder/RethinkServer.java b/src/main/java/com/bbn/hadder/RethinkServer.java index c15c448..04b11ca 100644 --- a/src/main/java/com/bbn/hadder/RethinkServer.java +++ b/src/main/java/com/bbn/hadder/RethinkServer.java @@ -24,15 +24,15 @@ public class RethinkServer { private Rethink rethink; - String accept_emote = ""; - String decline_emote = ""; + String accept_emote = null; + String decline_emote = null; String id; boolean invite_detect = false; - String message_id = ""; + String message_id = null; String neededstars = "3"; String prefix = "h."; - String role_id = ""; - String starboard = ""; + String role_id = null; + String starboard = null; public RethinkServer(JSONObject object, Rethink rethink) { for (Field field : this.getClass().getDeclaredFields()) { diff --git a/src/main/java/com/bbn/hadder/RethinkUser.java b/src/main/java/com/bbn/hadder/RethinkUser.java index 26376f5..eba7564 100644 --- a/src/main/java/com/bbn/hadder/RethinkUser.java +++ b/src/main/java/com/bbn/hadder/RethinkUser.java @@ -27,7 +27,7 @@ public class RethinkUser { String id; String prefix = "h."; String language = "en"; - String blacklisted = "none"; + String blacklisted = null; public RethinkUser(JSONObject object, Rethink rethink) { for (Field field : this.getClass().getDeclaredFields()) { diff --git a/src/main/java/com/bbn/hadder/core/CommandHandler.java b/src/main/java/com/bbn/hadder/core/CommandHandler.java index 5f2029c..99fe82a 100644 --- a/src/main/java/com/bbn/hadder/core/CommandHandler.java +++ b/src/main/java/com/bbn/hadder/core/CommandHandler.java @@ -69,7 +69,7 @@ public class CommandHandler { boolean run = true; String blacklisted = rethinkUser.getBlacklisted(); - if (!"none".equals(blacklisted)) { + if (!(null == blacklisted)) { for (String BLLabel : blacklisted.split(",")) { if (Arrays.asList(cmd.labels()).contains(BLLabel)) { run = false;