Skip to content
Snippets Groups Projects
Commit 736a0e88 authored by Ash's avatar Ash Committed by tec
Browse files

use enumerate() and fix indents

parent 310851d7
No related merge requests found
...@@ -101,9 +101,7 @@ pub fn sync_all_role_reactions(ctx: &Context) { ...@@ -101,9 +101,7 @@ pub fn sync_all_role_reactions(ctx: &Context) {
let mut roles_to_remove: HashMap<UserId, Vec<RoleId>> = let mut roles_to_remove: HashMap<UserId, Vec<RoleId>> =
HashMap::from_iter(all_members.iter().map(|m| (m.user_id(), Vec::new()))); HashMap::from_iter(all_members.iter().map(|m| (m.user_id(), Vec::new())));
let mut i = 0; for (i, (message, mapping)) in messages_with_role_mappings.iter().enumerate() {
for (message, mapping) in messages_with_role_mappings {
i += 1;
info!(" Sync: prossessing message #{}", i); info!(" Sync: prossessing message #{}", i);
for react in &message.reactions { for react in &message.reactions {
let react_as_string = get_string_from_react(&react.reaction_type); let react_as_string = get_string_from_react(&react.reaction_type);
...@@ -120,7 +118,7 @@ pub fn sync_all_role_reactions(ctx: &Context) { ...@@ -120,7 +118,7 @@ pub fn sync_all_role_reactions(ctx: &Context) {
warn!(" need to implement react removal"); warn!(" need to implement react removal");
} }
} }
for (react, role) in mapping { for (react, role) in *mapping {
info!(" message #{}: processing react '{}'", i, react); info!(" message #{}: processing react '{}'", i, react);
// TODO: proper pagination for the unlikely scenario that there are more than 100 (255?) reactions? // TODO: proper pagination for the unlikely scenario that there are more than 100 (255?) reactions?
let reaction_type = get_react_from_string(react.clone(), guild.clone()); let reaction_type = get_react_from_string(react.clone(), guild.clone());
......
...@@ -64,7 +64,7 @@ impl Commands { ...@@ -64,7 +64,7 @@ impl Commands {
"Error sending message: {:?}", "Error sending message: {:?}",
// TODO convert to email // TODO convert to email
msg.channel_id msg.channel_id
.say(&ctx.http, format!("Hey {} here's that token you ordered: {}\nIf this wasn't you just ignore this.", account_name, generate_token(&msg.author, account_name))) .say(&ctx.http, format!("Hey {} here's that token you ordered: {}\nIf this wasn't you just ignore this.", account_name, generate_token(&msg.author, account_name)))
); );
e!("Error deleting register message: {:?}", msg.delete(ctx)); e!("Error deleting register message: {:?}", msg.delete(ctx));
} }
......
...@@ -265,9 +265,9 @@ fn update_motion( ...@@ -265,9 +265,9 @@ fn update_motion(
); );
let update_status = |e: &mut serenity::builder::CreateEmbed, let update_status = |e: &mut serenity::builder::CreateEmbed,
status: &str, status: &str,
last_status_full: String, last_status_full: String,
topic: &str| { topic: &str| {
let last_status = last_status_full.lines().next().expect("No previous status"); let last_status = last_status_full.lines().next().expect("No previous status");
if last_status == status { if last_status == status {
e.field("Status", last_status_full, true); e.field("Status", last_status_full, true);
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment