fix: refactor some stuff part 1

Author: Fleeym

src/database/repository/mod.rs

@@ -1 +1,2 @@
-pub mod mod_tags;
+pub mod mod_tags;
+pub mod mods;

src/database/repository/mods.rs

@@ -0,0 +1,45 @@
+use crate::types::api::ApiError;
+use sqlx::PgConnection;
+
+pub async fn exists(id: &str, conn: &mut PgConnection) -> Result<bool, ApiError> {
+    Ok(sqlx::query!("SELECT id FROM mods WHERE id = $1", id)
+        .fetch_optional(&mut *conn)
+        .await
+        .map_err(|e| {
+            log::error!("Failed to check if mod {} exists: {}", id, e);
+            ApiError::DbError
+        })?
+        .is_some())
+}
+
+pub async fn get_logo(id: &str, conn: &mut PgConnection) -> Result<Option<Vec<u8>>, ApiError> {
+    struct QueryResult {
+        image: Option<Vec<u8>>,
+    }
+
+    let vec = sqlx::query_as!(
+        QueryResult,
+        "SELECT
+            m.image
+        FROM mods m
+        INNER JOIN mod_versions mv ON mv.mod_id = m.id
+        INNER JOIN mod_version_statuses mvs ON mvs.mod_version_id = mv.id
+        WHERE m.id = $1",
+        id
+    )
+    .fetch_optional(&mut *conn)
+    .await
+    .map_err(|e| {
+        log::error!("Failed to fetch mod logo for {}: {}", id, e);
+        ApiError::DbError
+    })?
+    .map(|optional| optional.image)
+    .flatten();
+
+    // Empty vec is basically no image
+    if vec.as_ref().is_some_and(|v| v.is_empty()) {
+        Ok(None)
+    } else {
+        Ok(vec)
+    }
+}

src/endpoints/mods.rs

@@ -212,8 +212,10 @@ pub async fn get_logo(
     data: web::Data<AppData>,
     path: web::Path<String>,
 ) -> Result<impl Responder, ApiError> {
+    use crate::database::repository::*;
     let mut pool = data.db.acquire().await.or(Err(ApiError::DbAcquireError))?;
-    let image = Mod::get_logo_for_mod(&path, &mut pool).await?;
+    let image = mods::get_logo(&path.into_inner(), &mut pool).await?;
+
     match image {
         Some(i) => {
             if i.is_empty() {