Problems interacting strangely with mods such as wooden hopper

[citric-sulfur]

debug.log
latest.log

It is not composted and is stored.
Destroying the composter in this state does not drop the seeds.

Must use wooden hopper again to eject.

All items will be stored.

The same problem occurs with similar blocks in different mods.

[yurisuika]

Ah, okay it didn't occur to me when you said wooden hopper that you were referring to a mod. For some reason the wooden part of the composter and the hopper amalgamated in my mind and it just went through me.

So, on Forge for some reason it is all screwy and I had to disable insertHook of Forge's VanillaInventoryCodeHooks from running because this was happening with the vanilla hopper. I cannot figure out what exactly goes wrong with it, so just stopping this if statement from returning true makes it work with the vanilla hopper. I don't know if it has any side effects, but it was enough to get it going.

Unless I can figure out what can actually be fixed with that, you will have to use a version of the mod prior to 1.2.0 if you want it to work with this mod, This is only a problem with the Forge version, the Fabric one did not need this workaround.

[yurisuika]

Good news! I finally figured out the issue. It had to use Forge's ItemStackHandler for an inventory. Will probably have the update out tomorrow.

[yurisuika]

Well it turns out I was wrong, it actually didn't fix it because I was testing the Fabric version on Forge. I was dead tired at the time so you can blame it on that haha. But after a bit more research today I figured out the real issue and have confirmed it to now work! We'll see, I might still be able to get a release out tonight.

[yurisuika]

The issue was specifically that the blockentity did not override the isValid method, which is basically the canInsert method without the side parameter. It wasn't till I was looking through another mod that uses mojmaps instead of yarn that this realization came to me, as those methods are called canPlaceItem and canPlaceItemThroughFace there.