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

Resizing snitch fields? #52

Open TealNerd opened 8 years ago

TealNerd commented 8 years ago

Some people were bugging me about wanting this but wanted to check in here and get thoughts on it before doing work to implement it. So... thoughts?

Maxopoly commented 8 years ago

Why would we need this? The current one is completly fine imo. Also isnt this just a config value?

TealNerd commented 8 years ago

The old snitch plugin precious stones had the ability to resize the field ingame so you would be limited to 12167 meters cubed but could have it long and flat or super tall and skinny to, for instance, fit the exact bounds of a building

Maxopoly commented 8 years ago

Hmm, sounds like a good idea imo.

erocs commented 8 years ago

The field calculates from the snitch block creating odd lengths for the sides.

An edge shouldn't be smaller than 3 blocks.

Validate the heck out of the provided lengths. Precious stones could be manipulated to create fields of millions of blocks which crashed the server at least once.

People have lived without it for over 2 years. What would it enable? On Jan 11, 2016 2:37 AM, "Max Baumstark" notifications@github.com wrote:

Hmm, sounds like a good idea imo.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/issues/52#issuecomment-170503230.

ProgrammerDan commented 8 years ago

There's a lot of subtle complexities here. We could put it as a P5 and if we get some bandwidth later review it then.

With Triggers I fear this would be a bit OP; I could create some nasty focused traps using a trigger-enabled juke that only scanned a 3x3x3 cube to the side of the snitch, for instance ...