Rx3/MassiveCore
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Yet another econ fix

Browse Source 
Yes, I am not doing that well
This commit is contained in:
Magnus Ulf 2019-01-24 19:23:31 +01:00
parent a74b5a9d35
commit c07c0974fc
1 changed files with 16 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/com/massivecraft/massivecore/money/MoneyMixinVault.java
View File

@ -3,6 +3,7 @@ package com.massivecraft.massivecore.money;
import com.massivecraft.massivecore.util.IdUtil; import com.massivecraft.massivecore.util.IdUtil;
import com.massivecraft.massivecore.util.MUtil; import com.massivecraft.massivecore.util.MUtil;
import net.milkbowl.vault.economy.Economy; import net.milkbowl.vault.economy.Economy;
import net.milkbowl.vault.economy.EconomyResponse;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
import org.bukkit.OfflinePlayer; import org.bukkit.OfflinePlayer;
import org.bukkit.plugin.RegisteredServiceProvider; import org.bukkit.plugin.RegisteredServiceProvider;
@ -180,8 +181,13 @@ public class MoneyMixinVault extends MoneyMixinAbstract
toId = temp; toId = temp;
} }
OfflinePlayer offlinePlayerFrom = IdUtil.getOfflinePlayer(fromId); Object objectFrom = getEconomyObject(fromId);
OfflinePlayer offlinePlayerTo = IdUtil.getOfflinePlayer(toId); OfflinePlayer opFrom = objectFrom instanceof OfflinePlayer ? (OfflinePlayer) objectFrom : null;
String nameFrom = objectFrom instanceof String ? (String) objectFrom : null;
Object objectTo = getEconomyObject(toId);
OfflinePlayer opTo = objectTo instanceof OfflinePlayer ? (OfflinePlayer) objectTo : null;
String nameTo = objectTo instanceof String ? (String) objectTo : null;
// Ensure the accounts exist // Ensure the accounts exist
if (fromId != null) this.ensureExists(fromId); if (fromId != null) this.ensureExists(fromId);
@ -190,7 +196,8 @@ public class MoneyMixinVault extends MoneyMixinAbstract
// Subtract From // Subtract From
if (fromId != null) if (fromId != null)
{ {
if (!economy.withdrawPlayer(offlinePlayerFrom, amount).transactionSuccess()) EconomyResponse response = opTo != null ? economy.withdrawPlayer(opFrom, amount) : economy.withdrawPlayer(nameFrom, amount);
if (!response.transactionSuccess())
{ {
return false; return false;
} }
@ -199,12 +206,15 @@ public class MoneyMixinVault extends MoneyMixinAbstract
// Add To // Add To
if (toId != null) if (toId != null)
{ {
if (!economy.depositPlayer(offlinePlayerTo, amount).transactionSuccess()) EconomyResponse response = opTo != null ? economy.depositPlayer(opTo, amount) : economy.depositPlayer(nameTo, amount);
if (!response.transactionSuccess())
{ {
if (fromId != null) if (fromId != null)
{ {
// Undo the withdraw // Undo the withdraw
economy.depositPlayer(offlinePlayerFrom, amount); if (opFrom != null) economy.depositPlayer(opFrom, amount);
if (nameFrom != null) economy.depositPlayer(nameFrom, amount);
throw new RuntimeException();
} }
return false; return false;
} }