servertap icon indicating copy to clipboard operation
servertap copied to clipboard

Incorrect item id in Inventory Route when using Forge + Spigot

Open ATechAdventurer opened this issue 5 years ago • 2 comments

When using any forge + (bukkit/spigot/paper) environment (MCPC+, Cauldron, CatServer) the /player inventory route returns all items with minecraft: this is not correct.

The resolution to this problem requires a way to determine if an item is provided by a mod that is outside of vanilla Minecraft.

This is not a huge issue and will likely not be fixed because servertap does not support Forge server environments. But if a solution is possible it might be valuable to address this issue.

This is mostly just here for reference purposes.

ATechAdventurer avatar Aug 24 '20 22:08 ATechAdventurer

I do think this can be fixed by checking the 1st part of the colon separated name, and removing it if it equals minecraft. I will try to make a PR.

ItsViolet avatar Sep 25 '22 13:09 ItsViolet

I don't believe it should be removed, as with Forge, items have different "providers" and in game it is already being referenced as minecraft:.

zoeleu avatar Sep 29 '22 13:09 zoeleu