phogue / Potato

Procon 2 is a (long promised and often postponed) second release of the Procon admin tool for gameservers. Once released, it will allow managing several games and automating tasks via plugins.
https://myrcon.com/
Apache License 2.0
20 stars 4 forks source link

Command.ToConfigCommand nulling scope #51

Closed phogue closed 10 years ago

phogue commented 10 years ago

When saving the config the plugins that are enabled are not being reenabled on load. This is because ToConfigCommand nulls out the scope.

Need a couple unit tests to validate the config can be saved/loaded for the plugin controller correctly.

Investigate why ToConfigCommand is nulling out the scope. I think it's just to make config's look prettier so the scope is not filled with lot's of empty GUID's for no reason.

Perhaps change over so if all GUID's in scope are empty then the scope can be nulled, otherwise the command requires it's scope.