cleanup code

Author: thdaele

public/develop.js

@@ -164,11 +164,6 @@ import {
     versionSelectorInput.addEventListener("input", async _ => await updateOrnitheDependencies())
 
     allowSnapshotsCheck.addEventListener("change", async _ => {
-        if (allowSnapshotsCheck.checked) {
-            possibleVersions = minecraftAllVersions;
-        } else {
-            possibleVersions = minecraftStableVersions;
-        }
         updateVersionList();
         await updateOrnitheDependencies();
     })
@@ -178,28 +173,26 @@ import {
         minecraftStableVersions = await getMinecraftStableVersions(gen);
         minecraftAllVersions = await getMinecraftVersions(gen);
 
-        // Update the version list based on snapshot checkbox
-        if (allowSnapshotsCheck.checked) {
-            possibleVersions = minecraftAllVersions;
-        } else {
-            possibleVersions = minecraftStableVersions;
-        }
         updateVersionList();
         // Update the dependencies message since it depends on the feather gen
         await updateOrnitheDependencies();
     })
 
     function updateVersionList() {
-        const list = possibleVersions;
+        if (allowSnapshotsCheck.checked) {
+            possibleVersions = minecraftAllVersions;
+        } else {
+            possibleVersions = minecraftStableVersions;
+        }
+
         while (versionListElement.firstChild) versionListElement.removeChild(versionListElement.lastChild);
-        list.forEach(e => {
+        possibleVersions.forEach(e => {
             const opt = new Option();
             opt.value = e;
             versionListElement.appendChild(opt);
         });
     }
 
-    possibleVersions = minecraftStableVersions;
     updateVersionList()
     await updateOrnitheDependencies()