bug fixes

src/defaults.yml

@@ -1,3 +1,4 @@
 maxBankDataSize: 1000000
+overloadedWarningMessage: "&cAttempted to exceed bank storage size. Contact a staff member for more information."
 worlds:
   - survival3

src/no/runsafe/runsafebank/BankHandler.java

@@ -29,6 +29,7 @@ public BankHandler(BankRepository bankRepository, IDebug output, IScheduler sche
 	public void OnConfigurationChanged(IConfiguration config)
 	{
 		this.maxBankDataSize = config.getConfigValueAsInt("maxBankDataSize");
+		this.overloadedWarningMessage = config.getConfigValueAsString("overloadedWarningMessage");
 	}
 
 	public void openBank(IPlayer viewer, IPlayer owner)
@@ -56,6 +57,7 @@ private void loadBank(IPlayer owner)
 	private void saveLoadedBanks()
 	{
 		List<IPlayer> oldBanks = new ArrayList<>();
+		List<IPlayer> suspiciousBanks = new ArrayList<>();
 		for (Map.Entry<IPlayer, RunsafeInventory> bank : this.loadedBanks.entrySet())
 		{
 			RunsafeInventory bankInventory = bank.getValue();
@@ -67,7 +69,11 @@ private void saveLoadedBanks()
 					"Player attempted to exceed size limit of bank. Player: " + bankOwner.getName() +
 					" Size: " + bankDataSize
 				);
-				this.debugger.debugFine("Bank inv size too big. Could not save bank for: " + bankOwner.getName());
+				this.debugger.debugFine(
+					"Bank inv size too big. Could not save bank for: " + bankOwner.getName() +
+					" Size: " + bankDataSize
+				);
+				suspiciousBanks.add(bankOwner);
 				continue;
 			}
 
@@ -79,6 +85,13 @@ private void saveLoadedBanks()
 				oldBanks.add(bankOwner);
 		}
 
+		for (IPlayer suspect: suspiciousBanks)
+		{
+			suspect.sendColouredMessage(overloadedWarningMessage);
+			suspect.closeInventory();
+			loadedBanks.remove(suspect);
+		}
+
 		for (IPlayer owner: oldBanks)
 		{
 			this.loadedBanks.remove(owner);
@@ -106,6 +119,7 @@ public void OnPluginDisabled()
 		this.saveLoadedBanks();
 	}
 
+	private String overloadedWarningMessage;
 	private int maxBankDataSize;
 	private final ConcurrentHashMap<IPlayer, RunsafeInventory> loadedBanks = new ConcurrentHashMap<>();
 	private final BankRepository bankRepository;

src/no/runsafe/runsafebank/Plugin.java

@@ -2,6 +2,7 @@
 
 import no.runsafe.framework.RunsafePlugin;
 import no.runsafe.framework.features.Commands;
+import no.runsafe.framework.features.Configuration;
 import no.runsafe.framework.features.Database;
 import no.runsafe.framework.features.Events;
 import no.runsafe.runsafebank.commands.ViewBank;
@@ -17,6 +18,7 @@ protected void pluginSetup()
 		addComponent(Commands.class);
 		addComponent(Events.class);
 		addComponent(Database.class);
+		addComponent(Configuration.class);
 
 		// Plugin components
 		addComponent(Interact.class);