Only update if already in map

This commit is contained in:
vemacs
2015-06-26 11:22:23 -04:00
committed by drtshock
parent bcf9aacd3b
commit eeb72ed1bb
2 changed files with 10 additions and 3 deletions
@@ -24,7 +24,8 @@ import java.util.zip.ZipFile;
* <b>AT ALL</b>. <br> If you fail to include this option in your config, your plugin will be <b>REJECTED</b> when you * <b>AT ALL</b>. <br> If you fail to include this option in your config, your plugin will be <b>REJECTED</b> when you
* attempt to submit it to dev.bukkit.org. * attempt to submit it to dev.bukkit.org.
* <p/> * <p/>
* An example of a good configuration option would be something similar to 'auto-update: true' - if this value is set to * An example of a good configuration option would be something similar to 'auto-update: true' - if this value is set
* to
* false you may NOT run the auto-updater. <br> If you are unsure about these rules, please read the plugin submission * false you may NOT run the auto-updater. <br> If you are unsure about these rules, please read the plugin submission
* guidelines: http://goo.gl/8iU5l * guidelines: http://goo.gl/8iU5l
* *
@@ -39,6 +39,7 @@ public class UUIDVaultManager {
* @param inventory The inventory to be saved. * @param inventory The inventory to be saved.
* @param player The player of whose file to save to. * @param player The player of whose file to save to.
* @param number The vault number. * @param number The vault number.
*
* @throws java.io.IOException Uh oh! * @throws java.io.IOException Uh oh!
*/ */
public void saveVault(Inventory inventory, UUID player, int number) throws IOException { public void saveVault(Inventory inventory, UUID player, int number) throws IOException {
@@ -148,6 +149,7 @@ public class UUIDVaultManager {
* @param playerFile the YamlConfiguration file. * @param playerFile the YamlConfiguration file.
* @param size the size of the vault. * @param size the size of the vault.
* @param number the vault number. * @param number the vault number.
*
* @return inventory if exists, otherwise null. * @return inventory if exists, otherwise null.
*/ */
private Inventory getInventory(YamlConfiguration playerFile, int size, int number, String title) { private Inventory getInventory(YamlConfiguration playerFile, int size, int number, String title) {
@@ -168,6 +170,7 @@ public class UUIDVaultManager {
* *
* @param holder The holder of the vault. * @param holder The holder of the vault.
* @param number The vault number. * @param number The vault number.
*
* @return The inventory of the specified holder and vault number. * @return The inventory of the specified holder and vault number.
*/ */
public Inventory getVault(UUID holder, int number) { public Inventory getVault(UUID holder, int number) {
@@ -198,6 +201,7 @@ public class UUIDVaultManager {
* @param sender The sender of whom to send messages to. * @param sender The sender of whom to send messages to.
* @param holder The vault holder. * @param holder The vault holder.
* @param number The vault number. * @param number The vault number.
*
* @throws IOException Uh oh! * @throws IOException Uh oh!
*/ */
public void deleteVault(CommandSender sender, final UUID holder, final int number) throws IOException { public void deleteVault(CommandSender sender, final UUID holder, final int number) throws IOException {
@@ -245,6 +249,7 @@ public class UUIDVaultManager {
* Get the holder's vault file. Create if doesn't exist. * Get the holder's vault file. Create if doesn't exist.
* *
* @param holder The vault holder. * @param holder The vault holder.
*
* @return The holder's vault config file. * @return The holder's vault config file.
*/ */
public YamlConfiguration getPlayerVaultFile(UUID holder) { public YamlConfiguration getPlayerVaultFile(UUID holder) {
@@ -274,10 +279,11 @@ public class UUIDVaultManager {
* *
* @param holder The vault holder of whose file to save. * @param holder The vault holder of whose file to save.
* @param yaml The config to save. * @param yaml The config to save.
*
* @throws IOException Uh oh! * @throws IOException Uh oh!
*/ */
public void saveFile(final UUID holder, final YamlConfiguration yaml) { public void saveFile(final UUID holder, final YamlConfiguration yaml) {
if (Bukkit.getPlayer(holder) != null) { if (cachedVaultFiles.containsKey(holder)) {
cachedVaultFiles.put(holder, yaml); cachedVaultFiles.put(holder, yaml);
} }
final boolean backups = PlayerVaults.getInstance().isBackupsEnabled(); final boolean backups = PlayerVaults.getInstance().isBackupsEnabled();
@@ -298,7 +304,7 @@ public class UUIDVaultManager {
} }
public void saveFileSync(final UUID holder, final YamlConfiguration yaml) { public void saveFileSync(final UUID holder, final YamlConfiguration yaml) {
if (Bukkit.getPlayer(holder) != null) { if (cachedVaultFiles.containsKey(holder)) {
cachedVaultFiles.put(holder, yaml); cachedVaultFiles.put(holder, yaml);
} }
final boolean backups = PlayerVaults.getInstance().isBackupsEnabled(); final boolean backups = PlayerVaults.getInstance().isBackupsEnabled();