ppi / framework

src/Debug/ExceptionHandler.php

Doc Comments +7 added lines, -3 removed lines

@@ -67,7 +67,7 @@ 
     }
 
     /**
-     * @param $appVersion
+     * @param string $appVersion
      *
      * @return $this
      */
@@ -128,7 +128,7 @@ 
      * This method uses plain PHP functions like header() and echo to output
      * the response.
      *
-     * @param \Exception|FlattenException $exception An \Exception instance
+     * @param \Exception $exception An \Exception instance
      */
     public function sendPhpResponse($exception)
     {
@@ -147,7 +147,7 @@ 
     /**
      * Creates the error Response associated with the given Exception.
      *
-     * @param \Exception|FlattenException $exception An \Exception instance
+     * @param \Exception $exception An \Exception instance
      *
      * @return Response A Response instance
      */
@@ -299,6 +299,10 @@ 
 EOT;
     }
 
+    /**
+     * @param string $content
+     * @param string $css
+     */
     private function decorate($content, $css)
     {
         $baseStylesheet = $this->getBaseStylesheet();

Switch Indentation +5 added lines, -5 removed lines

@@ -180,11 +180,11 @@
     public function getContent(FlattenException $exception, $showAll = true)
     {
         switch ($exception->getStatusCode()) {
-            case 404:
-                $title = "The page you are looking for could not be found";
-                break;
-            default:
-                $title = "Oh noes, something's broken";
+        case 404:
+            $title = "The page you are looking for could not be found";
+            break;
+        default:
+            $title = "Oh noes, something's broken";
         }
 
         $content = '';

Spacing +1 added lines, -1 removed lines

@@ -367,7 +367,7 @@
                 $formattedValue = sprintf("<em>object</em>(%s)", $this->abbrClass($item[1]));
             } elseif ('array' === $item[0]) {
                 $formattedValue = sprintf("<em>array</em>(%s)", is_array($item[1]) ? $this->formatArgs($item[1]) : $item[1]);
-            } elseif ('string'  === $item[0]) {
+            } elseif ('string' === $item[0]) {
                 $formattedValue = sprintf("'%s'", htmlspecialchars($item[1], ENT_QUOTES | ENT_SUBSTITUTE, $this->charset));
             } elseif ('null' === $item[0]) {
                 $formattedValue = '<em>null</em>';

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\View;
 

src/Http/Request.php

Doc Comments +1 added lines, -1 removed lines

@@ -592,7 +592,7 @@
      *
      * @throws InvalidArgumentException on invalid HTTP method.
      *
-     * @return bool
+     * @return boolean|null
      */
     private function validateMethod($method)
     {

Spacing +6 added lines, -6 removed lines

@@ -173,7 +173,7 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
@@ -212,13 +212,13 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
         }
 
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return $this->withHeader($header, $value);
         }
 
@@ -246,7 +246,7 @@ 
      */
     public function withoutHeader($header)
     {
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return clone $this;
         }
 
@@ -599,14 +599,14 @@ 
         if (null === $method) {
             return true;
         }
-        if (! is_string($method)) {
+        if (!is_string($method)) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported HTTP method; must be a string, received %s',
                 (is_object($method) ? get_class($method) : gettype($method))
             ));
         }
         $method = strtoupper($method);
-        if (! in_array($method, $this->validMethods, true)) {
+        if (!in_array($method, $this->validMethods, true)) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported HTTP method "%s" provided',
                 $method

src/Http/Response.php

Doc Comments +3 added lines, -3 removed lines

@@ -443,7 +443,7 @@ 
     }
 
     /**
-     * @param $reasonPhrase
+     * @param string $reasonPhrase
      *
      * @return $this
      */
@@ -466,7 +466,7 @@ 
      * @link http://tools.ietf.org/html/rfc7231#section-6
      * @link http://www.iana.org/assignments/http-status-codes/http-status-codes.xhtml
      *
-     * @return string|null Reason phrase, or null if unknown.
+     * @return string Reason phrase, or null if unknown.
      */
     public function getReasonPhrase()
     {
@@ -535,7 +535,7 @@ 
     /**
      * Validate a status code.
      *
-     * @param int|string $code
+     * @param integer $code
      *
      * @throws InvalidArgumentException on an invalid status code.
      */

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\Router;
 

Spacing +9 added lines, -9 removed lines

@@ -163,7 +163,7 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
@@ -202,13 +202,13 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
         }
 
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return $this->withHeader($header, $value);
         }
 
@@ -236,7 +236,7 @@ 
      */
     public function withoutHeader($header)
     {
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return clone $this;
         }
 
@@ -257,7 +257,7 @@ 
      */
     public function setBody($body = 'php://memory')
     {
-        if (! is_string($body) && ! is_resource($body) && ! $body instanceof StreamInterface) {
+        if (!is_string($body) && !is_resource($body) && !$body instanceof StreamInterface) {
             throw new InvalidArgumentException(
                 'Stream must be a string stream resource identifier, '
                 . 'an actual stream resource, '
@@ -499,15 +499,15 @@ 
     {
         $headerNames = $headers = array();
         foreach ($originalHeaders as $header => $value) {
-            if (! is_string($header)) {
+            if (!is_string($header)) {
                 continue;
             }
 
-            if (! is_array($value) && ! is_string($value)) {
+            if (!is_array($value) && !is_string($value)) {
                 continue;
             }
 
-            if (! is_array($value)) {
+            if (!is_array($value)) {
                 $value = array($value);
             }
 
@@ -542,7 +542,7 @@ 
      */
     private function validateStatus($code)
     {
-        if (! is_numeric($code)
+        if (!is_numeric($code)
             || is_float($code)
             || $code < 100
             || $code >= 600

src/Http/Stream.php

Doc Comments +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
      *
      * After the stream has been detached, the stream is in an unusable state.
      *
-     * @return resource|null
+     * @return resource
      */
     public function detach()
     {

Spacing +16 added lines, -16 removed lines

@@ -67,7 +67,7 @@ 
      */
     public function __toString()
     {
-        if (! $this->isReadable()) {
+        if (!$this->isReadable()) {
             return '';
         }
 
@@ -79,7 +79,7 @@ 
      */
     public function close()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return;
         }
 
@@ -116,8 +116,8 @@ 
     public function attach($resource, $mode = 'r')
     {
         $error = null;
-        if (! is_resource($resource) && is_string($resource)) {
-            set_error_handler(function ($e) use (&$error) {
+        if (!is_resource($resource) && is_string($resource)) {
+            set_error_handler(function($e) use (&$error) {
                 $error = $e;
             }, E_WARNING);
             $resource = fopen($resource, $mode);
@@ -128,7 +128,7 @@ 
             throw new InvalidArgumentException('Invalid stream reference provided');
         }
 
-        if (! is_resource($resource)) {
+        if (!is_resource($resource)) {
             throw new InvalidArgumentException(
                 'Invalid stream provided; must be a string stream identifier or resource'
             );
@@ -160,7 +160,7 @@ 
      */
     public function tell()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return false;
         }
 
@@ -174,7 +174,7 @@ 
      */
     public function eof()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return true;
         }
 
@@ -188,7 +188,7 @@ 
      */
     public function isSeekable()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return false;
         }
 
@@ -214,7 +214,7 @@ 
      */
     public function seek($offset, $whence = SEEK_SET)
     {
-        if (! $this->resource || ! $this->isSeekable()) {
+        if (!$this->resource || !$this->isSeekable()) {
             return false;
         }
 
@@ -230,7 +230,7 @@ 
      */
     public function rewind()
     {
-        if (! $this->isSeekable()) {
+        if (!$this->isSeekable()) {
             return false;
         }
 
@@ -246,7 +246,7 @@ 
      */
     public function isWritable()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return false;
         }
 
@@ -265,7 +265,7 @@ 
      */
     public function write($string)
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return false;
         }
 
@@ -279,7 +279,7 @@ 
      */
     public function isReadable()
     {
-        if (! $this->resource) {
+        if (!$this->resource) {
             return false;
         }
 
@@ -302,7 +302,7 @@ 
      */
     public function read($length)
     {
-        if (! $this->resource || ! $this->isReadable()) {
+        if (!$this->resource || !$this->isReadable()) {
             return false;
         }
 
@@ -320,7 +320,7 @@ 
      */
     public function getContents()
     {
-        if (! $this->isReadable()) {
+        if (!$this->isReadable()) {
             return '';
         }
 
@@ -341,7 +341,7 @@ 
         }
 
         $metadata = stream_get_meta_data($this->resource);
-        if (! array_key_exists($key, $metadata)) {
+        if (!array_key_exists($key, $metadata)) {
             return;
         }
 

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\Router;
 

src/Http/Uri.php

Doc Comments +1 added lines

@@ -589,6 +589,7 @@
 
     /**
      * Parse a URI into its parts, and set the properties.
+     * @param string $uri
      */
     private function parseUri($uri)
     {

Spacing +21 added lines, -21 removed lines

@@ -109,18 +109,18 @@ 
      */
     public function __construct($uri = '')
     {
-        if (! is_string($uri)) {
+        if (!is_string($uri)) {
             throw new InvalidArgumentException(sprintf(
                 'URI passed to constructor must be a string; received "%s"',
                 (is_object($uri) ? get_class($uri) : gettype($uri))
             ));
         }
 
-        $this->urlEncode = function (array $matches) {
+        $this->urlEncode = function(array $matches) {
             return rawurlencode($matches[0]);
         };
 
-        if (! empty($uri)) {
+        if (!empty($uri)) {
             $this->parseUri($uri);
         }
     }
@@ -210,7 +210,7 @@ 
         }
 
         $authority = $this->host;
-        if (! empty($this->userInfo)) {
+        if (!empty($this->userInfo)) {
             $authority = $this->userInfo . '@' . $authority;
         }
 
@@ -436,7 +436,7 @@ 
      */
     public function withPort($port)
     {
-        if (! (is_integer($port) || (is_string($port) && is_numeric($port)))) {
+        if (!(is_integer($port) || (is_string($port) && is_numeric($port)))) {
             throw new InvalidArgumentException(sprintf(
                 'Invalid port "%s" specified; must be an integer or integer string',
                 (is_object($port) ? get_class($port) : gettype($port))
@@ -482,7 +482,7 @@ 
      */
     public function withPath($path)
     {
-        if (! is_string($path)) {
+        if (!is_string($path)) {
             throw new InvalidArgumentException(
                 'Invalid path provided; must be a string'
             );
@@ -533,7 +533,7 @@ 
      */
     public function withQuery($query)
     {
-        if (! is_string($query)) {
+        if (!is_string($query)) {
             throw new InvalidArgumentException(
                 'Query string must be a string'
             );
@@ -594,12 +594,12 @@ 
     {
         $parts = parse_url($uri);
 
-        $this->scheme    = isset($parts['scheme'])   ? $this->filterScheme($parts['scheme']) : '';
-        $this->userInfo  = isset($parts['user'])     ? $parts['user']     : '';
-        $this->host      = isset($parts['host'])     ? $parts['host']     : '';
-        $this->port      = isset($parts['port'])     ? $parts['port']     : null;
-        $this->path      = isset($parts['path'])     ? $this->filterPath($parts['path']) : '';
-        $this->query     = isset($parts['query'])    ? $this->filterQuery($parts['query']) : '';
+        $this->scheme    = isset($parts['scheme']) ? $this->filterScheme($parts['scheme']) : '';
+        $this->userInfo  = isset($parts['user']) ? $parts['user'] : '';
+        $this->host      = isset($parts['host']) ? $parts['host'] : '';
+        $this->port      = isset($parts['port']) ? $parts['port'] : null;
+        $this->path      = isset($parts['path']) ? $this->filterPath($parts['path']) : '';
+        $this->query     = isset($parts['query']) ? $this->filterQuery($parts['query']) : '';
         $this->fragment  = isset($parts['fragment']) ? $this->filterFragment($parts['fragment']) : '';
 
         if (isset($parts['pass'])) {
@@ -622,11 +622,11 @@ 
     {
         $uri = '';
 
-        if (! empty($scheme)) {
+        if (!empty($scheme)) {
             $uri .= sprintf('%s://', $scheme);
         }
 
-        if (! empty($authority)) {
+        if (!empty($authority)) {
             $uri .= $authority;
         }
 
@@ -656,15 +656,15 @@ 
      */
     private function isNonStandardPort($scheme, $host, $port)
     {
-        if (! $scheme) {
+        if (!$scheme) {
             return true;
         }
 
-        if (! $host || ! $port) {
+        if (!$host || !$port) {
             return false;
         }
 
-        return ! isset($this->allowedSchemes[$scheme]) || $port !== $this->allowedSchemes[$scheme];
+        return !isset($this->allowedSchemes[$scheme]) || $port !== $this->allowedSchemes[$scheme];
     }
 
     /**
@@ -683,7 +683,7 @@ 
             return '';
         }
 
-        if (! array_key_exists($scheme, $this->allowedSchemes)) {
+        if (!array_key_exists($scheme, $this->allowedSchemes)) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported scheme "%s"; must be any empty string or in the set (%s)',
                 $scheme,
@@ -728,7 +728,7 @@ 
      */
     private function filterQuery($query)
     {
-        if (! empty($query) && strpos($query, '?') === 0) {
+        if (!empty($query) && strpos($query, '?') === 0) {
             $query = substr($query, 1);
         }
 
@@ -779,7 +779,7 @@ 
             $fragment = '';
         }
 
-        if (! empty($fragment) && strpos($fragment, '#') === 0) {
+        if (!empty($fragment) && strpos($fragment, '#') === 0) {
             $fragment = substr($fragment, 1);
         }
 

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\Router;
 

src/MicroApp.php

Doc Comments +3 added lines

@@ -40,6 +40,9 @@
         $this->add($uri, 'post', $callback);
     }
 
+    /**
+     * @param string $method
+     */
     public function add($uri, $method, $callback)
     {
         $this->routes[md5($uri . $method)] = array(

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\View;
 

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Symfony\Component\Routing\Matcher\UrlMatcher as UrlMatcher;
 use Symfony\Component\Routing\Route as Route;
 use Symfony\Component\Routing\RouteCollection;
-use Symfony\Component\Routing\Router;
 
 /**
  * The PPI MicroApp bootstrap class.

src/Module/Controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -458,7 +458,7 @@
      * Get an option from the controller.
      *
      * @param string $option  The option name
-     * @param null   $default The default value if the option does not exist
+     * @param string   $default The default value if the option does not exist
      *
      * @return mixed
      */

Switch Indentation +28 added lines, -28 removed lines

@@ -254,34 +254,34 @@
     {
         switch ($key = strtolower($key)) {
 
-            case 'ajax':
-                if (!isset($this->isCache['ajax'])) {
-                    return $this->isCache['ajax'] = $this->getService('Request')->isXmlHttpRequest();
-                }
-
-                return $this->isCache['ajax'];
-
-            case 'put':
-            case 'delete':
-            case 'post':
-            case 'patch':
-                if (!isset($this->isCache['requestMethod'][$key])) {
-                    $this->isCache['requestMethod'][$key] = $this->getService('Request')->getMethod() === strtoupper($key);
-                }
-
-                return $this->isCache['requestMethod'][$key];
-
-            case 'ssl':
-            case 'https':
-            case 'secure':
-                if (!isset($this->isCache['secure'])) {
-                    $this->isCache['secure'] = $this->getService('Request')->isSecure();
-                }
-
-                return $this->isCache['secure'];
-
-            default:
-                throw new \InvalidArgumentException("Invalid 'is' key supplied: {$key}");
+        case 'ajax':
+            if (!isset($this->isCache['ajax'])) {
+                return $this->isCache['ajax'] = $this->getService('Request')->isXmlHttpRequest();
+            }
+
+            return $this->isCache['ajax'];
+
+        case 'put':
+        case 'delete':
+        case 'post':
+        case 'patch':
+            if (!isset($this->isCache['requestMethod'][$key])) {
+                $this->isCache['requestMethod'][$key] = $this->getService('Request')->getMethod() === strtoupper($key);
+            }
+
+            return $this->isCache['requestMethod'][$key];
+
+        case 'ssl':
+        case 'https':
+        case 'secure':
+            if (!isset($this->isCache['secure'])) {
+                $this->isCache['secure'] = $this->getService('Request')->isSecure();
+            }
+
+            return $this->isCache['secure'];
+
+        default:
+            throw new \InvalidArgumentException("Invalid 'is' key supplied: {$key}");
 
         }
     }

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2012 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2012 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\View\Twig\Loader;
 

src/Router/LaravelRouter.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     /**
      * Given a request object, find the matching route
      *
-     * @param Request $request
+     * @param SymfonyRequest $request
      * @return \Illuminate\Routing\Route
      */
     public function matchRequest(SymfonyRequest $request)

src/ServiceManager/Config/TemplatingConfig.php

Unused Use Statements -1 removed lines

@@ -22,7 +22,6 @@
 use PPI\Framework\View\TemplateLocator;
 use PPI\Framework\View\TemplateNameParser;
 // Mustache
-use Symfony\Bundle\FrameworkBundle\Templating\Loader\FilesystemLoader;
 use Symfony\Component\Templating\Helper\AssetsHelper;
 // Twig
 

Indentation +8 added lines, -8 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright   Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license     http://opensource.org/licenses/mit-license.php MIT
- *
- * @link        http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright   Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license     http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link        http://www.ppi.io
+     */
 
 namespace PPI\Framework\ServiceManager\Config;
 
@@ -124,7 +124,7 @@ 
                     $serviceManager->get('templating.helper.assets'),
                     new RouterHelper($serviceManager->get('router')),
                     new SessionHelper($serviceManager->get('session')),
-                 )
+                    )
             );
 
             $engine->addGlobal('app', $serviceManager->get('templating.globals'));

Spacing +10 added lines, -10 removed lines

@@ -75,7 +75,7 @@ 
         /*
          * Templating Locator.
          */
-        $serviceManager->setFactory('templating.locator', function ($serviceManager) use ($appCacheDir) {
+        $serviceManager->setFactory('templating.locator', function($serviceManager) use ($appCacheDir) {
             return new TemplateLocator(
                 $serviceManager->get('file_locator'),
                 $appCacheDir
@@ -85,28 +85,28 @@ 
         /*
          * Templating Name Parser.
          */
-        $serviceManager->setFactory('templating.name_parser', function ($serviceManager) {
+        $serviceManager->setFactory('templating.name_parser', function($serviceManager) {
             return new TemplateNameParser($serviceManager->get('modulemanager'));
         });
 
         /*
          * Filesystem Loader.
          */
-        $serviceManager->setFactory('templating.loader.filesystem', function ($serviceManager) {
+        $serviceManager->setFactory('templating.loader.filesystem', function($serviceManager) {
             return new FileSystemLoader($serviceManager->get('templating.locator'));
         });
 
         /*
          * Templating assets helper.
          */
-        $serviceManager->setFactory('templating.helper.assets', function ($serviceManager) {
+        $serviceManager->setFactory('templating.helper.assets', function($serviceManager) {
             return new AssetsHelper($serviceManager->get('request')->getBasePath());
         });
 
         /*
          * Templating globals.
          */
-        $serviceManager->setFactory('templating.globals', function ($serviceManager) {
+        $serviceManager->setFactory('templating.globals', function($serviceManager) {
             return new GlobalVariables($serviceManager->get('servicemanager'));
         });
 
@@ -115,7 +115,7 @@ 
          *
          * TODO: Migrate to Symfony\Bundle\FrameworkBundle\Templating\PhpEngine
          */
-        $serviceManager->setFactory('templating.engine.php', function ($serviceManager) use ($appCharset) {
+        $serviceManager->setFactory('templating.engine.php', function($serviceManager) use ($appCharset) {
             $engine = new PhpEngine(
                 $serviceManager->get('templating.name_parser'),
                 $serviceManager->get('templating.loader'),
@@ -136,7 +136,7 @@ 
         /*
          * Twig Engine
          */
-        $serviceManager->setFactory('templating.engine.twig', function ($serviceManager) {
+        $serviceManager->setFactory('templating.engine.twig', function($serviceManager) {
 
             if (!class_exists('Twig_Environment')) {
                 throw new \Exception('PPI\Framework\TwigModule not found. Composer require: ppi/twig-module');
@@ -159,7 +159,7 @@ 
         /*
          * Smarty Engine.
          */
-        $serviceManager->setFactory('templating.engine.smarty', function ($serviceManager) use ($appCacheDir) {
+        $serviceManager->setFactory('templating.engine.smarty', function($serviceManager) use ($appCacheDir) {
 
             if (!class_exists('NoiseLabs\Bundle\SmartyBundle\SmartyEngine')) {
                 throw new \Exception('PPI\Framework\SmartyModule not found. Composer require: ppi/smarty-module');
@@ -188,7 +188,7 @@ 
         });
 
         // Mustache Engine
-        $serviceManager->setFactory('templating.engine.mustache', function ($serviceManager, $appCacheDir) {
+        $serviceManager->setFactory('templating.engine.mustache', function($serviceManager, $appCacheDir) {
 
             if (!class_exists('Mustache_Engine')) {
                 throw new \Exception('PPI\Framework\MustacheModule not found. Composer require: ppi/mustache-module');
@@ -206,7 +206,7 @@ 
         /*
          * Delegating Engine.
          */
-        $serviceManager->setFactory('templating.engine.delegating', function ($serviceManager) use ($engineIds) {
+        $serviceManager->setFactory('templating.engine.delegating', function($serviceManager) use ($engineIds) {
             $delegatingEngine = new DelegatingEngine();
             // @todo - lazy load this
             foreach ($engineIds as $id) {