Make setFromObject private or internal

Author: JulianVennen

src/main/java/com/exaroton/api/server/Server.java

@@ -279,8 +279,7 @@ public CompletableFuture<Server> get() throws IOException {
         return client.request(new GetServerRequest(this.client, this.gson, this.id))
                 .thenApply(data -> {
                     this.fetched = true;
-                    this.setFromObject(data);
-                    return this;
+                    return this.setFromObject(data);
                 });
     }
 
@@ -345,10 +344,7 @@ public CompletableFuture<Server> start() throws IOException {
      */
     public CompletableFuture<Server> start(boolean useOwnCredits) throws IOException {
         return client.request(new StartServerRequest(this.client, this.gson, this.id, useOwnCredits))
-                .thenApply(data -> {;
-                    this.setFromObject(data);
-                    return this;
-                });
+                .thenApply(this::setFromObject);
     }
 
     /**
@@ -359,10 +355,7 @@ public CompletableFuture<Server> start(boolean useOwnCredits) throws IOException
      */
     public CompletableFuture<Server> stop() throws IOException {
         return client.request(new StopServerRequest(this.client, this.gson, this.id))
-                .thenApply(server -> {
-                    this.setFromObject(server);
-                    return this;
-                });
+                .thenApply(this::setFromObject);
     }
 
     /**
@@ -373,10 +366,7 @@ public CompletableFuture<Server> stop() throws IOException {
      */
     public CompletableFuture<Server> restart() throws IOException {
         return client.request(new RestartServerRequest(this.client, this.gson, this.id))
-                .thenApply(server -> {
-                    this.setFromObject(server);
-                    return this;
-                });
+                .thenApply(this::setFromObject);
     }
 
     /**
@@ -429,6 +419,7 @@ public PlayerList getPlayerList(String name) {
      * @param server server fetched from the API
      * @return updated server object
      */
+    @ApiStatus.Internal
     public Server setFromObject(Server server) {
         this.id = server.getId();
         this.name = server.getName();

src/main/java/com/exaroton/api/server/ServerFile.java

@@ -67,10 +67,7 @@ public ServerFile(
      */
     public CompletableFuture<ServerFile> get() throws IOException {
         return client.request(new GetFileInfoRequest(this.client, this.gson, this.server.getId(), this.path))
-                .thenApply(data -> {
-                    this.setFromObject(data);
-                    return this;
-                });
+                .thenApply(this::setFromObject);
     }
 
     /**
@@ -269,7 +266,7 @@ public void setPath(String path) {
      * @param file file fetched from the API
      * @return updated file object
      */
-    public ServerFile setFromObject(ServerFile file) {
+    private ServerFile setFromObject(ServerFile file) {
         if (file.path != null) {
             this.setPath(path);
         }

src/main/java/com/exaroton/api/ws/WebSocketManager.java

@@ -89,8 +89,7 @@ public void handleData(String type, String message) {
                 break;
 
             case "status":
-                Server oldServer = new Server(server.getClient(), gson, server.getId())
-                        .setFromObject(server);
+                Server oldServer = new Server(server.getClient(), gson, server.getId()).setFromObject(server);
                 this.server.setFromObject(gson.fromJson(message, ServerStatusStreamData.class).getData());
 
                 //start/stop streams based on status