Botania (Fabric/Quilt)

Botania (Fabric/Quilt)

5M Downloads

Terra Shatterer doesn't mine Astral Sorcery marble in multi-mining mode

351n opened this issue ยท 5 comments

commented

Version Information

Forge version: 31.2.42
Botania version: Botania-r1.15-388
MC version: 1.15.2
Modpack: Valhelsia: Origins - 1.1.2
Multiplayer: Yes

Further Information

Link to video with bug: https://streamable.com/o5xfoa

Steps to reproduce:

  1. Get Astral Sorcery marble
  2. Get Terra Shatterer
  3. Enable multi mining
  4. Mine marble

What I expected to happen:
I expected marble to be mined just like stone.

What happened instead:
Single block of marble was mined instead of column.

commented

For some reason AS doesn't define the material of marble as rock (or any of these:) https://github.com/Vazkii/Botania/blob/dad99557eb53830ccef04ba248522a2380922342/src/main/java/vazkii/botania/common/item/equipment/tool/terrasteel/ItemTerraPick.java#L61-L63

Instead, it has its own material defined, and the terra shatterer doesn't have that on the list of materials to use.

commented

This issue is stale because it has been open for a while with no activity. Remove stale label or comment or this will be closed soon.

commented

Perhaps check for material is in list || tool canHarvestBlock?
That way if the block's material isn't in the list, but the tool can still harvest the block, then it'll work okay.

commented

I think b7ebacb fixes this? @Hubry

commented

Nope, the terra shatterer has its own list of materials that I haven't touched.