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

Base (ingredient) foods from Pam’s HarvestCraft 2 - Crops and Trees have no nutritional values #21

Open
Nevrai opened this issue Aug 8, 2022 · 0 comments

Comments

@Nevrai
Copy link

Nevrai commented Aug 8, 2022

None of the fruits and edible seeds from Pam’s HarvestCraft 2 - Crops and Pam’s HarvestCraft 2 - Trees that don’t have crafting recipes don’t have any nutritional values, even though they do have the proper Forge item tags.

One example is the peach from Pam’s HarvestCraft 2 - Trees, which has no nutritional value despite having the tag #forge:fruits/peach. The same goes for the plum, which has #forge:fruits/plum. The edible seeds are understandable as they don’t have the #forge:fruits tag, but the fruits do. Nutritional Balance’s server config for the world I’m playing does have #forge:fruits listed in fruits_item_list, as it does by default. I’m playing in a single-player world.

The user can fix this manually by adding to the server config. I added #forge:foods/fruits and a few other tags just in case, including #forge:crops/grape, #forge:seeds/grape, #forge:plums, and #forge:peaches. A lot of the Pam’s seeds, even non-grape, non-peach, and non-plum ones, have nutrient values now, which is nice. However, some don’t, such as Pam - Crops’ pineapple and pineapple seed. It seems like these lack the forge#fruits tag. This is an issue on Pam - Crops’ end, though, but when it comes to other fruits as previously mentioned, they already have that tag and yet still had no nutritional balance despite #forge:fruits being in Nutritional Balance’s config.

Additionally, on a very trivial note, the config mistakenly says “List of proteins” above fruit_item_list. This has no effect on gameplay, though.

Versions

  • Minecraft version: 1.18.2
  • Forge version: 40.1.69 (latest for 1.18.2)
  • Nutritional Balance: 3.1.1 (latest for 1.18.2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant