Wolvereness/PhysicalShop — pull request #14 by Benja-gipsy-king (2012-05-28 11:35:09+00:00)
Title: add deadbolt, always check for protection and ownership if such plugin was found, even if protect-existing-chest is false

Right now, as far as I understand it, protection works like this:
ON: A player can only put a sign over an existing chest if the chest is protected by a plugin AND "owned" by the player. Good.
OFF: A player can put a sign over any existing chests, including chests that are protected and owned by someone else. The latter means that a player could put a sign over someone elses chest, blocking the other playing from using it (it could even be exploitable, if the player knows or guesses the chests contets and writes a matching buy or sell line).

What I propose, is to also check for protection on OFF. We'd only deny if the chest is protected by someone else.
To achieve that, the checks should return false ONLY when the plugin is enabled, the block is protected, and the player is not the owner.

Deadbolt.jar can be downloaded from here: http://dev.bukkit.org/server-mods/deadbolt/files/54-deadbolt/