]> Untitled Git - lemmy.git/commitdiff
Running clippy --fix (#1647)
authorDessalines <dessalines@users.noreply.github.com>
Mon, 5 Jul 2021 16:07:26 +0000 (12:07 -0400)
committerGitHub <noreply@github.com>
Mon, 5 Jul 2021 16:07:26 +0000 (16:07 +0000)
73 files changed:
Cargo.lock
crates/api/Cargo.toml
crates/api/src/comment.rs
crates/api/src/comment_report.rs
crates/api/src/community.rs
crates/api/src/lib.rs
crates/api/src/local_user.rs
crates/api/src/post.rs
crates/api/src/post_report.rs
crates/api/src/private_message.rs
crates/api/src/site.rs
crates/api/src/websocket.rs
crates/api_common/src/lib.rs
crates/api_crud/Cargo.toml
crates/api_crud/src/comment/create.rs
crates/api_crud/src/comment/delete.rs
crates/api_crud/src/comment/read.rs
crates/api_crud/src/comment/update.rs
crates/api_crud/src/community/create.rs
crates/api_crud/src/community/delete.rs
crates/api_crud/src/community/read.rs
crates/api_crud/src/community/update.rs
crates/api_crud/src/lib.rs
crates/api_crud/src/post/create.rs
crates/api_crud/src/post/delete.rs
crates/api_crud/src/post/read.rs
crates/api_crud/src/post/update.rs
crates/api_crud/src/private_message/create.rs
crates/api_crud/src/private_message/delete.rs
crates/api_crud/src/private_message/read.rs
crates/api_crud/src/private_message/update.rs
crates/api_crud/src/site/create.rs
crates/api_crud/src/site/read.rs
crates/api_crud/src/site/update.rs
crates/api_crud/src/user/create.rs
crates/api_crud/src/user/delete.rs
crates/api_crud/src/user/read.rs
crates/apub/Cargo.toml
crates/apub/src/activities/send/comment.rs
crates/apub/src/activities/send/community.rs
crates/apub/src/activities/send/person.rs
crates/apub/src/activities/send/post.rs
crates/apub/src/extensions/signatures.rs
crates/apub/src/fetcher/community.rs
crates/apub/src/fetcher/fetch.rs
crates/apub/src/fetcher/search.rs
crates/apub/src/objects/comment.rs
crates/apub/src/objects/community.rs
crates/apub/src/objects/mod.rs
crates/apub/src/objects/post.rs
crates/apub_receive/src/activities/receive/mod.rs
crates/apub_receive/src/activities/receive/private_message.rs
crates/apub_receive/src/http/community.rs
crates/apub_receive/src/http/mod.rs
crates/apub_receive/src/http/person.rs
crates/apub_receive/src/inbox/community_inbox.rs
crates/apub_receive/src/inbox/mod.rs
crates/apub_receive/src/inbox/person_inbox.rs
crates/apub_receive/src/inbox/receive_for_community.rs
crates/apub_receive/src/inbox/shared_inbox.rs
crates/db_queries/src/lib.rs
crates/db_queries/src/source/activity.rs
crates/db_queries/src/source/local_user.rs
crates/db_queries/src/source/password_reset_request.rs
crates/routes/src/feeds.rs
crates/utils/Cargo.toml
crates/utils/src/claims.rs
crates/utils/src/request.rs
crates/utils/src/test.rs
crates/utils/src/utils.rs
crates/websocket/src/chat_server.rs
src/code_migrations.rs
src/scheduled_tasks.rs

index b5c654131e02908915841305d32e2a76a4fa577e..286efdf5bb766568026a67ee075709584f552a32 100644 (file)
@@ -3438,18 +3438,18 @@ dependencies = [
 
 [[package]]
 name = "thiserror"
-version = "1.0.23"
+version = "1.0.26"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "76cc616c6abf8c8928e2fdcc0dbfab37175edd8fb49a4641066ad1364fdab146"
+checksum = "93119e4feac1cbe6c798c34d3a53ea0026b0b1de6a120deef895137c0529bfe2"
 dependencies = [
  "thiserror-impl",
 ]
 
 [[package]]
 name = "thiserror-impl"
-version = "1.0.23"
+version = "1.0.26"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9be73a2caec27583d0046ef3796c3794f868a5bc813db689eed00c7631275cd1"
+checksum = "060d69a0afe7796bf42e9e2ff91f5ee691fb15c53d38b4b62a9a53eb23164745"
 dependencies = [
  "proc-macro2 1.0.24",
  "quote 1.0.8",
index 1aaa0528e68306a26f8867ce448cc2239e574edc..a22116642d17e2a944d3a35bac057e18f69262a0 100644 (file)
@@ -45,6 +45,6 @@ sha2 = "0.9.3"
 async-trait = "0.1.42"
 captcha = "0.0.8"
 anyhow = "1.0.38"
-thiserror = "1.0.23"
+thiserror = "1.0.26"
 background-jobs = "0.8.0"
 reqwest = { version = "0.10.10", features = ["json"] }
index dd373cb107503cea2a246af01c70b93641747176..ff1010fb1bfb997c04f24e5cd0f4ec7337aa7a33 100644 (file)
@@ -23,12 +23,12 @@ impl Perform for MarkCommentAsRead {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &MarkCommentAsRead = &self;
+    let data: &MarkCommentAsRead = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let comment_id = data.comment_id;
     let orig_comment = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
@@ -79,7 +79,7 @@ impl Perform for SaveComment {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &SaveComment = &self;
+    let data: &SaveComment = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let comment_saved_form = CommentSavedForm {
@@ -123,7 +123,7 @@ impl Perform for CreateCommentLike {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &CreateCommentLike = &self;
+    let data: &CreateCommentLike = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let mut recipient_ids = Vec::<LocalUserId>::new();
@@ -133,7 +133,7 @@ impl Perform for CreateCommentLike {
 
     let comment_id = data.comment_id;
     let orig_comment = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
index 5cdf697c23dc85f5431873f5060c28fcd3c0f015..83aebfc526f59638ce770ab58924492f09d94e0a 100644 (file)
@@ -31,7 +31,7 @@ impl Perform for CreateCommentReport {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CreateCommentReportResponse, LemmyError> {
-    let data: &CreateCommentReport = &self;
+    let data: &CreateCommentReport = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // check size of report and check for whitespace
@@ -46,7 +46,7 @@ impl Perform for CreateCommentReport {
     let person_id = local_user_view.person.id;
     let comment_id = data.comment_id;
     let comment_view = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
@@ -95,12 +95,12 @@ impl Perform for ResolveCommentReport {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<ResolveCommentReportResponse, LemmyError> {
-    let data: &ResolveCommentReport = &self;
+    let data: &ResolveCommentReport = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let report_id = data.report_id;
     let report = blocking(context.pool(), move |conn| {
-      CommentReportView::read(&conn, report_id)
+      CommentReportView::read(conn, report_id)
     })
     .await??;
 
@@ -148,7 +148,7 @@ impl Perform for ListCommentReports {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<ListCommentReportsResponse, LemmyError> {
-    let data: &ListCommentReports = &self;
+    let data: &ListCommentReports = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.person.id;
index b00160538b07ab0372ac55e5b22edfe0885f7db5..017ef84934dc3280d3f4e06deeb1aef134137ef3 100644 (file)
@@ -42,7 +42,7 @@ impl Perform for FollowCommunity {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityResponse, LemmyError> {
-    let data: &FollowCommunity = &self;
+    let data: &FollowCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let community_id = data.community_id;
@@ -116,7 +116,7 @@ impl Perform for BanFromCommunity {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<BanFromCommunityResponse, LemmyError> {
-    let data: &BanFromCommunity = &self;
+    let data: &BanFromCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let community_id = data.community_id;
@@ -246,7 +246,7 @@ impl Perform for AddModToCommunity {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<AddModToCommunityResponse, LemmyError> {
-    let data: &AddModToCommunity = &self;
+    let data: &AddModToCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let community_id = data.community_id;
@@ -333,7 +333,7 @@ impl Perform for TransferCommunity {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetCommunityResponse, LemmyError> {
-    let data: &TransferCommunity = &self;
+    let data: &TransferCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let site_creator_id = blocking(context.pool(), move |conn| {
index 0ba3fbea21370913f7b533ddb1afaec718fb1576..6234e756566c9247107b401bbd1df7a47f978e0f 100644 (file)
@@ -154,7 +154,7 @@ where
   for<'de> Data: Deserialize<'de> + 'a,
   Data: Perform,
 {
-  let parsed_data: Data = serde_json::from_str(&data)?;
+  let parsed_data: Data = serde_json::from_str(data)?;
   let res = parsed_data
     .perform(&web::Data::new(context), Some(id))
     .await?;
@@ -212,7 +212,7 @@ mod tests {
 
     // The check should fail, since the validator time is now newer than the jwt issue time
     let updated_local_user =
-      LocalUser::update_password(&conn, inserted_local_user.id, &"password111").unwrap();
+      LocalUser::update_password(&conn, inserted_local_user.id, "password111").unwrap();
     let check_after = check_validator_time(&updated_local_user.validator_time, &claims);
     assert!(check_after.is_err());
 
index 7fb4b83f9f4874dc77176ba151470a61f9b6209d..47d87d3f617d558cffd32ccbd49ab2d2d0410f55 100644 (file)
@@ -79,7 +79,7 @@ impl Perform for Login {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &Login = &self;
+    let data: &Login = self;
 
     // Fetch that username / email
     let username_or_email = data.username_or_email.clone();
@@ -160,7 +160,7 @@ impl Perform for SaveUserSettings {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &SaveUserSettings = &self;
+    let data: &SaveUserSettings = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let avatar = diesel_option_overwrite_to_url(&data.avatar)?;
@@ -279,7 +279,7 @@ impl Perform for ChangePassword {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &ChangePassword = &self;
+    let data: &ChangePassword = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     password_length_check(&data.new_password)?;
@@ -322,7 +322,7 @@ impl Perform for AddAdmin {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<AddAdminResponse, LemmyError> {
-    let data: &AddAdmin = &self;
+    let data: &AddAdmin = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Make sure user is an admin
@@ -384,7 +384,7 @@ impl Perform for BanPerson {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<BanPersonResponse, LemmyError> {
-    let data: &BanPerson = &self;
+    let data: &BanPerson = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Make sure user is an admin
@@ -460,7 +460,7 @@ impl Perform for GetReplies {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetRepliesResponse, LemmyError> {
-    let data: &GetReplies = &self;
+    let data: &GetReplies = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let sort: Option<SortType> = from_opt_str_to_opt_enum(&data.sort);
@@ -497,7 +497,7 @@ impl Perform for GetPersonMentions {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetPersonMentionsResponse, LemmyError> {
-    let data: &GetPersonMentions = &self;
+    let data: &GetPersonMentions = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let sort: Option<SortType> = from_opt_str_to_opt_enum(&data.sort);
@@ -531,7 +531,7 @@ impl Perform for MarkPersonMentionAsRead {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<PersonMentionResponse, LemmyError> {
-    let data: &MarkPersonMentionAsRead = &self;
+    let data: &MarkPersonMentionAsRead = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_mention_id = data.person_mention_id;
@@ -574,7 +574,7 @@ impl Perform for MarkAllAsRead {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetRepliesResponse, LemmyError> {
-    let data: &MarkAllAsRead = &self;
+    let data: &MarkAllAsRead = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.person.id;
@@ -629,7 +629,7 @@ impl Perform for PasswordReset {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<PasswordResetResponse, LemmyError> {
-    let data: &PasswordReset = &self;
+    let data: &PasswordReset = self;
 
     // Fetch that email
     let email = data.email.clone();
@@ -672,7 +672,7 @@ impl Perform for PasswordChange {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &PasswordChange = &self;
+    let data: &PasswordChange = self;
 
     // Fetch the user_id from the token
     let token = data.token.clone();
@@ -712,7 +712,7 @@ impl Perform for GetReportCount {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<GetReportCountResponse, LemmyError> {
-    let data: &GetReportCount = &self;
+    let data: &GetReportCount = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.person.id;
@@ -768,7 +768,7 @@ impl Perform for GetFollowedCommunities {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetFollowedCommunitiesResponse, LemmyError> {
-    let data: &GetFollowedCommunities = &self;
+    let data: &GetFollowedCommunities = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.person.id;
index f0e4d91faced00535d55aabd64e9c8a97e909762..4a55b7e05b876a06f27a5db07bfe2410ec317ec6 100644 (file)
@@ -25,7 +25,7 @@ impl Perform for CreatePostLike {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &CreatePostLike = &self;
+    let data: &CreatePostLike = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Don't do a downvote if site has downvotes disabled
@@ -102,7 +102,7 @@ impl Perform for LockPost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &LockPost = &self;
+    let data: &LockPost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let post_id = data.post_id;
@@ -172,7 +172,7 @@ impl Perform for StickyPost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &StickyPost = &self;
+    let data: &StickyPost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let post_id = data.post_id;
@@ -246,7 +246,7 @@ impl Perform for SavePost {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &SavePost = &self;
+    let data: &SavePost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let post_saved_form = PostSavedForm {
index b2ebbaf243cc7120327e21a0830a48fcc43ea34b..e718b775a48cced4f3eb4851928ba4e546655e1c 100644 (file)
@@ -38,7 +38,7 @@ impl Perform for CreatePostReport {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CreatePostReportResponse, LemmyError> {
-    let data: &CreatePostReport = &self;
+    let data: &CreatePostReport = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // check size of report and check for whitespace
@@ -53,7 +53,7 @@ impl Perform for CreatePostReport {
     let person_id = local_user_view.person.id;
     let post_id = data.post_id;
     let post_view = blocking(context.pool(), move |conn| {
-      PostView::read(&conn, post_id, None)
+      PostView::read(conn, post_id, None)
     })
     .await??;
 
@@ -104,12 +104,12 @@ impl Perform for ResolvePostReport {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<ResolvePostReportResponse, LemmyError> {
-    let data: &ResolvePostReport = &self;
+    let data: &ResolvePostReport = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let report_id = data.report_id;
     let report = blocking(context.pool(), move |conn| {
-      PostReportView::read(&conn, report_id)
+      PostReportView::read(conn, report_id)
     })
     .await??;
 
@@ -156,7 +156,7 @@ impl Perform for ListPostReports {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<ListPostReportsResponse, LemmyError> {
-    let data: &ListPostReports = &self;
+    let data: &ListPostReports = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.person.id;
index d0163dcf1345c101e9227b4aa4aab007f19c62d3..54bc3eb0cb4fa79d6690d1fd863769ce139354a3 100644 (file)
@@ -20,7 +20,7 @@ impl Perform for MarkPrivateMessageAsRead {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PrivateMessageResponse, LemmyError> {
-    let data: &MarkPrivateMessageAsRead = &self;
+    let data: &MarkPrivateMessageAsRead = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Checking permissions
index dab4583041ef455141d350679c4461adcd3f7da7..53f9bd30510e14837e9a589f5bc40c08568b9896 100644 (file)
@@ -60,7 +60,7 @@ impl Perform for GetModlog {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetModlogResponse, LemmyError> {
-    let data: &GetModlog = &self;
+    let data: &GetModlog = self;
 
     let community_id = data.community_id;
     let mod_person_id = data.mod_person_id;
@@ -134,7 +134,7 @@ impl Perform for Search {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<SearchResponse, LemmyError> {
-    let data: &Search = &self;
+    let data: &Search = self;
 
     match search_by_apub_id(&data.q, context).await {
       Ok(r) => return Ok(r),
@@ -191,7 +191,7 @@ impl Perform for Search {
       }
       SearchType::Comments => {
         comments = blocking(context.pool(), move |conn| {
-          CommentQueryBuilder::create(&conn)
+          CommentQueryBuilder::create(conn)
             .sort(sort)
             .listing_type(listing_type)
             .search_term(q)
@@ -347,7 +347,7 @@ impl Perform for TransferSite {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetSiteResponse, LemmyError> {
-    let data: &TransferSite = &self;
+    let data: &TransferSite = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     is_admin(&local_user_view)?;
@@ -410,7 +410,7 @@ impl Perform for GetSiteConfig {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetSiteConfigResponse, LemmyError> {
-    let data: &GetSiteConfig = &self;
+    let data: &GetSiteConfig = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Only let admins read this
@@ -431,7 +431,7 @@ impl Perform for SaveSiteConfig {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetSiteConfigResponse, LemmyError> {
-    let data: &SaveSiteConfig = &self;
+    let data: &SaveSiteConfig = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Only let admins read this
index 683b4c4d4ce3049c2f7a82dcb724378d6d972ca6..e69ffdd2421ce4c73d6765bf9c18620ce6e28c6e 100644 (file)
@@ -16,7 +16,7 @@ impl Perform for UserJoin {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<UserJoinResponse, LemmyError> {
-    let data: &UserJoin = &self;
+    let data: &UserJoin = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     if let Some(ws_id) = websocket_id {
@@ -39,7 +39,7 @@ impl Perform for CommunityJoin {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityJoinResponse, LemmyError> {
-    let data: &CommunityJoin = &self;
+    let data: &CommunityJoin = self;
 
     if let Some(ws_id) = websocket_id {
       context.chat_server().do_send(JoinCommunityRoom {
@@ -61,7 +61,7 @@ impl Perform for ModJoin {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<ModJoinResponse, LemmyError> {
-    let data: &ModJoin = &self;
+    let data: &ModJoin = self;
 
     if let Some(ws_id) = websocket_id {
       context.chat_server().do_send(JoinModRoom {
@@ -83,7 +83,7 @@ impl Perform for PostJoin {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostJoinResponse, LemmyError> {
-    let data: &PostJoin = &self;
+    let data: &PostJoin = self;
 
     if let Some(ws_id) = websocket_id {
       context.chat_server().do_send(JoinPostRoom {
index f2b41be50c8c027e619b4c93d4e7ac95bec07a67..c59bc500b98bac71eb521b2380ea9f041a7c0589 100644 (file)
@@ -112,7 +112,7 @@ fn do_send_local_notifs(
     .filter(|m| m.is_local() && m.name.ne(&person.name))
     .collect::<Vec<&MentionData>>()
   {
-    if let Ok(mention_user_view) = LocalUserView::read_from_name(&conn, &mention.name) {
+    if let Ok(mention_user_view) = LocalUserView::read_from_name(conn, &mention.name) {
       // TODO
       // At some point, make it so you can't tag the parent creator either
       // This can cause two notifications, one for reply and the other for mention
@@ -126,7 +126,7 @@ fn do_send_local_notifs(
 
       // Allow this to fail softly, since comment edits might re-update or replace it
       // Let the uniqueness handle this fail
-      PersonMention::create(&conn, &user_mention_form).ok();
+      PersonMention::create(conn, &user_mention_form).ok();
 
       // Send an email to those local users that have notifications on
       if do_send_email {
@@ -143,11 +143,11 @@ fn do_send_local_notifs(
   // Send notifs to the parent commenter / poster
   match comment.parent_id {
     Some(parent_id) => {
-      if let Ok(parent_comment) = Comment::read(&conn, parent_id) {
+      if let Ok(parent_comment) = Comment::read(conn, parent_id) {
         // Don't send a notif to yourself
         if parent_comment.creator_id != person.id {
           // Get the parent commenter local_user
-          if let Ok(parent_user_view) = LocalUserView::read_person(&conn, parent_comment.creator_id)
+          if let Ok(parent_user_view) = LocalUserView::read_person(conn, parent_comment.creator_id)
           {
             recipient_ids.push(parent_user_view.local_user.id);
 
@@ -166,7 +166,7 @@ fn do_send_local_notifs(
     // Its a post
     None => {
       if post.creator_id != person.id {
-        if let Ok(parent_user_view) = LocalUserView::read_person(&conn, post.creator_id) {
+        if let Ok(parent_user_view) = LocalUserView::read_person(conn, post.creator_id) {
           recipient_ids.push(parent_user_view.local_user.id);
 
           if do_send_email {
@@ -208,7 +208,7 @@ pub fn send_email_to_user(
       comment_content,
       Settings::get().get_protocol_and_hostname()
     );
-    match send_email(subject, &user_email, &local_user_view.person.name, html) {
+    match send_email(subject, user_email, &local_user_view.person.name, html) {
       Ok(_o) => _o,
       Err(e) => error!("{}", e),
     };
@@ -261,7 +261,7 @@ pub async fn get_local_user_view_from_jwt(
   jwt: &str,
   pool: &DbPool,
 ) -> Result<LocalUserView, LemmyError> {
-  let claims = Claims::decode(&jwt)
+  let claims = Claims::decode(jwt)
     .map_err(|_| ApiError::err("not_logged_in"))?
     .claims;
   let local_user_id = LocalUserId(claims.sub);
@@ -304,7 +304,7 @@ pub async fn get_local_user_settings_view_from_jwt(
   jwt: &str,
   pool: &DbPool,
 ) -> Result<LocalUserSettingsView, LemmyError> {
-  let claims = Claims::decode(&jwt)
+  let claims = Claims::decode(jwt)
     .map_err(|_| ApiError::err("not_logged_in"))?
     .claims;
   let local_user_id = LocalUserId(claims.sub);
index deebc12fa2c82a91d788c3f0e7ddb93cadee5ad7..236623451a869a9a014c934bbd2b735dbc591fe3 100644 (file)
@@ -39,6 +39,6 @@ uuid = { version = "0.8.2", features = ["serde", "v4"] }
 sha2 = "0.9.3"
 async-trait = "0.1.42"
 anyhow = "1.0.38"
-thiserror = "1.0.23"
+thiserror = "1.0.26"
 background-jobs = "0.8.0"
 reqwest = { version = "0.10.10", features = ["json"] }
index 4f1037c3ffac224acd29508434ccc6ba5d345ab3..7923b0836efcc1863691c7b0ac6387e8eebc61d9 100644 (file)
@@ -29,7 +29,7 @@ impl PerformCrud for CreateComment {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &CreateComment = &self;
+    let data: &CreateComment = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let content_slurs_removed = remove_slurs(&data.content.to_owned());
@@ -48,7 +48,7 @@ impl PerformCrud for CreateComment {
     // If there's a parent_id, check to make sure that comment is in that post
     if let Some(parent_id) = data.parent_id {
       // Make sure the parent comment exists
-      let parent = blocking(context.pool(), move |conn| Comment::read(&conn, parent_id))
+      let parent = blocking(context.pool(), move |conn| Comment::read(conn, parent_id))
         .await?
         .map_err(|_| ApiError::err("couldnt_create_comment"))?;
       if parent.post_id != post_id {
@@ -67,7 +67,7 @@ impl PerformCrud for CreateComment {
     // Create the comment
     let comment_form2 = comment_form.clone();
     let inserted_comment = blocking(context.pool(), move |conn| {
-      Comment::create(&conn, &comment_form2)
+      Comment::create(conn, &comment_form2)
     })
     .await?
     .map_err(|_| ApiError::err("couldnt_create_comment"))?;
@@ -78,7 +78,7 @@ impl PerformCrud for CreateComment {
       blocking(context.pool(), move |conn| -> Result<Comment, LemmyError> {
         let apub_id =
           generate_apub_endpoint(EndpointType::Comment, &inserted_comment_id.to_string())?;
-        Ok(Comment::update_ap_id(&conn, inserted_comment_id, apub_id)?)
+        Ok(Comment::update_ap_id(conn, inserted_comment_id, apub_id)?)
       })
       .await?
       .map_err(|_| ApiError::err("couldnt_create_comment"))?;
@@ -108,7 +108,7 @@ impl PerformCrud for CreateComment {
       score: 1,
     };
 
-    let like = move |conn: &'_ _| CommentLike::like(&conn, &like_form);
+    let like = move |conn: &'_ _| CommentLike::like(conn, &like_form);
     if blocking(context.pool(), like).await?.is_err() {
       return Err(ApiError::err("couldnt_like_comment").into());
     }
@@ -119,7 +119,7 @@ impl PerformCrud for CreateComment {
 
     let person_id = local_user_view.person.id;
     let mut comment_view = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, inserted_comment.id, Some(person_id))
+      CommentView::read(conn, inserted_comment.id, Some(person_id))
     })
     .await??;
 
index 8f928096576bd9602d5123374f404fc213dbe845..f58f3b0e41659b26c2a656b699efcdbb8d91b70b 100644 (file)
@@ -24,12 +24,12 @@ impl PerformCrud for DeleteComment {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &DeleteComment = &self;
+    let data: &DeleteComment = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let comment_id = data.comment_id;
     let orig_comment = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
@@ -110,12 +110,12 @@ impl PerformCrud for RemoveComment {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &RemoveComment = &self;
+    let data: &RemoveComment = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let comment_id = data.comment_id;
     let orig_comment = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
index 12ccbd5a1a6b6b6b7427d66ac3c44d7072673da9..767311d2a64d7fc04efd25971b1c1f8667f2093a 100644 (file)
@@ -15,7 +15,7 @@ impl PerformCrud for GetComments {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetCommentsResponse, LemmyError> {
-    let data: &GetComments = &self;
+    let data: &GetComments = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
 
     let show_bot_accounts = local_user_view
index e26a9884e0f64cf397c4dfc7c4ce8fe09f0dc64f..e6ed6ab59c80be69be81e810d7c2ee7ea16a79c9 100644 (file)
@@ -28,12 +28,12 @@ impl PerformCrud for EditComment {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommentResponse, LemmyError> {
-    let data: &EditComment = &self;
+    let data: &EditComment = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let comment_id = data.comment_id;
     let orig_comment = blocking(context.pool(), move |conn| {
-      CommentView::read(&conn, comment_id, None)
+      CommentView::read(conn, comment_id, None)
     })
     .await??;
 
index 88f4415bfcc97d91149a5bcf37dcf9490d07c644..5c55e149e5e407849569c0a554a31595c0b90463 100644 (file)
@@ -44,7 +44,7 @@ impl PerformCrud for CreateCommunity {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityResponse, LemmyError> {
-    let data: &CreateCommunity = &self;
+    let data: &CreateCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let site = blocking(context.pool(), move |conn| Site::read(conn, 0)).await??;
index 9511a2caaf3facc4ab38baa96547428b3be31204..2208a348b752b80cf141f89c1298aa339122ea5d 100644 (file)
@@ -23,7 +23,7 @@ impl PerformCrud for DeleteCommunity {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityResponse, LemmyError> {
-    let data: &DeleteCommunity = &self;
+    let data: &DeleteCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Fetch the community mods
@@ -87,7 +87,7 @@ impl PerformCrud for RemoveCommunity {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityResponse, LemmyError> {
-    let data: &RemoveCommunity = &self;
+    let data: &RemoveCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Verify its an admin (only an admin can remove a community)
index 7836878e96654d0c9624dc85bd32d4b09d64515d..eb11cb39759b6b6b1f333afff14df3185393ca61 100644 (file)
@@ -24,7 +24,7 @@ impl PerformCrud for GetCommunity {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetCommunityResponse, LemmyError> {
-    let data: &GetCommunity = &self;
+    let data: &GetCommunity = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
     let person_id = local_user_view.map(|u| u.person.id);
 
@@ -79,7 +79,7 @@ impl PerformCrud for ListCommunities {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<ListCommunitiesResponse, LemmyError> {
-    let data: &ListCommunities = &self;
+    let data: &ListCommunities = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.to_owned().map(|l| l.person.id);
index f28f771195f3c4ee04f5bb31760293f17e63bd2c..b17a223ee4b47ee250c98cb03ddb787c4fb4e330 100644 (file)
@@ -28,7 +28,7 @@ impl PerformCrud for EditCommunity {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<CommunityResponse, LemmyError> {
-    let data: &EditCommunity = &self;
+    let data: &EditCommunity = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     check_slurs_opt(&data.title)?;
index c6385d050b23c0d6ae29d703c50cfa0e31b1c2b8..f77632f908e97ee43406e95eae368ec4d10b1ec8 100644 (file)
@@ -124,7 +124,7 @@ where
   for<'de> Data: Deserialize<'de> + 'a,
   Data: PerformCrud,
 {
-  let parsed_data: Data = serde_json::from_str(&data)?;
+  let parsed_data: Data = serde_json::from_str(data)?;
   let res = parsed_data
     .perform(&web::Data::new(context), Some(id))
     .await?;
index 4ce7c0644607e9da3676b5b9ca19feae0b5b8a79..dc99f3ecf4f81052bbbe203691e6d7512b9442ac 100644 (file)
@@ -29,7 +29,7 @@ impl PerformCrud for CreatePost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &CreatePost = &self;
+    let data: &CreatePost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     check_slurs(&data.name)?;
index 32b4e9ff043f590dfa9b5979ab7365f412235044..e7fcb2fd679f9dbcf20aeb807cb2e69c1ed171b7 100644 (file)
@@ -23,7 +23,7 @@ impl PerformCrud for DeletePost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &DeletePost = &self;
+    let data: &DeletePost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let post_id = data.post_id;
@@ -88,7 +88,7 @@ impl PerformCrud for RemovePost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &RemovePost = &self;
+    let data: &RemovePost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let post_id = data.post_id;
index d49764bba269783b09970ad4425aee4eeb09be50..4a25b3a8d0442ef6a3973df34f8cde893264ac38 100644 (file)
@@ -22,7 +22,7 @@ impl PerformCrud for GetPost {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetPostResponse, LemmyError> {
-    let data: &GetPost = &self;
+    let data: &GetPost = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
 
     let show_bot_accounts = local_user_view
@@ -92,7 +92,7 @@ impl PerformCrud for GetPosts {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetPostsResponse, LemmyError> {
-    let data: &GetPosts = &self;
+    let data: &GetPosts = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
 
     let person_id = local_user_view.to_owned().map(|l| l.person.id);
index 59f8e446a170ad3a22c2dfaa24c1b2705a462d23..f215da7d03dacc3780a8e2aa69b2774b74d2fc39 100644 (file)
@@ -23,7 +23,7 @@ impl PerformCrud for EditPost {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PostResponse, LemmyError> {
-    let data: &EditPost = &self;
+    let data: &EditPost = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     check_slurs_opt(&data.name)?;
index 2dc223ec01173dac476fff3f0e33cba767a168e5..3e3074ad83290fe78f50dbd28d13d05f8e281d5a 100644 (file)
@@ -22,7 +22,7 @@ impl PerformCrud for CreatePrivateMessage {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PrivateMessageResponse, LemmyError> {
-    let data: &CreatePrivateMessage = &self;
+    let data: &CreatePrivateMessage = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     let content_slurs_removed = remove_slurs(&data.content.to_owned());
@@ -54,7 +54,7 @@ impl PerformCrud for CreatePrivateMessage {
           &inserted_private_message_id.to_string(),
         )?;
         Ok(PrivateMessage::update_ap_id(
-          &conn,
+          conn,
           inserted_private_message_id,
           apub_id,
         )?)
index 65c0022ceba656deb0e351a53acd182e8f54a1ad..6befb75da8e9e29d7a5b5c0182c2555c75b02edf 100644 (file)
@@ -21,7 +21,7 @@ impl PerformCrud for DeletePrivateMessage {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PrivateMessageResponse, LemmyError> {
-    let data: &DeletePrivateMessage = &self;
+    let data: &DeletePrivateMessage = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Checking permissions
index 79bc85f45bd391078e97694aca6e68946b2f13ae..6706bf419b9c53df0808f9c6e1aed761a668c82d 100644 (file)
@@ -18,7 +18,7 @@ impl PerformCrud for GetPrivateMessages {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<PrivateMessagesResponse, LemmyError> {
-    let data: &GetPrivateMessages = &self;
+    let data: &GetPrivateMessages = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
     let person_id = local_user_view.person.id;
 
@@ -26,7 +26,7 @@ impl PerformCrud for GetPrivateMessages {
     let limit = data.limit;
     let unread_only = data.unread_only;
     let messages = blocking(context.pool(), move |conn| {
-      PrivateMessageQueryBuilder::create(&conn, person_id)
+      PrivateMessageQueryBuilder::create(conn, person_id)
         .page(page)
         .limit(limit)
         .unread_only(unread_only)
index b073c928c0b98973d3e9073bd93f22a6cb271006..35a9806a8fdb643d50651d6b3a18c670346c3cff 100644 (file)
@@ -21,7 +21,7 @@ impl PerformCrud for EditPrivateMessage {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<PrivateMessageResponse, LemmyError> {
-    let data: &EditPrivateMessage = &self;
+    let data: &EditPrivateMessage = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Checking permissions
index 2b876c2ff6cd7a8b8ea4d7c3f17662b40e60b16d..b84adcdb4e88d3342bdd5b3a82cbfc55365d904c 100644 (file)
@@ -32,7 +32,7 @@ impl PerformCrud for CreateSite {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<SiteResponse, LemmyError> {
-    let data: &CreateSite = &self;
+    let data: &CreateSite = self;
 
     let read_site = move |conn: &'_ _| Site::read_simple(conn);
     if blocking(context.pool(), read_site).await?.is_ok() {
index 9b73c12f83666c88cf4b5b91095106a816300f9c..da379e322cb732c69bf7d8957030d81e3686ed5c 100644 (file)
@@ -22,7 +22,7 @@ impl PerformCrud for GetSite {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<GetSiteResponse, LemmyError> {
-    let data: &GetSite = &self;
+    let data: &GetSite = self;
 
     let site_view = match blocking(context.pool(), move |conn| SiteView::read(conn)).await? {
       Ok(site_view) => Some(site_view),
index 14716e782282ff2433df17112969ab616ee63a90..781ea00812041bda6afc9f99c25e0c5e72683e55 100644 (file)
@@ -29,7 +29,7 @@ impl PerformCrud for EditSite {
     context: &Data<LemmyContext>,
     websocket_id: Option<ConnectionId>,
   ) -> Result<SiteResponse, LemmyError> {
-    let data: &EditSite = &self;
+    let data: &EditSite = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     check_slurs_opt(&data.name)?;
index d7aae65f024d80dd41f43b41a21ddb895b5cad3f..ec227fefaa73a5b0f4e7b9b0ab09f00318a18d7d 100644 (file)
@@ -46,7 +46,7 @@ impl PerformCrud for Register {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &Register = &self;
+    let data: &Register = self;
 
     // Make sure site has open registration
     if let Ok(site) = blocking(context.pool(), move |conn| Site::read_simple(conn)).await? {
@@ -151,7 +151,7 @@ impl PerformCrud for Register {
 
         // If the local user creation errored, then delete that person
         blocking(context.pool(), move |conn| {
-          Person::delete(&conn, inserted_person.id)
+          Person::delete(conn, inserted_person.id)
         })
         .await??;
 
index ca88830c38672114c97bcb0b5d32437cec952ca8..fb9480dc7e74052871cf892b4f55c1b9998a4603 100644 (file)
@@ -16,7 +16,7 @@ impl PerformCrud for DeleteAccount {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<LoginResponse, LemmyError> {
-    let data: &DeleteAccount = &self;
+    let data: &DeleteAccount = self;
     let local_user_view = get_local_user_view_from_jwt(&data.auth, context.pool()).await?;
 
     // Verify the password
index 132bdd78f4e32d633332f098b97fac9b081aea14..f7275ef701af5e0b4b781f00eb48660dcb181b22 100644 (file)
@@ -21,7 +21,7 @@ impl PerformCrud for GetPersonDetails {
     context: &Data<LemmyContext>,
     _websocket_id: Option<ConnectionId>,
   ) -> Result<GetPersonDetailsResponse, LemmyError> {
-    let data: &GetPersonDetails = &self;
+    let data: &GetPersonDetails = self;
     let local_user_view = get_local_user_view_from_jwt_opt(&data.auth, context.pool()).await?;
 
     let show_nsfw = local_user_view.as_ref().map(|t| t.local_user.show_nsfw);
index 7d413f8511f9f50b90a73e22ad82bbf97a7db92e..35601db1ebb8ef92ed7f64225674fc5e90f9350d 100644 (file)
@@ -46,7 +46,7 @@ uuid = { version = "0.8.2", features = ["serde", "v4"] }
 sha2 = "0.9.3"
 async-trait = "0.1.42"
 anyhow = "1.0.38"
-thiserror = "1.0.23"
+thiserror = "1.0.26"
 background-jobs = "0.8.0"
 reqwest = { version = "0.10.10", features = ["json"] }
 backtrace = "0.3.56"
index b272557557f3508b91ce81fddae3ab513078d3b8..b93f9e1c9b90d2350194321746cffc766f8e495f 100644 (file)
@@ -57,7 +57,7 @@ impl ApubObjectType for Comment {
     })
     .await??;
 
-    let maa = collect_non_local_mentions(&self, &community, context).await?;
+    let maa = collect_non_local_mentions(self, &community, context).await?;
 
     let mut create = Create::new(
       creator.actor_id.to_owned().into_inner(),
@@ -71,8 +71,8 @@ impl ApubObjectType for Comment {
       // Set the mention tags
       .set_many_tags(maa.get_tags()?);
 
-    send_to_community(create.clone(), &creator, &community, None, context).await?;
-    send_comment_mentions(&creator, maa.inboxes, create, context).await?;
+    send_to_community(create.clone(), creator, &community, None, context).await?;
+    send_comment_mentions(creator, maa.inboxes, create, context).await?;
     Ok(())
   }
 
@@ -90,7 +90,7 @@ impl ApubObjectType for Comment {
     })
     .await??;
 
-    let maa = collect_non_local_mentions(&self, &community, context).await?;
+    let maa = collect_non_local_mentions(self, &community, context).await?;
 
     let mut update = Update::new(
       creator.actor_id.to_owned().into_inner(),
@@ -104,8 +104,8 @@ impl ApubObjectType for Comment {
       // Set the mention tags
       .set_many_tags(maa.get_tags()?);
 
-    send_to_community(update.clone(), &creator, &community, None, context).await?;
-    send_comment_mentions(&creator, maa.inboxes, update, context).await?;
+    send_to_community(update.clone(), creator, &community, None, context).await?;
+    send_comment_mentions(creator, maa.inboxes, update, context).await?;
     Ok(())
   }
 
@@ -129,7 +129,7 @@ impl ApubObjectType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(delete, &creator, &community, None, context).await?;
+    send_to_community(delete, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -169,7 +169,7 @@ impl ApubObjectType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(undo, &creator, &community, None, context).await?;
+    send_to_community(undo, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -193,7 +193,7 @@ impl ApubObjectType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(remove, &mod_, &community, None, context).await?;
+    send_to_community(remove, mod_, &community, None, context).await?;
     Ok(())
   }
 
@@ -233,7 +233,7 @@ impl ApubObjectType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(undo, &mod_, &community, None, context).await?;
+    send_to_community(undo, mod_, &community, None, context).await?;
     Ok(())
   }
 }
@@ -260,7 +260,7 @@ impl ApubLikeableType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(like, &creator, &community, None, context).await?;
+    send_to_community(like, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -284,7 +284,7 @@ impl ApubLikeableType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(dislike, &creator, &community, None, context).await?;
+    send_to_community(dislike, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -323,7 +323,7 @@ impl ApubLikeableType for Comment {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(undo, &creator, &community, None, context).await?;
+    send_to_community(undo, creator, &community, None, context).await?;
     Ok(())
   }
 }
index c5112f58cac0229645e9ffaabd988964a91e87d4..378c5bd60fa345302495511a2b46b8cf3ff97c21 100644 (file)
@@ -328,7 +328,7 @@ impl CommunityType for Community {
       .set_many_ccs(vec![self.actor_id()])
       .set_target(generate_moderators_url(&self.actor_id)?.into_inner());
 
-    send_to_community(remove, &actor, self, Some(removed_mod.actor_id()), context).await?;
+    send_to_community(remove, actor, self, Some(removed_mod.actor_id()), context).await?;
     Ok(())
   }
 
@@ -345,7 +345,7 @@ impl CommunityType for Community {
       .set_to(public())
       .set_many_ccs(vec![self.actor_id()]);
 
-    send_to_community(block, &actor, self, Some(blocked_user.actor_id()), context).await?;
+    send_to_community(block, actor, self, Some(blocked_user.actor_id()), context).await?;
     Ok(())
   }
 
@@ -370,7 +370,7 @@ impl CommunityType for Community {
       .set_to(public())
       .set_many_ccs(vec![self.actor_id()]);
 
-    send_to_community(undo, &actor, self, Some(unblocked_user.actor_id()), context).await?;
+    send_to_community(undo, actor, self, Some(unblocked_user.actor_id()), context).await?;
     Ok(())
   }
 }
index 286778c7826e7fbfe7c2d383ab6863ff291425a3..1e72a8578cea548ffd5f943f6ee286a798d564ee 100644 (file)
@@ -69,7 +69,7 @@ impl UserType for Person {
       person_id: self.id,
       pending: true,
     };
-    blocking(&context.pool(), move |conn| {
+    blocking(context.pool(), move |conn| {
       CommunityFollower::follow(conn, &community_follower_form).ok()
     })
     .await?;
index 0af7369ac73a7cc6395979029ee0943b283c2bd8..c51d6f2d6b27eb3b0671e44888f03542b51dd0d2 100644 (file)
@@ -211,7 +211,7 @@ impl ApubLikeableType for Post {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(like, &creator, &community, None, context).await?;
+    send_to_community(like, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -232,7 +232,7 @@ impl ApubLikeableType for Post {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(dislike, &creator, &community, None, context).await?;
+    send_to_community(dislike, creator, &community, None, context).await?;
     Ok(())
   }
 
@@ -268,7 +268,7 @@ impl ApubLikeableType for Post {
       .set_to(public())
       .set_many_ccs(vec![community.actor_id()]);
 
-    send_to_community(undo, &creator, &community, None, context).await?;
+    send_to_community(undo, creator, &community, None, context).await?;
     Ok(())
   }
 }
index 92e2a306d3fc34e2ace14411b9542e834c3a7058..47b8e5a2b4e7b9343332e1c0852682844e34a9d3 100644 (file)
@@ -80,7 +80,7 @@ pub fn verify_signature(request: &HttpRequest, actor: &dyn ActorType) -> Result<
       );
       let public_key = PKey::public_key_from_pem(public_key.as_bytes())?;
       let mut verifier = Verifier::new(MessageDigest::sha256(), &public_key)?;
-      verifier.update(&signing_string.as_bytes())?;
+      verifier.update(signing_string.as_bytes())?;
       Ok(verifier.verify(&base64::decode(signature)?)?)
     })?;
 
index 6187ac10cf3fe58a33b9c74de7b2f34c680d003f..f0a759c54592530321a91b9a1ce49d96b3652dbb 100644 (file)
@@ -16,10 +16,7 @@ use anyhow::Context;
 use diesel::result::Error::NotFound;
 use lemmy_api_common::blocking;
 use lemmy_db_queries::{source::community::Community_, ApubObject, Joinable};
-use lemmy_db_schema::{
-  source::community::{Community, CommunityModerator, CommunityModeratorForm},
-  DbUrl,
-};
+use lemmy_db_schema::source::community::{Community, CommunityModerator, CommunityModeratorForm};
 use lemmy_db_views_actor::community_moderator_view::CommunityModeratorView;
 use lemmy_utils::{location_info, LemmyError};
 use lemmy_websocket::LemmyContext;
@@ -102,12 +99,12 @@ async fn update_community_mods(
   let new_moderators = fetch_community_mods(context, group, request_counter).await?;
   let community_id = community.id;
   let current_moderators = blocking(context.pool(), move |conn| {
-    CommunityModeratorView::for_community(&conn, community_id)
+    CommunityModeratorView::for_community(conn, community_id)
   })
   .await??;
   // Remove old mods from database which arent in the moderators collection anymore
   for mod_user in &current_moderators {
-    if !new_moderators.contains(&&mod_user.moderator.actor_id.clone().into()) {
+    if !new_moderators.contains(&mod_user.moderator.actor_id.clone().into()) {
       let community_moderator_form = CommunityModeratorForm {
         community_id: mod_user.community.id,
         person_id: mod_user.moderator.id,
@@ -122,12 +119,13 @@ async fn update_community_mods(
   // Add new mods to database which have been added to moderators collection
   for mod_uri in new_moderators {
     let mod_user = get_or_fetch_and_upsert_person(&mod_uri, context, request_counter).await?;
-    let current_mod_uris: Vec<DbUrl> = current_moderators
+
+    if !current_moderators
       .clone()
       .iter()
       .map(|c| c.moderator.actor_id.clone())
-      .collect();
-    if !current_mod_uris.contains(&mod_user.actor_id) {
+      .any(|x| x == mod_user.actor_id)
+    {
       let community_moderator_form = CommunityModeratorForm {
         community_id: community.id,
         person_id: mod_user.id,
index 03f9e3fb2e70a8726916aa7f20656cf966974333..128ccf1f77132e6a89a1a70566d29bf3bf505f8f 100644 (file)
@@ -60,7 +60,7 @@ where
   if *recursion_counter > MAX_REQUEST_NUMBER {
     return Err(LemmyError::from(anyhow!("Maximum recursion depth reached")).into());
   }
-  check_is_apub_id_valid(&url, false)?;
+  check_is_apub_id_valid(url, false)?;
 
   let timeout = Duration::from_secs(60);
 
index d2d00f3245b2c8ae41f329c32ebf4f434b445cf1..5a09fd4307532e09d71b66d02592b07a4715b360 100644 (file)
@@ -89,7 +89,7 @@ pub async fn search_by_apub_id(
     );
     Url::parse(&url)?
   } else {
-    Url::parse(&query)?
+    Url::parse(query)?
   };
 
   let recursion_counter = &mut 0;
@@ -124,7 +124,7 @@ async fn build_response(
     SearchAcceptedObjects::Person(p) => {
       let person_uri = p.inner.id(domain)?.context("person has no id")?;
 
-      let person = get_or_fetch_and_upsert_person(&person_uri, context, recursion_counter).await?;
+      let person = get_or_fetch_and_upsert_person(person_uri, context, recursion_counter).await?;
 
       response.users = vec![
         blocking(context.pool(), move |conn| {
index ae16b76fe0f85bf188234e9e7abe7cb52c64e64d..361da462977e626c8217bd8b7164b4252df98744 100644 (file)
@@ -169,7 +169,7 @@ impl FromApubToForm<NoteExt> for CommentForm {
 
     // This post, or the parent comment might not yet exist on this server yet, fetch them.
     let post = Box::pin(get_or_fetch_and_insert_post(
-      &post_ap_id,
+      post_ap_id,
       context,
       request_counter,
     ))
@@ -181,7 +181,7 @@ impl FromApubToForm<NoteExt> for CommentForm {
       Some(parent_comment_uri) => {
         let parent_comment_ap_id = &parent_comment_uri?;
         let parent_comment = Box::pin(get_or_fetch_and_insert_comment(
-          &parent_comment_ap_id,
+          parent_comment_ap_id,
           context,
           request_counter,
         ))
index a72575d079363e34400722b1b7b266237b5f4640..34a15793b9195aaa5d11101f4244af0f5ed20987 100644 (file)
@@ -45,7 +45,7 @@ impl ToApub for Community {
   async fn to_apub(&self, pool: &DbPool) -> Result<GroupExt, LemmyError> {
     let id = self.id;
     let moderators = blocking(pool, move |conn| {
-      CommunityModeratorView::for_community(&conn, id)
+      CommunityModeratorView::for_community(conn, id)
     })
     .await??;
     let moderators: Vec<Url> = moderators
index 351ece1fcd35c1e73402173739c3ba0988470570..c87fc043c97ed16311ef23193a5ff2d926e6b0e7 100644 (file)
@@ -199,7 +199,7 @@ where
   // otherwise parse and insert, assuring that it comes from the right domain
   else {
     let to_form = ToForm::from_apub(
-      &from,
+      from,
       context,
       expected_domain,
       request_counter,
index 1c984d84dee050bbf48e9796525285d256f49964..19bfe8aabae631405bf32a71b2286bef86f68a39 100644 (file)
@@ -71,7 +71,7 @@ impl ToApub for Post {
       .set_attributed_to(creator.actor_id.into_inner());
 
     if let Some(body) = &self.body {
-      set_content_and_source(&mut page, &body)?;
+      set_content_and_source(&mut page, body)?;
     }
 
     if let Some(url) = &self.url {
index afad2dfc942523fa2013718b42c1b83af46b70fe..f421a45e92db072841e4e99fb56c769ac56d7359 100644 (file)
@@ -39,7 +39,7 @@ where
 {
   let actor = activity.actor()?;
   let person_uri = actor.as_single_xsd_any_uri().context(location_info!())?;
-  get_or_fetch_and_upsert_person(&person_uri, context, request_counter).await
+  get_or_fetch_and_upsert_person(person_uri, context, request_counter).await
 }
 
 /// Ensure that the ID of an incoming activity comes from the same domain as the actor. Optionally
index a0dcb81ea518e4aacfc3a542b48737b2cff29f5f..3d82c5c2b10ae25683b213de3142b5d87b484400 100644 (file)
@@ -41,7 +41,7 @@ pub(crate) async fn receive_create_private_message(
   let private_message =
     PrivateMessage::from_apub(&note, context, expected_domain, request_counter, false).await?;
 
-  let message = blocking(&context.pool(), move |conn| {
+  let message = blocking(context.pool(), move |conn| {
     PrivateMessageView::read(conn, private_message.id)
   })
   .await??;
@@ -88,7 +88,7 @@ pub(crate) async fn receive_update_private_message(
     PrivateMessage::from_apub(&note, context, expected_domain, request_counter, false).await?;
 
   let private_message_id = private_message.id;
-  let message = blocking(&context.pool(), move |conn| {
+  let message = blocking(context.pool(), move |conn| {
     PrivateMessageView::read(conn, private_message_id)
   })
   .await??;
@@ -128,8 +128,8 @@ pub(crate) async fn receive_delete_private_message(
   })
   .await??;
 
-  let message = blocking(&context.pool(), move |conn| {
-    PrivateMessageView::read(&conn, deleted_private_message.id)
+  let message = blocking(context.pool(), move |conn| {
+    PrivateMessageView::read(conn, deleted_private_message.id)
   })
   .await??;
 
@@ -173,8 +173,8 @@ pub(crate) async fn receive_undo_delete_private_message(
   })
   .await??;
 
-  let message = blocking(&context.pool(), move |conn| {
-    PrivateMessageView::read(&conn, deleted_private_message.id)
+  let message = blocking(context.pool(), move |conn| {
+    PrivateMessageView::read(conn, deleted_private_message.id)
   })
   .await??;
 
@@ -222,7 +222,7 @@ where
     .context(location_info!())?;
   check_is_apub_id_valid(&person_id, false)?;
   // check that the sender is a person, not a community
-  get_or_fetch_and_upsert_person(&person_id, &context, request_counter).await?;
+  get_or_fetch_and_upsert_person(&person_id, context, request_counter).await?;
 
   Ok(())
 }
index 1d5ae906e67273ef7fb06c552f6dc86c9647164e..30daa3b18ccf3b4e0eb7ba01400212974566f77c 100644 (file)
@@ -52,13 +52,13 @@ pub(crate) async fn get_apub_community_followers(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let community = blocking(context.pool(), move |conn| {
-    Community::read_from_name(&conn, &info.community_name)
+    Community::read_from_name(conn, &info.community_name)
   })
   .await??;
 
   let community_id = community.id;
   let community_followers = blocking(context.pool(), move |conn| {
-    CommunityFollowerView::for_community(&conn, community_id)
+    CommunityFollowerView::for_community(conn, community_id)
   })
   .await??;
 
@@ -77,7 +77,7 @@ pub(crate) async fn get_apub_community_outbox(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let community = blocking(context.pool(), move |conn| {
-    Community::read_from_name(&conn, &info.community_name)
+    Community::read_from_name(conn, &info.community_name)
   })
   .await??;
 
@@ -106,7 +106,7 @@ pub(crate) async fn get_apub_community_inbox(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let community = blocking(context.pool(), move |conn| {
-    Community::read_from_name(&conn, &info.community_name)
+    Community::read_from_name(conn, &info.community_name)
   })
   .await??;
 
@@ -122,7 +122,7 @@ pub(crate) async fn get_apub_community_moderators(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let community = blocking(context.pool(), move |conn| {
-    Community::read_from_name(&conn, &info.community_name)
+    Community::read_from_name(conn, &info.community_name)
   })
   .await??;
 
@@ -132,7 +132,7 @@ pub(crate) async fn get_apub_community_moderators(
   // ignore that for now
   let cid = community.id;
   let moderators = blocking(context.pool(), move |conn| {
-    CommunityModeratorView::for_community(&conn, cid)
+    CommunityModeratorView::for_community(conn, cid)
   })
   .await??;
 
index 531c37890ab3f8001f8839d2cf226db568ac2046..4f3328497b429342d9acc186a7ee76bc3f519e95 100644 (file)
@@ -55,7 +55,7 @@ pub(crate) async fn get_activity(
   ))?
   .into();
   let activity = blocking(context.pool(), move |conn| {
-    Activity::read_from_apub_id(&conn, &activity_id)
+    Activity::read_from_apub_id(conn, &activity_id)
   })
   .await??;
 
index 69598ec2af906895d592e56dad97eefa279d0e12..0d0b8f7651bd07b525308d0d0b1be0ba56db6c8c 100644 (file)
@@ -44,7 +44,7 @@ pub(crate) async fn get_apub_person_outbox(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let person = blocking(context.pool(), move |conn| {
-    Person::find_by_name(&conn, &info.user_name)
+    Person::find_by_name(conn, &info.user_name)
   })
   .await??;
   // TODO: populate the person outbox
@@ -62,7 +62,7 @@ pub(crate) async fn get_apub_person_inbox(
   context: web::Data<LemmyContext>,
 ) -> Result<HttpResponse<Body>, LemmyError> {
   let person = blocking(context.pool(), move |conn| {
-    Person::find_by_name(&conn, &info.user_name)
+    Person::find_by_name(conn, &info.user_name)
   })
   .await??;
 
index dfea860553810d91edd193b9e4659643da98a736..d851b7c0e31aa73a1b4b2412a0adba452ec168bf 100644 (file)
@@ -84,12 +84,12 @@ pub async fn community_inbox(
 
   // Check if the activity is actually meant for us
   let path = path.into_inner();
-  let community = blocking(&context.pool(), move |conn| {
-    Community::read_from_name(&conn, &path)
+  let community = blocking(context.pool(), move |conn| {
+    Community::read_from_name(conn, &path)
   })
   .await??;
   let to_and_cc = get_activity_to_and_cc(&activity);
-  if !to_and_cc.contains(&&community.actor_id()) {
+  if !to_and_cc.contains(&community.actor_id()) {
     return Err(anyhow!("Activity delivered to wrong community").into());
   }
 
@@ -117,8 +117,8 @@ pub(crate) async fn community_receive_message(
   // Only persons can send activities to the community, so we can get the actor as person
   // unconditionally.
   let actor_id = actor.actor_id();
-  let person = blocking(&context.pool(), move |conn| {
-    Person::read_from_apub_id(&conn, &actor_id.into())
+  let person = blocking(context.pool(), move |conn| {
+    Person::read_from_apub_id(conn, &actor_id.into())
   })
   .await??;
   check_community_or_site_ban(&person, to_community.id, context.pool()).await?;
@@ -142,7 +142,7 @@ pub(crate) async fn community_receive_message(
         any_base.clone(),
         person,
         &to_community,
-        &context,
+        context,
       ))
       .await?;
       false
@@ -282,8 +282,8 @@ async fn handle_follow(
   };
 
   // This will fail if they're already a follower, but ignore the error.
-  blocking(&context.pool(), move |conn| {
-    CommunityFollower::follow(&conn, &community_follower_form).ok()
+  blocking(context.pool(), move |conn| {
+    CommunityFollower::follow(conn, &community_follower_form).ok()
   })
   .await?;
 
@@ -304,7 +304,7 @@ async fn handle_undo(
     .is_single_kind(&FollowType::Follow.to_string());
   let any_base = activity.into_any_base()?;
   if inner_kind {
-    handle_undo_follow(any_base, actor_url, to_community, &context).await?;
+    handle_undo_follow(any_base, actor_url, to_community, context).await?;
     Ok(false)
   } else {
     receive_undo_for_community(context, any_base, None, &actor_url, request_counter).await?;
@@ -326,8 +326,8 @@ async fn handle_undo_follow(
   let follow = Follow::from_any_base(object)?.context(location_info!())?;
   verify_activity_domains_valid(&follow, &person_url, false)?;
 
-  let person = blocking(&context.pool(), move |conn| {
-    Person::read_from_apub_id(&conn, &person_url.into())
+  let person = blocking(context.pool(), move |conn| {
+    Person::read_from_apub_id(conn, &person_url.into())
   })
   .await??;
   let community_follower_form = CommunityFollowerForm {
@@ -337,8 +337,8 @@ async fn handle_undo_follow(
   };
 
   // This will fail if they aren't a follower, but ignore the error.
-  blocking(&context.pool(), move |conn| {
-    CommunityFollower::unfollow(&conn, &community_follower_form).ok()
+  blocking(context.pool(), move |conn| {
+    CommunityFollower::unfollow(conn, &community_follower_form).ok()
   })
   .await?;
 
index dce2c279433dee806fad12cfbfaec5468de68d1d..6b6f7b3fe57930471536abd45ff02ea9d6102f5e 100644 (file)
@@ -48,7 +48,7 @@ pub(crate) async fn is_activity_already_known(
 ) -> Result<bool, LemmyError> {
   let activity_id = activity_id.to_owned().into();
   let existing = blocking(pool, move |conn| {
-    Activity::read_from_apub_id(&conn, &activity_id)
+    Activity::read_from_apub_id(conn, &activity_id)
   })
   .await?;
   match existing {
@@ -86,7 +86,7 @@ where
     .single_xsd_any_uri()
     .context(location_info!())?;
   check_is_apub_id_valid(&actor_id, false)?;
-  let actor = get_or_fetch_and_upsert_actor(&actor_id, &context, request_counter).await?;
+  let actor = get_or_fetch_and_upsert_actor(&actor_id, context, request_counter).await?;
   verify_signature(&request, actor.as_ref())?;
   Ok(actor)
 }
@@ -98,8 +98,8 @@ pub(crate) async fn is_addressed_to_local_person(
 ) -> Result<bool, LemmyError> {
   for url in to_and_cc {
     let url = url.to_owned();
-    let person = blocking(&pool, move |conn| {
-      Person::read_from_apub_id(&conn, &url.into())
+    let person = blocking(pool, move |conn| {
+      Person::read_from_apub_id(conn, &url.into())
     })
     .await?;
     if let Ok(u) = person {
@@ -119,9 +119,9 @@ pub(crate) async fn is_addressed_to_community_followers(
 ) -> Result<Option<Community>, LemmyError> {
   for url in to_and_cc {
     let url = url.to_owned().into();
-    let community = blocking(&pool, move |conn| {
+    let community = blocking(pool, move |conn| {
       // ignore errors here, because the current url might not actually be a followers url
-      Community::read_from_followers_url(&conn, &url).ok()
+      Community::read_from_followers_url(conn, &url).ok()
     })
     .await?;
     if let Some(c) = community {
index 02d1d8be9ebb15482f3f82b92621e11e086e05dc..080e143c65fb6b1507a62f51f34d537a0e82d589 100644 (file)
@@ -102,13 +102,13 @@ pub async fn person_inbox(
 
   // Check if the activity is actually meant for us
   let username = path.into_inner();
-  let person = blocking(&context.pool(), move |conn| {
-    Person::find_by_name(&conn, &username)
+  let person = blocking(context.pool(), move |conn| {
+    Person::find_by_name(conn, &username)
   })
   .await??;
   let to_and_cc = get_activity_to_and_cc(&activity);
   // TODO: we should also accept activities that are sent to community followers
-  if !to_and_cc.contains(&&person.actor_id()) {
+  if !to_and_cc.contains(&person.actor_id()) {
     return Err(anyhow!("Activity delivered to wrong person").into());
   }
 
@@ -150,7 +150,7 @@ pub(crate) async fn person_receive_message(
   match kind {
     PersonValidTypes::Accept => {
       receive_accept(
-        &context,
+        context,
         any_base,
         actor,
         to_person.expect("person provided"),
@@ -159,11 +159,11 @@ pub(crate) async fn person_receive_message(
       .await?;
     }
     PersonValidTypes::Announce => {
-      Box::pin(receive_announce(&context, any_base, actor, request_counter)).await?
+      Box::pin(receive_announce(context, any_base, actor, request_counter)).await?
     }
     PersonValidTypes::Create => {
       Box::pin(receive_create(
-        &context,
+        context,
         any_base,
         actor_url,
         request_counter,
@@ -172,7 +172,7 @@ pub(crate) async fn person_receive_message(
     }
     PersonValidTypes::Update => {
       Box::pin(receive_update(
-        &context,
+        context,
         any_base,
         actor_url,
         request_counter,
@@ -217,7 +217,7 @@ async fn is_for_person_inbox(
   let community = is_addressed_to_community_followers(&to_and_cc, context.pool()).await?;
   if let Some(c) = community {
     let community_id = c.id;
-    let has_local_followers = blocking(&context.pool(), move |conn| {
+    let has_local_followers = blocking(context.pool(), move |conn| {
       CommunityFollower::has_local_followers(conn, community_id)
     })
     .await??;
@@ -261,7 +261,7 @@ async fn receive_accept(
   let community_id = community.id;
   let person_id = person.id;
   // This will throw an error if no follow was requested
-  blocking(&context.pool(), move |conn| {
+  blocking(context.pool(), move |conn| {
     CommunityFollower::follow_accepted(conn, community_id, person_id)
   })
   .await??;
@@ -377,7 +377,7 @@ async fn receive_create(
   if verify_is_addressed_to_public(&create).is_ok() {
     receive_create_comment(create, context, request_counter).await
   } else {
-    receive_create_private_message(&context, create, expected_domain, request_counter).await
+    receive_create_private_message(context, create, expected_domain, request_counter).await
   }
 }
 
@@ -394,7 +394,7 @@ async fn receive_update(
   if verify_is_addressed_to_public(&update).is_ok() {
     receive_update_comment(update, context, request_counter).await
   } else {
-    receive_update_private_message(&context, update, expected_domain, request_counter).await
+    receive_update_private_message(context, update, expected_domain, request_counter).await
   }
 }
 
@@ -436,7 +436,7 @@ async fn receive_remove(
     Community::read_from_apub_id(conn, &object_uri.into())
   })
   .await??;
-  receive_remove_community(&context, community).await
+  receive_remove_community(context, community).await
 }
 
 async fn receive_undo(
index 970f80ddd210087771ee1d55ef5394538ebd5afc..4bb8d32503ef6b2e041fad273b8061e81b529db4 100644 (file)
@@ -128,7 +128,7 @@ pub(in crate::inbox) async fn receive_create_for_community(
   request_counter: &mut i32,
 ) -> Result<(), LemmyError> {
   let create = Create::from_any_base(activity)?.context(location_info!())?;
-  verify_activity_domains_valid(&create, &expected_domain, true)?;
+  verify_activity_domains_valid(&create, expected_domain, true)?;
   verify_is_addressed_to_public(&create)?;
 
   let kind = create
@@ -151,7 +151,7 @@ pub(in crate::inbox) async fn receive_update_for_community(
   request_counter: &mut i32,
 ) -> Result<(), LemmyError> {
   let update = Update::from_any_base(activity.to_owned())?.context(location_info!())?;
-  verify_activity_domains_valid(&update, &expected_domain, false)?;
+  verify_activity_domains_valid(&update, expected_domain, false)?;
   verify_is_addressed_to_public(&update)?;
   verify_modification_actor_instance(&update, &announce, context, request_counter).await?;
 
@@ -179,14 +179,14 @@ pub(in crate::inbox) async fn receive_like_for_community(
   request_counter: &mut i32,
 ) -> Result<(), LemmyError> {
   let like = Like::from_any_base(activity)?.context(location_info!())?;
-  verify_activity_domains_valid(&like, &expected_domain, false)?;
+  verify_activity_domains_valid(&like, expected_domain, false)?;
   verify_is_addressed_to_public(&like)?;
 
   let object_id = like
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  match fetch_post_or_comment_by_id(&object_id, context, request_counter).await? {
+  match fetch_post_or_comment_by_id(object_id, context, request_counter).await? {
     PostOrComment::Post(post) => receive_like_post(like, *post, context, request_counter).await,
     PostOrComment::Comment(comment) => {
       receive_like_comment(like, *comment, context, request_counter).await
@@ -210,14 +210,14 @@ pub(in crate::inbox) async fn receive_dislike_for_community(
   }
 
   let dislike = Dislike::from_any_base(activity)?.context(location_info!())?;
-  verify_activity_domains_valid(&dislike, &expected_domain, false)?;
+  verify_activity_domains_valid(&dislike, expected_domain, false)?;
   verify_is_addressed_to_public(&dislike)?;
 
   let object_id = dislike
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  match fetch_post_or_comment_by_id(&object_id, context, request_counter).await? {
+  match fetch_post_or_comment_by_id(object_id, context, request_counter).await? {
     PostOrComment::Post(post) => {
       receive_dislike_post(dislike, *post, context, request_counter).await
     }
@@ -248,11 +248,11 @@ pub(in crate::inbox) async fn receive_delete_for_community(
 
   match find_object_by_id(context, object).await {
     Ok(Object::Post(p)) => {
-      verify_activity_domains_valid(&delete, &expected_domain, true)?;
+      verify_activity_domains_valid(&delete, expected_domain, true)?;
       receive_delete_post(context, *p).await
     }
     Ok(Object::Comment(c)) => {
-      verify_activity_domains_valid(&delete, &expected_domain, true)?;
+      verify_activity_domains_valid(&delete, expected_domain, true)?;
       receive_delete_comment(context, *c).await
     }
     Ok(Object::Community(c)) => {
@@ -281,7 +281,7 @@ pub(in crate::inbox) async fn receive_remove_for_community(
       .object()
       .as_single_xsd_any_uri()
       .context(location_info!())?;
-    let remove_mod = get_or_fetch_and_upsert_person(&remove_mod, context, request_counter).await?;
+    let remove_mod = get_or_fetch_and_upsert_person(remove_mod, context, request_counter).await?;
     let form = CommunityModeratorForm {
       community_id: community.id,
       person_id: remove_mod.id,
@@ -388,11 +388,11 @@ pub(in crate::inbox) async fn receive_undo_delete_for_community(
     .context(location_info!())?;
   match find_object_by_id(context, object).await {
     Ok(Object::Post(p)) => {
-      verify_activity_domains_valid(&delete, &expected_domain, true)?;
+      verify_activity_domains_valid(&delete, expected_domain, true)?;
       receive_undo_delete_post(context, *p).await
     }
     Ok(Object::Comment(c)) => {
-      verify_activity_domains_valid(&delete, &expected_domain, true)?;
+      verify_activity_domains_valid(&delete, expected_domain, true)?;
       receive_undo_delete_comment(context, *c).await
     }
     Ok(Object::Community(c)) => {
@@ -413,7 +413,7 @@ pub(in crate::inbox) async fn receive_undo_remove_for_community(
 ) -> Result<(), LemmyError> {
   let remove = Remove::from_any_base(undo.object().to_owned().one().context(location_info!())?)?
     .context(location_info!())?;
-  verify_activity_domains_valid(&remove, &expected_domain, false)?;
+  verify_activity_domains_valid(&remove, expected_domain, false)?;
   verify_is_addressed_to_public(&remove)?;
   verify_undo_remove_actor_instance(&undo, &remove, &announce, context).await?;
 
@@ -439,14 +439,14 @@ pub(in crate::inbox) async fn receive_undo_like_for_community(
 ) -> Result<(), LemmyError> {
   let like = Like::from_any_base(undo.object().to_owned().one().context(location_info!())?)?
     .context(location_info!())?;
-  verify_activity_domains_valid(&like, &expected_domain, false)?;
+  verify_activity_domains_valid(&like, expected_domain, false)?;
   verify_is_addressed_to_public(&like)?;
 
   let object_id = like
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  match fetch_post_or_comment_by_id(&object_id, context, request_counter).await? {
+  match fetch_post_or_comment_by_id(object_id, context, request_counter).await? {
     PostOrComment::Post(post) => {
       receive_undo_like_post(&like, *post, context, request_counter).await
     }
@@ -474,7 +474,7 @@ pub(in crate::inbox) async fn receive_add_for_community(
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  let new_mod = get_or_fetch_and_upsert_person(&new_mod, context, request_counter).await?;
+  let new_mod = get_or_fetch_and_upsert_person(new_mod, context, request_counter).await?;
 
   // If we had to refetch the community while parsing the activity, then the new mod has already
   // been added. Skip it here as it would result in a duplicate key error.
@@ -515,14 +515,14 @@ pub(in crate::inbox) async fn receive_undo_dislike_for_community(
 ) -> Result<(), LemmyError> {
   let dislike = Dislike::from_any_base(undo.object().to_owned().one().context(location_info!())?)?
     .context(location_info!())?;
-  verify_activity_domains_valid(&dislike, &expected_domain, false)?;
+  verify_activity_domains_valid(&dislike, expected_domain, false)?;
   verify_is_addressed_to_public(&dislike)?;
 
   let object_id = dislike
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  match fetch_post_or_comment_by_id(&object_id, context, request_counter).await? {
+  match fetch_post_or_comment_by_id(object_id, context, request_counter).await? {
     PostOrComment::Post(post) => {
       receive_undo_dislike_post(&dislike, *post, context, request_counter).await
     }
@@ -548,8 +548,7 @@ pub(crate) async fn receive_block_user_for_community(
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  let blocked_user =
-    get_or_fetch_and_upsert_person(&blocked_user, context, request_counter).await?;
+  let blocked_user = get_or_fetch_and_upsert_person(blocked_user, context, request_counter).await?;
 
   let community_user_ban_form = CommunityPersonBanForm {
     community_id: community.id,
@@ -587,7 +586,7 @@ pub(crate) async fn receive_undo_block_user_for_community(
   let block = Block::from_any_base(object)?.context(location_info!())?;
   let community = extract_community_from_cc(&block, context).await?;
 
-  verify_activity_domains_valid(&block, &expected_domain, false)?;
+  verify_activity_domains_valid(&block, expected_domain, false)?;
   verify_is_addressed_to_public(&block)?;
   verify_undo_remove_actor_instance(&undo, &block, &announce, context).await?;
 
@@ -595,8 +594,7 @@ pub(crate) async fn receive_undo_block_user_for_community(
     .object()
     .as_single_xsd_any_uri()
     .context(location_info!())?;
-  let blocked_user =
-    get_or_fetch_and_upsert_person(&blocked_user, context, request_counter).await?;
+  let blocked_user = get_or_fetch_and_upsert_person(blocked_user, context, request_counter).await?;
 
   let community_user_ban_form = CommunityPersonBanForm {
     community_id: community.id,
@@ -646,8 +644,8 @@ where
     .flatten()
     .context(location_info!())?;
   let community_id: DbUrl = community_id.to_owned().into();
-  let community = blocking(&context.pool(), move |conn| {
-    Community::read_from_apub_id(&conn, &community_id)
+  let community = blocking(context.pool(), move |conn| {
+    Community::read_from_apub_id(conn, &community_id)
   })
   .await??;
   Ok(community)
@@ -672,8 +670,8 @@ where
     .as_single_xsd_any_uri()
     .context(location_info!())?
     .to_owned();
-  let actor = blocking(&context.pool(), move |conn| {
-    Person::read_from_apub_id(&conn, &actor.into())
+  let actor = blocking(context.pool(), move |conn| {
+    Person::read_from_apub_id(conn, &actor.into())
   })
   .await??;
 
index db060247c92e687731b4b735197bf6441ffc48a1..17691d4f407a76d2a8b8149650893d0a5edb6f68 100644 (file)
@@ -137,8 +137,8 @@ async fn extract_local_community_from_destinations(
 ) -> Result<Option<Community>, LemmyError> {
   for url in to_and_cc {
     let url = url.to_owned();
-    let community = blocking(&pool, move |conn| {
-      Community::read_from_apub_id(&conn, &url.into())
+    let community = blocking(pool, move |conn| {
+      Community::read_from_apub_id(conn, &url.into())
     })
     .await?;
     if let Ok(c) = community {
index 0665ac95d00a7261e8518cf15a5010eb2e9e3037..fbd508c150a6c261568ee278fc5ec46469f6eaca 100644 (file)
@@ -314,7 +314,7 @@ mod tests {
     let example_url = "https://example.com";
     assert!(matches!(
       diesel_option_overwrite_to_url(&Some(example_url.to_string())),
-      Ok(Some(Some(url))) if url == Url::parse(&example_url).unwrap().into()
+      Ok(Some(Some(url))) if url == Url::parse(example_url).unwrap().into()
     ));
   }
 }
index d1fc0622f90a1f389f047571d8d87a7cc930ce1d..f87cf73306390b61ed03d0b0e0ca181695d70703 100644 (file)
@@ -78,7 +78,7 @@ impl Activity_ for Activity {
       sensitive,
       updated: None,
     };
-    let result = Activity::create(&conn, &activity_form);
+    let result = Activity::create(conn, &activity_form);
     match result {
       Ok(s) => Ok(s),
       Err(e) => Err(IoError::new(
index 2cd347e5cba7533ca72900c303c688908bda5c34..abfc4e0be87abcf763303aa6ec2fabfdbee3d83a 100644 (file)
@@ -70,7 +70,7 @@ impl LocalUser_ for LocalUser {
       hash(&form.password_encrypted, DEFAULT_COST).expect("Couldn't hash password");
     edited_user.password_encrypted = password_hash;
 
-    Self::create(&conn, &edited_user)
+    Self::create(conn, &edited_user)
   }
 
   fn update_password(
index 6fe4ee3a5b645e29a8c46879d4367457d19ebd40..d04a1ac4bc39a9c3a51670320dd3fed23df7ee30 100644 (file)
@@ -53,7 +53,7 @@ impl PasswordResetRequest_ for PasswordResetRequest {
       token_encrypted: token_hash,
     };
 
-    Self::create(&conn, &form)
+    Self::create(conn, &form)
   }
   fn read_from_token(conn: &PgConnection, token: &str) -> Result<PasswordResetRequest, Error> {
     let mut hasher = Sha256::new();
index 06b9ac13aef395940803d911cb0e3691029b0f12..8250d3c6e50c9e683f2c61f0b074bead4450cace 100644 (file)
@@ -88,10 +88,10 @@ async fn get_feed_data(
   listing_type: ListingType,
   sort_type: SortType,
 ) -> Result<HttpResponse, LemmyError> {
-  let site_view = blocking(context.pool(), move |conn| SiteView::read(&conn)).await??;
+  let site_view = blocking(context.pool(), move |conn| SiteView::read(conn)).await??;
 
   let posts = blocking(context.pool(), move |conn| {
-    PostQueryBuilder::create(&conn)
+    PostQueryBuilder::create(conn)
       .listing_type(listing_type)
       .sort(sort_type)
       .list()
@@ -169,10 +169,10 @@ fn get_feed_user(
   sort_type: &SortType,
   user_name: String,
 ) -> Result<ChannelBuilder, LemmyError> {
-  let site_view = SiteView::read(&conn)?;
-  let person = Person::find_by_name(&conn, &user_name)?;
+  let site_view = SiteView::read(conn)?;
+  let person = Person::find_by_name(conn, &user_name)?;
 
-  let posts = PostQueryBuilder::create(&conn)
+  let posts = PostQueryBuilder::create(conn)
     .listing_type(ListingType::All)
     .sort(*sort_type)
     .creator_id(person.id)
@@ -195,10 +195,10 @@ fn get_feed_community(
   sort_type: &SortType,
   community_name: String,
 ) -> Result<ChannelBuilder, LemmyError> {
-  let site_view = SiteView::read(&conn)?;
-  let community = Community::read_from_name(&conn, &community_name)?;
+  let site_view = SiteView::read(conn)?;
+  let community = Community::read_from_name(conn, &community_name)?;
 
-  let posts = PostQueryBuilder::create(&conn)
+  let posts = PostQueryBuilder::create(conn)
     .listing_type(ListingType::All)
     .sort(*sort_type)
     .community_id(community.id)
@@ -225,14 +225,14 @@ fn get_feed_front(
   sort_type: &SortType,
   jwt: String,
 ) -> Result<ChannelBuilder, LemmyError> {
-  let site_view = SiteView::read(&conn)?;
+  let site_view = SiteView::read(conn)?;
   let local_user_id = LocalUserId(Claims::decode(&jwt)?.claims.sub);
-  let local_user = LocalUser::read(&conn, local_user_id)?;
+  let local_user = LocalUser::read(conn, local_user_id)?;
   let person_id = local_user.person_id;
   let show_bot_accounts = local_user.show_bot_accounts;
   let show_read_posts = local_user.show_read_posts;
 
-  let posts = PostQueryBuilder::create(&conn)
+  let posts = PostQueryBuilder::create(conn)
     .listing_type(ListingType::Subscribed)
     .my_person_id(person_id)
     .show_bot_accounts(show_bot_accounts)
@@ -257,22 +257,22 @@ fn get_feed_front(
 }
 
 fn get_feed_inbox(conn: &PgConnection, jwt: String) -> Result<ChannelBuilder, LemmyError> {
-  let site_view = SiteView::read(&conn)?;
+  let site_view = SiteView::read(conn)?;
   let local_user_id = LocalUserId(Claims::decode(&jwt)?.claims.sub);
-  let local_user = LocalUser::read(&conn, local_user_id)?;
+  let local_user = LocalUser::read(conn, local_user_id)?;
   let person_id = local_user.person_id;
   let show_bot_accounts = local_user.show_bot_accounts;
 
   let sort = SortType::New;
 
-  let replies = CommentQueryBuilder::create(&conn)
+  let replies = CommentQueryBuilder::create(conn)
     .recipient_id(person_id)
     .my_person_id(person_id)
     .show_bot_accounts(show_bot_accounts)
     .sort(sort)
     .list()?;
 
-  let mentions = PersonMentionQueryBuilder::create(&conn)
+  let mentions = PersonMentionQueryBuilder::create(conn)
     .recipient_id(person_id)
     .my_person_id(person_id)
     .sort(sort)
index f8c2012dba0187b18948debe151537d04bac5839..fc323fce0e66e8f0343f0f6bb3e3f94be6c33db0 100644 (file)
@@ -18,7 +18,7 @@ rand = "0.8.3"
 percent-encoding = "2.1.0"
 serde = { version = "1.0.123", features = ["derive"] }
 serde_json = { version = "1.0.61", features = ["preserve_order"] }
-thiserror = "1.0.23"
+thiserror = "1.0.26"
 comrak = { version = "0.9.0", default-features = false }
 lazy_static = "1.4.0"
 openssl = "0.10.32"
index 3a444b9a4f3bab6a9d2bb573cd5d89f1111699b2..8cd5796ef4b594d9702f3deed2faa4a8f66e175c 100644 (file)
@@ -21,7 +21,7 @@ impl Claims {
       ..Validation::default()
     };
     decode::<Claims>(
-      &jwt,
+      jwt,
       &DecodingKey::from_secret(Settings::get().jwt_secret().as_ref()),
       &v,
     )
index 66d0b11d1906ab104edd6439339645bbe2345e09..f14a6c1ac23c7bf941c3c80065ca01e573adbe80 100644 (file)
@@ -134,7 +134,7 @@ pub async fn fetch_iframely_and_pictrs_data(
           }
         },
         // Try to generate a small thumbnail if iframely is not supported
-        None => match fetch_pictrs(client, &url).await {
+        None => match fetch_pictrs(client, url).await {
           Ok(res) => Some(res.files[0].file.to_owned()),
           Err(e) => {
             error!("pictrs err: {}", e);
index da38299a810f92d7fe477c38005754f43af392cc..2d9d417f399ec8c3f1eba27f41741aa528c1ab63 100644 (file)
@@ -71,7 +71,7 @@ fn test_slur_filter() {
       "faggot test kike tranny cocksucker retardeds. Capitalized Niggerz. This is a bunch of other safe text.";
   let slur_free = "No slurs here";
   assert_eq!(
-      remove_slurs(&test),
+      remove_slurs(test),
       "*removed* test *removed* *removed* *removed* *removed*. Capitalized *removed*. This is a bunch of other safe text."
         .to_string()
     );
index c6f1aaaf0647b455c7e09321f589a0a6b97d54b6..ee02891432e71b483a3370201b61a44b9a9288e4 100644 (file)
@@ -14,7 +14,7 @@ lazy_static! {
         slurs.push('|');
         slurs.push_str(&additional_slurs);
     };
-    RegexBuilder::new(&&slurs).case_insensitive(true).build().expect("compile regex")
+    RegexBuilder::new(&slurs).case_insensitive(true).build().expect("compile regex")
   };
 
 
index e08aa94a5d4a523b7e369cec037b4b4946c9e6de..89ef2fc129c743874ff2a73b93006b3ec3ba1b33 100644 (file)
@@ -461,7 +461,7 @@ impl ChatServer {
         message: "Unknown op type".to_string(),
       })?;
 
-      if let Ok(user_operation_crud) = UserOperationCrud::from_str(&op) {
+      if let Ok(user_operation_crud) = UserOperationCrud::from_str(op) {
         let fut = (message_handler_crud)(context, msg.id, user_operation_crud.clone(), data);
         match user_operation_crud {
           UserOperationCrud::Register => rate_limiter.register().wrap(ip, fut).await,
@@ -470,7 +470,7 @@ impl ChatServer {
           _ => rate_limiter.message().wrap(ip, fut).await,
         }
       } else {
-        let user_operation = UserOperation::from_str(&op)?;
+        let user_operation = UserOperation::from_str(op)?;
         let fut = (message_handler)(context, msg.id, user_operation.clone(), data);
         rate_limiter.message().wrap(ip, fut).await
       }
index d8563b51bda293a79d2f40e73c524dd0ab395c32..cebf12936efe9c061e5369ad12926aa6eaede0fd 100644 (file)
@@ -28,13 +28,13 @@ use lemmy_utils::{apub::generate_actor_keypair, settings::structs::Settings, Lem
 use log::info;
 
 pub fn run_advanced_migrations(conn: &PgConnection) -> Result<(), LemmyError> {
-  user_updates_2020_04_02(&conn)?;
-  community_updates_2020_04_02(&conn)?;
-  post_updates_2020_04_03(&conn)?;
-  comment_updates_2020_04_03(&conn)?;
-  private_message_updates_2020_05_05(&conn)?;
-  post_thumbnail_url_updates_2020_07_27(&conn)?;
-  apub_columns_2021_02_02(&conn)?;
+  user_updates_2020_04_02(conn)?;
+  community_updates_2020_04_02(conn)?;
+  post_updates_2020_04_03(conn)?;
+  comment_updates_2020_04_03(conn)?;
+  private_message_updates_2020_05_05(conn)?;
+  post_thumbnail_url_updates_2020_07_27(conn)?;
+  apub_columns_2021_02_02(conn)?;
 
   Ok(())
 }
@@ -62,7 +62,7 @@ fn user_updates_2020_04_02(conn: &PgConnection) -> Result<(), LemmyError> {
       ..PersonForm::default()
     };
 
-    Person::update(&conn, cperson.id, &form)?;
+    Person::update(conn, cperson.id, &form)?;
   }
 
   info!("{} person rows updated.", incorrect_persons.len());
@@ -106,7 +106,7 @@ fn community_updates_2020_04_02(conn: &PgConnection) -> Result<(), LemmyError> {
       shared_inbox_url: None,
     };
 
-    Community::update(&conn, ccommunity.id, &form)?;
+    Community::update(conn, ccommunity.id, &form)?;
   }
 
   info!("{} community rows updated.", incorrect_communities.len());
@@ -127,7 +127,7 @@ fn post_updates_2020_04_03(conn: &PgConnection) -> Result<(), LemmyError> {
 
   for cpost in &incorrect_posts {
     let apub_id = generate_apub_endpoint(EndpointType::Post, &cpost.id.to_string())?;
-    Post::update_ap_id(&conn, cpost.id, apub_id)?;
+    Post::update_ap_id(conn, cpost.id, apub_id)?;
   }
 
   info!("{} post rows updated.", incorrect_posts.len());
@@ -148,7 +148,7 @@ fn comment_updates_2020_04_03(conn: &PgConnection) -> Result<(), LemmyError> {
 
   for ccomment in &incorrect_comments {
     let apub_id = generate_apub_endpoint(EndpointType::Comment, &ccomment.id.to_string())?;
-    Comment::update_ap_id(&conn, ccomment.id, apub_id)?;
+    Comment::update_ap_id(conn, ccomment.id, apub_id)?;
   }
 
   info!("{} comment rows updated.", incorrect_comments.len());
@@ -169,7 +169,7 @@ fn private_message_updates_2020_05_05(conn: &PgConnection) -> Result<(), LemmyEr
 
   for cpm in &incorrect_pms {
     let apub_id = generate_apub_endpoint(EndpointType::PrivateMessage, &cpm.id.to_string())?;
-    PrivateMessage::update_ap_id(&conn, cpm.id, apub_id)?;
+    PrivateMessage::update_ap_id(conn, cpm.id, apub_id)?;
   }
 
   info!("{} private message rows updated.", incorrect_pms.len());
index 4751e9ea205a5588e041b3be7d632c4f63686114..a3b5f75dbdec21254cdebd655cd6ca3ac508b534 100644 (file)
@@ -41,7 +41,7 @@ fn reindex_aggregates_tables(conn: &PgConnection) {
     "comment_aggregates",
     "community_aggregates",
   ] {
-    reindex_table(&conn, &table_name);
+    reindex_table(conn, table_name);
   }
 }
 
@@ -55,7 +55,7 @@ fn reindex_table(conn: &PgConnection, table_name: &str) {
 /// Clear old activities (this table gets very large)
 fn clear_old_activities(conn: &PgConnection) {
   info!("Clearing old activities...");
-  Activity::delete_olds(&conn).expect("clear old activities");
+  Activity::delete_olds(conn).expect("clear old activities");
   info!("Done.");
 }