Fix $response->getBody()->getContents() returns always empty content #48

(#80)
8p · Dec 12, 2016 · 88e1ed0 · 88e1ed0
1 parent 2b6eb8a
commit 88e1ed0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/Log/LogResponse.php b/Log/LogResponse.php
@@ -60,17 +60,11 @@ public function save(ResponseInterface $response)
     {
         $this->setStatusCode($response->getStatusCode());
         $this->setStatusPhrase($response->getReasonPhrase());
+        $this->setBody($response->getBody()->getContents());
 
-        // rewind to previous position after response body
-        $readPosition = null;
-        if($response->getBody()->isSeekable()) {
-            $readPosition = $response->getBody()->tell();
-        }
-
-        $this->setBody($response->getBody()->__toString());
-
-        if($readPosition) {
-            $response->getBody()->seek($readPosition);
+        // rewind to previous position after reading response body
+        if ($response->getBody()->isSeekable()) {
+            $response->getBody()->rewind();
         }
 
         $this->setHeaders($response->getHeaders());

diff --git a/Tests/Log/LogResponseTest.php b/Tests/Log/LogResponseTest.php
@@ -41,7 +41,7 @@ public function setUp()
                          ->disableOriginalConstructor()
                          ->getMock();
 
-        $bodyMock->method('__toString')->willReturn('test body');
+        $bodyMock->method('getContents')->willReturn('test body');
 
         $this->response->method('getStatusCode')->willReturn(200);
         $this->response->method('getHeaders')->willReturn($this->headers);