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

[Bug]: Biome dictionary tags are PARTIALLY case-sensitive #350

Open
1 task done
CombatZAK opened this issue Jul 28, 2022 · 0 comments
Open
1 task done

[Bug]: Biome dictionary tags are PARTIALLY case-sensitive #350

CombatZAK opened this issue Jul 28, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@CombatZAK
Copy link

Bug Description

It seems that using biome dictionary tags are case-insensitive when building a list of biomes where a bee might spawn to display in JEI or the beepedia, but case-sensitive when determining where bees can actually spawn.

For example, if using a tag like this for a custom bee:

"biomeWhitelist": "tag:forest",

then bees will spawn as expected in both vanilla Minecraft biomes, as well as biomes added by additional mods which use the dictionary forest tag.

However, a tag formatted like this:

"biomeWhitelist": "TAG:FOREST",

Then, while the list of biomes is correct in JEI, bees will only spawn in vanilla biomes, and they will never spawn in custom biomes which use the dictionary tag. Biomes O' Plenty was used to verify this behavior - I am uncertain if it is a specific bad interaction with this mod or a general case problem.

How to Reproduce?

  1. Create a custom be that uses biome dictionary tags for biomeWhitelist

  2. Set the dictionary tags to TAG:FOREST (all caps)

  3. Set the spawnWeight value to an arbitrarily high value (ex, 1000)

  4. Add custom biomes to the forest.json biome dictionary from another mod (e.g, biomes o' plenty)

  5. Load minecraft and observe valid bee biomes in JEI

  6. Search custom biome(s) for custom bee

  7. Find none

  8. Change biomeWhitelist value to tag:forest

  9. Create new world

  10. Search custom biomes for custom bee

  11. Find bee

Expected Behavior

Case sensitivity rules should be consistent between valid biomes displayed in JEI/beepedia and actual spawn rules.

Version

0.10.7

Mod Loader Version

Forge 1.16.5 - 36.2.34

Logs or additional context

This behavior was found by observation only, and without attaching a debugger or looking at code. I can do a deep analysis on the 1.16.5 branch to confirm if you would find it helpful.

Acknowledgement

  • I agree that I read the terms at the top of the page.
@CombatZAK CombatZAK added the bug Something isn't working label Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant