Merge branch 'hotfix/2660'

zendframework · Oct 3, 2012 · a29bbbc · a29bbbc
commit a29bbbc
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/Strategy/JsonStrategy.php b/src/Strategy/JsonStrategy.php
@@ -112,6 +112,8 @@ public function selectRenderer(ViewEvent $e)
             }
             return $this->renderer;
         }
+
+        return ($model instanceof Model\JsonModel) ? $this->renderer : null;
     }
 
     /**

diff --git a/test/Strategy/JsonStrategyTest.php b/test/Strategy/JsonStrategyTest.php
@@ -63,6 +63,16 @@ public function testJavascriptAcceptHeaderSelectsJsonStrategy()
         $this->assertFalse($result->hasJsonpCallback());
     }
 
+    public function testJsonModelMatchedAcceptHeaderMatchSelectsJsonStrategy()
+    {
+        $this->event->setModel(new JsonModel());
+        $request = new HttpRequest();
+        $request->getHeaders()->addHeaderLine('Accept', '*/*');
+        $this->event->setRequest($request);
+        $result = $this->strategy->selectRenderer($this->event);
+        $this->assertSame($this->renderer, $result);
+    }
+
     public function testJsonModelJavascriptAcceptHeaderSetsJsonpCallback()
     {
         $this->event->setModel(new JsonModel());