Json strategy should only be matched for application/json and applica…

…tion/javascript
zendframework · Jul 19, 2012 · c6ef70d · c6ef70d
7 parents e3b1be1 + 19f0ef6 + 33bf9c0 + 3729984 + a5ce396 + 698dbe0 + 0efca0b
commit c6ef70d
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/Strategy/JsonStrategy.php b/src/Strategy/JsonStrategy.php
@@ -107,17 +107,18 @@ public function selectRenderer(ViewEvent $e)
             return;
         }
 
-        if ($match->getFormat() == 'json') {
+        if ($match->getTypeString() == 'application/json') {
             // application/json Accept header found
             return $this->renderer;
         }
 
-        // application/javascript Accept header found
-        if (false != ($callback = $request->getQuery()->get('callback'))) {
-            $this->renderer->setJsonpCallback($callback);
+        if ($match->getTypeString() == 'application/javascript') {
+            // application/javascript Accept header found
+            if (false != ($callback = $request->getQuery()->get('callback'))) {
+                $this->renderer->setJsonpCallback($callback);
+            }
+            return $this->renderer;
         }
-
-        return $this->renderer;
     }
 
     /**