dragonfly icon indicating copy to clipboard operation
dragonfly copied to clipboard

add ability to enchant item with incompatible enchantments

Open FDUTCH opened this issue 3 months ago • 8 comments

I think it is better to have this ability

FDUTCH avatar Sep 18 '25 09:09 FDUTCH

I think it is better to have this ability Do you have an actual use case for this feature? Could you give an example?

I'm not going to argue, I know it's pretty pointless, but I just need it

FDUTCH avatar Sep 18 '25 10:09 FDUTCH

we do not have any need to restrict the developer from making items with incompatible enchantments anyway

Dasciam avatar Sep 18 '25 10:09 Dasciam

I think that's fair enough, but I do feel like this method deserves a better name. Currently it sounds like every enchantment you provided has to be incompatible. What do you think of something like WithIncompatibleEnchantmentsAllowed() or something similar?

TwistedAsylumMC avatar Sep 18 '25 10:09 TwistedAsylumMC

I think that's fair enough, but I do feel like this method deserves a better name. Currently it sounds like every enchantment you provided has to be incompatible. What do you think of something like WithIncompatibleEnchantmentsAllowed() or something similar?

maybe, I'm not too good at naming

FDUTCH avatar Sep 18 '25 10:09 FDUTCH

or have the WithEnchantments method be WithEnchantments(ignoreIncompatibles bool, enchantments Enchantment...). You would break the original usage for little gain tho...

Flonja avatar Sep 18 '25 15:09 Flonja

or have the WithEnchantments method be WithEnchantments(ignoreIncompatibles bool, enchantments Enchantment...). You would break the original usage for little gain tho...

I think it's better to create a separate method for this

FDUTCH avatar Sep 18 '25 15:09 FDUTCH

I guess it is ready to be merged

FDUTCH avatar Sep 18 '25 16:09 FDUTCH

maybe merge? :thinking:

FDUTCH avatar Nov 16 '25 14:11 FDUTCH