From 7f429f60e84096c2a362c3e9889a0bee77b62746 Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Wed, 26 Jun 2013 20:43:39 +0900
Subject: [PATCH] Add issue comment update in stub.

---
 src/main/scala/app/IssuesController.scala |  4 ++
 src/main/twirl/issues/edit.scala.html     | 48 ++++++++++++++---------
 src/main/twirl/issues/issue.scala.html    | 19 +++++++--
 3 files changed, 49 insertions(+), 22 deletions(-)

diff --git a/src/main/scala/app/IssuesController.scala b/src/main/scala/app/IssuesController.scala
index 411a449cb..321d1c4b8 100644
--- a/src/main/scala/app/IssuesController.scala
+++ b/src/main/scala/app/IssuesController.scala
@@ -87,6 +87,10 @@ trait IssuesControllerBase extends ControllerBase {
   // TODO Authenticator
   post("/:owner/:repository/issue_comments/:id"){
     // TODO update issue memo
+    contentType = formats("json")
+
+    org.json4s.jackson.Serialization.write(Map(
+        "content" -> view.Markdown.toHtml("* hoge memo", getRepository("root", "test", baseUrl).get, false, true, true)))
   }
 
   // TODO Authenticator
diff --git a/src/main/twirl/issues/edit.scala.html b/src/main/twirl/issues/edit.scala.html
index d78512a50..edca5b10c 100644
--- a/src/main/twirl/issues/edit.scala.html
+++ b/src/main/twirl/issues/edit.scala.html
@@ -1,29 +1,41 @@
 @(title: Option[String], content: String, key: Int, owner: String, repository: String)(implicit context: app.Context)
 @import context._
-@if(title.isDefined){
-<input type="text" style="width: 730px;" id="title" value="@title.get"/>
+@defining(title.isEmpty){ isComment =>
+@if(!isComment){
+<input type="text" style="width: 730px;" id="edit-title" value="@title.get"/>
 }
-<textarea style="width: 730px; height: 100px;" id="content-@key">@content</textarea>
-<input type="button" class="btn btn-small" value="Update @{if(title.isDefined) "Issue" else "Comment"}" id="update-@key"/>
-<span class="pull-right"><a class="btn btn-small btn-danger" href="#" id="cancel-@key">Cancel</a></span>
+<textarea style="width: 730px; height: 100px;" id="edit-content@if(isComment){"-" + key}">@content</textarea>
+<input type="button" class="btn btn-small" value="Update @{if(isComment) "Comment" else "Issue"}"/>
+<span class="pull-right"><a class="btn btn-small btn-danger" href="#">Cancel</a></span>
 <script>
 $(function(){
-  $('#update-@key').click(function(){
-    $.post('@path/@owner/@repository/@{if(title.isDefined) "issues" else "issue_comments"}/@key',
-    {
-      @if(title.isDefined){ title : $('#title').val(), }
-      content : $('#content-@key').val()
-    },
-    function(data){
-      @if(title.isDefined){
-        $('#issueTitle').empty().text(data.title);
-      }
-      $('#issueContent').empty().html(data.content);
-    });
+  $('@{if(isComment) "#commentContent-" + key else "#issueContent"} input.btn').click(function(){
+    @if(isComment){
+      var url = '@path/@owner/@repository/issue_comments/@key';
+      var param = {
+          content : $('#edit-content-@key').val()
+      };
+      var func = function(data){
+          $('#commentContent-@key').empty().html(data.content);
+      };
+    }else{
+      var url = '@path/@owner/@repository/issues/@key';
+      var param = {
+          title   : $('#edit-title').val(),
+          content : $('#edit-content-@key').val()
+      };
+      var func = function(data){
+          $('#issueTitle').empty().text(data.title);
+          $('#issueContent').empty().html(data.content);
+      };
+    }
+    $.post(url, param, func);
   });
 
-  $('#cancel-@key').click(function(){
+  $('#cancel').click(function(){
 
   });
 });
 </script>
+
+}
diff --git a/src/main/twirl/issues/issue.scala.html b/src/main/twirl/issues/issue.scala.html
index 5ec3b9f1f..97cee3b04 100644
--- a/src/main/twirl/issues/issue.scala.html
+++ b/src/main/twirl/issues/issue.scala.html
@@ -12,7 +12,7 @@
     <div class="span10">
       <div class="box">
         <div class="box-content">
-          <span class="pull-right"><a class="btn btn-small" href="#" id="editIssue">Edit</a></span>
+          <span class="pull-right"><a class="btn btn-small" href="#" id="edit">Edit</a></span>
           <div class="small"><a href="@path/@issue.openedUserName">@issue.openedUserName</a> opened this issue @datetime(issue.registeredDate)</div>
           <h4 id="issueTitle">@issue.title</h4>
         </div>
@@ -24,9 +24,12 @@
       <div class="box" id="comment-@comment.commentId">
         <div class="box-header-small">
           <a href="@path/@comment.commentedUserName">@comment.commentedUserName</a> commented
-          <span class="pull-right">@datetime(comment.registeredDate)</span>
+          <span class="pull-right">
+            @datetime(comment.registeredDate)
+            <a href="#" data-comment-id="@comment.commentId"><i class="icon-pencil"></i></a>
+          </span>
         </div>
-        <div class="box-content" style="background-color: #f5f5f5;">
+        <div class="box-content" style="background-color: #f5f5f5;" id="commentContent-@comment.commentId">
           @markdown(comment.content, repository, false, true, true)
         </div>
       </div>
@@ -55,7 +58,7 @@
 }
 <script>
 $(function(){
-  $('#editIssue').click(function(){
+  $('#edit').click(function(){
     $.get('@path/@repository.owner/@repository.name/issues/_data/@issue.issueId',
     function(data){
       $('#issueContent').empty().html(data);
@@ -63,5 +66,13 @@ $(function(){
     return false;
   });
 
+  $('i.icon-pencil').click(function(){
+    var id = $(this).closest('a').data('comment-id');
+    $.get('@path/@repository.owner/@repository.name/issue_comments/_data/' + id,
+    function(data){
+      $('#commentContent-' + id).empty().html(data);
+    });
+    return false;
+  });
 });
 </script>
\ No newline at end of file