Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redundant tags for raw storage blocks #919

Closed
KnightMiner opened this issue May 3, 2024 · 1 comment · Fixed by #1030
Closed

Redundant tags for raw storage blocks #919

KnightMiner opened this issue May 3, 2024 · 1 comment · Fixed by #1030
Labels
1.20.6 Targeted at Minecraft 1.20.6 cleanup Change that isn't an enhancement or a bug fix

Comments

@KnightMiner
Copy link
Contributor

Minecraft Version: 1.20.6

NeoForge Version: 20.6.24-beta

Description of issue:

Neo currently has two sets of tags for raw ore blocks: storage_blocks/raw_ (with a block form) and raw_blocks/ (only item form, no block). Both just contain the vanilla raw blocks, and neither has any javadocs explaining the distinction, leading to modders being unsure of which one to use.

I think the storage block tag pretty clearly means it can be converted to and from the raw ore item, just like other storage block tags. However, raw blocks is unclear as we lack examples that are not vanilla.

  • The best answer I could get was that raw blocks are blocks that can drop 1 or more raw ore when mined, however that definition was unsatisfying to me as raw ore blocks do not drop raw ore (they drop themselves) and regular ores drop 1 or more raw ore items (but are not tagged as raw blocks). Plus, raw_blocks/ is exclusively an item tag, talking about block drops with an item tag does not make sense.
  • I was also told the tags exist because they came from Fabric, however Fabric lacked a mirror to the raw storage block tags suggesting the tags potentially should have just been merged with the forge tag for these vanilla block.

Thus, I propose we take one of the following actions:

  • Remove the raw_blocks/ tags as they lack any conventions tied to their usage. Since are not in a breaking change window, that would consist of deprecating for removal until the next breaking changes window.
  • Document exactly what it means to be a raw block, and how that is different from raw storage blocks and ore blocks and potentially make it a block tag as well. Make it clear what guarantees the tag provides and what other tags are required to make the tag meaningful (e.g. do we need to do tag intersection like with ores for sizes?)
@TelepathicGrunt
Copy link
Contributor

FabricMC/fabric#3755

@sciwhiz12 sciwhiz12 added cleanup Change that isn't an enhancement or a bug fix 1.20.6 Targeted at Minecraft 1.20.6 and removed triage Needs triaging and confirmation labels May 3, 2024
TelepathicGrunt added a commit to TelepathicGrunt/NeoForge that referenced this issue May 27, 2024
TelepathicGrunt added a commit to TelepathicGrunt/NeoForge that referenced this issue Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.20.6 Targeted at Minecraft 1.20.6 cleanup Change that isn't an enhancement or a bug fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants