XP From Harvest

XP From Harvest

16M Downloads

Quark Compat/Harvest Tool Check

SaltSouls opened this issue ยท 2 comments

commented

The mod doesn't seem to work with quarks implementation of simple harvest. This is a little bit of an issue as this mods implementation of it doesn't check for the tool currently being used to harvest the crop and adjusting the loot tables accordingly. This probably wouldn't be an issue with most other farming mods, but causes issues with farmers delight which adds alternate loot tables to both wheat and rice if the player is holding a knife.

I would also like to suggest expanding the range of harvestable blocks and including blocks that already use the right click harvest mechanics to the xp pool. These would be things such as beehives/nests, berry bushes and so on. What I mean on expanding the range is adding things like custom replant rules that can be set for specific blocks such as farmers respite's tea bush. Blocks like these aren't able to be replanted using the normal method and would need to have some way of specifying the block to replant when harvesting.
Example: "Block{farmersrespite:tea_bush}[age=3],Replant{farmersrespite:small_tea_bush}"

commented

Of course it does not support simple harvest from Quark and it will never be for any mod, because those mods never send any messages about simple harvest being done, thus there's no way to detect that harvest happened and give player XP. That's why I had to reinvent the wheel and add such mechanics into the mod.

As for second idea it's actually not bad IMO, I'll think about implementing it.

commented

I'll look towards for the RightClick option too! Farmer's Delights Tomatoes does not give XP at the moment because of this.