[plugin] SeparateOptions

[plugin] SeparateOptions

SpringRTS Perl Autohost for Dedicated Server

Moderators: Moderators, Lobby Developers, SPADS AutoHost

Post Reply
User avatar
Jools
XTA Developer
Posts: 2804
Joined: 23 Feb 2009, 16:29

[plugin] SeparateOptions

Post by Jools » 29 Jan 2015, 20:21

I made a package that adds the !mapoptions and !modoptions commands.

It's not 100% perfect: for some reason it misses some lines of the allowed values, althought the original !list bsettings doesn't.

The code used to generate the options is basically copied from the main spads module.

Would be maybe better to extend the functionality of the original function instead, it would be easy. Do you accept pull requests?
Attachments
SeparateOptions.zip
(3.07 KiB) Downloaded 5 times
0 x

User avatar
bibim
Lobby Developer
Posts: 901
Joined: 06 Dec 2007, 11:12

Re: SPADS AutoHost

Post by bibim » 30 Jan 2015, 12:52

Jools wrote:Would be maybe better to extend the functionality of the original function instead, it would be easy. Do you accept pull requests?
Yes of course, if I agree with the change :)
But in this case I will just make "!list bSettings" accept a new parameter which can take values "map","mod","engine". The change is very simple so I don't think it's needed.

You will then be able to add "!mapoptions" alias for "!list bSettings map" if you want for example.
0 x

User avatar
Jools
XTA Developer
Posts: 2804
Joined: 23 Feb 2009, 16:29

Re: SPADS AutoHost

Post by Jools » 30 Jan 2015, 16:47

bibim wrote:
Jools wrote:Would be maybe better to extend the functionality of the original function instead, it would be easy. Do you accept pull requests?
Yes of course, if I agree with the change :)
But in this case I will just make "!list bSettings" accept a new parameter which can take values "map","mod","engine". The change is very simple so I don't think it's needed.

You will then be able to add "!mapoptions" alias for "!list bSettings map" if you want for example.
That's exactly what I was thinking about doing too :)
0 x

User avatar
bibim
Lobby Developer
Posts: 901
Joined: 06 Dec 2007, 11:12

Re: SPADS AutoHost

Post by bibim » 17 Feb 2015, 16:33

bibim wrote:I will just make "!list bSettings" accept a new parameter which can take values "map","mod","engine".
Done in SPADS 0.11.24a.
0 x

Post Reply

Return to “SPADS AutoHost”