search

AE2-UEL / Applied-Energistics-2

AE2UEL, a Fork of a Minecraft Mod about Matter, Energy and using them to conquer the world.
https://legacy.curseforge.com/minecraft/mc-mods/ae2-extended-life
Other
75 stars 71 forks source link

fix locking mode in interface #478

Open GlodBlock opened 4 months ago

GlodBlock commented 4 months ago

close https://github.com/AE2-UEL/Applied-Energistics-2/issues/476

zeng-github01 commented 4 months ago

@GlodBlock 你改了流体接口的流体触发器嘛？我给流体也加了一个类似的触发器。用来给隔壁的AE2FC适配的。忘记改的话可能二合一接口要出问题。我怕你把流体接口也改了...

看起来流体接口没有改，还是用了原来的逻辑的样子

Did you make similar changes to the fluid interface? There is an support for AE2FC in Lock Crafting PR. If so, please don't forget to fix it as well.

commit

GlodBlock commented 4 months ago

明天弄

zeng-github01 commented 4 months ago

明天弄

OK，还好管理员还没合并。合并了想再打补丁就有点难了。而且现有的构建Action有点毛病。不知道为什么它的版本号被Forge识别成了无效的。我交了个PR临时修复了一下

顺便现有的修复方案还能再改改。可以不考虑重新调用那个触发器了。接口Tile里面不是还有一个实际功能的Tile类吗 那个类里面有一个OnStackReturnNetwork(IAEItemStack) 的方法。那个是实际的解锁函数。你看看能不能直接call到这个函数。省的重新用那个InventoryChange了。要是Call不到的话TileInterface里面也有一个嵌套的实现。直接Call那个也行，正好那个还是接口方法。不是具体实现。还能解耦。

至于流体接口，也有一个同名的解锁函数。但是在流体接口的内部实现类里面没有实现。FluidChange是直接调用的上层IFluidInterfaceHost的接口方法。毕竟流体接口没需求 只是留个位置给AE2FC