EpicKnarvik97/mcMMO
EpicKnarvik97/mcMMO
1
0
Fork 0
mirror of https://github.com/mcMMO-Dev/mcMMO.git synced 2024-11-22 21:26:46 +01:00
Code Issues Projects Releases Wiki Activity

Less spammy corrupt data reporting

Browse Source
This commit is contained in:
nossr50 2021-03-11 12:16:33 -08:00
parent 7755875dbf
commit d77c8c88a3
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/main/java/com/gmail/nossr50/database/FlatFileDatabaseManager.java
View File

@ -299,6 +299,7 @@ public final class FlatFileDatabaseManager implements DatabaseManager {
BufferedReader in = null; BufferedReader in = null;
FileWriter out = null; FileWriter out = null;
String usersFilePath = mcMMO.getUsersFilePath(); String usersFilePath = mcMMO.getUsersFilePath();
boolean corruptDataFound = false;
synchronized (fileWritingLock) { synchronized (fileWritingLock) {
try { try {
@ -312,7 +313,12 @@ public final class FlatFileDatabaseManager implements DatabaseManager {
while ((line = in.readLine()) != null) { while ((line = in.readLine()) != null) {
//Check for incomplete or corrupted data //Check for incomplete or corrupted data
if(!line.contains(":")) { if(!line.contains(":")) {
mcMMO.p.getLogger().severe("mcMMO found some unexpected data in mcmmo.users and is removing it");
if(!corruptDataFound) {
mcMMO.p.getLogger().severe("mcMMO found some unexpected or corrupted data in mcmmo.users and is removing it, it is possible some data has been lost.");
corruptDataFound = true;
}
continue; continue;
} }
@ -321,7 +327,12 @@ public final class FlatFileDatabaseManager implements DatabaseManager {
//This would be rare, but check the splitData for having enough entries to contain a username //This would be rare, but check the splitData for having enough entries to contain a username
if(splitData.length < USERNAME_INDEX) { //UUID have been in mcMMO DB for a very long time so any user without if(splitData.length < USERNAME_INDEX) { //UUID have been in mcMMO DB for a very long time so any user without
//Something is wrong if we don't have enough split data to have an entry for a username //Something is wrong if we don't have enough split data to have an entry for a username
mcMMO.p.getLogger().severe("mcMMO found some corrupted data in mcmmo.users and is removing it.");
if(!corruptDataFound) {
mcMMO.p.getLogger().severe("mcMMO found some unexpected or corrupted data in mcmmo.users and is removing it, it is possible some data has been lost.");
corruptDataFound = true;
}
continue; continue;
} }