Fix/lectern

[ShaneBeee]

• Allow all players to read books in lecterns
• Prevent players without container trust from removing books from/placing books into lectern

References #537

NOTE:
Don't ask me why, maybe a Spigot/Bukkit mistake, but when trying to put a book into a lectern, it calls the BlockPlaceEvent (weird hey?!?)

[Jikoo]

Re: BlockPlaceEvent - I believe it follows the same logic as log stripping, etc. where the block's data is modified even though it isn't necessarily actually placed. I don't think it's a particularly great way to handle it, but it does at least seem somewhat consistent, though it is inconsistent with blocks like note blocks and jukeboxes.

[RoboMWM]

I believe I filed a bug report re: blockplaceevent. https://hub.spigotmc.org/jira/browse/SPIGOT-4387

Hmm, it's resolved as fixed. Don't recall getting an update about it. Was the PR pulled? I don't have access to see PRs.

[ShaneBeee]

I believe I filed a bug report re: blockplaceevent. https://hub.spigotmc.org/jira/browse/SPIGOT-4387

Hmm, it's resolved as fixed. Don't recall getting an update about it. Was the PR pulled? I don't have access to see PRs.

Looks like the PR was never pulled, its still open.
Here is the commit that fixes the bug report you posted. Looks like it just fixed the items but did nothing regarding the action firing of the wrong event

[RoboMWM]

Looks good to me at a glance.