SONARJAVA-5719 S1176 Should have a separate message for undocumented type parameters.

java-checks-test-sources/default/src/main/files/non-compiling/checks/api/undocumentedAPI/UndocumentedApi.java

@@ -67,7 +67,7 @@ public A() { // Noncompliant {{Document this public constructor by adding an exp
 /**
  * This is a Javadoc comment
  */
-public class MyClass<T> implements Runnable { // Noncompliant {{Document the parameter(s): <T>}}
+public class MyClass<T> implements Runnable { // Noncompliant {{Document the type parameter(s): <T>}}
 
  private int status;                            // Compliant - not public
 

java-checks-test-sources/default/src/main/java/checks/api/undocumentedAPI/UndocumentedApiJava23.java

@@ -51,7 +51,7 @@ public String missingThrows(int value) throws NumberFormatException { // Noncomp
   }
 
   /// Documented, but not the type.
-  public class SomethingGenericBad<T> { // Noncompliant {{Document the parameter(s): <T>}}
+  public class SomethingGenericBad<T> { // Noncompliant {{Document the type parameter(s): <T>}}
   }
 
   /// Documented.

java-checks/src/main/java/org/sonar/java/checks/UndocumentedApiCheck.java

@@ -129,7 +129,8 @@ private void visitNode(Tree tree, Tree reportTree, SymbolMetadata symbolMetadata
       } else {
         Set<String> undocumentedParameters = javadoc.undocumentedParameters();
         if (!undocumentedParameters.isEmpty()) {
-          context.reportIssue(this, reportTree, "Document the parameter(s): " + undocumentedParameters.stream().collect(Collectors.joining(", ")));
+          String label = getParamLabel(tree);
+          context.reportIssue(this, reportTree, "Document the " + label + undocumentedParameters.stream().collect(Collectors.joining(", ")));
         }
         if (hasNonVoidReturnType(tree) && javadoc.noReturnDescription()) {
           context.reportIssue(this, reportTree, "Document this method return value.");
@@ -153,6 +154,10 @@ private boolean isNonVoidMethodWithNoParameter(Tree tree, Javadoc javadoc) {
       && !javadoc.noReturnDescription();
   }
 
+  private static String getParamLabel(Tree tree) {
+    return (tree.is(Kind.CLASS) || tree.is(Kind.INTERFACE)) ? "type parameter(s): " : "parameter(s): ";
+  }
+
   private static String getType(Tree tree) {
     switch (tree.kind()) {
       case CONSTRUCTOR: