Merge branch 'hotfix/151'

Close #151
Fixes #93

Author: weierophinney

CHANGELOG.md

@@ -31,6 +31,8 @@ All notable changes to this project will be documented in this file, in reverse
 - [#148](https://github.com/zendframework/zend-diactoros/pull/148) fixes several
   stream operations to ensure they raise exceptions when the internal pointer
   is at an invalid position.
+- [#151](https://github.com/zendframework/zend-diactoros/pull/151) ensures
+  URI fragments are properly encoded.
 
 ## 1.3.3 - 2016-01-04
 

src/Uri.php

@@ -627,7 +627,7 @@ private function splitQueryValue($value)
     private function filterFragment($fragment)
     {
         if (! empty($fragment) && strpos($fragment, '#') === 0) {
-            $fragment = substr($fragment, 1);
+            $fragment = '%23' . substr($fragment, 1);
         }
 
         return $this->filterQueryOrFragment($fragment);

test/UriTest.php

@@ -388,11 +388,11 @@ public function testStripsQueryPrefixIfPresent()
         $this->assertEquals('foo=bar', $new->getQuery());
     }
 
-    public function testStripsFragmentPrefixIfPresent()
+    public function testEncodeFragmentPrefixIfPresent()
     {
         $uri = new Uri('http://example.com');
         $new = $uri->withFragment('#/foo/bar');
-        $this->assertEquals('/foo/bar', $new->getFragment());
+        $this->assertEquals('%23/foo/bar', $new->getFragment());
     }
 
     public function standardSchemePortCombinations()