BlueMond / LifeMC

MIT License
1 stars 5 forks source link

MINOR: Vault check missing from "/lifemc cost" command #4

Closed BlueMond closed 4 years ago

BlueMond commented 4 years ago

An error occurs when executing the "/lifemc cost" command due to a missing prior check for the availability of the vault plugin. EDIT: Only when the vault plugin does not exist.

BlueMond commented 4 years ago

The "cost" command may have been overlooked in the most recent code refractor. Currently, there no longer appears to be any preemptive method of querying the cost of a life.

Staartvin commented 4 years ago

I removed this command on purpose. Instead, when a player tries to buy a life, he'll get notified how much their balance needs to increase to be able to purchase it.

BlueMond commented 4 years ago

But what if the price of a life is steep, they do have enough, and only figure out the cost of it after they make the purchase?

Edit: Maybe we could integrate it into the buy command if they don't include any arguments?

Staartvin commented 4 years ago

Integrating would be nice, yes. Perhaps a good way for you to find out how commands work :P

BlueMond commented 4 years ago

Added cost check to buy command without arguments