[plugin] SeparateOptions

[plugin] SeparateOptions

SpringRTS Perl Autohost for Dedicated Server

Moderators: Moderators, Lobby Developers, SPADS AutoHost, Moderators, Lobby Developers, SPADS AutoHost

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

[plugin] SeparateOptions

Post by Jools »

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: 913
Joined: 06 Dec 2007, 11:12

Re: SPADS AutoHost

Post by bibim »

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: 2806
Joined: 23 Feb 2009, 16:29

Re: SPADS AutoHost

Post by Jools »

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: 913
Joined: 06 Dec 2007, 11:12

Re: SPADS AutoHost

Post by bibim »

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”

cron