Mekanism

Mekanism

111M Downloads

Mekanism axes can't be enchanted with Sharpness

DBotThePony opened this issue · 13 comments

commented

2017-01-04_13 04 49
2017-01-04_13 04 54

commented

@ThatGamerBlue What version do you use? I use latest version on 1.10.2

commented

With working as intended he means that the item is non-enchantable.

commented

You can enchant vanilla axe with Sharpness, but you can't enchant Mekanism axe with Sharpness. For me this is unexpected behaviour

commented

Are we sure they are supposed to be non-enchantable? The config file has an "enchantability" setting that I think defaults to not 0.

In Hermitpack, Obsidian is set to have an enchantability of 40 (which I assume is the default). For comparison, gold tools have an enchantability of 22.

commented

As roboderpy said , you can enchant vanilla axes with sharpness enchant. I thought we could also do this in prior versions of mekanism.. Has it been removed ?

commented

Working as intended. @aidancbrady /close

commented

I stand corrected, Mekanism items are enchantable.

commented
commented

I know that, I initially thought they where not enchantable at all. Currently the axes look like an ItemTool for Minecraft and thus only allows ItemTool enchantments. Somewhere in Minecraft/forge must be a check: if axe, allow sharpness. I'll take a look where it is and try to fix the axe :)

commented

Took a look at how enchantments are dealt with in Forge/Minecraft. Looks like (in 1.10) it's hardcoded to instances of ItemAxe. I only can think of a dirty way to fix it, not a nice and clean one.

commented

This looks better right?
Was easier than expected.

commented
commented

Can be closed when the 1.10 branch is merged to 1.11.