BentoBoxWorld / BentoBox

Expandable Minecraft server plugin for island-type games like SkyBlock or AcidIsland.
https://bentobox.world
Eclipse Public License 2.0
334 stars 136 forks source link

Reset health, hunger and XP when resetting the island #958

Closed Aytho closed 5 years ago

Aytho commented 5 years ago

Description

Describe the bug

When you reset an island, the heal and xp and hunger do not reset. The money does not come at a value we would like

Steps to reproduce the behavior

You just have to reset the island, for example if the player took damage or lost hunger, if he reset his island he will always have his stats on his new island.

Screenshots and videos

Capture1 Capture2

Expected behavior

I do not think this is a bug but a configuration line that is not in my configuration file.

Environment

Server

Plugins

Plugin (32) : AntiAFK, ArmorStandTools, asCMD, AutoMessage, BentoBox, ChestCommands, ComAytopia, CraftingStore, CrateReloaded, CrazyAuctions, Essentials, EssentialsChat, EssentialsGeoIP, EssentialsSpawn, MineableSpawners, Multiverse-Core, Multiverse-NetherPortals, Multiverse-Portals, Multiverse-Portals, Parkour, PermissionEx, PlaceHolderAPI, Scoreboard-revision, SimplePortals, TAB, TempFly, uShop, Vault, VoidSpawn, Votifier, VotingPlugin, WorldEdit, WorldGuard

BentoBox setup

BentoBox and Addons
Running PAPER 1.14.4.
BentoBox version: 1.6.0
bentobox.version.database
Mondes de jeu chargés:
skytopia (SkyTopia): Overworld, Nether, End
Addons chargés:
BSkyBlock 1.6.0 (ENABLED)
Challenges 0.8.0-SNAPSHOT-#323 (ENABLED)
Level 1.6.0 (ENABLED)
Limits 0.2.2 (ENABLED)
MagicCobblestoneGenerator 1.5.0.1 (ENABLED)
WelcomeWarps 1.6.1 (ENABLED)
Configuration

Additional context

tastybento commented 5 years ago

For xp and health, that would be a feature request - both of those are not reset, no. For money, it is just a plain release (zero), so that's correct, isn't it?

Aytho commented 5 years ago

Thanks for the reply, For the money, I would like that there is a basic money when we reset the island (2000 for example). And where I can post a feature request for the xp, health and hunger ?

Poslovitch commented 5 years ago

And where I can post a feature request for the xp, health and hunger ?

You already did. This is the one.

For the money, I would like that there is a basic money when we reset the island (2000 for example).

So you want an option (@tastybento I guess it should be gamemode based?) to specify a default amount of money?

@tastybento This might cause issues with our current implementation, with which we do not support world-specific money, whereas Vault can do. We might need to slightly adapt our VaultHook to suit the needs of this feature request.

Aytho commented 5 years ago

Yes exactly ^^ for exemple a line on the configuration file ("starting-money: " for example)

tastybento commented 5 years ago

We had starting money in ASkyBlock. Let me noodle on this.

Poslovitch commented 5 years ago

@tastybento So, I guess we're not doing world-based economy support for now? I'll open a ticket then.

tastybento commented 5 years ago

That's a good point. Unless there is a world-based economy in place, then any setting of money will affect all game modes. Actually, in terms of xp and health, the InvSwitcher addon handles those between worlds. So, resetting them could be fine.

Poslovitch commented 5 years ago

Actually, in terms of xp and health, the InvSwitcher addon handles those between worlds. So, resetting them could be fine.

XP, hunger and health sound okay to me.

Unless there is a world-based economy in place, then any setting of money will affect all game modes.

Vault can support that, but I'll definitely open a ticket for it, as it might require some more thoughts.

Poslovitch commented 5 years ago

Another concern of this issue: it will require to add new methods in the WorldSettings, so it might break gamemode addons snapshots, once again ^^'.

I'll also open another issue for the "starter-money" feature request @tastybento.

tastybento commented 5 years ago

Agreed. 1.8.0 has already crossed that point so it’s okay to do it again. Let’s just time it so that the addons are updated concurrently.

Poslovitch commented 5 years ago

Gamemode Addons will need to implement the newly added WorldSettings methods. @tastybento @BONNe

Poslovitch commented 5 years ago

Implemented.

While working on this, I stumbled upon excessive code duplication. We'll need to address that at some point.