search

zlainsama / CosmeticArmorReworked

This mod allows you to wear two sets of armor, one for display, one for function.
https://minecraft.curseforge.com/projects/cosmetic-armor-reworked
Other
17 stars 18 forks source link

Duplicate method signature when using forge mappings in dev environment #62

Closed Quarris closed 2 years ago

Quarris commented 2 years ago

In InventoryCosArmor There is a duplicate method signature error when using forge mappings, as IInvetory#getItem(int) from official mappings, maps to IInventory#getStackInSlot(int) which clashes with IItemHandler#getStackInSlot(int) causing a loading error in dev environments that use forge mappings.

I was hoping that you'd be able to remove InventoryCosArmor#getStackInSlot(int) as it looks like its not needed there, which should fix this problem.

Thanks!