Vault

Vault

7M Downloads

Add support for timers

Liz4v opened this issue · 5 comments

commented

Perms, Econ, and Chat are great! I'd like to ask for support for timers too.

Context: I use both Essentials and HomeSpawnPlus, both providing a few teleport commands. These teleport commands have warm-up and cooldown timers. If I use Essentials' teleport features, further commands will share a single warm-up and cooldown that is independent of HSP's teleport features, that have their own warm-up and cooldown timers.

I think both plugins should share the same timers and I believe Vault would be the best way to provide that.

commented

make timers yourself? there's no reason to request this when java already has builtin timers.
EDIT: apparently I mis-understood the full request initially, see response below.

commented

That was pretty stupid response. Java also has builtin addition and subtraction and yet there's an economy support in Vault.

Oh well.

commented

Java however does not support storing references to specific individuals that are in minecraft hence the necessity for economy. Timers however are 100% contained within java as they are a generic utility methods for executing blocks of code at a given time. Please read the java tutorials on Timers and TimerTasks.

Vault is a layer for Minecraft, not a java utility library. If you have the inclination to create a timer utility for use in Bukkit, by all means go for it. I see no point in having one, others however might. But I wont be putting it into Vault as it has no direct correlation to MC. I will give the same response to someone that would want me to include a database implementation for use in vault plugins... there's little difference here.

In addition, this was a feature request indicating you had no intention of coding one yourself, but would like me to code one for you and others. I wold decline simply for that with no other reason being necessary.

Furthermore, it's presumptuous to assume essentials would do anything with said timers once they were in Vault. They haven't requested them in Vault, and from previous discussions with the Essentials team they only use their own code for critical features. The only reason Essentials supports Vault is because it's a non-required plugin.

commented

Ok, I was too generic in the title there. The need is specifically for a
pair of timers, one per player, which is warm-up and cool down, and there
are different plugins that support this idea of methods of teleport with
warm-up and cool down times. This is very minecraft specific or at least
gaming specific (since this is a concept borrowed from WoW as far as I
know).

On Wed, Dec 5, 2012 at 6:04 PM, Nick Minkler [email protected]:

Java however does not support storing references to specific individuals
that are in minecraft hence the necessity for economy. Timers however are
100% contained within java as they are a generic utility methods for
executing blocks of code at a given time. Please read the java tutorials on
Timers and TimerTasks. Vault is a layer for Minecraft, not a java utility
library. If you have the inclination to create a timer utility for use in
Bukkit, by all means go for it. I see no point in having one, others
however might. But I wont be putting it into Vault as it has no direct
correlation to MC. I will give the same response to someone that would want
me to include a database implementation for use in vault plugins... there's
little difference here.


Reply to this email directly or view it on GitHubhttps://github.com//issues/312#issuecomment-11060410.

commented

I am not going to code this as I don't think it would actually be used, nor do I want it included in Vault as Vault is concerned only with chat, Economy, and Permissions. If you want plugins to adopt something like this you will need to code it for use then get everyone to start using it.