FakeFishGames / Barotrauma

A 2D online multiplayer game taking place in a submarine travelling through the icy depths of Jupiter's moon Europa.
http://www.barotraumagame.com/
1.73k stars 401 forks source link

Various Servers are Considered "Playstyle-less", Affecting Server Filters #3612

Closed hubismal closed 4 years ago

hubismal commented 4 years ago

Some servers are considered "without a playstyle", and will show up for some reason when absolutely all playstyles are filtered out. They take on the appearance of Serious servers. A few Discord members are complaining about them since this means that these "serious" servers will start showing up in every other single category filter (ex "serious" servers will show up in just roleplay, just casual, just rampage, etc). Reproduce: Open server browser, turn off all the inclusive filters for playstyles. Should see something like below: image

EtiamNullam commented 4 years ago

I can confirm. At the time I checked all of them listed when no playstyle was selected were empty, so they all are dedicated servers - server hosted over steam p2p can't be empty.

Regalis11 commented 4 years ago

Fixed in https://github.com/Regalis11/Barotrauma-development/commit/ba824f75e6f3e040ebe3de05b5a12824e74a897e

Rokvach commented 4 years ago

I think we should also not show the servers if a playstyle has been selected, is it a bug that the servers doesn't have any?

One way to do this would be to add a "None" playstyle and being able to toggle those servers off from the list

@Regalis11

Game modes seems to have similar issue where certain servers are always shown even tho you don't have any game modes toggled on, as well as showing servers doing campaign when the list has been filtered to only show mission game mode.

Regalis11 commented 4 years ago

Those should be fixed now: https://github.com/Regalis11/Barotrauma-development/commit/40057c29f43ad01b06ed054acb64fa78603f26c4

Normally the servers should always have a play style set, but there's a bug in the current version that causes dedicated servers to not report the playstyle to the master server.

Since there should be no "playstyle-less" servers after the fix goes out, instead of a "None" filtering option I decided to only show them when all playstyles have been selected. The "Playstyle: Serious" text should also now be hidden.

Rokvach commented 4 years ago

Tested, didn't find any issues. Closing.

EtiamNullam commented 4 years ago

Still does not work in 0.10.5 (unstable), my dedicated server is always considered "playstyle-less" - after last fix unchecking any of the playstyle in server browser filtering will make my server disappear from the list.

ekqvist commented 4 years ago

Reopened and moved back to to-do

Regalis11 commented 4 years ago

Third time's a charm? https://github.com/Regalis11/Barotrauma-development/commit/d00035b12d3a2a8fa75cf26a11844aa674c35e35

Rokvach commented 4 years ago

No issues, closing.