Mtihc / TreasureChest

The TreasureChest plugin for CraftBukkit
dev.bukkit.org/server-mods/treasurechest
10 stars 39 forks source link

Why doesn't setforget work on unlimited chests? #74

Open swills1 opened 10 years ago

swills1 commented 10 years ago

This plugin is really great, but some of the stuff just seems irrelevant. (Like multiple commands that do one thing.)

One thing I don't understand is why setforget doesn't work on unlimited chests. It kind of takes away from what this plugin is supposed to do.

Each chest can have a message set to it, and there are 3 types of messages.

Once you apply setforget to a chest the contents are going to regenerate after a set amount of time. So these chests are basically unlimited just not classified as unlimited.

However, since these chests aren't actually unlimited chests they have two messages applied to them. One for the first discovery and one for each discovery after that.

It isn't really necessary to say 'You found a treasure!' and then say 'You already found this treasure.' when the chest has reset and the entire point was to find it again.

Anyone wanting to set a message to their chests have to set two messages for a chest that is meant to act unlimited. It would be much more convenient for setforget to only work on unlimited chests. That way you just have to set one message.

This would give players two options. They can have unlimited chests that regenerate immediately or after time. Then they can have regular chests that give notice when found and all ready found and do all of the counting and keeping track of how many were found.

It just seems silly to say you've found treasure 6 out of 6 when it regenerates and there is more treasure. Unlimited should have setforget and regular chests shouldn't so they can work better with how the plugin is set up.

Mtihc commented 10 years ago

You are confused about the difference between normal TreasureChest (which are indeed, like you said, also kindof unlimited) and Unlimited TreasureChest. It's semantics.

One thing I don't understand is why setforget doesn't work on unlimited chests. It kind of takes away from what this plugin is supposed to do.

Maybe this will clarify: It started out with the idea of normal chests: Every player can loot it individually 1 time. Then came the setforget command: Every player can loot it individually.. and multiple times with a cooldown. Then came the unlimited chest. No forget time. No individuality. No nothing. Just a chest that spawns a set of items when you open it.

Once you apply setforget to a chest the contents are going to regenerate after a set amount of time. So these chests are basically unlimited just not classified as unlimited.

I would say.. they are TreasureChests with a cooldown. You have a point, maybe "unlimited" is not the right word.

It isn't really necessary to say 'You found a treasure!' and then say 'You already found this treasure.' when the chest has reset and the entire point was to find it again.

If I remember correctly, the message resets back to "You found a treasure!" when the forget-time has passed for that player. The message "You already found this treasure" is to indicate that the forget-time has not passed yet. Maybe im wrong? I forget.

Anyone wanting to set a message to their chests have to set two messages for a chest that is meant to act unlimited. It would be much more convenient for setforget to only work on unlimited chests. That way you just have to set one message.

This would give players two options. They can have unlimited chests that regenerate immediately or after time. Then they can have regular chests that give notice when found and all ready found and do all of the counting and keeping track of how many were found.

I see you have a different idea for this plugin.

There are many ways you can use this system. You use it in 1 particular way. The naming of the commands might be confusing. Maybe we could add more/better message. But this is how it works. Deal with it.

Mtihc commented 10 years ago

This would give players two options. They can have unlimited chests that regenerate immediately or after time. Then they can have regular chests that give notice when found and all ready found and do all of the counting and keeping track of how many were found.

You added a forget-time to the unlimited chest. And you distinguish unlimited and regular chests by keeping track and stuff. But that's not how it works.

Mtihc commented 10 years ago

This would give players two options. They can have unlimited chests that regenerate immediately or after time. Then they can have regular chests that give notice when found and all ready found and do all of the counting and keeping track of how many were found.

I regret dismissing what you said so easily. I made these decisions about commands on a whim. It's because features have been added over time.

You make a very good point at the least. I have to read what you said a couple more times.

... hmm we have 3 types of chests. In your words they are:

So you moved the setforget command to unlimited chests instead of regular. This makes sense, considering the name "unlimited" like you explained.

Hmm, this would have consequences for the message configuration. The only type of chest that requires no second message is the unlimited without a cooldown. The other 2 types of chest do require a second message! That's the big difference.

But that's not a good reason for my choice of names. Your view makes more sense!

swills1 commented 10 years ago

You've basically nailed what I meant. I'd just like to see an unlimited chest with a cool down, but there is absolutely nothing wrong with how it is. I mean everything I want can easily be accomplished right now. Thanks again.