bzflag icon indicating copy to clipboard operation
bzflag copied to clipboard

Complete bz_eGetWorldEvent

Open allejo opened this issue 9 years ago • 4 comments

Currently, the bz_eGetWorldEvent allows you change the map the bzfs instance without killing the actual process but the event does not parse the options block to update the respective settings. In other words, if you start with a map that's FFA and try to use this event to switch to a CTF map, it'll remain FFA. This applies to everything in the options block, so BZDB variables, map settings, etc.

This would most likely require a refactor of how the options blocked is parsed since there's no function or class to parse a world file.

allejo avatar May 28 '15 03:05 allejo

This change is probably a bit too big and risky for a point release and should be a goal for 2.6.

blast007 avatar Jun 18 '15 00:06 blast007

What are some possible risks involved? Not that I'm against the idea of moving this to 2.6, I'm just curious.

allejo avatar Jun 18 '15 04:06 allejo

It would need to touch a lot of code and require a significant amount of testing to ensure we're not getting into some wacky state that's some crazy hybrid of, for instance, CTF and FFA. I'd rather such a feature be done during a longer development/testing phase like a new major release.

blast007 avatar Jun 18 '15 10:06 blast007

That seems fair. 2.6 it is then. We can make use of this event to add an official mapchange plug-in as well without the weird bash script setup

allejo avatar Jun 19 '15 13:06 allejo