[1.13.2] com.google.gson.JsonSyntaxException: Expected item to be a JsonObject, was "str...ld"

[PanSzelescik]

[10mar2019 18:39:56.581] [Server thread/ERROR] [net.minecraftforge.common.crafting.CraftingHelper/CRAFTHELPER]: Parsing error loading constants strangetrees:recipes/_constants.json
com.google.gson.JsonSyntaxException: Expected item to be a JsonObject, was "str...ld"
    at net.minecraft.util.JsonUtils.getJsonObject(JsonUtils.java:247) ~[?:?]
    at net.minecraft.util.JsonUtils.getJsonObject(JsonUtils.java:253) ~[?:?]

What here is wrong?

_constants.json

Edited March 10, 2019 by PanSzelescik

[Cadiboo]

Check your file for weird characters and make sure you have the correct encoding

[LexManos]

Protip: All of your 'constants' are bad. The point of the constants json was to prevent massive duplication. If you're just redirecting to a single line for a tag. Then you're just wasting load time by making it a constant.

You should convert your 'emerald's to tags and skip _constants.json entirely.

[PanSzelescik]

11 hours ago, LexManos said:

Protip: All of your 'constants' are bad. The point of the constants json was to prevent massive duplication. If you're just redirecting to a single line for a tag. Then you're just wasting load time by making it a constant.

You should convert your 'emerald's to tags and skip _constants.json entirely.

Okay I converted, but the same error is still there
 

Spoiler

[11mar2019 15:19:32.988] [Server thread/ERROR] [net.minecraftforge.common.crafting.CraftingHelper/CRAFTHELPER]: Parsing error loading constants strangetrees:recipes/_constants.json
com.google.gson.JsonSyntaxException: Expected item to be a JsonObject, was "str...ld"
    at net.minecraft.util.JsonUtils.getJsonObject(JsonUtils.java:247) ~[?:?]
    at net.minecraft.util.JsonUtils.getJsonObject(JsonUtils.java:253) ~[?:?]
    at net.minecraftforge.common.crafting.CraftingHelper.reloadConstants(CraftingHelper.java:339) [?:?]
    at net.minecraftforge.common.extensions.ForgeRecipeManager.onResourceManagerReload(ForgeRecipeManager.java:48) [?:?]
    at net.minecraft.item.crafting.RecipeManager.onResourceManagerReload(RecipeManager.java:39) [?:?]
    at net.minecraft.resources.SimpleReloadableResourceManager.triggerReloadListeners(SimpleReloadableResourceManager.java:125) [?:?]
    at net.minecraft.resources.SimpleReloadableResourceManager.reload(SimpleReloadableResourceManager.java:101) [?:?]
    at net.minecraft.server.MinecraftServer.loadDataPacks(MinecraftServer.java:1518) [?:?]
    at net.minecraft.server.MinecraftServer.loadDataPacks(MinecraftServer.java:400) [?:?]
    at net.minecraft.server.integrated.IntegratedServer.loadAllWorlds(IntegratedServer.java:76) [?:?]
    at net.minecraft.server.integrated.IntegratedServer.init(IntegratedServer.java:99) [?:?]
    at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:595) [?:?]
    at java.lang.Thread.run(Thread.java:748) [?:1.8.0_201]

Updated _constants.json

[LexManos]

Again your issue is that you're being stupid and using a constant when a standard item/tag would do:

    {
        "name": "strangetrees:super_infusion_emerald",
        "item": "strangetrees:super_infusion_emerald"
    },

                    else if (json.has("item"))
                        tmp.put(name, new StackList(Lists.newArrayList(getItemStack(JsonUtils.getJsonObject(json, "item"), true))));

Follow the correct spec for _constants if you're gunna use them.

 

Basically, _constants was created as a precursor to tags. 

If you're just using ONE item use the item directly.

If you have basic list items that have no meta/nbt. Then use a tag.

ONLY use _constants if you're using custom ingredients/nbt.