Fix deduplication

marcelropos · Feb 10, 2024 · c72b1e9 · c72b1e9
1 parent 687e55d
commit c72b1e9
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/bot/commands/subject.rs b/src/bot/commands/subject.rs
@@ -1,5 +1,7 @@
 //! TODO: Permissions
 
+use std::collections::HashSet;
+
 use futures::future::join_all;
 use poise::serenity_prelude::{GuildId, RoleId};
 use sqlx::{MySql, Pool};
@@ -154,11 +156,11 @@ async fn get_available_subjects(
 
     let subjects = join_all(subjects).await;
 
-    let flattened_subjects: Vec<DatabaseSubject> = subjects.iter()
-        .flatten().flatten().cloned().dedup()
-        .collect();
-
-    flattened_subjects
+    subjects.iter()
+        .flatten().flatten().cloned()
+        .sorted_by_key(|subject| subject.id)
+        .dedup()
+        .collect()
 }
 
 /// Parses subject names/ids from user input, then