Pick / Inventory events not implemented?

Miwarre · Dec 29th 2016

I understand this is a busy moment, still I have to raise these points.

As far as I can tell, neither the PlayerPickupItemEvent or the PlayerInventoryAddEvent do fire, at least not for the actions I would them expect to. Picking a plant up (tomatoes, broccoli, etc.) or an object laying on the ground does not fire either event.

Is this intended?

As an aside, may I expect the events which where present in the LUA binding (related to vegetation, construction objects, chests, etc.) to be added to the Java API too?

Miwarre · Feb 15th 2017

This seems to be still true.

In addition also the new PlayerObjectInteractionEvent did not fire for the few examples I tried (mostly crafting stations).

Any plan about them? Thanks!

red51 · Feb 15th 2017

The Inventory events will probably be implemented after the next update. About the PlayerObjectInteractionEvent: Make sure you call Player.setListenForObjectInteractions(true) first, otherwise this event will not be triggered (if it's set to true, it will listen to interactions with all kind of objects, even objects which are not interactable by default)

Miwarre · Feb 15th 2017

Correct! By using Player.setListenForObjectInteractions(true), the event is correctly reported. My fault!

It is not cancellable, though; so, in the area protection perspective, anybody will be able, for instance, to open any door, regardless of areas...

red51 · Feb 15th 2017

Yeah, this event it's cancellable unfortunately. In Lua, door access was restricted by cancelling the "PlayerObjectStatusChange" event, however, this event does not exist in the plugin API yet.
Making the object interactions cancellable is a little bit tricky... this requires the server to confirm every player-object interaction (even accessing a workbench or sitting down on a chair). I'm afraid that this may have a negative impact on the responsiveness of the gui (e.g. accessing a workbench in multiplayer, especially when having a high ping)... I will do some tests, maybe the lag wouldn't be so bad at all, so eventually the interaction event becomes cancellable in the future

Participate now!