Exultant/Citadel — pull request #94 by pruby (2012-12-27 03:59:03+00:00)
Title: Change to speed up ctf use. Untested but small.

This is a fixed version of pull request https://github.com/Exultant/Citadel/pull/93 .

I've managed to compile it with Maven (though produced jar doesn't have the plugin.xml and such?). In doing so found that I had a method name wrong. Compiles but haven't managed to run.

Original message:

While doing a bunch of building in Civcraft I've been annoyed by a couple of little things with /ctf:
* You can't change reinforcement material without turning fortification mode off and on again.
* You can't change security level or leave fortification mode without holding the original material (or using /ctoff, which also messes up /ctbypass state).

I've added a condition that if you're already in fortification mode, /ctf will check these conditions:
* If holding a different, valid reinforcement material, will switch to that material without turning off fortification mode.
* If not holding a reinforcement material and selecting a different security level, will change security level and leave material as-is.
* If not holding a reinforcement material and using same security level, will turn off.

If not in fortification mode, behaves as it currently does.

I'm afraid I don't have a bukkit set-up to test this on sorry - hope worth your time to test. Please let me know if any issues with it.