Harvest Scythes

Harvest Scythes

7M Downloads

Reason for using mixin instead of overriding use or useOnBlock?

SilverAndro opened this issue ยท 5 comments

commented

Hey there, I was recently asked to add support for your mod (RPGStats#29) and as I went to look for where I could mixin or listen for your item being used, I noticed that your injecting at Item.use, is there any reason for this?

It makes compatibility harder (but not impossible) so I was wondering if there was a reason behind it, as opposed to simply putting your behavior in your already existing item class

commented

Ah, thanks for mentioning this. The scythe tools were part of my first (unreleased) mod, so I just moved everything from there without changing too much.
I'm leaving some parts of the mixin, because I want to check for NBT data, so it's compatible with Smithee, until I think of something better.
I will push a new version later tonight. I just gotta fix some other things.

commented

Alright, thanks for the explanation!

commented

I pushed the update on git and uploaded a build on CF. Hope this helps :)

commented

having some issues with jitpack but it def looks far easier (Looks like ill only need 2 files instead of 5-7 :p), thanks much!

commented

I recommend CurseMaven. That's what I use personally.