From b103f1d66aacd57897ee2107787eacc0bff061a1 Mon Sep 17 00:00:00 2001
From: tobinio <Tobias.frischmann1@gmail.com>
Date: Thu, 15 Aug 2024 08:12:16 +0200
Subject: [PATCH] ignore loader for file update if not applicable

---
 src/routes/v3/version_file.rs | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/routes/v3/version_file.rs b/src/routes/v3/version_file.rs
index 01e9cddb..130095dc 100644
--- a/src/routes/v3/version_file.rs
+++ b/src/routes/v3/version_file.rs
@@ -1,6 +1,7 @@
 use super::ApiError;
 use crate::auth::checks::{filter_visible_versions, is_visible_version};
 use crate::auth::{filter_visible_projects, get_user_from_headers};
+use crate::database::models::loader_fields::Loader;
 use crate::database::redis::RedisPool;
 use crate::models::ids::VersionId;
 use crate::models::pats::Scopes;
@@ -149,6 +150,8 @@ pub async fn get_update_from_hash(
         if let Some(project) =
             database::models::Project::get_id(file.project_id, &**pool, &redis).await?
         {
+            let all_loaders_data: Vec<Loader> = Loader::list(&**pool, &redis).await?;
+
             let versions = database::models::Version::get_many(&project.versions, &**pool, &redis)
                 .await?
                 .into_iter()
@@ -160,7 +163,22 @@ pub async fn get_update_from_hash(
                             .any(|y| y.as_str() == x.inner.version_type);
                     }
                     if let Some(loaders) = &update_data.loaders {
-                        bool &= x.loaders.iter().any(|y| loaders.contains(y));
+                        let loaders = loaders
+                            .iter()
+                            .filter(|loader| {
+                                let loader_data = all_loaders_data
+                                    .iter()
+                                    .find(|game_loader| game_loader.loader == **loader)
+                                    .unwrap();
+
+                                loader_data
+                                    .supported_project_types
+                                    .iter()
+                                    .any(|project_type| x.project_types.contains(project_type))
+                            })
+                            .collect_vec();
+
+                        bool &= x.loaders.iter().any(|y| loaders.contains(&y));
                     }
                     if let Some(loader_fields) = &update_data.loader_fields {
                         for (key, values) in loader_fields {