Botania (Fabric/Quilt)

Botania (Fabric/Quilt)

6M Downloads

Natura Pylons do not work for mana enchanter

deleteitmatt opened this issue ยท 6 comments

commented

Mod Loader

Forge

Minecraft Version

1.18.2

Botania version

1.18.2-435

Modloader version

Forge: 0.212.4.7222

Modpack info

https://www.curseforge.com/minecraft/modpacks/all-the-mods-7

The latest.log file

https://pastebin.com/jjn1nh8b

Issue description

Tried making the mana enchanter using natura pylons, i saw online this was possible but the multiblock never worked. ended up making more mana pylons and it works like a charm. is this intended?

Steps to reproduce

  1. build out multiblock enchanter
  2. use natura pylons instead of mana pylons
  3. no feedback, multiblock doesn't create

Other information

No response

commented

https://ftb.fandom.com/wiki/Mana_Enchanter
FTB wiki says you can use pylons other than the mana ones!

if that's not the way it is intended to work that's fine. i think i just assumed that they would provide a higher enchanting bonus? after actually using the system a couple of times i think i realize that it isn't actually necessary, it just might be worth specifying it in the lexicon

commented

FTB wiki can be incorrect.

They don't provide any higher enchanting power than mana pylons, no.

commented

From what I can tell, the overlay never supported non-mana pylons.

commented

From an older branch...

@Override
public float getEnchantPowerBonus(World world, BlockPos pos) {
IBlockState state = world.getBlockState(pos);
if (state.getBlock() != this || state.getValue(BotaniaStateProps.PYLON_VARIANT) == PylonVariant.MANA) {
return 8;
} else {
return 15;
}
}

commented

that's for the enchanting table, I'm pretty sure OP is talking about the Mana Enchanter seeing as this issue revolves around it.

commented

i was talking about the Mana Enchanter, but it isn't an issue if it is only intended to work with mana pylons. it seems like that's the case!