From 28272032e08aeb78aa980d4684b758bbe8703e1b Mon Sep 17 00:00:00 2001 From: ryanhamshire Date: Tue, 7 Oct 2014 19:28:56 -0700 Subject: [PATCH] Fixed players charged blocks twice for same claim. Side effect: likely faster plugin boot due to reduced number of reads and writes. --- src/me/ryanhamshire/GriefPrevention/DataStore.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/me/ryanhamshire/GriefPrevention/DataStore.java b/src/me/ryanhamshire/GriefPrevention/DataStore.java index 524ce5a..d4d6a1e 100644 --- a/src/me/ryanhamshire/GriefPrevention/DataStore.java +++ b/src/me/ryanhamshire/GriefPrevention/DataStore.java @@ -205,7 +205,10 @@ public abstract class DataStore { newClaim.parent.children.add(newClaim); newClaim.inDataStore = true; - this.saveClaim(newClaim); + if(writeToStorage) + { + this.saveClaim(newClaim); + } return; } @@ -228,7 +231,7 @@ public abstract class DataStore newClaim.inDataStore = true; //except for administrative claims (which have no owner), update the owner's playerData with the new claim - if(!newClaim.isAdminClaim()) + if(!newClaim.isAdminClaim() && writeToStorage) { PlayerData ownerData = this.getPlayerData(newClaim.ownerID); ownerData.claims.add(newClaim); @@ -334,7 +337,7 @@ public abstract class DataStore Claim claim = this.claims.get(i); if(playerID.equals(claim.ownerID)) { - playerData.claims.add(claim); + playerData.claims.add(claim); } }