Merge fix

This commit is contained in:
Oleg Agafonov 2023-11-23 09:32:07 +04:00
parent a0ed89035f
commit 5691ffa40e
5 changed files with 9 additions and 15 deletions

View file

@ -49,8 +49,7 @@ public interface MageServer {
List<ExpansionInfo> syncGetMissingExpansionData(List<String> codes);
List<CardInfo> syncGetMissingCardsData(List<String> classNames);
// server state methods
ServerState serverGetState() throws MageException;
ServerState getServerState() throws MageException; // TODO: need stable update process, so rename it after few releases
UUID serverGetMainRoomId() throws MageException;

View file

@ -8,18 +8,10 @@ import mage.utils.MageVersion;
*/
public class MageVersionException extends MageException {
private final MageVersion serverVersion;
public MageVersionException(MageVersion clientVersion, MageVersion serverVersion) {
super("Wrong client version."
+ "<br/>Your version: " + clientVersion
+ "<br/>Server version: " + serverVersion
+ "<br/>Release app download: http://xmage.de"
+ "<br/>BETA app download: http://xmage.today");
this.serverVersion = serverVersion;
}
public MageVersion getServerVersion() {
return serverVersion;
+ "<br/>Server version: " + (serverVersion == null ? "unknown" : serverVersion)
+ "<br/>App download: http://xmage.today");
}
}

View file

@ -257,7 +257,10 @@ public class SessionImpl implements Session {
}
if (result) {
serverState = server.serverGetState();
serverState = server.getServerState();
if (serverState == null) {
throw new MageVersionException(client.getVersion(), null);
}
// client side check for incompatible versions
if (client.getVersion().compareTo(serverState.getVersion()) != 0) {