No need to use mb functions here

When mb functions are used and a large response is given the OOM happens as mb functions will use large quantities of memory.
symfony · Apr 18, 2014 · 4e8ba13 · 4e8ba13
1 parent 5b72e5a
commit 4e8ba13
Showing 1 changed file with 2 additions and 10 deletions.
diff --git a/src/Symfony/Bundle/WebProfilerBundle/EventListener/WebDebugToolbarListener.php b/src/Symfony/Bundle/WebProfilerBundle/EventListener/WebDebugToolbarListener.php
@@ -96,16 +96,8 @@ public function onKernelResponse(FilterResponseEvent $event)
      */
     protected function injectToolbar(Response $response)
     {
-        if (function_exists('mb_stripos')) {
-            $posrFunction   = 'mb_strripos';
-            $substrFunction = 'mb_substr';
-        } else {
-            $posrFunction   = 'strripos';
-            $substrFunction = 'substr';
-        }
-
         $content = $response->getContent();
-        $pos = $posrFunction($content, '</body>');
+        $pos = strripos($content, '</body>');
 
         if (false !== $pos) {
             $toolbar = "\n".str_replace("\n", '', $this->twig->render(
@@ -115,7 +107,7 @@ protected function injectToolbar(Response $response)
                     'token' => $response->headers->get('X-Debug-Token'),
                 )
             ))."\n";
-            $content = $substrFunction($content, 0, $pos).$toolbar.$substrFunction($content, $pos);
+            $content = substr($content, 0, $pos).$toolbar.substr($content, $pos);
             $response->setContent($content);
         }
     }