AgriCraft

AgriCraft

30M Downloads

Cannot affect number of drops a plant gives in config [a17]

shadowbolt79 opened this issue ยท 9 comments

commented

I was fixing mystical agriculture plants in the config files. I got everything almost fixed, but the 5 tiers of inferium plants were totally messed up.

There seed items were not pointing to the correct seeds, tier 5 was named wrong so it wasn't even pulling the right kind of seed texture. Renamed the seeds and plants, and of course, the widely known spawning of plants in crop sticks. Fixed all of that.

However, the different tiers of inferium plants are supposed to drop different numbers of inferium essence. A 1,1,1 should drop their tier in essences. However, all plants, despite changing both min and max on higher tiers, drop the standard 1-5 number of drops. Attached is my config file for t5 inferium plants in case I am doing something wrong:

tier5_inferium_plant.json

commented

Just as a heads up all the fixes that you made are on the current agriplants repo, and if I had to guess will be in the next release. As for the different tiers of inferium plants I have not tested changing the min or max product output. Will take a look at mine when I get home.

commented

@TokkenGmr Typo still exists in your tier 5 inferium plants in two places. You added an extra n.

9: "item": "mysticalagriculture:tier5_infernium_seeds",
60: "seed_texture": "mysticalagriculture:items/infernium_seeds",

Which makes sense. That's how I naturally want to spell it, lol.

commented

I did not make the JSON files for these seeds but you are correct it is an easy mistake to make, also that typo is fixed in the current agriplants repo it was corrected 4 days ago

commented

I also noticed the unchangeable amount of dropped essences today when trying to adjust the inferium seeds to act like when planted normally. Actually after changing the configs, I only got one essence at a time. With the default values, I got one to 5 essences at a time.

commented

@TokkenGmr Ahh,right. But those two typos still exist in the current repo: https://github.com/TokkenGmr/AgriPlants/blob/master/mod_mysticalagriculture/plants/tier5_inferium_plant.json
The other things seem to have been fixed. It is just the t5 plants that have the issue right now.

commented

@shadowbolt79 Have a look right here is that not the correct information? Also you are referencing my agriplants repo which does not have the fix as I have not updated my repo. Also if you check here it will take you to the correct repo. That same fix also added the crops from mystical agriculture additions.

@Meduris Thank you for taking the time do do those updates

commented

@TokkenGmr Ah, ok. That's where my confusion was.

commented

This looks like it has been resolved.

commented

@RlonRyan when was this resolved? There is still a bug in which the config can't affect number of drops by a plant