[1.15.2] Sack of Holding can pick up reinforced blocks
Quu1337 opened this issue ยท 5 comments
Cyclic Sack of Holding can pick up reinforced blocks, making reinforcing blocks bit useless.
Is it possible to disable that, or is it do i just need to wait for cyclic to hopefully fix it?
1.15.2
forge 31.1.14
securitycraft 1.8.16
I would assume that the Cyclic dev needs to fix it on his end, but depending on how the item works, we might be able to do something against that as well. What does the sack of holding do?
You explained it well enough for me to understand what it does. I'll take a look at what I can do.
Not 100% sure (havent checked the code, not good enough to do it anyways) but i think it can pick up ANY block/entity with NBT-tags. cant pick vanilla blocks, but can chests and furnaces and stuff.
Allows you to pick up eg. reinforced oak planks, gives you Sack of Holding with "reinforced oak planks" "enchant" or whatever, and allows you to place them anywhere you want.
Sorry, really bad at explaining stuff i dont understand fully.
Hey dev of cyclic here.
Should i check if its "unbreakable", or whats the best way to check "Can player harvest this"
if(tileEntity instanceof IOwnable && ((IOwnable)tileEntity).getOwner().isOwner(playerWhoTriesToUse))
. Only the owner of the block should be able to pick it up with your item.
IOwnable is an interface that all our tile entities implement which defines that a tile entity is ownable. More info here: https://github.com/Geforce132/SecurityCraft/blob/1.15.x/src/main/java/net/geforcemods/securitycraft/api/IOwnable.java and here https://github.com/Geforce132/SecurityCraft/blob/1.15.x/src/main/java/net/geforcemods/securitycraft/api/Owner.java