-use crate::PerformCrud;
use actix_web::web::Data;
-use lemmy_api_common::{blocking, check_community_ban, get_local_user_view_from_jwt, post::*};
-use lemmy_apub::activities::{post::create_or_update::CreateOrUpdatePost, CreateOrUpdateType};
-use lemmy_db_queries::{source::post::Post_, Crud, DeleteableOrRemoveable};
-use lemmy_db_schema::{naive_now, source::post::*};
-use lemmy_db_views::post_view::PostView;
+use lemmy_api_common::{
+ post::{EditPost, PostResponse},
+ request::fetch_site_data,
+ utils::{
+ blocking,
+ check_community_ban,
+ check_community_deleted_or_removed,
+ get_local_user_view_from_jwt,
+ },
+};
+use lemmy_apub::protocol::activities::{
+ create_or_update::post::CreateOrUpdatePost,
+ CreateOrUpdateType,
+};
+use lemmy_db_schema::{
+ source::post::{Post, PostForm},
+ traits::Crud,
+ utils::naive_now,
+};
use lemmy_utils::{
- request::fetch_iframely_and_pictrs_data,
- utils::{check_slurs_opt, clean_url_params, is_valid_post_title},
- ApiError,
+ error::LemmyError,
+ utils::{check_slurs_opt, clean_optional_text, clean_url_params, is_valid_post_title},
ConnectionId,
- LemmyError,
};
-use lemmy_websocket::{messages::SendPost, LemmyContext, UserOperationCrud};
+use lemmy_websocket::{send::send_post_ws_message, LemmyContext, UserOperationCrud};
+
+use crate::PerformCrud;
#[async_trait::async_trait(?Send)]
impl PerformCrud for EditPost {
type Response = PostResponse;
+ #[tracing::instrument(skip(context, websocket_id))]
async fn perform(
&self,
context: &Data<LemmyContext>,
websocket_id: Option<ConnectionId>,
) -> Result<PostResponse, LemmyError> {
let data: &EditPost = self;
- let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
+ let local_user_view =
+ get_local_user_view_from_jwt(&data.auth, context.pool(), context.secret()).await?;
- check_slurs_opt(&data.name)?;
- check_slurs_opt(&data.body)?;
+ let slur_regex = &context.settings().slur_regex();
+ check_slurs_opt(&data.name, slur_regex)?;
+ check_slurs_opt(&data.body, slur_regex)?;
if let Some(name) = &data.name {
if !is_valid_post_title(name) {
- return Err(ApiError::err("invalid_post_title").into());
+ return Err(LemmyError::from_message("invalid_post_title"));
}
}
context.pool(),
)
.await?;
+ check_community_deleted_or_removed(orig_post.community_id, context.pool()).await?;
// Verify that only the creator can edit
if !Post::is_post_creator(local_user_view.person.id, orig_post.creator_id) {
- return Err(ApiError::err("no_post_edit_allowed").into());
+ return Err(LemmyError::from_message("no_post_edit_allowed"));
}
- // Fetch Iframely and Pictrs cached image
+ // Fetch post links and Pictrs cached image
let data_url = data.url.as_ref();
- let (iframely_response, pictrs_thumbnail) =
- fetch_iframely_and_pictrs_data(context.client(), data_url).await?;
- let (embed_title, embed_description, embed_html) = iframely_response
- .map(|u| (u.title, u.description, u.html))
- .unwrap_or((None, None, None));
+ let (metadata_res, thumbnail_url) =
+ fetch_site_data(context.client(), &context.settings(), data_url).await;
+ let (embed_title, embed_description, embed_video_url) = metadata_res
+ .map(|u| (u.title, u.description, u.embed_video_url))
+ .unwrap_or_default();
let post_form = PostForm {
creator_id: orig_post.creator_id.to_owned(),
community_id: orig_post.community_id,
name: data.name.to_owned().unwrap_or(orig_post.name),
url: data_url.map(|u| clean_url_params(u.to_owned()).into()),
- body: data.body.to_owned(),
+ body: clean_optional_text(&data.body),
nsfw: data.nsfw,
updated: Some(naive_now()),
embed_title,
embed_description,
- embed_html,
- thumbnail_url: pictrs_thumbnail.map(|u| u.into()),
+ embed_video_url,
+ thumbnail_url,
..PostForm::default()
};
"couldnt_update_post"
};
- return Err(ApiError::err(err_type).into());
+ return Err(LemmyError::from_error_message(e, err_type));
}
};
// Send apub update
CreateOrUpdatePost::send(
- &updated_post,
- &local_user_view.person,
+ updated_post.into(),
+ &local_user_view.person.clone().into(),
CreateOrUpdateType::Update,
context,
)
.await?;
- let post_id = data.post_id;
- let mut post_view = blocking(context.pool(), move |conn| {
- PostView::read(conn, post_id, Some(local_user_view.person.id))
- })
- .await??;
-
- // Blank out deleted info
- if post_view.post.deleted || post_view.post.removed {
- post_view.post = post_view.post.blank_out_deleted_or_removed_info();
- }
-
- let res = PostResponse { post_view };
-
- context.chat_server().do_send(SendPost {
- op: UserOperationCrud::EditPost,
- post: res.clone(),
+ send_post_ws_message(
+ data.post_id,
+ UserOperationCrud::EditPost,
websocket_id,
- });
-
- Ok(res)
+ Some(local_user_view.person.id),
+ context,
+ )
+ .await
}
}