avatar module launched even if missing memeber

src/main.rs

@@ -75,60 +75,108 @@ fn main() {
         Commands::Set { members, discord, fedi } => {
             res = set_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), discord, fedi);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(feature = "discord", not(feature = "fedi")))]
         Commands::Set { members, discord } => {
             res = set_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), discord, false);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(not(feature = "discord"), feature = "fedi"))]
         Commands::Set { members, fedi} => {
             res = set_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), false, fedi);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(not(feature = "discord"), not(feature = "fedi")))]
         Commands::Set { members } => {
             res = set_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), false, false);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
 
         // ADD MEMBER
         #[cfg(all(feature = "discord", feature = "fedi"))]
         Commands::Add { members, discord, fedi } => {
             res = add_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), discord, fedi);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(feature = "discord", not(feature = "fedi")))]
         Commands::Add { members, discord } => {
             res = add_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), discord, false);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(not(feature = "discord"), feature = "fedi"))]
         Commands::Add { members, fedi} => {
             res = add_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), false, fedi);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
         #[cfg(all(not(feature = "discord"), not(feature = "fedi")))]
         Commands::Add { members } => {
             res = add_member(config_path.clone(), members);
 
-            #[cfg(feature = "avatar")]
+            #[cfg(feature = "avatar")] {
+                match res {
+                    Ok(_) => {
                         let _ = update_avatars(config_path.clone(), false, false);
                     },
+                    Err(_) => (),
+                }
+            }
+        },
 
         // Get MEMBER
         #[cfg(all(feature = "discord", feature = "fedi"))]
@@ -270,6 +318,7 @@ fn set_member(config_path: String, tf_members: Vec<String>) -> Result<(), &'stat
         println!("One or more members were not found. Known members:\n--------------------------");
         let _ = memberlist(config_path);
         println!("--------------------------\nIf a member is missing from the system try running \"pluralsync sync\" to refresh the local database");
+        return Err("Missing member");
     }
     Ok(())
 }
@@ -307,6 +356,7 @@ fn add_member(config_path: String, tf_members: Vec<String>) -> Result<(), &'stat
         println!("One or more members were not found. Known members:\n--------------------------");
         let _ = memberlist(config_path);
         println!("--------------------------\nIf a member is missing from the system try running \"pluralsync sync\" to refresh the local database");
+        return Err("Missing member");
     }
     Ok(())
 }