Correcting bug in "SessionClient::listSessions()"

dcarbone · Jul 21, 2016 · 5169215 · 5169215
1 parent d33b92b
commit 5169215
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 7 deletions.
diff --git a/src/HttpRequest.php b/src/HttpRequest.php
@@ -132,12 +132,23 @@ public function setWriteOptions(WriteOptions $writeOptions = null)
      */
     public function execute()
     {
-        $this->url = sprintf(
-            '%s/%s?%s',
-            $this->_Config->compileAddress(),
-            ltrim(trim($this->path), "/"),
-            $this->params
-        );
+        if (0 === count($this->params))
+        {
+            $this->url = sprintf(
+                '%s/%s',
+                $this->_Config->compileAddress(),
+                ltrim(trim($this->path), "/")
+            );
+        }
+        else
+        {
+            $this->url = sprintf(
+                '%s/%s?%s',
+                $this->_Config->compileAddress(),
+                ltrim(trim($this->path), "/"),
+                $this->params
+            );
+        }
 
         Logger::log('debug', 'Executing '.$this->method.' request '.$this->url.($this->body ? ' with body "'.$this->body.'"':''));
 

diff --git a/src/Session/SessionClient.php b/src/Session/SessionClient.php
@@ -257,7 +257,7 @@ public function node($node, QueryOptions $queryOptions = null)
      */
     public function listSessions(QueryOptions $queryOptions = null)
     {
-        $r = new HttpRequest('get', 'v1/sesion/list', $this->_Config);
+        $r = new HttpRequest('get', 'v1/session/list', $this->_Config);
         $r->setQueryOptions($queryOptions);
 
         list($duration, $response, $err) = $this->requireOK($this->doRequest($r));