Merge branch 'hotfix/79'

Close #79
Fixes #78

Author: weierophinney

CHANGELOG.md

@@ -21,6 +21,9 @@ All notable changes to this project will be documented in this file, in reverse
 - [#71](https://github.com/zendframework/zend-diactoros/pull/71) fixes the
   docblock of the `JsonResponse` constructor to typehint the `$data` argument
   as `mixed`.
+- [#73](https://github.com/zendframework/zend-diactoros/pull/73) changes the
+  behavior in `Request` such that if it marshals a stream during instantiation,
+  the stream is marked as writeable (specifically, mode `wb+`).
 
 ## 1.1.2 - 2015-07-12
 

src/RequestTrait.php

@@ -82,7 +82,7 @@ private function initialize($uri = null, $method = null, $body = 'php://memory',
 
         $this->method = $method ?: '';
         $this->uri    = $uri ?: new Uri();
-        $this->stream = ($body instanceof StreamInterface) ? $body : new Stream($body, 'r');
+        $this->stream = ($body instanceof StreamInterface) ? $body : new Stream($body, 'wb+');
 
         list($this->headerNames, $headers) = $this->filterHeaders($headers);
         $this->assertHeaders($headers);

test/RequestTest.php

@@ -92,6 +92,14 @@ public function testConstructorCanAcceptAllMessageParts()
         }
     }
 
+    public function testDefaultStreamIsWritable()
+    {
+        $request = new Request();
+        $request->getBody()->write("test");
+
+        $this->assertSame("test", (string)$request->getBody());
+    }
+
     public function invalidRequestUri()
     {
         return [