Eredrim / pvparena

Controlled violence since 2011
https://www.spigotmc.org/resources/pvp-arena.16584/
GNU General Public License v3.0
23 stars 36 forks source link

Prevent player from item drop #173

Open tsingyayin opened 1 month ago

tsingyayin commented 1 month ago

Now the plugin allows players in the game to drop items, which can be used to obtain limited items in the game. For example, they can use many means to transfer items to players in the field who have not joined the game to transfer illegal items. Can you add a configuration option to prevent players from throwing items from the inventory at any time?

tsingyayin commented 1 month ago

This is very important, which fundamentally shakes the security of the arena

tsingyayin commented 1 month ago

I tried to fix it. At present, this minor change is in line with my own expectations. I'm not sure if I need to change something else. #174

Eredrim commented 1 month ago

Hello, These mechanics are already implemented in the plugin with region protections. In your arena region, you can enable drop protection to prevent (or at the opposite allow) item dropping. https://github.com/Eredrim/pvparena/blob/master/doc/regions.md#region-protection

tsingyayin commented 1 month ago

However, players can leave the protected area in various ways. I know that there are flags in the protection that can restrict players from leaving, but at present, the players on my server are very good at sidestepping, and there are various ways to try to avoid these judgments.

Personally, I think it is not impossible to add a config option in the file that directly prevents all items from falling in the FIGHT state.

In a word, my PR has implemented this small part. If you like, you can merge it. It is regarded as adding a new feature.

Eredrim commented 1 month ago

PVPArena is designed to create PVP/minigames arenas, not PVP worlds. Following this principle, players should not be able to leave the battle area. In addition, leaving the battlefield area is prevent by default in the plugin, so there is no need for specific configuration.

I agree with you, adding a config option is not impossible but it's not the topic. I have to keep consistency in the way that the plugin works and, to date, most of settings about user interactions with their own inventory are in region config.