]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-3755 Update java web service client to use latest ruby API
authorJulien Lancelot <julien.lancelot@gmail.com>
Fri, 12 Apr 2013 14:59:22 +0000 (16:59 +0200)
committerJulien Lancelot <julien.lancelot@gmail.com>
Fri, 12 Apr 2013 14:59:22 +0000 (16:59 +0200)
sonar-server/src/main/webapp/WEB-INF/app/views/issues/_list.html.erb
sonar-ws-client/src/main/java/org/sonar/wsclient/services/IssueQuery.java
sonar-ws-client/src/test/java/org/sonar/wsclient/services/IssueQueryTest.java

index a4d40056d61eb3143e746068b1533fc80311d620..de2809a8f0ec9a89e143e36f87eddd95324f2a94 100644 (file)
           <%= h(issue.message) %>
         </td>
         <td>
-          <span class="nowrap"><%= h issue.component_key -%></span>
+          <%= h issue.component_key -%>
         </td>
         <td>
           <%= issue.rule_repository_key + ":" + issue.rule_key %>
         </td>
         <td>
-          <%= issue.cost -%>
+          <%= issue.line -%>
         </td>
         <td>
           <%= issue.cost -%>
index faee242c8fa87a404bf822a8a706b70390b29329..988df66f72311985127a8bcf3aafb1869996a3e6 100644 (file)
@@ -24,9 +24,8 @@ package org.sonar.wsclient.services;
  */
 public final class IssueQuery extends Query<Issue> {
 
-  public static final String BASE_URL = "/api/issues";
+  public static final String BASE_URL = "/api/issues/search";
 
-  private String key;
   private String[] keys;
   private String[] severities;
   private String minSeverity;
@@ -49,19 +48,6 @@ public final class IssueQuery extends Query<Issue> {
     return new IssueQuery();
   }
 
-  public static IssueQuery byKey(String key) {
-    return new IssueQuery().setKey(key);
-  }
-
-  public String getKey() {
-    return key;
-  }
-
-  public IssueQuery setKey(String key) {
-    this.key = key;
-    return this;
-  }
-
   public String[] getKeys() {
     return keys;
   }
@@ -155,10 +141,6 @@ public final class IssueQuery extends Query<Issue> {
   @Override
   public String getUrl() {
     StringBuilder url = new StringBuilder(BASE_URL);
-    if (key != null && !"".equals(key)) {
-      url.append("/");
-      url.append(encode(key));
-    }
     url.append('?');
     appendUrlParameter(url, "keys", keys);
     appendUrlParameter(url, "severities", severities);
index 180af81f149fa2123616ae4d698365fb6575680a..1d620e483e28eb385a8c865dc049bdeb9695dbf1 100644 (file)
@@ -29,7 +29,7 @@ public class IssueQueryTest extends QueryTestCase {
   @Test
   public void get_all_issues() {
     IssueQuery query = IssueQuery.create();
-    assertThat(query.getUrl()).isEqualTo("/api/issues?");
+    assertThat(query.getUrl()).isEqualTo("/api/issues/search?");
   }
 
   @Test
@@ -45,16 +45,9 @@ public class IssueQueryTest extends QueryTestCase {
         .setStatus("status1", "status2")
         .setSeverities("sev1", "sev2")
         .setUserLogins("userLogin1", "userLogin2");
-    assertThat(query.getUrl()).isEqualTo("/api/issues?keys=key1,key2&severities=sev1,sev2&minSeverity=minSev&status=status1,status2&" +
+    assertThat(query.getUrl()).isEqualTo("/api/issues/search?keys=key1,key2&severities=sev1,sev2&minSeverity=minSev&status=status1,status2&" +
         "resolutions=resoltion1,resolution2&components=component1,component2&rules=rule1,rule2&userLogins=userLogin1,userLogin2&" +
         "assigneeLogins=assigneeLogin1,assigneeLogin2&limit=1&");
   }
 
-  @Test
-  public void get_issue_by_key() {
-    IssueQuery query = IssueQuery.byKey("issue_key");
-    assertThat(query.getUrl()).isEqualTo("/api/issues/issue_key?");
-    assertThat(query.getKey()).isEqualTo("issue_key");
-  }
-
 }