search

Civcraft / JukeAlert

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/JukeAlert
BSD 3-Clause "New" or "Revised" License
5 stars 15 forks source link

Breaking snitches now displays who owns them and what group they were on #50

Closed TealNerd closed 8 years ago

TealNerd commented 8 years ago

The logic should still work but i can add the extra parenthesis just to be sure. as for the config thing, those are definitely parenthesis...

rourke750 commented 8 years ago

For the if statement lets say some breaks a Jukebox wouldnt and config was set to false: true || false && false == true || false making the statement true even though it should be false or am I messing up?

rourke750 commented 8 years ago

ok to test

rourke750 commented 8 years ago

ok to test

CivcraftBot commented 8 years ago

Can one of the admins verify this patch? Type 'ok to test' to test.

rourke750 commented 8 years ago

ok to test

TealNerd commented 8 years ago

Also I think you meant to say they weren't quotes not they weren't parenthesis @rourke750

rourke750 commented 8 years ago

Lol yep.

rourke750 commented 8 years ago

ok to test

TealNerd commented 8 years ago

@rourke750 can we merge this?

Maxopoly commented 8 years ago

Did you test this?

ttk2 commented 8 years ago

bump.

On Mon, Dec 14, 2015 at 8:21 AM, Max Baumstark notifications@github.com wrote:

Did you test this?

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/pull/50#issuecomment-164434451.

TealNerd commented 8 years ago

yes