Mekanism

Mekanism

111M Downloads

[Request] Add Forge ITEM_HANDLER_CAPABILITY support for 1.10 and 1.11

Landmaster opened this issue ยท 7 comments

commented

Peeking at the source code, it seems that the cables (and most other items from this mod) do not support the new Forge ITEM_HANDLER_CAPABILITY. Support for this should be added; otherwise, modders would be forced to use IInventory in order to have generic support for item pipes.

commented

Looks like it has already been added in c12f1c1.

Nice work @aidancbrady!

commented

@aidancbrady Will this be backported to 1.10?

commented

Maybe a nice idea to add to this request: use the ItemCapability to interact with other mods. Currently all tileentities that interact with other blocks use IInventory (for example logistical pipes and the push of machines).
There amount of mods that only support capabilities is increasing. Would be nice to have a better cross mod interaction.

commented

First part done by @aidancbrady at b97b718

commented

Tested latest version (okey, version of 12 hours ago). Tested with Actually Additions crate as that only supports the inventory capability and not the IInventory.
Working:

  • Digital miner output to crate and vanilla chests
  • All processing machines automatic output to crate and vanilla chests
  • Pipes extracting from crate
  • Pipes inserting and extracting from chests

Not working:

  • Pipes inserting into crate

To test:

  • Something like RFTools inventory viewer (forgot the name) to test if Mekanism's capability is exposed correctly
  • Normal operations of non capability providing blocks (vanilla exposes both capability and IInventory)
commented

Should be done!

commented

Working flawless!!