MinecraftWars / Gringotts

Item based economy plugin for Bukkit
BSD 2-Clause "Simplified" License
37 stars 77 forks source link

Other plugins tend to lose money when accounts exceed capacity #44

Open bezeek opened 11 years ago

bezeek commented 11 years ago

Some plugins, (Factions and RealShop2 in my experience,) do not take appropriate action when an account reaches its capacity, and any excess money is lost forever.

I propose an option "account.canoverflow," to store excess money in the DB, and split the account into "available" and "unavailable" balances.

I hope you find this viable! I'm not much of a Java guy :grin:

jastice commented 11 years ago

Thanks, this sounds like a reasonable workaround to plugins ignoring failed transfer messages. I like the tax idea.

ryantheleach commented 11 years ago

As an addendum, if a vault is destroyed, is it possible for some/all of the overflow to drop as items? would prevent people using overflow as unraidable money for factions etc.

bezeek commented 11 years ago

Awesome job on the fix! RealShop2 now (silently, as it tends to do) denies sale to a shop if your vaults are full. I can't speak for Factions though - it seems to have broken sometime after 1.4.0.

Edit: It was bouncing transactions because I wiped out the shop owner's account. facepalm

jastice commented 11 years ago

There is no global fix for this yet :) Perhaps RealShop2 has implemented a fix. I did also submit a fix to Factions, but it's probably not in the release version yet. The latest source version should have it: https://github.com/MassiveCraft/Factions