Use tasklist flag.

Author: KN4CK3R

routers/web/repo/issue.go

@@ -2127,13 +2127,6 @@ func UpdateCommentContent(ctx *context.Context) {
 		return
 	}
 
-	if comment.Type == models.CommentTypeComment {
-		if err := comment.LoadAttachments(); err != nil {
-			ctx.ServerError("LoadAttachments", err)
-			return
-		}
-	}
-
 	if !ctx.IsSigned || (ctx.User.ID != comment.PosterID && !ctx.Repo.CanWriteIssuesOrPulls(comment.Issue.IsPull)) {
 		ctx.Error(http.StatusForbidden)
 		return
@@ -2155,6 +2148,17 @@ func UpdateCommentContent(ctx *context.Context) {
 		return
 	}
 
+	if ctx.FormString("action") == "tasklist" {
+		return
+	}
+
+	if comment.Type == models.CommentTypeComment {
+		if err := comment.LoadAttachments(); err != nil {
+			ctx.ServerError("LoadAttachments", err)
+			return
+		}
+	}
+
 	if err := updateAttachments(comment, ctx.FormStrings("files[]")); err != nil {
 		ctx.ServerError("UpdateAttachments", err)
 		return

web_src/js/markup/tasklist.js

@@ -43,20 +43,13 @@ export function initMarkupTasklist() {
 
         try {
           const editContentZone = container.querySelector('.edit-content-zone');
-          const {updateUrl, context, attachmentUrl} = editContentZone.dataset;
-
-          const resp = await fetch(attachmentUrl);
-          const attachments = await resp.json();
-          const files = [];
-          for (const attachment of attachments) {
-            files.push(attachment.uuid);
-          }
+          const {updateUrl, context} = editContentZone.dataset;
 
           await $.post(updateUrl, {
+            action: 'tasklist',
             _csrf: window.config.csrf,
             content: newContent,
-            context,
-            files
+            context
           });
 
           rawContent.textContent = newContent;