ppi / framework

PPI/ServiceManager/Config/RouterConfig.php

Unused Use Statements +6 added lines, -6 removed lines

@@ -8,12 +8,12 @@
  */
 namespace PPI\ServiceManager\Config;
 
-use Zend\ServiceManager\Config,
-    Zend\ServiceManager\ServiceManager,
-    Symfony\Component\Routing\RequestContext,
-    Symfony\Component\Routing\RouteCollection,
-    PPI\Module\Routing\Router,
-    PPI\Module\Routing\RoutingHelper;
+use Zend\ServiceManager\Config;
+use Zend\ServiceManager\ServiceManager;
+use Symfony\Component\Routing\RequestContext;
+use Symfony\Component\Routing\RouteCollection;
+use PPI\Module\Routing\Router;
+use PPI\Module\Routing\RoutingHelper;
 
 /**
  * ServiceManager configuration for the Router component.

PPI/ServiceManager/ServiceManager.php

Doc Comments -1 removed lines

@@ -69,7 +69,6 @@
      *
      * This method is an alias to $this->get().
      *
-     * @param string  $cName
      * @param boolean $usePeeringServiceManagers
      *
      * @return object|array

Unused Use Statements +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
  */
 namespace PPI\ServiceManager;
 
-use PPI\ServiceManager\Options\OptionsInterface,
-    Zend\ServiceManager\ServiceManager as BaseServiceManager;
+use PPI\ServiceManager\Options\OptionsInterface;
+use Zend\ServiceManager\ServiceManager as BaseServiceManager;
 
 /**
  * ServiceManager implements the Service Locator design pattern.

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
             $parameters = parent::matchRequest($request);
             $this->matchedRouteRequest = $request;
             return $parameters;
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             throw $e;
         }
     }

PPI/Templating/Helper/RouterHelper.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
  */
 namespace PPI\Templating\Helper;
 
-use Symfony\Component\Templating\Helper\Helper,
-    Symfony\Component\Routing\Generator\UrlGeneratorInterface;
+use Symfony\Component\Templating\Helper\Helper;
+use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 
 /**
  * RouterHelper manages links between pages in a template context.

PPI/Templating/Helper/SessionHelper.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
  */
 namespace PPI\Templating\Helper;
 
-use Symfony\Component\Templating\Helper\Helper,
-    Symfony\Component\HttpFoundation\Session\SessionInterface;
+use Symfony\Component\Templating\Helper\Helper;
+use Symfony\Component\HttpFoundation\Session\SessionInterface;
 
 /**
  * SessionHelper provides read-only access to the session attributes.

PPI/Templating/Mustache/MustacheEngine.php

Doc Comments +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
     /**
      * Constructor.
      *
-     * @param Mustache_Engine             $mustache A \Mustache_Engine instance
+     * @param \Mustache_Engine             $mustache A \Mustache_Engine instance
      * @param TemplateNameParserInterface $parser   A TemplateNameParserInterface instance
      */
     public function __construct(\Mustache_Engine $mustache, TemplateNameParserInterface $parser)
@@ -40,7 +40,7 @@ 
     /**
      * Renders a template.
      *
-     * @param mixed $name       A template name
+     * @param string $name       A template name
      * @param array $parameters An array of parameters to pass to the template
      *
      * @return string The evaluated template as a string

PPI/Templating/Php/FileSystemLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
      *
      * @param FileLocatorInterface $locator A FileLocatorInterface instance
      *
-     * @return void
+     * @return LoaderInterface
      */
     public function __construct(FileLocatorInterface $locator)
     {
@@ -67,7 +67,7 @@ 
      * @param TemplateReferenceInterface $template The template name as an array
      * @param integer                    $time     The last modification time of the cached template (timestamp)
      *
-     * @return integer|boolean
+     * @return boolean
      */
     public function isFresh(TemplateReferenceInterface $template, $time)
     {

Unused Use Statements +4 added lines, -4 removed lines

@@ -8,10 +8,10 @@
  */
 namespace PPI\Templating\Php;
 
-use Symfony\Component\Templating\Storage\FileStorage,
-    Symfony\Component\Templating\Loader\LoaderInterface,
-    Symfony\Component\Config\FileLocatorInterface,
-    Symfony\Component\Templating\TemplateReferenceInterface;
+use Symfony\Component\Templating\Storage\FileStorage;
+use Symfony\Component\Templating\Loader\LoaderInterface;
+use Symfony\Component\Config\FileLocatorInterface;
+use Symfony\Component\Templating\TemplateReferenceInterface;
 
 /**
  * FilesystemLoader is a loader that read templates from the filesystem.

PPI/Templating/Smarty/Extension/AssetsExtension.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
      *
      * @param AssetsHelper $assetsHelper
      *
-     * @return void
+     * @return \NoiseLabs\Bundle\SmartyBundle\Extension\ExtensionInterface
      */
     public function __construct(AssetsHelper $assetsHelper)
     {
@@ -130,7 +130,7 @@ 
      * @param array                     $parameters
      * @param \Smarty_Internal_Template $template
      *
-     * @return integer
+     * @return string
      */
     public function getAssetsVersion(array $parameters = array(), \Smarty_Internal_Template $template)
     {

Unused Use Statements +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
  */
 namespace PPI\Templating\Smarty\Extension;
 
-use NoiseLabs\Bundle\SmartyBundle\Extension\AssetsExtension as BaseAssetsExtension,
-    Symfony\Component\Templating\Helper\AssetsHelper;
+use NoiseLabs\Bundle\SmartyBundle\Extension\AssetsExtension as BaseAssetsExtension;
+use Symfony\Component\Templating\Helper\AssetsHelper;
 
 /**
  * Provides helper functions to link to assets (images, Javascript,

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
      * @return mixed
      */
     public function __call($method, array $params = array()) {
-        if(isset($this->funRemap[$method])) {
+        if (isset($this->funRemap[$method])) {
             return call_user_func_array(array($this, $this->funRemap[$method]), $params);
         }
         throw new \BadMethodCallException('Method ' . $method . ' does not exist');

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?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\Exception;
 
 /**

PPI/Templating/Smarty/SmartyEngine.php

Unused Use Statements +5 added lines, -5 removed lines

@@ -8,11 +8,11 @@
  */
 namespace PPI\Templating\Smarty;
 
-use PPI\Templating\GlobalVariables,
-    PPI\Templating\TemplateLocator,
-    NoiseLabs\Bundle\SmartyBundle\SmartyEngine as BaseSmartyEngine,
-    Symfony\Component\Templating\Loader\LoaderInterface,
-    Symfony\Component\Templating\TemplateNameParserInterface;
+use PPI\Templating\GlobalVariables;
+use PPI\Templating\TemplateLocator;
+use NoiseLabs\Bundle\SmartyBundle\SmartyEngine as BaseSmartyEngine;
+use Symfony\Component\Templating\Loader\LoaderInterface;
+use Symfony\Component\Templating\TemplateNameParserInterface;
 
 /**
  * SmartyEngine is an engine able to render Smarty templates.

Indentation +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
          * of your custom error handler, please call muteExpectedErrors() after
          * you've registered your custom error handler.
          *
-        * muteExpectedErrors() registers a custom error handler using
+         * muteExpectedErrors() registers a custom error handler using
          * set_error_handler(). The error handler merely inspects $errno and
          * $errfile to determine if the given error was produced deliberately
          * and must be ignored, or should be passed on to the next error handler.

PPI/Templating/TemplateLocator.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
  */
 namespace PPI\Templating;
 
-use PPI\Templating\TemplateReference,
-    Symfony\Component\Config\FileLocatorInterface,
-    Symfony\Component\Templating\TemplateReferenceInterface;
+use PPI\Templating\TemplateReference;
+use Symfony\Component\Config\FileLocatorInterface;
+use Symfony\Component\Templating\TemplateReferenceInterface;
 
 /**
  * The PPI Template Locator.

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function __construct(FileLocatorInterface $locator, $cacheDir = null)
     {
-        if (null !== $cacheDir && is_file($cache = $cacheDir.'/templates.php')) {
+        if (null !== $cacheDir && is_file($cache = $cacheDir . '/templates.php')) {
             $this->cache = require $cache;
         }
 
@@ -103,7 +103,7 @@ 
      */
     public function getAppPath()
     {
-        return $this->locator->appPath.DIRECTORY_SEPARATOR.TemplateReference::APP_VIEWS_DIRECTORY;
+        return $this->locator->appPath . DIRECTORY_SEPARATOR . TemplateReference::APP_VIEWS_DIRECTORY;
     }
 
     /**
@@ -116,7 +116,7 @@ 
         $paths = $this->locator->getModulesPath();
 
         foreach (array_keys($paths) as $module) {
-            $paths[$module] .= DIRECTORY_SEPARATOR.TemplateReference::MODULE_VIEWS_DIRECTORY;
+            $paths[$module] .= DIRECTORY_SEPARATOR . TemplateReference::MODULE_VIEWS_DIRECTORY;
         }
 
         return $paths;