use actix_web::web::Data;
use anyhow::Context;
use lemmy_api_common::{
- blocking,
community::{GetCommunityResponse, TransferCommunity},
- get_local_user_view_from_jwt,
+ context::LemmyContext,
+ utils::{get_local_user_view_from_jwt, is_admin, is_top_mod},
};
use lemmy_db_schema::{
source::{
},
traits::{Crud, Joinable},
};
-use lemmy_db_views_actor::{
- community_moderator_view::CommunityModeratorView,
- community_view::CommunityView,
- person_view::PersonViewSafe,
-};
-use lemmy_utils::{location_info, ConnectionId, LemmyError};
-use lemmy_websocket::LemmyContext;
+use lemmy_db_views_actor::structs::{CommunityModeratorView, CommunityView};
+use lemmy_utils::{error::LemmyError, location_info, ConnectionId};
// TODO: we dont do anything for federation here, it should be updated the next time the community
// gets fetched. i hope we can get rid of the community creator role soon.
let local_user_view =
get_local_user_view_from_jwt(&data.auth, context.pool(), context.secret()).await?;
- let admins = blocking(context.pool(), PersonViewSafe::admins).await??;
-
// Fetch the community mods
let community_id = data.community_id;
- let mut community_mods = blocking(context.pool(), move |conn| {
- CommunityModeratorView::for_community(conn, community_id)
- })
- .await??;
+ let mut community_mods =
+ CommunityModeratorView::for_community(context.pool(), community_id).await?;
// Make sure transferrer is either the top community mod, or an admin
- if local_user_view.person.id != community_mods[0].moderator.id
- && !admins
- .iter()
- .map(|a| a.person.id)
- .any(|x| x == local_user_view.person.id)
+ if !(is_top_mod(&local_user_view, &community_mods).is_ok()
+ || is_admin(&local_user_view).is_ok())
{
return Err(LemmyError::from_message("not_an_admin"));
}
// Delete all the mods
let community_id = data.community_id;
- blocking(context.pool(), move |conn| {
- CommunityModerator::delete_for_community(conn, community_id)
- })
- .await??;
+
+ CommunityModerator::delete_for_community(context.pool(), community_id).await?;
// TODO: this should probably be a bulk operation
// Re-add the mods, in the new order
person_id: cmod.moderator.id,
};
- let join = move |conn: &'_ _| CommunityModerator::join(conn, &community_moderator_form);
- blocking(context.pool(), join)
- .await?
+ CommunityModerator::join(context.pool(), &community_moderator_form)
+ .await
.map_err(|e| LemmyError::from_error_message(e, "community_moderator_already_exists"))?;
}
mod_person_id: local_user_view.person.id,
other_person_id: data.person_id,
community_id: data.community_id,
- removed: Some(false),
};
- blocking(context.pool(), move |conn| {
- ModTransferCommunity::create(conn, &form)
- })
- .await??;
+
+ ModTransferCommunity::create(context.pool(), &form).await?;
let community_id = data.community_id;
let person_id = local_user_view.person.id;
- let community_view = blocking(context.pool(), move |conn| {
- CommunityView::read(conn, community_id, Some(person_id))
- })
- .await?
- .map_err(|e| LemmyError::from_error_message(e, "couldnt_find_community"))?;
+ let community_view = CommunityView::read(context.pool(), community_id, Some(person_id), None)
+ .await
+ .map_err(|e| LemmyError::from_error_message(e, "couldnt_find_community"))?;
let community_id = data.community_id;
- let moderators = blocking(context.pool(), move |conn| {
- CommunityModeratorView::for_community(conn, community_id)
- })
- .await?
- .map_err(|e| LemmyError::from_error_message(e, "couldnt_find_community"))?;
+ let moderators = CommunityModeratorView::for_community(context.pool(), community_id)
+ .await
+ .map_err(|e| LemmyError::from_error_message(e, "couldnt_find_community"))?;
// Return the jwt
Ok(GetCommunityResponse {
site: None,
moderators,
online: 0,
+ discussion_languages: vec![],
+ default_post_language: None,
})
}
}