EngineHub / EngineHub/CraftBook

Call InventoryMoveItemEvent

Open
#1,235 2 comments 0 reactions 0 assignees View on GitHub
status:accepted type:feature-request version:five
Dominant language
Java
Stars
320
Forks
166
Avg merge
6d 21h
Merged PRs (30d)
2

Description

**Is your feature request related to a problem? Please describe.**
If a Minecart gets put into a chest/dispenser there is no InventoryMoveItemEvent called, plugins like Chests++ rely on this to synchronize Chestlinks(virtual Inventorys)

**Describe the solution you'd like**
Move the item from an virtual hopper into the Inventory using the InventoryMoveItemEvent

**Describe alternatives you've considered**
As far as I know my first solution is hacky so instead there could be a small class with a onMove method that can be extended by other Plugins.
**Additional context**
Solution 1 would be the best for compatibility because the other Plugins dont need to be updated. And no solution would force the plugins to check every inventory for new Items and produce unnecessary CPU load.

I hope you understand my point and look into this soon.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.