PlayerEvent.HarvestCheck is not BlockPos-aware

[anti344]

Soo, the only thing which calls ForgeEventFactory#doPlayerHarvestCheck (which sends the event) is EntityPlayer#canHarvesBlock and the only thing which calls that is ForgeHooks#canHarvestBlock which has a BlockPos, but does not pass it to the event.

 

Also, notice this forge->mc->forge jump

[LexManos]

Yes, it does a Forge->mc->Forge->mc->Forge->*** jump a lot in Forge, that's how programming works.

The position isnt there to keep compatibility with vanilla and OTHER MODs that call this function without location information.

We're not the only loader out there we still have to maintain compatibility with people who do dumb things.