diff options
Diffstat (limited to 'routers')
| -rw-r--r-- | routers/web/repo/issue.go | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/routers/web/repo/issue.go b/routers/web/repo/issue.go index 56561c6168..59ffe1edb1 100644 --- a/routers/web/repo/issue.go +++ b/routers/web/repo/issue.go @@ -1727,9 +1727,12 @@ func UpdateIssueContent(ctx *context.Context) { return } - if err := updateAttachments(issue, ctx.FormStrings("files[]")); err != nil { - ctx.ServerError("UpdateAttachments", err) - return + // when update the request doesn't intend to update attachments (eg: change checkbox state), ignore attachment updates + if !ctx.FormBool("ignore_attachments") { + if err := updateAttachments(issue, ctx.FormStrings("files[]")); err != nil { + ctx.ServerError("UpdateAttachments", err) + return + } } content, err := markdown.RenderString(&markup.RenderContext{ @@ -2148,10 +2151,6 @@ func UpdateCommentContent(ctx *context.Context) { return } - if ctx.FormBool("ignore_attachments") { - return - } - if comment.Type == models.CommentTypeComment { if err := comment.LoadAttachments(); err != nil { ctx.ServerError("LoadAttachments", err) @@ -2159,9 +2158,12 @@ func UpdateCommentContent(ctx *context.Context) { } } - if err := updateAttachments(comment, ctx.FormStrings("files[]")); err != nil { - ctx.ServerError("UpdateAttachments", err) - return + // when the update request doesn't intend to update attachments (eg: change checkbox state), ignore attachment updates + if !ctx.FormBool("ignore_attachments") { + if err := updateAttachments(comment, ctx.FormStrings("files[]")); err != nil { + ctx.ServerError("UpdateAttachments", err) + return + } } content, err := markdown.RenderString(&markup.RenderContext{ |