Add delete and update for GHIssueComment

henryju · henryju · commit cce02aec3d5d · 2015-06-16T09:27:55.000+02:00
diff --git a/src/main/java/org/kohsuke/github/GHIssue.java b/src/main/java/org/kohsuke/github/GHIssue.java
@@ -141,8 +141,9 @@ public URL getApiURL(){
     /**
      * Updates the issue by adding a comment.
      */
-    public void comment(String message) throws IOException {
-        new Requester(root).with("body",message).to(getIssuesApiRoute() + "/comments");
+    public GHIssueComment comment(String message) throws IOException {
+        GHIssueComment r = new Requester(root).with("body",message).to(getIssuesApiRoute() + "/comments", GHIssueComment.class);
+        return r.wrapUp(this);
     }
 
     private void edit(String key, Object value) throws IOException {
diff --git a/src/main/java/org/kohsuke/github/GHIssueComment.java b/src/main/java/org/kohsuke/github/GHIssueComment.java
@@ -79,4 +79,23 @@ public GHUser getUser() throws IOException {
     public URL getHtmlUrl() {
         return null;
     }
+    
+    /**
+     * Updates the body of the issue comment.
+     */
+    public void update(String body) throws IOException {
+        new Requester(owner.root).with("body", body).method("PATCH").to(getCommentApiTail(), GHIssueComment.class);
+        this.body = body;
+    }
+
+    /**
+     * Deletes this issue comment.
+     */
+    public void delete() throws IOException {
+        new Requester(owner.root).method("DELETE").to(getCommentApiTail());
+    }
+    
+    private String getCommentApiTail() {
+        return "/repos/"+owner.getRepository().getOwnerName()+"/"+owner.getRepository().getName()+"/issues/comments/" + id;
+    }
 }
diff --git a/src/main/java/org/kohsuke/github/GHPullRequestReviewComment.java b/src/main/java/org/kohsuke/github/GHPullRequestReviewComment.java
@@ -94,6 +94,7 @@ protected String getApiRoute() {
      */
     public void update(String body) throws IOException {
         new Requester(owner.root).method("PATCH").with("body", body).to(getApiRoute(),this);
+        this.body = body;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,7 @@ protected String getApiRoute() {`
`94`	`94`	`*/`
`95`	`95`	`public void update(String body) throws IOException {`
`96`	`96`	`new Requester(owner.root).method("PATCH").with("body", body).to(getApiRoute(),this);`
	`97`	`+ this.body = body;`
`97`	`98`	`}`
`98`	`99`
`99`	`100`	`/**`