Fix #126

Author: theRookieCoder

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Clean up imports
 - Fix [#131](https://github.com/gorilla-devs/ferium/issues/131), clean up `main.rs`'s fetching of the current profile and modpack
 - Fix [#130](https://github.com/gorilla-devs/ferium/issues/130)
+- Fix [#126](https://github.com/gorilla-devs/ferium/issues/126). Removed `sort` subcommand, all profiles' mods are sorted when writing to config
 
 ## `v4.1.2`
 ### 08.06.2022

src/cli.rs

@@ -75,8 +75,6 @@ Optionally, provide a list of names of the mods to remove"
         #[clap(help("A case-insensitive list of names of a mods to remove"))]
         mod_names: Vec<String>,
     },
-    #[clap(about("Sort all your mods in alphabetical order"))]
-    Sort,
     #[clap(about("Download and install the latest version of the mods specified"))]
     Upgrade,
 }

src/main.rs

@@ -282,9 +282,6 @@ async fn actual_main(cli_app: Ferium) -> Result<()> {
             check_empty_profile(profile)?;
             subcommands::remove(profile, mod_names)?;
         },
-        SubCommands::Sort => get_active_profile(&mut config)?
-            .mods
-            .sort_by_cached_key(|mod_| mod_.name.to_lowercase()),
         SubCommands::Upgrade => {
             check_internet().await?;
             let profile = get_active_profile(&mut config)?;
@@ -293,6 +290,11 @@ async fn actual_main(cli_app: Ferium) -> Result<()> {
         },
     };
 
+    config.profiles.iter_mut().for_each(|profile| {
+        profile
+            .mods
+            .sort_by_cached_key(|mod_| mod_.name.to_lowercase())
+    });
     // Update config file with possibly edited config
     config::write_file(&mut config_file, &config).await?;