Lembas-Modding-Team / pvp-mode

A Minecraft mod offering various options to manage availability for and ability to PvP.
GNU Lesser General Public License v3.0
5 stars 3 forks source link

No partial inv loss on /suicide from deathcraft #335

Open DwarfyAssassin opened 5 years ago

DwarfyAssassin commented 5 years ago

So when players use /suicide from deathcraft they still lose items. Could there be a config to disable this if possible?

CraftedMods commented 5 years ago

I'll verify it tomorrow. If I can reproduce it, we'll add a fix to 2.0.0-BETA.2.

CraftedMods commented 5 years ago

Ah, it's not really a bug. It would allow players to bypass the partial inventory loss for PvP and PvE, though #331 would solve the PvP death problem. Are you really sure that that would make sence? We'll add a configuration option if you really want the feature (though it'll be disabled by default), but I don't really see the gain it would have. It just makes bypassing certain things more easy.

AlteOgre commented 5 years ago

Dwary, do you really, really want this? The /suicide option is in fact a mechanic by-pass no server owner would wish to have nor encourage to be used imo. When putting in effort to encourage its use, it will only be used more often, making the effort coding this configurable compatibility feature a waste of effort imo.

DwarfyAssassin commented 5 years ago

From what I have noticed is that most of the time /suicide is used to exit a place when you're stuck and can't use any of the other teleport/fast travel options. I know that it could be used as a way to cheat the system but I have it blocked for PvP and if they want to use it for PvE they just ruin their own fun.

AlteOgre commented 5 years ago

Sounds okay. For this purpose a general configurable compatibility option regarding the use of /suicide independent of which plugin or mod it triggers, could be created allowing server owner to by-pass partial inventory loss upon 'death' through use of that command.