Add org.sonar.api.server.ws.Request#paramAsStrings(String)

2 files changed, 21 insertions, 0 deletions

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/server/ws/Request.java b/sonar-plugin-api/src/main/java/org/sonar/api/server/ws/Request.java
index 73ab0a8b939..eacbb069512 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/server/ws/Request.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/server/ws/Request.java
@@ -19,9 +19,12 @@
  */
 package org.sonar.api.server.ws;
 
+import com.google.common.base.Splitter;
+import com.google.common.collect.Lists;
 import org.apache.commons.lang.StringUtils;
 
 import javax.annotation.CheckForNull;
+import java.util.List;
 
 /**
  * @since 4.2
@@ -88,6 +91,15 @@ public abstract class Request {
   }
 
   @CheckForNull
+  public List<String> paramAsStrings(String key) {
+    String s = param(key);
+    if (s == null) {
+      return null;
+    }
+    return Lists.newArrayList(Splitter.on(',').omitEmptyStrings().trimResults().split(s));
+  }
+
+  @CheckForNull
   public Integer paramAsInt(String key) {
     String s = param(key);
     return s == null ? null : Integer.parseInt(s);
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
index 8003e7577ee..ef28b62c2c8 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
@@ -129,4 +129,13 @@ public class RequestTest {
     assertThat(request.setParam("foo", "true").paramAsBoolean("xxx", true)).isTrue();
     assertThat(request.setParam("foo", "true").paramAsBoolean("xxx", false)).isFalse();
   }
+
+  @Test
+  public void param_as_strings() throws Exception {
+    assertThat(request.paramAsStrings("foo")).isNull();
+    assertThat(request.setParam("foo", "").paramAsStrings("foo")).isEmpty();
+    assertThat(request.setParam("foo", "bar").paramAsStrings("foo")).containsExactly("bar");
+    assertThat(request.setParam("foo", "bar,baz").paramAsStrings("foo")).containsExactly("bar", "baz");
+    assertThat(request.setParam("foo", "bar , baz").paramAsStrings("foo")).containsExactly("bar", "baz");
+  }
 }