diff options
author | Joris | 2023-04-17 21:10:48 +0200 |
---|---|---|
committer | Joris | 2023-04-17 21:10:48 +0200 |
commit | 459016e70dd4933a8082d27748097de81a3e53ff (patch) | |
tree | 0ed14ecc8762d62a55bf8c05356bcc984b1a0cea /src/controller/balance.rs | |
parent | 3932daa26360d6e03807381d0b8ffa2d0e704847 (diff) | |
download | budget-459016e70dd4933a8082d27748097de81a3e53ff.tar.gz budget-459016e70dd4933a8082d27748097de81a3e53ff.tar.bz2 budget-459016e70dd4933a8082d27748097de81a3e53ff.zip |
Follow clippy indications
Diffstat (limited to 'src/controller/balance.rs')
-rw-r--r-- | src/controller/balance.rs | 11 |
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); |