I thought inventories were working

Socratic_Phoenix · November 19, 2015, 12:28am

I had thought that inventories were partially implemented, yet with the lasted version of sponge, and forge, I get the following AbstractMethodError. Is the implementation just not done yet, or am I just doing it wrong?

gist.github.com

https://gist.github.com/SocraticPhoenix/9220fb226713f285c919

stacktrace.txt

Could not pass DisplaceEntityEvent$Move$TargetPlayer$Impl to SpongePlugin:Split Inventories{unknown}
java.lang.AbstractMethodError: Method net/minecraft/entity/player/EntityPlayerMP.getInventory()Lorg/spongepowered/api/item/inventory/type/CarriedInventory; is abstract
	at net.minecraft.entity.player.EntityPlayerMP.getInventory(EntityPlayerMP.java) ~[qw.class:?]
	at com.gmail.socraticphoenix.sponge.splitinventories.SplitInventories.exactInventorySave(SplitInventories.java:156) ~[SplitInventories.class:?]
	at com.gmail.socraticphoenix.sponge.splitinventories.SplitInventories.saveInventory(SplitInventories.java:141) ~[SplitInventories.class:?]
	at com.gmail.socraticphoenix.sponge.splitinventories.SplitInventories.onWorldChange(SplitInventories.java:117) ~[SplitInventories.class:?]
	at org.spongepowered.common.event.listener.MoveListener_SplitInventories_onWorldChange10.handle(Unknown Source) ~[?:?]
	at org.spongepowered.common.event.RegisteredListener.handle(RegisteredListener.java:92) ~[RegisteredListener.class:1.8-1568-2.1-DEV-827]
	at org.spongepowered.common.event.SpongeEventManager.post(SpongeEventManager.java:242) [SpongeEventManager.class:1.8-1568-2.1-DEV-827]
	at org.spongepowered.mod.event.SpongeModEventManager.post(SpongeModEventManager.java:269) [SpongeModEventManager.class:1.8-1568-2.1-DEV-827]

This file has been truncated. show original

clienthax · November 19, 2015, 12:29am

No they are not implemented yet.
you will have to use nms for the time being

gabizou · November 19, 2015, 12:29am

The Inventory API is not implemented, but the inventory events are implemented.

Socratic_Phoenix · November 19, 2015, 12:31am

@gabizou
Oh, ok. Thanks… I guess my plugin competition submission will have to wait.
Any ETA for the API? If I’m not mistaken its @mumfrey whoes working on it, right?

gabizou · November 19, 2015, 12:32am

Correct, he’s working on it, it’s not the easiest thing to implement sanely.

Socratic_Phoenix · November 19, 2015, 12:34am

Well ok. Thanks for the info.

clienthax · November 19, 2015, 12:40am

Theres no reason why you cant use base nms to work with inventory.

Socratic_Phoenix · November 19, 2015, 12:55am

@clienthax
I have a reason: its evil

Rather then skirt the API I think I can afford to wait. Thanks for the suggestion though.