Call InventoryMoveItemEvent

Open
#1,235 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
java
Domain
game-dev

Research direction

Start by locating the virtual hopper inventory-transfer path and any existing InventoryMoveItemEvent handling. Done means moving an item from a virtual hopper into a chest or dispenser also fires InventoryMoveItemEvent, while preserving compatibility for plugins such as Chests++.

Written by the indexing model from the issue text.

Description

status:accepted type:feature-request version:five

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.

Dominant language
Java
Stars
320
Forks
165
Avg merge
6d 21h
Merged PRs (30d)
2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from EngineHub/CraftBook

All issues in EngineHub/CraftBook

Similar issues

More Java issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.