search

Civcraft / PrisonPearl

Minecraft plugin for civcraft which allows players to imprison other players inside ender pearls
BSD 3-Clause "New" or "Revised" License
4 stars 16 forks source link

Log successful verification #19

Closed erocs closed 9 years ago

erocs commented 9 years ago

https://github.com/Civcraft/PrisonPearl/blob/master/src/com/untamedears/PrisonPearl/PrisonPearl.java#L267

Could we log why a verification was successful at this code location? It will need a bit of refactoring to get the reason for both successes and failures.

rourke750 commented 9 years ago

Oh fun.

spaceFountain commented 9 years ago

Wops I thought I'd already written something here.

What exactly do you want added to the log? It seems it already logs some of the reasons in verifyHolder.

erocs commented 9 years ago

VerifyHolder logs locations where the pearl wasn't found. I'd like to know where the pearl is for a successful verification.

On Fri, Jan 23, 2015 at 10:04 AM, spaceFountain notifications@github.com wrote:

Wops I thought I'd already written something here.

What exactly do you want added to the log? It seems it already logs some of the reasons in verifyHolder.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/PrisonPearl/issues/19#issuecomment-71235720.

spaceFountain commented 9 years ago

Does this do what you need?

rourke750 commented 9 years ago

https://github.com/Civcraft/PrisonPearl/commit/b4d66ab67b79d1265e4a1c78ff00b7394fdf27ff