BlakeBr0 / MysticalCustomization

Allows modpack creators to add new content and modify existing content in Mystical Agriculture.
https://blakesmods.com/mystical-customization
MIT License
6 stars 6 forks source link

Can't use mysticalagradditions:6 tier when creating a new crop #27

Closed Nevrai closed 1 year ago

Nevrai commented 1 year ago

Describe the bug

I’ve tried to create custom seeds, but it’s not working. I’ve also tried including the parameters that weren’t necessary, but it still doesn’t work. No seeds are being loaded by Mystical Customization.

{
  "name": "Cyanite",
  "type": "mysticalagriculture:resource",
  "tier": "mysticalagriculture:4",
  "color": "32a8a8",
  "ingredient": {
    "tag": "forge:ingots/cyanite"
  },
  "textures": {
    "flower": "mysticalagriculture:block/flower_ingot",
    "essence": "mysticalagriculture:item/essence_ingot",
    "seeds": "mysticalagriculture:item/mystical_seeds"
  },
  "enabled": true
}

Versions

BlakeBr0 commented 1 year ago

Are there any errors in the logs? Is the file in the correct folder?

Nevrai commented 1 year ago

@BlakeBr0 I figured out the issue with the cyanite seeds: Mystical Agriculture already has cyanite seeds. So now, I just added recipes for them and enabled them in JEI.

However, with my other seeds, which are using Agradditions’ tier 6, they are not working. I’m getting this error: com.google.gson.JsonSyntaxException: Invalid crop tier provided: mysticalagradditions:6

This seems like a bug, as I made sure to use proper spelling and syntax, as well as check the in-game /mysticalcustomization tiers command, yet it’s not recognizing that tier 6 exists.

Here is the JSON file:

{
  "type": "mysticalagriculture:resource",
  "tier": "mysticalagradditions:6",
  "color": "501075",
  "ingredient": {
    "tag": "forge:gems/void"
  }
}

The file is in the correct location (config\mysticalcustomization\crops). Here’s the full error from latest.log.

BlakeBr0 commented 1 year ago

If you were to set it to one of the default tiers then update it to tier 6 in the update file, does that work? Maybe it's a load order issue?

Nevrai commented 1 year ago

Yes, that worked! Wonderful. Thanks for the tip.

Hopefully you can figure out what was causing this, though.

BlakeBr0 commented 1 year ago

Fixed by ce35f444851e9bf167017dab213da98502e5ba87

Nevrai commented 1 year ago

Thank you for the fix. Could this be backported to 1.18.2? Would be wonderful

BlakeBr0 commented 1 year ago

If I do another release for 1.18 I'll be sure to add this fix in.