hook or etc for more control over what enchantments are valid for items.

[sigurd4]

here's a link to my modder support post that i made for asking if this was already a thing: http://www.minecraftforge.net/forum/index.php/topic,33398.0.html

what i basically wanted to do was to allow vanilla, armor-specific enchantments to be applied to an item that doesn't extend ItemArmor. this is handled in net.minecraft.enchantment.EnumEnchantmentType.canEnchantItem() and there's nothing i can do about it because there are no hooks, events, interfaces or anything for it.

 

so i suggest:

 

• 
  a function in net.minecraft.item.Item kind of like this, that can be overridden and done whatever with (and would be called instead of EnumEnchantmentType.canEnchantItem()):
   
  

public boolean isEnchantmentValid(Enchantment ench)

{

return ench.type.canEnchantItem(this);

}

[/Code]

 

• 
  or an event that is called after EnumEnchantmentType.canEnchantItem() is, where one can change the value of it. the advantage of this would be that a mod could change the value for items that aren't a part of said mod (vanilla items or from other mods).
   
  

 

• 
  or an interface that would basically work the same as the hook described in the first point, except it's an interface. EnumEnchantmentType.canEnchantItem() would now look like this:
   
  

[Code]
public boolean canEnchantItem(Item p_77557_1_)
{
if(p_77557_1_ instanceof IItemCustomEnchantsSomethingabob)
{
return ((IItemCustomEnchantsSomethingabob)p_77557_1_ ).isEnchantmentValid(this) //here the argument is an EnumEnchantmentType, not an Enchantment cuz why not
}

and then all the other stuff...
[/Code]

 

it would really make my life a whole lot easier.

[LexManos]

Why, overriding other peoples code is just going to cause issues. Why is your armor not a itemarmor? You screw a lot of other things up unrelated to enchantments when you donthat.

[sigurd4]

My item is a bauble, and i want to have full control over what properties it has and doesn't have. There are things that vanilla armor does that i may not want mine to do. Regardless, i figure this would be useful for a whole lot of other stuff where one might want an enchantment to be available for an item that doesn't extend any of the classes that are checked for. It would just be really nice to have control over this kind of thing. Also, i don't see how this would break anything. It should be perfectly capatible with earlier mods, and there are already plenty of similar hooks for other things to allow more control over otherwise unchangable vanilla functionalities.