Remove $blockClicked from Block::place()

[dktapps]

Description

This parameter:

• is used in very few places, which can all be migrated to use blockReplace
• is a giant footgun (usually blockReplace->getSide() should be used instead - doing otherwise causes bugs when the resulting facing of the block doesn't match the face clicked)

This is, of course, BC breaking.

The only irreplaceable use case for it is to detect when the block replaced is the block clicked (e.g. when clicking on grass). However, this case is currently buggy anyway due to usage of dodgy Facing values (try placing a bell next to a wall by clicking on the side of grass), and the signature needs to be changed to avoid passing dodgy values into place() anyway.

Alternative methods