EMC values for enchantments

[BigXplosion]

I'm wondering if it is possible to take the enchantments into account for the tools? Best way to do this is probably calculating an extra EMC value on top of the original one with the use of the enchantment weight and level. Maybe something like this:

public int getAdditionEnchantmentEMC(ItemStack stack) {
        int result = 0;

        Map<Integer, Integer> enchantments = EnchantmentHelper.getEnchantments(stack);

        if (!enchantments.isEmpty())
        {
            for (Map.Entry<Integer, Integer> entry : enchantments.entrySet())
            {
                Enchantment enchantment = Enchantment.enchantmentsList[entry.getKey()];

                if (enchantment.getWeight() == 0)
                {
                    continue;
                }

                result += 5000 / enchantment.getWeight() * entry.getValue();
            }
        }

        return result;
}

If this would be added I would say to not make the enchanted item learnable (if this isn't already the case)

[MatanShahar]

I don't know if fits the theme of the mod all that well. Enchantments are not exactly matter. Regardless it seems that you have some working knowledge in java, you're more than welcome to make a pull request. It's up to Pahimar to accept it though.

[BigXplosion]

i wanted to ask first before PRing it in

[pahimar]

This would definitely be possible, but not sure about how I feel about it. Currently (because you can set EMC values for items that have NBT data) you could set a value on an enchanted item. However, that is per item. This idea would support EMC values per enchantment.

Open to discussion on this.

[SonOfTheStars]

Great idea! There is defenitly time and effort put into enchants so you should get more out of it AND pay more for creating them

[GeoTerra]

I would be in support of this. Reasons being that, as Bitterholz said, to get enchants (in a vanilla environment anyways) you need to invest the time and effort, and lore-wise, enchants can be considered as additional energy imbued into the item.

[MaPePeR]

ProjectE does this in a way, so I guess it was also part of EE2?

It's a fixed amount of emc for the level of the enchantment, I think.

It caused some problems in the past, though.

[tahg]

As far as I know, no, this was not a part of EE2.

[MaPePeR]

I don't know for sure. It was part of ProjectE since forever and was included in the code MozeIntel wrote.

I have to correct myself: The part that caused the real problems was the "Reduce amount of emc for damaged tools" and not the enchantment bonus.