1 files changed, 5 insertions, 6 deletions
diff --git a/src/controller/balance.rs b/src/controller/balance.rs
index 228ff04..adde8a9 100644
--- a/src/controller/balance.rs
+++ b/src/controller/balance.rs
@@ -24,8 +24,7 @@ pub async fn get(wallet: &Wallet) -> Response<Body> {
     let user_payments = db::payments::repartition(&wallet.pool).await;
     let template_user_payments =
         get_template_user_payments(&users, &user_payments);
-    let total_payments: i64 =
-        user_payments.clone().into_iter().map(|p| p.1).sum();
+    let total_payments: i64 = user_payments.iter().map(|p| p.1).sum();
 
     let exceeding_payers =
         payer::exceeding(&users, &user_incomes, &user_payments);
@@ -47,11 +46,11 @@ pub async fn get(wallet: &Wallet) -> Response<Body> {
 }
 
 fn get_template_user_payments(
-    users: &Vec<User>,
+    users: &[User],
     user_payments: &HashMap<i64, i64>,
 ) -> Vec<(String, i64)> {
     let mut user_payments: Vec<(String, i64)> = users
-        .into_iter()
+        .iter()
         .map(|u| (u.name.clone(), *user_payments.get(&u.id).unwrap_or(&0)))
         .collect();
     user_payments.sort_by_key(|i| i.1);
@@ -59,11 +58,11 @@ fn get_template_user_payments(
 }
 
 fn get_template_user_incomes(
-    users: &Vec<User>,
+    users: &[User],
     user_incomes: &HashMap<i64, i64>,
 ) -> Vec<(String, i64)> {
     let mut user_incomes: Vec<(String, i64)> = users
-        .into_iter()
+        .iter()
         .map(|u| (u.name.clone(), *user_incomes.get(&u.id).unwrap_or(&0)))
         .collect();
     user_incomes.sort_by_key(|i| i.1);