Make functions work with both connection and pool (#3420)

[lemmy.git] / crates / db_views_moderator / src / mod_remove_comment_view.rs

diff --git a/crates/db_views_moderator/src/mod_remove_comment_view.rs b/crates/db_views_moderator/src/mod_remove_comment_view.rs
index 2ee230d112afe7a959f0ad4f64a574a015ffba23..946e0eb37cbf17fd729c09eedd20c1f0a07f7f85 100644 (file)
--- a/crates/db_views_moderator/src/mod_remove_comment_view.rs
+++ b/crates/db_views_moderator/src/mod_remove_comment_view.rs
@@ -1,30 +1,41 @@
 use crate::structs::{ModRemoveCommentView, ModlogListParams};
-use diesel::{result::Error, *};
+use diesel::{
+  result::Error,
+  BoolExpressionMethods,
+  ExpressionMethods,
+  IntoSql,
+  JoinOnDsl,
+  NullableExpressionMethods,
+  QueryDsl,
+};
+use diesel_async::RunQueryDsl;
 use lemmy_db_schema::{
   newtypes::PersonId,
-  schema::{comment, community, mod_remove_comment, person, person_alias_1, post},
+  schema::{comment, community, mod_remove_comment, person, post},
   source::{
     comment::Comment,
-    community::{Community, CommunitySafe},
+    community::Community,
     moderator::ModRemoveComment,
-    person::{Person, PersonAlias1, PersonSafe, PersonSafeAlias1},
+    person::Person,
     post::Post,
   },
-  traits::{ToSafe, ViewToVec},
-  utils::limit_and_offset,
+  traits::JoinView,
+  utils::{get_conn, limit_and_offset, DbPool},
 };
 
 type ModRemoveCommentViewTuple = (
   ModRemoveComment,
-  Option<PersonSafe>,
+  Option<Person>,
   Comment,
-  PersonSafeAlias1,
+  Person,
   Post,
-  CommunitySafe,
+  Community,
 );
 
 impl ModRemoveCommentView {
-  pub fn list(conn: &PgConnection, params: ModlogListParams) -> Result<Vec<Self>, Error> {
+  pub async fn list(pool: &mut DbPool<'_>, params: ModlogListParams) -> Result<Vec<Self>, Error> {
+    let conn = &mut get_conn(pool).await?;
+    let person_alias_1 = diesel::alias!(lemmy_db_schema::schema::person as person1);
     let admin_person_id_join = params.mod_person_id.unwrap_or(PersonId(-1));
     let show_mod_names = !params.hide_modlog_names;
     let show_mod_names_expr = show_mod_names.as_sql::<diesel::sql_types::Bool>();
@@ -35,16 +46,16 @@ impl ModRemoveCommentView {
     let mut query = mod_remove_comment::table
       .left_join(person::table.on(admin_names_join))
       .inner_join(comment::table)
-      .inner_join(person_alias_1::table.on(comment::creator_id.eq(person_alias_1::id)))
+      .inner_join(person_alias_1.on(comment::creator_id.eq(person_alias_1.field(person::id))))
       .inner_join(post::table.on(comment::post_id.eq(post::id)))
       .inner_join(community::table.on(post::community_id.eq(community::id)))
       .select((
         mod_remove_comment::all_columns,
-        Person::safe_columns_tuple().nullable(),
+        person::all_columns.nullable(),
         comment::all_columns,
-        PersonAlias1::safe_columns_tuple(),
+        person_alias_1.fields(person::all_columns),
         post::all_columns,
-        Community::safe_columns_tuple(),
+        community::all_columns,
       ))
       .into_boxed();
 
@@ -57,7 +68,7 @@ impl ModRemoveCommentView {
     };
 
     if let Some(other_person_id) = params.other_person_id {
-      query = query.filter(person_alias_1::id.eq(other_person_id));
+      query = query.filter(person_alias_1.field(person::id).eq(other_person_id));
     };
 
     let (limit, offset) = limit_and_offset(params.page, params.limit)?;
@@ -66,26 +77,24 @@ impl ModRemoveCommentView {
       .limit(limit)
       .offset(offset)
       .order_by(mod_remove_comment::when_.desc())
-      .load::<ModRemoveCommentViewTuple>(conn)?;
+      .load::<ModRemoveCommentViewTuple>(conn)
+      .await?;
 
-    let results = Self::from_tuple_to_vec(res);
+    let results = res.into_iter().map(Self::from_tuple).collect();
     Ok(results)
   }
 }
 
-impl ViewToVec for ModRemoveCommentView {
-  type DbTuple = ModRemoveCommentViewTuple;
-  fn from_tuple_to_vec(items: Vec<Self::DbTuple>) -> Vec<Self> {
-    items
-      .into_iter()
-      .map(|a| Self {
-        mod_remove_comment: a.0,
-        moderator: a.1,
-        comment: a.2,
-        commenter: a.3,
-        post: a.4,
-        community: a.5,
-      })
-      .collect::<Vec<Self>>()
+impl JoinView for ModRemoveCommentView {
+  type JoinTuple = ModRemoveCommentViewTuple;
+  fn from_tuple(a: Self::JoinTuple) -> Self {
+    Self {
+      mod_remove_comment: a.0,
+      moderator: a.1,
+      comment: a.2,
+      commenter: a.3,
+      post: a.4,
+      community: a.5,
+    }
   }
 }