EcoCityCraft/Spigot — pull request #1 by agaricusb (2013-02-11 04:07:03+00:00)
Title: Fix orebfuscator enabled log message

Simple cosmetic fix to the per-world enable state log message for Spigot orebfuscator

#spigot
(Zeph) He confirmed it works for obfuscation, but theres an inconstistancy when you load up the server. Even if you set it to false in the bukkit.yml, the startup console message still says that its enabled.
(Zeph) Even though the functionality in game is actually disabled
(Zeph) It might only be saying it per world, assuming theres a way to configure it to work per world. Maybe its not caring what the global false says. But thats just a guess
(agaricus) well the code does have a global enable/disable, and a per-world disable setting
(agaricus) obfuscated = !world.getServer().orebfuscatorDisabledWorlds.contains(name);
(agaricus) so its logging whether orebfuscation isn't disabled for that world, disregarding the global enable
(Zeph) Aww yeah, perfect guess
(agaricus) should be: obfuscated = !world.getServer().orebfuscatorDisabledWorlds.contains(name) && world.getServer().orebfuscatorEnabled
(agaricus) but the ObfuscatorManager properly checks: if (world.getServer().orebfuscatorEnabled && world.getWorld().obfuscated) {
(Zeph) Yeah, and so it functions correctly in game