Iron Tanks

Iron Tanks

663k Downloads

Stacking tanks of different types doesn't update side texture

Indemnity83 opened this issue ยท 5 comments

commented

When you stack BuildCraft tanks, the side texture changes so that the bottom line of the upper tanks doesn't show.

This works fine for any of the extended tanks when stacking with themselves, but when you mix any various tanks together the side texture is not updated as expected.

commented

After some brief investigation, this is a problem with the logic in the BuildCraft BlockTank class that's being extended. BuildCraft/BuildCraft#3047 initiated and accepted by the BuildCraft team should resolve this in the next release of BuildCraft. I'm leaving this issue open until the patch is released.

commented

There are tons of things in BuildCraft I or the previous developers hadn't thought of, admittedly. It's a giant codebase where nobody is 100% sure of what is going on everywhere, sadly.

commented

Oh certainly; I'm not trying to take anything away from the BuildCraft code, its wasn't ever designed to be extended like this I'm sure. But thats the beauty of open source ๐Ÿ‘

commented

Yes! Yes it is! Generally, I'm trying to make BuildCraft more extensible as I go.

commented

Pushed in BC 7.1.6.