These should all be 0 too, for consistancy

This commit is contained in:
T00thpick1 2013-06-29 23:42:28 -04:00
parent d6b39a11e7
commit 36b09421e8

View File

@ -105,10 +105,10 @@ public final class SQLDatabaseManager implements DatabaseManager {
public void saveUser(PlayerProfile profile) { public void saveUser(PlayerProfile profile) {
checkConnected(); checkConnected();
int userId = readId(profile.getPlayerName()); int userId = readId(profile.getPlayerName());
if (userId == -1) { if (userId == 0) {
newUser(profile.getPlayerName()); newUser(profile.getPlayerName());
userId = readId(profile.getPlayerName()); userId = readId(profile.getPlayerName());
if (userId == -1) { if (userId == 0) {
mcMMO.p.getLogger().log(Level.WARNING, "Failed to save user " + profile.getPlayerName()); mcMMO.p.getLogger().log(Level.WARNING, "Failed to save user " + profile.getPlayerName());
return; return;
} }
@ -1043,7 +1043,7 @@ public final class SQLDatabaseManager implements DatabaseManager {
} }
private int readId(String playerName) { private int readId(String playerName) {
int id = -1; int id = 0;
try { try {
PreparedStatement statement = connection.prepareStatement("SELECT id FROM " + tablePrefix + "users WHERE user = ?"); PreparedStatement statement = connection.prepareStatement("SELECT id FROM " + tablePrefix + "users WHERE user = ?");