Reliquary Reincarnations

Reliquary Reincarnations

51M Downloads

Shears of winter can not be put in ender io farming station

Peach774 opened this issue ยท 8 comments

commented

As title says, i expect them to go in and essentialyl, function as regular shears but without durability.

commented

They wouldn't go into the ender io farming station, because the ender io farming station will only take modded shears that extend from ItemShears, and the Shears of Winter extend ItemBase, not ItemShears, so they wouldn't be able to be put in the ender io farming station.

commented

Shouldn't they extend itemShears since they are shears?

commented

https://github.com/P3pp3rF1y/Reliquary/blob/1.10.2/src/main/java/xreliquary/items/ItemShearsOfWinter.java#L39 , As you can see by that line of code they don't extend ItemShears, but extends ItemBase, which is Reliquary's base item.

commented

That doesn't explain anything to me, I don't speak code well. I'm moreso curious as to why they shouldn't extend ItemShears.

commented

They could be easily switched to were they should extends ItemShears, and I almost created a pull request the moment I saw your issue to change it to extends ItemShears, but I didn't know if the underlying code in ItemBase is important enough to copy-paste into the newly extended file.

commented

Sorry that I didn't comment on this for quite some time. I actually wasn't sure if I want shears of winter to be usable in farming station. But now I am pretty sure I do.

Converting them to extend ItemShears will have a benefit of them being usable in pedestal as well without additional changes.

I will include this change in one of the next releases.

commented

Awesome, thank you so much!

commented

This change is in the latest version of Reliquary that I just uploaded to CF.