MarkusBordihn / BOs-Easy-NPC

Create easily NPC for your world or for your mod.
Other
20 stars 8 forks source link

[Feature Request] Bring back easy_npc-common config #368

Closed Yimi-t closed 6 days ago

Yimi-t commented 3 weeks ago

The newer versions of EasyNPC don't seem to have a type of toml. config that can prevent access to specific NPC options to different groups of players of varying permissions.

I run a server with EasyNPC version 4.5.1 on 1.20.1 Forge and I'm eager to update to the newest version to get rid of some of the bugs plagued by this older version, and also try out the new features and mod compatibilities. I allow my survival players to purchase NPCs spawn eggs, along with the wand, for world building and RP reasons, so naturally I disable access to Trading and Attribute changes to NPCs for non-OP players (to prevent trade dupes and the creation of broken NPCs with 999 health).

I'm requesting for the return of this feature or the implementation of something similar. I have tried applying the easy_npc-common.toml file from version 4.5.1 onto newer versions, but to no avail.

MarkusBordihn commented 2 weeks ago

Thank you for your feedback! To clarify, would you prefer the ability to fine-tune each individual option, or would you like a general “safe mode” for survival that automatically disables any options that could potentially be exploited for duplication or other unfair advantages?

The recent updates have introduced many new features, some of which could lead to unfair advantages in survival mode. However, it’s important to note that it’s nearly impossible to block all potential exploits completely. Players often find clever workarounds and typically don’t report them for fixes.

Yimi-t commented 2 weeks ago

Hi Markus! Thanks for the reply. I'd be happy with either option, though a 'safe mode' would seem more suitable if it was based on your judgement on what options should be disabled in Survival. If it isn't too much work, I would still like the ability to fine-tune each individual option, as all I really want is a config that disables Trading and Attribute options for non-OP players, regardless if they are in survival or creative. I understand that we're not aware of all the potential exploits that can be used in survival, but I'm willing to just handle any exploit issues that I encounter it if it means I can upgrade to newer versions. Thanks for the hard work :)