X-Git-Url: http://these/git/?a=blobdiff_plain;f=crates%2Fapi_crud%2Fsrc%2Fpost%2Fremove.rs;h=8feff19fc74fe04ea81d63be5842f6e93d7d4fd5;hb=38c621091273822f798dfd75046b9ce724cf4bc1;hp=8850349cd2be8147a65ff3cccb8e672958f0d55f;hpb=5d837780f5d149cb8d3b861c63a7dc4466a7cbf1;p=lemmy.git diff --git a/crates/api_crud/src/post/remove.rs b/crates/api_crud/src/post/remove.rs index 8850349c..8feff19f 100644 --- a/crates/api_crud/src/post/remove.rs +++ b/crates/api_crud/src/post/remove.rs @@ -1,48 +1,42 @@ use crate::PerformCrud; use actix_web::web::Data; use lemmy_api_common::{ + build_response::build_post_response_deleted_allowed, + context::LemmyContext, post::{PostResponse, RemovePost}, - utils::{check_community_ban, get_local_user_view_from_jwt, is_mod_or_admin}, + utils::{check_community_ban, is_mod_or_admin, local_user_view_from_jwt}, }; -use lemmy_apub::activities::deletion::{send_apub_delete_in_community, DeletableObjects}; use lemmy_db_schema::{ source::{ - community::Community, moderator::{ModRemovePost, ModRemovePostForm}, post::{Post, PostUpdateForm}, }, traits::Crud, }; -use lemmy_utils::{error::LemmyError, ConnectionId}; -use lemmy_websocket::{send::send_post_ws_message, LemmyContext, UserOperationCrud}; +use lemmy_utils::error::LemmyError; #[async_trait::async_trait(?Send)] impl PerformCrud for RemovePost { type Response = PostResponse; - #[tracing::instrument(skip(context, websocket_id))] - async fn perform( - &self, - context: &Data, - websocket_id: Option, - ) -> Result { + #[tracing::instrument(skip(context))] + async fn perform(&self, context: &Data) -> Result { let data: &RemovePost = self; - let local_user_view = - get_local_user_view_from_jwt(&data.auth, context.pool(), context.secret()).await?; + let local_user_view = local_user_view_from_jwt(&data.auth, context).await?; let post_id = data.post_id; - let orig_post = Post::read(context.pool(), post_id).await?; + let orig_post = Post::read(&mut context.pool(), post_id).await?; check_community_ban( local_user_view.person.id, orig_post.community_id, - context.pool(), + &mut context.pool(), ) .await?; // Verify that only the mods can remove is_mod_or_admin( - context.pool(), + &mut context.pool(), local_user_view.person.id, orig_post.community_id, ) @@ -51,8 +45,8 @@ impl PerformCrud for RemovePost { // Update the post let post_id = data.post_id; let removed = data.removed; - let updated_post = Post::update( - context.pool(), + Post::update( + &mut context.pool(), post_id, &PostUpdateForm::builder().removed(Some(removed)).build(), ) @@ -63,31 +57,16 @@ impl PerformCrud for RemovePost { mod_person_id: local_user_view.person.id, post_id: data.post_id, removed: Some(removed), - reason: data.reason.to_owned(), + reason: data.reason.clone(), }; - ModRemovePost::create(context.pool(), &form).await?; + ModRemovePost::create(&mut context.pool(), &form).await?; - let res = send_post_ws_message( - data.post_id, - UserOperationCrud::RemovePost, - websocket_id, - Some(local_user_view.person.id), - context, - ) - .await?; - - // apub updates - let community = Community::read(context.pool(), orig_post.community_id).await?; - let deletable = DeletableObjects::Post(Box::new(updated_post.into())); - send_apub_delete_in_community( - local_user_view.person, - community, - deletable, - data.reason.clone().or_else(|| Some("".to_string())), - removed, + build_post_response_deleted_allowed( context, + orig_post.community_id, + local_user_view.person.id, + post_id, ) - .await?; - Ok(res) + .await } }