ppi / framework

PPI/Console/Command/ModuleCommand.php

Doc Comments +4 added lines

@@ -57,6 +57,10 @@
         }
     }
 
+    /**
+     * @param string $src
+     * @param string $dst
+     */
     protected function copyRecursively($src, $dst)
     {
         $dir = opendir($src);

Indentation +3 added lines, -3 removed lines

@@ -27,9 +27,9 @@
     {
 
         $this->setName('module:create')
-             ->setDescription('Create a module')
-             ->addArgument('name', InputArgument::REQUIRED, 'What is your module name?')
-             ->addOption('dir', null, InputOption::VALUE_NONE, 'Specify the modules directory');
+                ->setDescription('Create a module')
+                ->addArgument('name', InputArgument::REQUIRED, 'What is your module name?')
+                ->addOption('dir', null, InputOption::VALUE_NONE, 'Specify the modules directory');
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

Spacing +5 added lines, -5 removed lines

@@ -61,12 +61,12 @@
     {
         $dir = opendir($src);
         @mkdir($dst);
-        while (false !== ( $file = readdir($dir)) ) {
-            if (( $file != '.' ) && ( $file != '..' )) {
-                if ( is_dir($src . '/' . $file) ) {
-                    $this->copyRecursively($src . '/' . $file,$dst . '/' . $file);
+        while (false !== ($file = readdir($dir))) {
+            if (($file != '.') && ($file != '..')) {
+                if (is_dir($src . '/' . $file)) {
+                    $this->copyRecursively($src . '/' . $file, $dst . '/' . $file);
                 } else {
-                    copy($src . '/' . $file,$dst . '/' . $file);
+                    copy($src . '/' . $file, $dst . '/' . $file);
                 }
             }
         }

PPI/DataSource/DataSourceInterface.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
     /**
      * Manufacturs datasource drivers.
      *
-     * @param string $key
      *
      * @return object
      *

PPI/DataSource/PDO.php

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

@@ -8,8 +8,8 @@
  */
 namespace PPI\DataSource;
 
-use Doctrine\DBAL\DriverManager,
-    Doctrine\DBAL\Configuration;
+use Doctrine\DBAL\DriverManager;
+use Doctrine\DBAL\Configuration;
 
 /**
  * PDO class

PPI/Exception/Log.php

Doc Comments +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
      *
      * @param array $options
      *
-     * @return void
+     * @return HandlerInterface
      */
     public function __construct(array $options = array())
     {
@@ -63,7 +63,7 @@ 
     /**
      * Get the log file path
      *
-     * @return mixed
+     * @return string
      */
     private function getLogFile()
     {

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
                 $e->getLine()
             );
 
-            if (file_put_contents($logFile, $logEntry , FILE_APPEND|LOCK_EX) > 0) {
+            if (file_put_contents($logFile, $logEntry, FILE_APPEND | LOCK_EX) > 0) {
                 return array(
                     'status'  => true,
                     'message' => 'Written to log file ' . $logFile

PPI/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 +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/Module/Listener/DefaultListenerAggregate.php

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

@@ -8,15 +8,15 @@
  */
 namespace PPI\Module\Listener;
 
-use PPI\ServiceManager\ServiceManager,
-    Zend\EventManager\EventManagerInterface,
-    Zend\Loader\ModuleAutoloader,
-    Zend\ModuleManager\Listener\AutoloaderListener,
-    Zend\ModuleManager\Listener\DefaultListenerAggregate as ZendDefaultListenerAggregate,
-    Zend\ModuleManager\Listener\InitTrigger,
-    Zend\ModuleManager\Listener\ModuleResolverListener,
-    Zend\ModuleManager\ModuleEvent,
-    Zend\Stdlib\ArrayUtils;
+use PPI\ServiceManager\ServiceManager;
+use Zend\EventManager\EventManagerInterface;
+use Zend\Loader\ModuleAutoloader;
+use Zend\ModuleManager\Listener\AutoloaderListener;
+use Zend\ModuleManager\Listener\DefaultListenerAggregate as ZendDefaultListenerAggregate;
+use Zend\ModuleManager\Listener\InitTrigger;
+use Zend\ModuleManager\Listener\ModuleResolverListener;
+use Zend\ModuleManager\ModuleEvent;
+use Zend\Stdlib\ArrayUtils;
 
 /**
  * DefaultListenerAggregate class

PPI/Module/Module.php

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

@@ -8,9 +8,9 @@
  */
 namespace PPI\Module;
 
-use PPI\Module\Routing\Loader\YamlFileLoader,
-    Symfony\Component\Config\FileLocator,
-    Symfony\Component\Yaml\Yaml as YamlParser;
+use PPI\Module\Routing\Loader\YamlFileLoader;
+use Symfony\Component\Config\FileLocator;
+use Symfony\Component\Yaml\Yaml as YamlParser;
 
 /**
  * The base PPI module class..

Spacing +1 added lines, -1 removed lines

@@ -224,7 +224,7 @@
     {
         if (!method_exists($this->_controller, $this->_actionName)) {
             throw new \Exception(sprintf(
-                'Unable to dispatch action: "%s" does not exist in controller '.
+                'Unable to dispatch action: "%s" does not exist in controller ' .
                 '"%s" within module "%s"',
                 $this->_actionName,
                 $this->_controllerName,

PPI/Module/Routing/Loader/YamlFileLoader.php

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

@@ -8,8 +8,8 @@
  */
 namespace PPI\Module\Routing\Loader;
 
-use Symfony\Component\Routing\Loader\YamlFileLoader as BaseYamlFileLoader,
-    Symfony\Component\Routing\RouteCollection;
+use Symfony\Component\Routing\Loader\YamlFileLoader as BaseYamlFileLoader;
+use Symfony\Component\Routing\RouteCollection;
 
 /**
  * YamlFileLoader class

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/Module/Routing/Router.php

Doc Comments +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
      * Constructor.
      *
      * @param RequestContext $requestContext
-     * @param type           $collection
+     * @param \Symfony\Component\Routing\RouteCollection           $collection
      * @param array          $options
      *
      * @return void
@@ -41,7 +41,7 @@ 
     /**
      * Set the route collection
      *
-     * @param type $collection
+     * @param \Symfony\Component\Routing\RouteCollection $collection
      *
      * @return void
      */

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

@@ -8,8 +8,8 @@
  */
 namespace PPI\Module\Routing;
 
-use Symfony\Component\Routing\Router as BaseRouter,
-    Symfony\Component\Routing\RequestContext;
+use Symfony\Component\Routing\Router as BaseRouter;
+use Symfony\Component\Routing\RequestContext;
 
 /**
  * The PPI router

Spacing +6 added lines, -6 removed lines

@@ -58,9 +58,9 @@ 
     public function isMatcherCached()
     {
         return file_exists(
-            $this->options['cache_dir'].
-            DIRECTORY_SEPARATOR.
-            $this->options['matcher_cache_class'].
+            $this->options['cache_dir'] .
+            DIRECTORY_SEPARATOR .
+            $this->options['matcher_cache_class'] .
             '.php'
         );
     }
@@ -73,9 +73,9 @@ 
     public function isGeneratorCached()
     {
         return file_exists(
-            $this->options['cache_dir'].
-            DIRECTORY_SEPARATOR.
-            $this->options['generator_cache_class'].
+            $this->options['cache_dir'] .
+            DIRECTORY_SEPARATOR .
+            $this->options['generator_cache_class'] .
             '.php'
         );
     }