]> source.dussan.org Git - gitea.git/commitdiff
Add attachments for PR reviews (#16075)
authorKN4CK3R <admin@oldschoolhack.me>
Tue, 15 Jun 2021 01:12:33 +0000 (03:12 +0200)
committerGitHub <noreply@github.com>
Tue, 15 Jun 2021 01:12:33 +0000 (21:12 -0400)
* First step for multiple dropzones per page.

* Allow attachments on review comments.

* Lint.

* Fixed accidental initialize of the review textarea.

* Initialize SimpleMDE textarea.

Co-authored-by: techknowlogick <techknowlogick@gitea.io>
15 files changed:
models/issue_comment.go
models/review.go
routers/api/v1/repo/pull_review.go
routers/web/repo/pull.go
routers/web/repo/pull_review.go
services/forms/repo_form.go
services/pull/review.go
templates/repo/diff/new_review.tmpl
templates/repo/editor/upload.tmpl
templates/repo/issue/comment_tab.tmpl
templates/repo/issue/view_content.tmpl
templates/repo/issue/view_content/comments.tmpl
templates/repo/release/new.tmpl
templates/repo/upload.tmpl
web_src/js/index.js

index 26bf122dc9835da4aff68d0e08b6e273e75a1ac9..1b98b248b1fccb1f859ccc503ea82f892a24fb48 100644 (file)
@@ -762,6 +762,8 @@ func updateCommentInfos(e *xorm.Session, opts *CreateCommentOptions, comment *Co
                        }
                }
                fallthrough
+       case CommentTypeReview:
+               fallthrough
        case CommentTypeComment:
                if _, err = e.Exec("UPDATE `issue` SET num_comments=num_comments+1 WHERE id=?", opts.Issue.ID); err != nil {
                        return err
index 343621c0fa5ab323cd4900da30806813039e419e..316cbe4da6426cb949f61d83c443efd64d520ebf 100644 (file)
@@ -347,7 +347,7 @@ func IsContentEmptyErr(err error) bool {
 }
 
 // SubmitReview creates a review out of the existing pending review or creates a new one if no pending review exist
-func SubmitReview(doer *User, issue *Issue, reviewType ReviewType, content, commitID string, stale bool) (*Review, *Comment, error) {
+func SubmitReview(doer *User, issue *Issue, reviewType ReviewType, content, commitID string, stale bool, attachmentUUIDs []string) (*Review, *Comment, error) {
        sess := x.NewSession()
        defer sess.Close()
        if err := sess.Begin(); err != nil {
@@ -419,12 +419,13 @@ func SubmitReview(doer *User, issue *Issue, reviewType ReviewType, content, comm
        }
 
        comm, err := createComment(sess, &CreateCommentOptions{
-               Type:     CommentTypeReview,
-               Doer:     doer,
-               Content:  review.Content,
-               Issue:    issue,
-               Repo:     issue.Repo,
-               ReviewID: review.ID,
+               Type:        CommentTypeReview,
+               Doer:        doer,
+               Content:     review.Content,
+               Issue:       issue,
+               Repo:        issue.Repo,
+               ReviewID:    review.ID,
+               Attachments: attachmentUUIDs,
        })
        if err != nil || comm == nil {
                return nil, nil, err
index 63179aa9907dbc972bc11ac252a81d114772fbee..35414e0a80c5323ecf6dd55483b4a927c52cfcb1 100644 (file)
@@ -359,7 +359,7 @@ func CreatePullReview(ctx *context.APIContext) {
        }
 
        // create review and associate all pending review comments
-       review, _, err := pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, pr.Issue, reviewType, opts.Body, opts.CommitID)
+       review, _, err := pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, pr.Issue, reviewType, opts.Body, opts.CommitID, nil)
        if err != nil {
                ctx.Error(http.StatusInternalServerError, "SubmitReview", err)
                return
@@ -447,7 +447,7 @@ func SubmitPullReview(ctx *context.APIContext) {
        }
 
        // create review and associate all pending review comments
-       review, _, err = pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, pr.Issue, reviewType, opts.Body, headCommitID)
+       review, _, err = pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, pr.Issue, reviewType, opts.Body, headCommitID, nil)
        if err != nil {
                ctx.Error(http.StatusInternalServerError, "SubmitReview", err)
                return
index 28f94c841701e9e47d6a394c7382b174fd2f5bbc..e5554e966444468c2680cf5bdba3852835f025e3 100644 (file)
@@ -694,6 +694,10 @@ func ViewPullFiles(ctx *context.Context) {
        getBranchData(ctx, issue)
        ctx.Data["IsIssuePoster"] = ctx.IsSigned && issue.IsPoster(ctx.User.ID)
        ctx.Data["HasIssuesOrPullsWritePermission"] = ctx.Repo.CanWriteIssuesOrPulls(issue.IsPull)
+
+       ctx.Data["IsAttachmentEnabled"] = setting.Attachment.Enabled
+       upload.AddUploadContext(ctx, "comment")
+
        ctx.HTML(http.StatusOK, tplPullFiles)
 }
 
index 9e505c3db373820e25e3384b51394fb5c428b6b0..36eee3f377b89086ea653baec78e90db879e0c87 100644 (file)
@@ -12,6 +12,7 @@ import (
        "code.gitea.io/gitea/modules/base"
        "code.gitea.io/gitea/modules/context"
        "code.gitea.io/gitea/modules/log"
+       "code.gitea.io/gitea/modules/setting"
        "code.gitea.io/gitea/modules/web"
        "code.gitea.io/gitea/services/forms"
        pull_service "code.gitea.io/gitea/services/pull"
@@ -211,7 +212,12 @@ func SubmitReview(ctx *context.Context) {
                }
        }
 
-       _, comm, err := pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, issue, reviewType, form.Content, form.CommitID)
+       var attachments []string
+       if setting.Attachment.Enabled {
+               attachments = form.Files
+       }
+
+       _, comm, err := pull_service.SubmitReview(ctx.User, ctx.Repo.GitRepo, issue, reviewType, form.Content, form.CommitID, attachments)
        if err != nil {
                if models.IsContentEmptyErr(err) {
                        ctx.Flash.Error(ctx.Tr("repo.issues.review.content.empty"))
index a40b0be9a76ddba6c3de652bb1374f00eff196f9..71a83a8be36e2eea53267d364084f25af0a7075b 100644 (file)
@@ -587,6 +587,7 @@ type SubmitReviewForm struct {
        Content  string
        Type     string `binding:"Required;In(approve,comment,reject)"`
        CommitID string
+       Files    []string
 }
 
 // Validate validates the fields
index 4b647722fcb2b140a501d6f7c94e9fab606861cd..b07e21fad97740ac4e0aa612c33fec6bb9e28780 100644 (file)
@@ -100,7 +100,7 @@ func CreateCodeComment(doer *models.User, gitRepo *git.Repository, issue *models
 
        if !isReview && !existsReview {
                // Submit the review we've just created so the comment shows up in the issue view
-               if _, _, err = SubmitReview(doer, gitRepo, issue, models.ReviewTypeComment, "", latestCommitID); err != nil {
+               if _, _, err = SubmitReview(doer, gitRepo, issue, models.ReviewTypeComment, "", latestCommitID, nil); err != nil {
                        return nil, err
                }
        }
@@ -215,7 +215,7 @@ func createCodeComment(doer *models.User, repo *models.Repository, issue *models
 }
 
 // SubmitReview creates a review out of the existing pending review or creates a new one if no pending review exist
-func SubmitReview(doer *models.User, gitRepo *git.Repository, issue *models.Issue, reviewType models.ReviewType, content, commitID string) (*models.Review, *models.Comment, error) {
+func SubmitReview(doer *models.User, gitRepo *git.Repository, issue *models.Issue, reviewType models.ReviewType, content, commitID string, attachmentUUIDs []string) (*models.Review, *models.Comment, error) {
        pr, err := issue.GetPullRequest()
        if err != nil {
                return nil, nil, err
@@ -240,7 +240,7 @@ func SubmitReview(doer *models.User, gitRepo *git.Repository, issue *models.Issu
                }
        }
 
-       review, comm, err := models.SubmitReview(doer, issue, reviewType, content, commitID, stale)
+       review, comm, err := models.SubmitReview(doer, issue, reviewType, content, commitID, stale, attachmentUUIDs)
        if err != nil {
                return nil, nil, err
        }
index 9e65d6d42053a15dacda78872f7a9bfe6ae1d5c6..cbaabe255e21b93ba261ed8040f89ce52b795d3b 100644 (file)
                                <div class="ui field">
                                        <textarea name="content" tabindex="0" rows="2" placeholder="{{$.i18n.Tr "repo.diff.review.placeholder"}}"></textarea>
                                </div>
+                               {{if .IsAttachmentEnabled}}
+                                       <div class="field">
+                                               {{template "repo/upload" .}}
+                                       </div>
+                               {{end}}
                                <div class="ui divider"></div>
                                <button type="submit" name="type" value="approve" {{ if and $.IsSigned ($.Issue.IsPoster $.SignedUser.ID) }} disabled {{ end }} class="ui submit green tiny button btn-submit">{{$.i18n.Tr "repo.diff.review.approve"}}</button>
                                <button type="submit" name="type" value="comment" class="ui submit tiny basic button btn-submit">{{$.i18n.Tr "repo.diff.review.comment"}}</button>
index 488465120ee790e49cb52dfff8a4d00baffb2277..fb00615abdb2adc0faedd7f263da79257206a145 100644 (file)
@@ -26,7 +26,6 @@
                                </div>
                        </div>
                        <div class="field">
-                               <div class="files"></div>
                                {{template "repo/upload" .}}
                        </div>
                        {{template "repo/editor/commit_form" .}}
index 77e82930dcf90e933e8184bd15b914f85cf1afcd..22e1d5af8456a98c6148f322335af5769af11f2e 100644 (file)
@@ -14,7 +14,6 @@
 </div>
 {{if .IsAttachmentEnabled}}
        <div class="field">
-               <div class="files"></div>
                {{template "repo/upload" .}}
        </div>
 {{end}}
index 00ce61921d0e6cbaeb83cd5d37eebffe24f2ccd5..d2928df3425706b1516c9fea8c9ab09bf56b9817 100644 (file)
                </div>
                {{if .IsAttachmentEnabled}}
                        <div class="field">
-                               <div class="comment-files"></div>
                                {{template "repo/upload" .}}
                        </div>
                {{end}}
index 53005cc82032ba476174508b15087bec2be7159b..de31430ce05994adea939e34b09d28da8e8ed054 100644 (file)
                                                                <span class="no-content">{{$.i18n.Tr "repo.issues.no_content"}}</span>
                                                        {{end}}
                                                </div>
+                                               {{if .Attachments}}
+                                                       {{template "repo/issue/view_content/attachments" Dict "ctx" $ "Attachments" .Attachments "Content" .RenderedContent}}
+                                               {{end}}
                                        </div>
                                </div>
                        </div>
index c4b36597c68fa52c141f3461077420b5ad912c5e..49759713aaca0d96ab6fbfe4450cec8615ed3e47 100644 (file)
@@ -76,7 +76,6 @@
                                {{end}}
                                {{if .IsAttachmentEnabled}}
                                        <div class="field">
-                                               <div class="files"></div>
                                                {{template "repo/upload" .}}
                                        </div>
                                {{end}}
index 9215da2b1967040418331a5ef6ab3e8381e5ab14..3dd40d1b27bcbef1dc3af08ea986f218349b15f9 100644 (file)
@@ -1,6 +1,5 @@
 <div
        class="ui dropzone"
-       id="dropzone"
        data-link-url="{{.UploadLinkUrl}}"
        data-upload-url="{{.UploadUrl}}"
        data-remove-url="{{.UploadRemoveUrl}}"
@@ -11,4 +10,6 @@
        data-invalid-input-type="{{.i18n.Tr "dropzone.invalid_input_type"}}"
        data-file-too-big="{{.i18n.Tr "dropzone.file_too_big"}}"
        data-remove-file="{{.i18n.Tr "dropzone.remove_file"}}"
-></div>
+>
+       <div class="files"></div>
+</div>
index 8818511e32cf2fbda29e5e7bf4f985f955eb36a3..e42a6640151ab86d4fc5d3c818696806ae32e75c 100644 (file)
@@ -327,11 +327,11 @@ function getPastedImages(e) {
   return files;
 }
 
-async function uploadFile(file) {
+async function uploadFile(file, uploadUrl) {
   const formData = new FormData();
   formData.append('file', file, file.name);
 
-  const res = await fetch($('#dropzone').data('upload-url'), {
+  const res = await fetch(uploadUrl, {
     method: 'POST',
     headers: {'X-Csrf-Token': csrf},
     body: formData,
@@ -345,24 +345,33 @@ function reload() {
 
 function initImagePaste(target) {
   target.each(function () {
-    this.addEventListener('paste', async (e) => {
-      for (const img of getPastedImages(e)) {
-        const name = img.name.substr(0, img.name.lastIndexOf('.'));
-        insertAtCursor(this, `![${name}]()`);
-        const data = await uploadFile(img);
-        replaceAndKeepCursor(this, `![${name}]()`, `![${name}](${AppSubUrl}/attachments/${data.uuid})`);
-        const input = $(`<input id="${data.uuid}" name="files" type="hidden">`).val(data.uuid);
-        $('.files').append(input);
-      }
-    }, false);
+    const dropzone = this.querySelector('.dropzone');
+    if (!dropzone) {
+      return;
+    }
+    const uploadUrl = dropzone.dataset.uploadUrl;
+    const dropzoneFiles = dropzone.querySelector('.files');
+    for (const textarea of this.querySelectorAll('textarea')) {
+      textarea.addEventListener('paste', async (e) => {
+        for (const img of getPastedImages(e)) {
+          const name = img.name.substr(0, img.name.lastIndexOf('.'));
+          insertAtCursor(textarea, `![${name}]()`);
+          const data = await uploadFile(img, uploadUrl);
+          replaceAndKeepCursor(textarea, `![${name}]()`, `![${name}](${AppSubUrl}/attachments/${data.uuid})`);
+          const input = $(`<input id="${data.uuid}" name="files" type="hidden">`).val(data.uuid);
+          dropzoneFiles.appendChild(input[0]);
+        }
+      }, false);
+    }
   });
 }
 
-function initSimpleMDEImagePaste(simplemde, files) {
+function initSimpleMDEImagePaste(simplemde, dropzone, files) {
+  const uploadUrl = dropzone.dataset.uploadUrl;
   simplemde.codemirror.on('paste', async (_, e) => {
     for (const img of getPastedImages(e)) {
       const name = img.name.substr(0, img.name.lastIndexOf('.'));
-      const data = await uploadFile(img);
+      const data = await uploadFile(img, uploadUrl);
       const pos = simplemde.codemirror.getCursor();
       simplemde.codemirror.replaceRange(`![${name}](${AppSubUrl}/attachments/${data.uuid})`, pos);
       const input = $(`<input id="${data.uuid}" name="files" type="hidden">`).val(data.uuid);
@@ -381,7 +390,7 @@ function initCommentForm() {
   autoSimpleMDE = setCommentSimpleMDE($('.comment.form textarea:not(.review-textarea)'));
   initBranchSelector();
   initCommentPreviewTab($('.comment.form'));
-  initImagePaste($('.comment.form textarea'));
+  initImagePaste($('.comment.form'));
 
   // Listsubmit
   function initListSubmits(selector, outerSelector) {
@@ -993,8 +1002,7 @@ async function initRepository() {
 
         let dz;
         const $dropzone = $editContentZone.find('.dropzone');
-        const $files = $editContentZone.find('.comment-files');
-        if ($dropzone.length > 0) {
+        if ($dropzone.length === 1) {
           $dropzone.data('saved', false);
 
           const filenameDict = {};
@@ -1020,7 +1028,7 @@ async function initRepository() {
                   submitted: false
                 };
                 const input = $(`<input id="${data.uuid}" name="files" type="hidden">`).val(data.uuid);
-                $files.append(input);
+                $dropzone.find('.files').append(input);
               });
               this.on('removedfile', (file) => {
                 if (!(file.name in filenameDict)) {
@@ -1042,7 +1050,7 @@ async function initRepository() {
               this.on('reload', () => {
                 $.getJSON($editContentZone.data('attachment-url'), (data) => {
                   dz.removeAllFiles(true);
-                  $files.empty();
+                  $dropzone.find('.files').empty();
                   $.each(data, function () {
                     const imgSrc = `${$dropzone.data('link-url')}/${this.uuid}`;
                     dz.emit('addedfile', this);
@@ -1055,7 +1063,7 @@ async function initRepository() {
                     };
                     $dropzone.find(`img[src='${imgSrc}']`).css('max-width', '100%');
                     const input = $(`<input id="${this.uuid}" name="files" type="hidden">`).val(this.uuid);
-                    $files.append(input);
+                    $dropzone.find('.files').append(input);
                   });
                 });
               });
@@ -1075,7 +1083,9 @@ async function initRepository() {
         $simplemde = setCommentSimpleMDE($textarea);
         commentMDEditors[$editContentZone.data('write')] = $simplemde;
         initCommentPreviewTab($editContentForm);
-        initSimpleMDEImagePaste($simplemde, $files);
+        if ($dropzone.length === 1) {
+          initSimpleMDEImagePaste($simplemde, $dropzone[0], $dropzone.find('.files'));
+        }
 
         $editContentZone.find('.cancel.button').on('click', () => {
           $renderContent.show();
@@ -1087,7 +1097,7 @@ async function initRepository() {
         $editContentZone.find('.save.button').on('click', () => {
           $renderContent.show();
           $editContentZone.hide();
-          const $attachments = $files.find('[name=files]').map(function () {
+          const $attachments = $dropzone.find('.files').find('[name=files]').map(function () {
             return $(this).val();
           }).get();
           $.post($editContentZone.data('update-url'), {
@@ -1369,6 +1379,13 @@ function initPullRequestReview() {
     $simplemde.codemirror.focus();
     assingMenuAttributes(form.find('.menu'));
   });
+
+  const $reviewBox = $('.review-box');
+  if ($reviewBox.length === 1) {
+    setCommentSimpleMDE($reviewBox.find('textarea'));
+    initImagePaste($reviewBox);
+  }
+
   // The following part is only for diff views
   if ($('.repository.pull.diff').length === 0) {
     return;
@@ -1656,6 +1673,10 @@ $.fn.getCursorPosition = function () {
 };
 
 function setCommentSimpleMDE($editArea) {
+  if ($editArea.length === 0) {
+    return null;
+  }
+
   const simplemde = new SimpleMDE({
     autoDownloadFontAwesome: false,
     element: $editArea[0],
@@ -1827,7 +1848,8 @@ function initReleaseEditor() {
   const $files = $editor.parent().find('.files');
   const $simplemde = setCommentSimpleMDE($textarea);
   initCommentPreviewTab($editor);
-  initSimpleMDEImagePaste($simplemde, $files);
+  const dropzone = $editor.parent().find('.dropzone')[0];
+  initSimpleMDEImagePaste($simplemde, dropzone, $files);
 }
 
 function initOrganization() {
@@ -2610,11 +2632,10 @@ $(document).ready(async () => {
   initLinkAccountView();
 
   // Dropzone
-  const $dropzone = $('#dropzone');
-  if ($dropzone.length > 0) {
+  for (const el of document.querySelectorAll('.dropzone')) {
     const filenameDict = {};
-
-    await createDropzone('#dropzone', {
+    const $dropzone = $(el);
+    await createDropzone(el, {
       url: $dropzone.data('upload-url'),
       headers: {'X-Csrf-Token': csrf},
       maxFiles: $dropzone.data('max-file'),
@@ -2633,7 +2654,7 @@ $(document).ready(async () => {
         this.on('success', (file, data) => {
           filenameDict[file.name] = data.uuid;
           const input = $(`<input id="${data.uuid}" name="files" type="hidden">`).val(data.uuid);
-          $('.files').append(input);
+          $dropzone.find('.files').append(input);
         });
         this.on('removedfile', (file) => {
           if (file.name in filenameDict) {