[MJAVADOC-814] handle parameters such packages with multi lines

Signed-off-by: Olivier Lamy <[email protected]>

Author: olamy

src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java

@@ -57,6 +57,7 @@
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import java.util.regex.PatternSyntaxException;
+import java.util.stream.Collectors;
 
 import org.apache.http.HttpHeaders;
 import org.apache.http.HttpHost;
@@ -223,12 +224,12 @@ protected static List<String> getExcludedPackages(
     protected static String quotedArgument(String value) {
         String arg = value;
 
+        List<String> list =  Arrays.stream(arg.split("\n")).map(String::trim).collect(Collectors.toList());
+        arg = String.join("", list);
+
         if (arg != null && !arg.isEmpty()) {
             arg = arg.replace("'", "\\'");
             arg = "'" + arg + "'";
-
-            // To prevent javadoc error
-            arg = arg.replace("\n", " ");
         }
 
         return arg;

src/test/java/org/apache/maven/plugins/javadoc/JavadocUtilTest.java

@@ -675,4 +675,19 @@ private void stopSilently(Server server) {
             // ignored
         }
     }
+
+    public void testQuotedArgument() throws Exception {
+
+        String value = "      org.apache.uima.analysis_component:\n" +
+                "      org.apache.uima.analysis_engine\n";
+
+        String arg = JavadocUtil.quotedArgument(value);
+        assertThat(arg).isEqualTo("'org.apache.uima.analysis_component:org.apache.uima.analysis_engine'");
+
+        value = "org.apache.uima.analysis_component:org.apache.uima.analysis_engine";
+
+        arg = JavadocUtil.quotedArgument(value);
+        assertThat(arg).isEqualTo("'org.apache.uima.analysis_component:org.apache.uima.analysis_engine'");
+
+    }
 }