fix: dont use option in MpJson
All checks were successful
ci/woodpecker/push/build-and-publish Pipeline was successful
ci/woodpecker/push/publish-docs Pipeline was successful

This commit is contained in:
Radical 2025-06-01 14:22:52 +02:00
parent 57f52d96df
commit 2f7fac8db5

View file

@ -46,7 +46,7 @@ struct NewInfo {
struct UploadForm { struct UploadForm {
#[multipart(limit = "100MB")] #[multipart(limit = "100MB")]
avatar: Option<TempFile>, avatar: Option<TempFile>,
json: MpJson<Option<NewInfo>>, json: MpJson<NewInfo>,
} }
#[patch("")] #[patch("")]
@ -64,11 +64,8 @@ pub async fn update(
let uuid = check_access_token(auth_header, &mut conn).await?; let uuid = check_access_token(auth_header, &mut conn).await?;
if form.avatar.is_some() if form.avatar.is_some()
|| form || form.json.username.is_some()
.json || form.json.display_name.is_some()
.0
.clone()
.is_some_and(|ni| ni.username.is_some() || ni.display_name.is_some())
{ {
global_checks(&data, uuid).await?; global_checks(&data, uuid).await?;
} }
@ -88,18 +85,16 @@ pub async fn update(
.await?; .await?;
} }
if let Some(new_info) = form.json.0 { if let Some(username) = &form.json.username {
if let Some(username) = &new_info.username { me.set_username(&data, username.clone()).await?;
me.set_username(&data, username.clone()).await?; }
}
if let Some(display_name) = &new_info.display_name { if let Some(display_name) = &form.json.display_name {
me.set_display_name(&data, display_name.clone()).await?; me.set_display_name(&data, display_name.clone()).await?;
} }
if let Some(email) = &new_info.email { if let Some(email) = &form.json.email {
me.set_email(&data, email.to_string()).await?; me.set_email(&data, email.clone()).await?;
}
} }
Ok(HttpResponse::Ok().finish()) Ok(HttpResponse::Ok().finish())