Carpet Extra

Carpet Extra

1M Downloads

Some property names are misleading

ehx-v1 opened this issue ยท 2 comments

commented

...due to describing features that are already in the game (but done differently by the property).

To clarify: This is not to discredit the property options as good alternatives. It's just that the name can be confusing to some players (if they don't read the Readme), who might think setting them to false disables the Vanilla option whereas in reality, setting them to true enables a new option.

The affected properties:

  • renewableIce: Both ice itself (by placing water at a high Y level or in a cold biome and letting it freeze) and dense forms of ice (by crafting them from 4 of the respective less dense form) are renewable. Proposed name: betterRenewableIce
  • renewableSand: Sand is technically already renewable through the Wandering Trader. While the option from Carpet Extras is actually viable whereas the Wandering Trader option isn't, it can still be misleading. Proposed name: betterRenewableSand
  • renewableWitherSkeletons: Wither Skeletons spawn repeatedly in Nether Fortresses. This case is unique in that the added option isn't significantly more viable than the Vanilla option (although it still feels like a good and fitting idea). Proposed name: electricWitherSkeletons
commented

The problem is that renaming them would mean that any worlds that have them enabled would get them disabled, likely breaking stuff in them.

There's currently no system to automatically migrate rules to new names.

commented

Well, that just means the issue is dependant on establishing such a system (e.g. an API version setup with backwards compatibility mixins) beforehand. Such a system would probably be helpful either way, as being able to rename properties between versions is a major plus in general. Plus, it's not like this issue is very likely to be high priority, so taking your time really is an option.