cc-restitched icon indicating copy to clipboard operation
cc-restitched copied to clipboard

Inventory peripherals don't support transfer api

Open MattiDragon opened this issue 2 years ago • 7 comments

If you try to interact with any block that uses the fabric transfer api using the inventory peripheral it simply won't be detected. Many mods have started exclusively using this api for their inventories and such compatibility will only get worse over time if this isn't fixed. I don't see logs being useful and for reference I'm using minecraft 1.18.2 with cc-restitched 1.100.5.

MattiDragon avatar Apr 26 '22 15:04 MattiDragon

Will look into it, been a while since I personally took a look at the code.

As always, PR's are always welcome

Merith-TK avatar Apr 26 '22 19:04 Merith-TK

Can you give an example of some blocks that you think should work but don't? It would be helpful.

toad-dev avatar Apr 26 '22 23:04 toad-dev

You can test it on many mods, but an easy one would be my drawers mod. Hoppers can interact with them due to a fapi patch (and so can other transfer mods), but cc can't.

MattiDragon avatar Apr 27 '22 06:04 MattiDragon

This seems to also affect Create belts, but that might be a separate issue?

leo60228 avatar Jul 06 '22 16:07 leo60228

try with this mod https://www.curseforge.com/minecraft/mc-mods/unlimitedperipheralworks

this is currently mod that adds support for Transfer API (functions kinda like the Fluid API according to documentation)

unfortunately we cannot properly add support for the Transfer API as of this moment as it will require basically an complete rewrite of the inventory code to my knowlede

Merith-TK avatar Jul 06 '22 17:07 Merith-TK

Can you give an example of some blocks that you think should work but don't? It would be helpful.

Some other examples of this are the Modern Industrialization Barrels & Machines, and the DankStorage Danks.

You can test this on AOF6. None of them will connect to the Modem.

luiz00martins avatar Mar 06 '23 13:03 luiz00martins

Transfer api recently added an optional slot system. Would it be possible to support that?

MattiDragon avatar Mar 06 '23 15:03 MattiDragon