diff --git a/src/main/scala/gitbucket/core/controller/RepositoryViewerController.scala b/src/main/scala/gitbucket/core/controller/RepositoryViewerController.scala index 3da3594..a1245a4 100644 --- a/src/main/scala/gitbucket/core/controller/RepositoryViewerController.scala +++ b/src/main/scala/gitbucket/core/controller/RepositoryViewerController.scala @@ -103,7 +103,8 @@ oldLineNumber: Option[Int], newLineNumber: Option[Int], content: String, - issueId: Option[Int] + issueId: Option[Int], + diff: Option[String] ) val uploadForm = mapping( @@ -138,7 +139,8 @@ "oldLineNumber" -> trim(label("Old line number", optional(number()))), "newLineNumber" -> trim(label("New line number", optional(number()))), "content" -> trim(label("Content", text(required))), - "issueId" -> trim(label("Issue Id", optional(number()))) + "issueId" -> trim(label("Issue Id", optional(number()))), + "diff" -> optional(text()) )(CommentForm.apply) /** @@ -613,6 +615,9 @@ form.newLineNumber, form.issueId ) + + println(form.diff) // TODO store diff into the database + val comment = getCommitComment(repository.owner, repository.name, commentId.toString).get form.issueId match { case Some(issueId) => diff --git a/src/main/twirl/gitbucket/core/repo/commentform.scala.html b/src/main/twirl/gitbucket/core/repo/commentform.scala.html index d8e69e7..57243fb 100644 --- a/src/main/twirl/gitbucket/core/repo/commentform.scala.html +++ b/src/main/twirl/gitbucket/core/repo/commentform.scala.html @@ -49,6 +49,15 @@ $($form.serializeArray()).each(function(i, v) { param[v.name] = v.value; }); + + @if(newLineNumber.isDefined){ + var diff = getDiffData($('table[filename="@fileName"] table.diff tr:has(th.line-num.newline[line-number=@newLineNumber])')); + param['diff'] = JSON.stringify(diff); + } else if(oldLineNumber.isDefined){ + var diff = getDiffData($('table[filename="@fileName"] table.diff tr:has(th.line-num.oldline[line-number=@oldLineNumber])')); + param['diff'] = JSON.stringify(diff); + } + $.ajax({ url: '@helpers.url(repository)/commit/@commitId/comment/_data/new', type: 'POST', @@ -96,7 +105,6 @@ }); function getInlineContainer() { - console.log(window.viewType); if (window.viewType == 0) { if(@newLineNumber.isDefined){ return $('