Skip to content

Commit 75b819a

Browse files
committed
fix: fix check for assigning devs to a mod
1 parent 49ccfc3 commit 75b819a

File tree

3 files changed

+10
-8
lines changed

3 files changed

+10
-8
lines changed

Cargo.lock

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "geode-index"
3-
version = "0.40.3"
3+
version = "0.40.4"
44
edition = "2021"
55

66
[dependencies]

src/types/models/mod_entity.rs

+8-6
Original file line numberDiff line numberDiff line change
@@ -1144,7 +1144,7 @@ impl Mod {
11441144
is_owner: bool,
11451145
}
11461146

1147-
sqlx::query_as!(
1147+
let assignment = sqlx::query_as!(
11481148
FetchedRow,
11491149
"SELECT md.developer_id, md.is_owner FROM mods_developers md
11501150
WHERE md.mod_id = $1
@@ -1161,11 +1161,13 @@ impl Mod {
11611161
e
11621162
);
11631163
ApiError::DbError
1164-
})?
1165-
.ok_or(ApiError::BadRequest(format!(
1166-
"This developer is already assigned on mod {}",
1167-
mod_id
1168-
)))?;
1164+
})?;
1165+
if assignment.is_some() {
1166+
return Err(ApiError::BadRequest(format!(
1167+
"This developer is already assigned on mod {}",
1168+
mod_id
1169+
)));
1170+
}
11691171

11701172
sqlx::query!(
11711173
"INSERT INTO mods_developers (mod_id, developer_id)

0 commit comments

Comments
 (0)