comodojo / dispatcher.framework

src/Comodojo/Dispatcher/Router/Route.php

Indentation +2 added lines, -2 removed lines

@@ -80,8 +80,8 @@
     protected $error_message;
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $parameters = [];
 
     /**

Spacing +4 added lines, -4 removed lines

@@ -357,13 +357,13 @@ 
     public function path($path) {
 
         // Because of the nature of the global regular expression, all the bits of the matched route are associated with a parameter key
-        foreach ($this->query as $key => $value) {
+        foreach ( $this->query as $key => $value ) {
 
             if ( isset($path[$key]) ) {
                 /* if it's available a bit associated with the parameter name, it is compared against
                  * it's regular expression in order to extrect backreferences
                  */
-                if ( preg_match('/^' . $value['regex'] . '$/', $path[$key], $matches) ) {
+                if ( preg_match('/^'.$value['regex'].'$/', $path[$key], $matches) ) {
 
                     if ( count($matches) == 1 ) $matches = $matches[0]; // This is the case where no backreferences are present or available.
 
@@ -372,7 +372,7 @@ 
 
                 }
 
-            } elseif ($value['required']) {
+            } elseif ( $value['required'] ) {
 
                 throw new DispatcherException(sprintf("Required parameter '%s' not specified.", $key), 1, null, 500);
 
@@ -391,7 +391,7 @@ 
      */
     public function serialize() {
 
-        return serialize( (object) [
+        return serialize((object)[
             'classname' => $this->classname,
             'type' => $this->type,
             'service' => $this->service,

src/Comodojo/Dispatcher/Router/Model.php

Spacing +12 added lines, -12 removed lines

@@ -107,11 +107,11 @@ 
 
     public function route(Request $request) {
 
-        $method = (string) $request->getMethod();
+        $method = (string)$request->getMethod();
 
         $methods = $this->configuration->get('allowed-http-methods');
 
-        if (!empty($methods) && in_array($method, $methods) === false) {
+        if ( !empty($methods) && in_array($method, $methods) === false ) {
 
             throw new DispatcherException("Method not allowed", 0, null, 405, array(
                 "Allow" => implode(",", $methods)
@@ -121,9 +121,9 @@ 
 
         $this->setRequest($request);
 
-        if ($this->bypass_routing === false) {
+        if ( $this->bypass_routing === false ) {
 
-            if (!$this->parse()) throw new DispatcherException("Unable to find a valid route for the specified uri", 0, null, 404);
+            if ( !$this->parse() ) throw new DispatcherException("Unable to find a valid route for the specified uri", 0, null, 404);
 
         }
 
@@ -135,10 +135,10 @@ 
 
         $class = $this->route->getClassName();
 
-        if (class_exists($class)) {
+        if ( class_exists($class) ) {
 
             // All the route parameters are also added to the query parameters
-            foreach ($this->route->getRequestParameters() as $parameter => $value) {
+            foreach ( $this->route->getRequestParameters() as $parameter => $value ) {
                 $this->getRequest()->getQuery()->set($parameter, $value);
             }
 
@@ -163,13 +163,13 @@ 
 
         $this->setResponse($response);
 
-        if (is_null($this->route)) {
+        if ( is_null($this->route) ) {
 
             throw new DispatcherException("Route has not been loaded!");
 
         }
 
-        if ($this->bypass_service) {
+        if ( $this->bypass_service ) {
 
             return;
 
@@ -177,7 +177,7 @@ 
 
         $service = $this->getServiceInstance();
 
-        if (!is_null($service)) {
+        if ( !is_null($service) ) {
 
             $result = "";
 
@@ -185,7 +185,7 @@ 
 
             $methods = $service->getImplementedMethods();
 
-            if (in_array($method, $methods)) {
+            if ( in_array($method, $methods) ) {
 
                 $callable = $service->getMethod($method);
 
@@ -225,10 +225,10 @@ 
 
         $path = urldecode($this->getRequest()->route());
 
-        foreach ($this->table->getRoutes() as $regex => $value) {
+        foreach ( $this->table->getRoutes() as $regex => $value ) {
 
             // The current uri is checked against all the global regular expressions associated with the routes
-            if (preg_match("/".$regex."/", $path, $matches)) {
+            if ( preg_match("/".$regex."/", $path, $matches) ) {
 
                 /* If a route is matched, all the bits of the route string are evalued in order to create
                  * new query parameters which will be available for the service class

src/Comodojo/Dispatcher/Router/Table.php

Spacing +9 added lines, -9 removed lines

@@ -41,7 +41,7 @@ 
      * Types of route that this table will accept
      * @var array
      */
-    const ALLOWED_ROUTES = [ "ROUTE", "REDIRECT", "ERROR" ];
+    const ALLOWED_ROUTES = ["ROUTE", "REDIRECT", "ERROR"];
 
     /**
      * Current repository of routes
@@ -141,7 +141,7 @@ 
 
         $regex = $this->regex($route);
 
-        if (isset($this->routes[$regex])) {
+        if ( isset($this->routes[$regex]) ) {
             return $this->routes[$regex];
         }
 
@@ -163,7 +163,7 @@ 
 
         $routes = $this->routes;
 
-        if (isset($routes[$regex])) {
+        if ( isset($routes[$regex]) ) {
 
             unset($routes[$regex]);
 
@@ -185,9 +185,9 @@ 
 
     public function load(array $routes) {
 
-        if (!empty($routes)) {
+        if ( !empty($routes) ) {
 
-            foreach ($routes as $name => $route) {
+            foreach ( $routes as $name => $route ) {
 
                 $this->add(
                     $route['route'],
@@ -249,11 +249,11 @@ 
 
     private function readCache() {
 
-        if ($this->configuration->get('routing-table-cache') !== true) return;
+        if ( $this->configuration->get('routing-table-cache') !== true ) return;
 
         $data = $this->cache->read();
 
-        if (is_null($data)) {
+        if ( is_null($data) ) {
 
             $this->routes = [];
 
@@ -269,11 +269,11 @@ 
 
     private function dumpCache() {
 
-        if ($this->configuration->get('routing-table-cache') !== true) return;
+        if ( $this->configuration->get('routing-table-cache') !== true ) return;
 
         $ttl = $this->configuration->get('routing-table-ttl');
 
-        if ($this->cache->dump($this->routes, $ttl)) {
+        if ( $this->cache->dump($this->routes, $ttl) ) {
             $this->logger->debug("Routing table saved to cache");
         } else {
             $this->logger->warning("Cannot save routing table to cache");

src/Comodojo/Dispatcher/Router/Parser.php

Spacing +7 added lines, -7 removed lines

@@ -41,14 +41,14 @@ 
     // the global regular expression against which all the request URI will be compared
     public function read($folders = array(), Route $value = null, $regex = '') {
 
-        if (is_null($value)) {
+        if ( is_null($value) ) {
 
             $value = new Route();
 
         }
 
         // if the first 'folder' is empty is removed
-        while (!empty($folders) && empty($folders[0])) {
+        while ( !empty($folders) && empty($folders[0]) ) {
 
             array_shift($folders);
 
@@ -56,7 +56,7 @@ 
 
         // if the 'folder' array is empty, the route has been fully analyzed
         // this is the exit condition from the recursive loop.
-        if (empty($folders)) {
+        if ( empty($folders) ) {
 
             return '^'.$regex.'[\/]?$';
 
@@ -68,7 +68,7 @@ 
             // All the parameters of the route must be json strings
             $decoded = json_decode($folder, true);
 
-            if (!is_null($decoded) && is_array($decoded)) {
+            if ( !is_null($decoded) && is_array($decoded) ) {
 
                 $param_regex = '';
 
@@ -84,7 +84,7 @@ 
                  *
                  * This is the reason of the following 'foreach'
                  */
-                foreach ($decoded as $key => $string) {
+                foreach ( $decoded as $key => $string ) {
 
                     $this->logger->debug("Route parser - parameter key: $key");
 
@@ -96,7 +96,7 @@ 
                      */
                     $param_regex .= $this->param($key, $string, $value);
 
-                    if ($value->isQueryRequired($key)) {
+                    if ( $value->isQueryRequired($key) ) {
                         $param_required = true;
                     }
 
@@ -132,7 +132,7 @@ 
         $field_required = false;
 
         // If the field name ends with a '*', the parameter is considered as required
-        if (preg_match('/^(.+)\*$/', $key, $bits)) {
+        if ( preg_match('/^(.+)\*$/', $key, $bits) ) {
 
             $key = $bits[1];
             $field_required = true;

src/Comodojo/Dispatcher/Response/Content.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 
     public function set($content = null) {
 
-        if (!is_scalar($content) && $content != null) {
+        if ( !is_scalar($content) && $content != null ) {
 
             throw new Exception("Invalid HTTP content");
 

src/Comodojo/Dispatcher/Response/Status.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
 
     public function set($code) {
 
-        if (!$this->codes->exists($code)) {
+        if ( !$this->codes->exists($code) ) {
 
             throw new Exception("Invalid HTTP Status Code $code");
 
@@ -60,7 +60,7 @@ 
 
     }
 
-    public function description($code=null) {
+    public function description($code = null) {
 
         if ( is_null($code) ) return $this->codes->getMessage($this->status_code);
 

src/Comodojo/Dispatcher/Service/AbstractService.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     use ResponseTrait;
     use ExtraTrait;
 
-    protected static $supported_methods = ['GET','PUT','POST','DELETE','OPTIONS','HEAD','TRACE','CONNECT','PURGE'];
+    protected static $supported_methods = ['GET', 'PUT', 'POST', 'DELETE', 'OPTIONS', 'HEAD', 'TRACE', 'CONNECT', 'PURGE'];
 
     public function __construct(
         Configuration $configuration,

src/Comodojo/Dispatcher/Request/File.php

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
 
         $file = $this->getTemporaryName();
 
-        if (file_exists($file)) {
+        if ( file_exists($file) ) {
             return file_get_contents($file);
         }
 
@@ -97,7 +97,7 @@ 
 
     public function load($slugOrControl) {
 
-        if (isset($_FILES) && isset($_FILES[$slugOrControl])) {
+        if ( isset($_FILES) && isset($_FILES[$slugOrControl]) ) {
 
             $this->loadFromUploadedFile($slugOrControl);
 
@@ -111,11 +111,11 @@ 
 
     public function save($path, $as_slug = false) {
 
-        if (!empty($path) && file_exists($path)) {
+        if ( !empty($path) && file_exists($path) ) {
 
             $local_name = "$path/".($as_slug ? $this->getSlug() : $this->getFileName());
 
-            if (file_exists($local_name)) {
+            if ( file_exists($local_name) ) {
 
                 $files = glob("$local_name*");
 
@@ -125,7 +125,7 @@ 
 
             }
 
-            if (move_uploaded_file($this->getTemporaryName(), $local_name)) {
+            if ( move_uploaded_file($this->getTemporaryName(), $local_name) ) {
 
                 // return file_exists($local_name);
                 return true;
@@ -167,7 +167,7 @@ 
 
         $files = array();
 
-        foreach ($_FILES as $idx => $data) {
+        foreach ( $_FILES as $idx => $data ) {
 
             $files[] = new File($idx, $repository);
 

src/Comodojo/Dispatcher/Request/Files.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
     public function __construct() {
 
-        foreach ($_FILES as $idx => $data) {
+        foreach ( $_FILES as $idx => $data ) {
 
             $this->files[] = new File($idx);