desht / ModularRouters

A Forge Mod for item routers with pluggable modules
20 stars 22 forks source link

[1.21] [Neoforge] Routers do not recognise Ars Pedestal/Platform inventories #239

Closed ffuentesm closed 4 months ago

ffuentesm commented 4 months ago

Describe the bug

The Ars pedestal is a single inventory display for items. You can place items on it by clicking on it with an item in hand. But when using a sender or puller module, they do not recognize the platform's inventory image

How to reproduce the bug

Try to set the sender module to the Pedestal or use a puller module to extract from the platform.

Expected behavior

It should set the pedestal as a valid inventory

Additional details

ars_nouveau-1.21.0-5.0.2-all modular-routers-13.0.3+mc1.21

Which Minecraft version are you using?

1.21

Crash log

No response

desht commented 4 months ago

Modular Routers just does a standard capability query on the block for an item handler. A quick check with a debugger shows that the pedestal doesn't have a registered item handler, so that's on Ars Nouveau to support, if the mod author wishes to do so.

(If you're wondering why the vanilla hopper works, it's because the pedestal implements Container, so looks like an inventory to vanilla block entities. But mods are under no obligation to support that, and I have no intention of supporting anything other than capability-based item transfer)