Change: prioritize servers without password over ones with in server listing (#39)

TrueBrain · web-flow · commit dcd8a1595393 · 2021-10-20T23:35:21.000+02:00
The in-game client already does this, and it makes a lot more sense
to show servers you are free to join.
diff --git a/webclient/pages/servers.py b/webclient/pages/servers.py
@@ -136,7 +136,7 @@ def _split_version(raw_version):
 def _sort_servers(servers):
     servers.sort(
         key=lambda x: _split_version(x["info"]["openttd_version"])
-        + [x["info"]["clients_on"], x["info"]["companies_on"]],
+        + ["0" if x["info"]["use_password"] else "1", x["info"]["clients_on"], x["info"]["companies_on"]],
         reverse=True,
     )
 

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ def _split_version(raw_version):`
`136`	`136`	`def _sort_servers(servers):`
`137`	`137`	`servers.sort(`
`138`	`138`	`key=lambda x: _split_version(x["info"]["openttd_version"])`
`139`		`- + [x["info"]["clients_on"], x["info"]["companies_on"]],`
	`139`	`+ + ["0" if x["info"]["use_password"] else "1", x["info"]["clients_on"], x["info"]["companies_on"]],`
`140`	`140`	`reverse=True,`
`141`	`141`	`)`
`142`	`142`