dannydjdk / Nutritional-Balance

A Minecraft mod that enhances the food system to encourage you to eat a balanced diet.
https://www.curseforge.com/minecraft/mc-mods/nutritional-balance
GNU General Public License v3.0
3 stars 5 forks source link

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

Open Nevrai opened 2 years ago

Nevrai commented 2 years ago

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