SONAR-3755 minor comments

diff --git a/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueFinder.java b/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueFinder.java
index 0e6c8f83b5cc3f1760f3a1374293a9c2f9d8f39f..3689c541f600d3b6c894a4ea48d2e65ddefeff76 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueFinder.java
+++ b/sonar-core/src/main/java/org/sonar/core/issue/DefaultIssueFinder.java
@@ -93,7 +93,6 @@ public class DefaultIssueFinder implements IssueFinder {
       }
     }));
 
-
     return new DefaultResults(issues);
   }
 

diff --git a/sonar-core/src/main/java/org/sonar/core/issue/IssueDao.java b/sonar-core/src/main/java/org/sonar/core/issue/IssueDao.java
index 8514ce2f75e2962c31959688573d049ee78fa6eb..2962518a8c431be59697afdef30bc164d26db887 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/issue/IssueDao.java
+++ b/sonar-core/src/main/java/org/sonar/core/issue/IssueDao.java
@@ -88,6 +88,8 @@ public class IssueDao implements BatchComponent, ServerComponent {
   public List<IssueDto> select(IssueQuery query) {
     SqlSession session = mybatis.openSession();
     try {
+      // TODO support ordering
+
       return session.selectList("org.sonar.core.issue.IssueMapper.select", query, new RowBounds(query.offset(), query.limit()));
 
     } finally {

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/issue/JRubyIssues.java b/sonar-plugin-api/src/main/java/org/sonar/api/issue/JRubyIssues.java
index 8ed376ab670a50f2170041b8f8c766d84fdf6026..3cb966e800a42c86df6f41474fc147d266b2af81 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/issue/JRubyIssues.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/issue/JRubyIssues.java
@@ -28,14 +28,22 @@ import java.util.Map;
  * <p>
  * Reference from Ruby code : <code>Api.issues</code>
  * </p>
+ *
+ * @since 3.6
  */
 public interface JRubyIssues extends ServerComponent {
 
   /**
-   * <code>Api.issues.find(hash)</code>
+   * Search for issues.
    *
-   * TODO document parameters
+   * <p>
+   *   Ruby: <code>Api.issues.find(hash)</code>
+   * </p>
    *
+   * <p>Parameters</p>
+   * <ul>
+   *   TODO document parameters
+   * </ul>
    */
   IssueFinder.Results find(Map<String, Object> parameters);