Mainmenu: Still support favorites if send_pre_v25_init is disabled 3998/head
authorest31 <MTest31@outlook.com>
Fri, 15 Apr 2016 02:13:53 +0000 (04:13 +0200)
committerest31 <MTest31@outlook.com>
Fri, 15 Apr 2016 12:40:31 +0000 (14:40 +0200)
commitbc4dc80c0166bd5c7f645bec1472101e4a9e20bb
treedd09c0072a3ab4361cc447853e17d14bcd1fef00
parentd82c5da0dccb1f30250725d17c3f373aacdda0e2
Mainmenu: Still support favorites if send_pre_v25_init is disabled

@SmallJoker has noted a bug that servers from the (local) main menu
favorites list can't be opened.

This commit fixes the bug by disabling any main menu based protocol
checks for servers from the favorite list.

Also, it fixes a second bug that happens when a server from the
public serverlist doesn't send its supported protocol versions,
most likely because its running a minetest older than commit [1].
Then we have shown an error msg that the server has enforced
one specific protocol version. This was most likely not the case.

Of course, we can't do anything better than do an assumption on
the protocol versions if they are not known. That assumption
should however be closest to the most often occuring case as
possible.

Also, some little cleanups.

[1]: 5a0ed780f56a5225b3d7c5f64099586e390e5f39 "Server: announce MIN/MAX protocol version supported to serverlist. Client: check serverlist"
builtin/mainmenu/common.lua
builtin/mainmenu/tab_multiplayer.lua
builtin/mainmenu/tab_simple_main.lua