Don't update comment_aggregates if updating path fails (#4281)

pull/4295/head
dullbananas 2023-12-18 02:31:39 -07:00 committed by GitHub
parent bc32b408b5
commit dcb89f52d6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions

View File

@ -87,7 +87,7 @@ impl Comment {
let updated_comment = diesel::update(comment.find(comment_id)) let updated_comment = diesel::update(comment.find(comment_id))
.set(path.eq(ltree)) .set(path.eq(ltree))
.get_result::<Self>(conn) .get_result::<Self>(conn)
.await; .await?;
// Update the child count for the parent comment_aggregates // Update the child count for the parent comment_aggregates
// You could do this with a trigger, but since you have to do this manually anyway, // You could do this with a trigger, but since you have to do this manually anyway,
@ -121,7 +121,7 @@ where ca.comment_id = c.id"
sql_query(update_child_count_stmt).execute(conn).await?; sql_query(update_child_count_stmt).execute(conn).await?;
} }
} }
updated_comment Ok(updated_comment)
}) as _ }) as _
}) })
.await .await