Move local check into function.

crates/api/src/lib.rs

@@ -174,6 +174,8 @@ pub(crate) async fn ban_nonlocal_user_from_local_communities(
   expires: &Option<i64>,
   context: &Data<LemmyContext>,
 ) -> LemmyResult<()> {
+  // Only run this code for federated users
+  if !target.local {
     let ids = Person::list_local_community_ids(&mut context.pool(), target.id).await?;
 
     for community_id in ids {
@@ -243,6 +245,7 @@ pub(crate) async fn ban_nonlocal_user_from_local_communities(
       )
       .await?;
     }
+  }
 
   Ok(())
 }

crates/api/src/local_user/ban_person.rs

@@ -72,7 +72,6 @@ pub async fn ban_from_site(
 
   let person_view = PersonView::read(&mut context.pool(), person.id).await?;
 
-  if !person.local {
   ban_nonlocal_user_from_local_communities(
     &local_user_view,
     &person,
@@ -83,7 +82,6 @@ pub async fn ban_from_site(
     &context,
   )
   .await?;
-  }
 
   ActivityChannel::submit_activity(
     SendActivityData::BanFromSite {