Message Event mocked tests pass

2 files changed, 30 insertions, 12 deletions

diff --git a/lib/SparkPost/APIResource.php b/lib/SparkPost/APIResource.php
index d0b8a5b..71c6a26 100644
--- a/lib/SparkPost/APIResource.php
+++ b/lib/SparkPost/APIResource.php
@@ -112,11 +112,6 @@ class APIResource {
    * @return array Result of the request
    */
   public function get( $resourcePath=null, Array $query=[] ) {
-    foreach($query as $element) {
-      if(is_array($element)) {
-        
-      }
-    }
     return $this->callResource( 'get', $resourcePath, ['query'=>$query] );
   }
 
@@ -135,7 +130,8 @@ class APIResource {
   /**
    * assembles a URL for a request
    * @param string $resourcePath path after the initial endpoint
-   * @param array $options array with an optional value of query with values to build a querystring from.
+   * @param array $options array with an optional value of query with values to build a querystring from.  Any
+   *                        query elements that are themselves arrays will be imploded into a comma separated list. 
    * @return string the assembled URL
    */
   private function buildUrl($resourcePath, $options) {
@@ -145,6 +141,12 @@ class APIResource {
     }
 
     if( !empty($options['query'])) {
+      foreach($options['query'] as &$element) {
+        if(is_array($element)) {
+          $element = implode(",", $element);
+        }
+      }
+
       $queryString = http_build_query($options['query']);
       $url .= '?'.$queryString;
     }
diff --git a/lib/SparkPost/MessageEvent.php b/lib/SparkPost/MessageEvent.php
index a0ab16e..a72fb49 100644
--- a/lib/SparkPost/MessageEvent.php
+++ b/lib/SparkPost/MessageEvent.php
@@ -1,11 +1,27 @@
 <?php
 namespace SparkPost;
 
+/**
+ * SDK class for querying message events API
+ * @package SparkPost
+ */
+class MessageEvent extends APIResource
+{
+  public $endpoint = 'message-events';
 
-class MessageEvent extends APIResource {
-	public $endpoint = 'message-events';
-
-	public function search(Array $queryParams) {
-		return $this->get(null, $queryParams);
-	}
+  /**
+   * Method for issuing search requests to the Message Events API.
+   *
+   * The method passes-through all of the query parameters listed at
+   * @link https://developers.sparkpost.com/api/#/reference/message-events/events-documentation/search-for-message-events
+   *
+   * @param array $queryParams  The query parameters.  Note that a query parameter containing an array
+   * is collapsed into a comma-separated list.
+   *
+   * @return array The result of the query.
+   */
+  public function search(Array $queryParams)
+  {
+    return $this->get(null, $queryParams);
+  }
 }
\ No newline at end of file