Small fix to use Paper/Travertine/Waterfall's updated API for downloading .jar files

Additionally fixes some formatting mistakes
Updates version to 1.4.1

src/main/java/net/knarcraft/minecraftserverlauncher/server/JarBuilder.java

@@ -1,10 +1,10 @@
-package net.knarcraft.minecraftserverlauncher.utility;
+package net.knarcraft.minecraftserverlauncher.server;
 
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
-import net.knarcraft.minecraftserverlauncher.server.ServerVersionContainer;
 import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
 import net.knarcraft.minecraftserverlauncher.userinterface.ServerLauncherGUI;
+import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
 import java.io.BufferedReader;
 import java.io.BufferedWriter;

src/main/java/net/knarcraft/minecraftserverlauncher/server/Server.java

@@ -377,7 +377,8 @@ public class Server {
                 if (Integer.parseInt(serverVersion.split("\\.")[1]) >= 17) {
                     return controller.getJavaCommand();
                 }
-            } catch (NumberFormatException ignored) {}
+            } catch (NumberFormatException ignored) {
+            }
         }
         return controller.getOldJavaCommand();
     }

src/main/java/net/knarcraft/minecraftserverlauncher/server/ServerVersionContainer.java

@@ -311,7 +311,7 @@ public class ServerVersionContainer {
     /**
      * Sets the current version for a given paper version
      *
-     * @param mapKey <p>The version key to set version for</p>
+     * @param mapKey   <p>The version key to set version for</p>
      * @param newValue <p>The new current version</p>
      */
     public void setPaperVersion(String mapKey, String newValue) {

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/CraftBukkit.java

@@ -2,8 +2,8 @@ package net.knarcraft.minecraftserverlauncher.server.servertypes;
 
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
+import net.knarcraft.minecraftserverlauncher.server.JarBuilder;
 import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
-import net.knarcraft.minecraftserverlauncher.utility.JarBuilder;
 
 import java.io.File;
 import java.nio.file.Paths;

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/Waterfall.java

@@ -50,13 +50,21 @@ public class Waterfall extends AbstractServerType {
     public boolean downloadJar(String folder, String version) throws IOException {
         String file = this.getName() + version + ".jar";
         File filePath = new File(folder + file);
-        String newestVersion = stringBetween(readRemoteFile(versionURL + version), srcStart, srcEnd);
-        String fullURL = downloadURL + version + "/" + newestVersion + "/download";
+        String versionFileContents = readRemoteFile(versionURL + version);
+        String[] versions = stringBetween(versionFileContents, srcStart, srcEnd).split(",");
+        String newestVersion = versions[versions.length - 1];
         String oldVersion = oldVersionFunction.apply(version);
+
         //The file is already the newest version
         if (filePath.isFile() && newestVersion.equals(oldVersion)) {
             return true;
         }
+
+        //Get necessary information for downloading the latest version
+        String newestVersionInfoURL = versionURL + version + "/builds/" + newestVersion;
+        String fileName = stringBetween(readRemoteFile(newestVersionInfoURL), "\"name\":\"", "\"");
+        String fullURL = downloadURL + version + "/builds/" + newestVersion + "/downloads/" + fileName;
+
         //The new jar file could not be downloaded
         if (!downloadFile(fullURL, Paths.get(filePath.toURI()))) {
             return false;
@@ -64,4 +72,5 @@ public class Waterfall extends AbstractServerType {
         versionUpdateFunction.accept(version, newestVersion);
         return true;
     }
+
 }

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/BackupGUI.java

@@ -51,7 +51,7 @@ public class BackupGUI implements ActionListener {
         JPanel panel = new JPanel();
         panel.setLayout(new BoxLayout(panel, BoxLayout.PAGE_AXIS));
         panel.add(progressTextArea);
-        panel.add(Box.createRigidArea(new Dimension(0,5)));
+        panel.add(Box.createRigidArea(new Dimension(0, 5)));
         panel.add(progressBar);
 
         JPanel buttonPane = new JPanel();
@@ -71,7 +71,7 @@ public class BackupGUI implements ActionListener {
     /**
      * Updates information about the backup progress
      *
-     * @param infoText <p>The text to display</p>
+     * @param infoText        <p>The text to display</p>
      * @param progressPercent <p>The new percent of the progress bar</p>
      */
     public static void updateProgress(String infoText, int progressPercent) {

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/CloseTabActionListener.java

@@ -17,6 +17,7 @@ public class CloseTabActionListener implements ActionListener {
 
     /**
      * Instantiates a new close tab action listener
+     *
      * @param tabPane <p>The tab pane containing all tabs</p>
      * @param tabName <p>The name of the tab connected to this action listener</p>
      */

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ControlPanelTab.java

@@ -33,6 +33,7 @@ public class ControlPanelTab implements ActionListener {
 
     /**
      * Sets the text of the status label
+     *
      * @param text <p>The new text of the status label</p>
      */
     public void setStatusText(String text) {

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerLauncherGUI.java

@@ -93,6 +93,7 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
 
     /**
      * Gets this GUI's control panel tab
+     *
      * @return <p>The control panel tab for this GUI</p>
      */
     public ControlPanelTab getControlPanelTab() {
@@ -168,7 +169,7 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
         addServerTabButton.addActionListener(this);
         addServerPaneButton = getAddServerButton(false);
         addServerPaneButton.addActionListener(this);
-        
+
         tabContentsPanel.add(addServerTabButton);
         tabPanel.add(addServerPaneButton);
         serversPane.setTabComponentAt(serversPane.getTabCount() - 1, tabPanel);
@@ -176,6 +177,7 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
 
     /**
      * Gets a button for adding a new server
+     *
      * @param displayButtonStyle <p>Whether to show or hide the button's style</p>
      * @return <p>A new add server button</p>
      */
@@ -193,7 +195,6 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
     }
 
     /**
-     *
      * @param serverName <p>The name of the server/tab to add a close button to</p>
      */
     private void addCloseButtonToServerTab(String serverName) {

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerLauncherMenu.java

@@ -164,7 +164,7 @@ public class ServerLauncherMenu implements ActionListener {
      * Creates a checkbox menu item
      *
      * @param itemName <p>The name of the new checkbox item</p>
-     * @param parent <p>The parent menu the item belongs to</p>
+     * @param parent   <p>The parent menu the item belongs to</p>
      * @return <p>The created checkbox menu item</p>
      */
     private JCheckBoxMenuItem createCheckBoxMenuItem(String itemName, JMenu parent) {
@@ -178,7 +178,7 @@ public class ServerLauncherMenu implements ActionListener {
      * Creates a menu item
      *
      * @param itemName <p>The name of the new item</p>
-     * @param parent <p>The parent menu the item belongs to</p>
+     * @param parent   <p>The parent menu the item belongs to</p>
      * @return <p>The created menu item</p>
      */
     private JMenuItem createMenuItem(String itemName, JMenu parent) {
@@ -190,6 +190,7 @@ public class ServerLauncherMenu implements ActionListener {
 
     /**
      * Asks the user for the new Java path
+     *
      * @param old <p>Whether asking for the path to the old java version</p>
      */
     private void configureJava(boolean old) {

src/main/java/net/knarcraft/minecraftserverlauncher/utility/BackupUtil.java

@@ -22,11 +22,15 @@ import java.util.stream.Stream;
 /**
  * A helper class for performing server backup
  */
-public class BackupUtil {
+public final class BackupUtil {
 
     private static boolean backupAborted;
     private static boolean backupRunning = false;
 
+    private BackupUtil() {
+        
+    }
+
     /**
      * Aborts the currently running backup
      */
@@ -44,7 +48,7 @@ public class BackupUtil {
      * @throws IOException <p>If we can't start a file stream</p>
      */
     private static long backupFolder(File source, File destination, long backupFileSize,
-                                                  long alreadyCopied) throws IOException {
+                                     long alreadyCopied) throws IOException {
         if (backupAborted) {
             return 0L;
         }
@@ -136,8 +140,9 @@ public class BackupUtil {
 
     /**
      * Performs the actual backup after checks have passed and necessary info is available
-     * @param gui <p>The GUI to use for informing the user</p>
-     * @param serverFolders <p>The folders of the servers to backup</p>
+     *
+     * @param gui            <p>The GUI to use for informing the user</p>
+     * @param serverFolders  <p>The folders of the servers to backup</p>
      * @param backupFileSize <p>The total size of the folders to backup</p>
      */
     private static void performBackup(GUI gui, List<List<File>> serverFolders, long backupFileSize) {
@@ -164,10 +169,10 @@ public class BackupUtil {
     /**
      * Backs up the files for a single server
      *
-     * @param gui <p>The GUI to use for informing the user</p>
-     * @param serverFolder <p>The server's input and output folders</p>
+     * @param gui            <p>The GUI to use for informing the user</p>
+     * @param serverFolder   <p>The server's input and output folders</p>
      * @param backupFileSize <p>The total size of the files to backup</p>
-     * @param alreadyCopied <p>The amount of bytes already copied</p>
+     * @param alreadyCopied  <p>The amount of bytes already copied</p>
      * @return <p>The new amount of bytes copied</p>
      */
     private static long backupServerFiles(GUI gui, List<File> serverFolder, long backupFileSize, long alreadyCopied) {

src/main/java/net/knarcraft/minecraftserverlauncher/utility/CommonFunctions.java

@@ -126,6 +126,7 @@ public final class CommonFunctions {
 
     /**
      * Gets a buffered writer for writing to a given file
+     *
      * @param path <p>The path to the file to write to</p>
      * @return <p>A buffered writer for writing to the file</p>
      * @throws FileNotFoundException <p>If the file does not exist</p>
@@ -149,8 +150,8 @@ public final class CommonFunctions {
     /**
      * Writes text to a file and adds a trailing newline
      *
-     * @param path               <p>The path of the file to write to</p>
-     * @param text               <p>The text to write</p>
+     * @param path <p>The path of the file to write to</p>
+     * @param text <p>The text to write</p>
      * @throws IOException <p>If unable to write to the file</p>
      */
     public static void writeFile(String path, String text) throws IOException {
@@ -243,7 +244,6 @@ public final class CommonFunctions {
     }
 
 
-
     /**
      * Validates that a name is not empty and does not contain invalid characters
      *
@@ -259,7 +259,7 @@ public final class CommonFunctions {
      *
      * @param target <p>The folder to delete from</p>
      */
-    static void removeFilesRecursively(File target) {
+    public static void removeFilesRecursively(File target) {
         File[] oldFiles = target.listFiles();
         if (oldFiles == null) {
             throw new IllegalArgumentException("Unable to list files in directory");