Completed
Push — master ( 8f6f9d...d708fb )
by Sinnarasa
02:41
created
src/Routing/Router.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -46,13 +46,13 @@  discard block
 block discarded – undo
46 46
      * @param ResponseInterface $response
47 47
      * @param Route $route
48 48
      */
49
-    public function __construct(RouteCollection $collection,ResponseInterface $response = null,Route $route = null)
49
+    public function __construct(RouteCollection $collection, ResponseInterface $response = null, Route $route = null)
50 50
     {
51 51
         $this->collection = $collection;
52
-        $this->response = is_null($response)? new Response() : $response;
52
+        $this->response = is_null($response) ? new Response() : $response;
53 53
         $this->response->setStatusCode(404);
54
-        $this->route = is_null($route)? new Route() : $route;
55
-        $this->config['di'] = function($class){
54
+        $this->route = is_null($route) ? new Route() : $route;
55
+        $this->config['di'] = function($class) {
56 56
             return new $class;
57 57
         };
58 58
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     /**
77 77
      * @param string $matcher
78 78
      */
79
-    public function addMatcher($matcher){
79
+    public function addMatcher($matcher) {
80 80
         $this->config['matcher'][] = $matcher;
81 81
     }
82 82
 
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
     public function callTarget()
124 124
     {
125 125
         $target = $this->route->getTarget('dispatcher');
126
-        if(!empty($target)) {
126
+        if (!empty($target)) {
127 127
             $this->dispatcher = new $target($this->route, $this->response);
128 128
             return call_user_func([$this->dispatcher, 'call']);
129 129
         }
@@ -157,12 +157,12 @@  discard block
 block discarded – undo
157 157
     {
158 158
         if (isset($this->route->getDetail()['response_templates']) && isset($this->route->getDetail()['response_templates'][$code = $this->response->getStatusCode()])) {
159 159
             $this->route->setCallback($this->route->getDetail()['response_templates'][$code]);
160
-            if(!isset($this->config['matcherInstance']))
160
+            if (!isset($this->config['matcherInstance']))
161 161
                 foreach ($this->config['matcher'] as $matcher)
162 162
                     $this->config['matcherInstance'][$matcher] = new $matcher($this);
163
-            foreach($this->config['matcherInstance'] as $instance) {
163
+            foreach ($this->config['matcherInstance'] as $instance) {
164 164
                 foreach (call_user_func([$instance, 'getMatcher']) as $match)
165
-                    if (call_user_func_array([$instance, $match],[$this->route->getCallback()])){ $this->callTarget(); break; }
165
+                    if (call_user_func_array([$instance, $match], [$this->route->getCallback()])) { $this->callTarget(); break; }
166 166
             }
167 167
             $this->response->setStatusCode($code);
168 168
         }
Please login to merge, or discard this patch.
src/Routing/RouteCollection.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -42,11 +42,11 @@  discard block
 block discarded – undo
42 42
      */
43 43
     public function addRoutes($routes = null, $options = [])
44 44
     {
45
-        if(!is_null($routes) && !is_array($routes)) {
45
+        if (!is_null($routes) && !is_array($routes)) {
46 46
             if (strpos($routes, '.php') === false) $routes = trim($routes, '/') . '/';
47 47
             if (is_file($routes . '/routes.php') && is_array($routesFile = include $routes . '/routes.php')) $routes = $routesFile;
48 48
             elseif (is_file($routes) && is_array($routesFile = include $routes)) $routes = $routesFile;
49
-            else throw new \InvalidArgumentException('Argument for "' . get_called_class() . '" constructor is not recognized. Expected argument array or file containing array but "'.$routes.'" given');
49
+            else throw new \InvalidArgumentException('Argument for "' . get_called_class() . '" constructor is not recognized. Expected argument array or file containing array but "' . $routes . '" given');
50 50
         }
51 51
         $this->routes['routes_' . $this->countRoutes] = is_array($routes) ? $routes : [];
52 52
         $this->routes['view_dir_' . $this->countRoutes] = (isset($options['view_dir']) && !empty($options['view_dir'])) ? rtrim($options['view_dir'], '/') . '/' : '';
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
      */
63 63
     public function getRoutes($key = null)
64 64
     {
65
-        if(!is_null($key))
66
-            return isset($this->routes[$key])?$this->routes[$key]:'';
65
+        if (!is_null($key))
66
+            return isset($this->routes[$key]) ? $this->routes[$key] : '';
67 67
         return $this->routes;
68 68
     }
69 69
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         } elseif (is_string($args))
80 80
             for ($i = 0; $i < $this->countRoutes; ++$i)
81 81
                 $this->routes['prefix_' . $i] = '/' . trim($args, '/');
82
-        if($this->countRoutes == 0)$this->countRoutes++;
82
+        if ($this->countRoutes == 0)$this->countRoutes++;
83 83
     }
84 84
 
85 85
     /**
@@ -89,15 +89,15 @@  discard block
 block discarded – undo
89 89
     {
90 90
         $nbrArgs = count($args);
91 91
         for ($i = 0; $i < $nbrArgs; ++$i) {
92
-            if(is_array($args[$i])){
92
+            if (is_array($args[$i])) {
93 93
                 $this->routes['block_' . $i] = (isset($args[$i]['block']) && !empty($args[$i]['block'])) ? rtrim($args[$i]['block'], '/') . '/' : '';
94 94
                 $this->routes['view_dir_' . $i] = (isset($args[$i]['view_dir']) && !empty($args[$i]['view_dir'])) ? rtrim($args[$i]['view_dir'], '/') . '/' : '';
95 95
                 $this->routes['ctrl_namespace_' . $i] = (isset($args[$i]['ctrl_namespace']) && !empty($args[$i]['ctrl_namespace'])) ? trim($args[$i]['ctrl_namespace'], '\\') . '\\' : '';
96
-                $this->routes['prefix_' . $i] = (isset($args[$i]['prefix']) && !empty($args[$i]['prefix'])) ? '/'.trim($args[$i]['prefix'], '/') : '';
97
-                if(!isset($this->routes['routes_' . $i]))$this->routes['routes_' . $i] = [];
96
+                $this->routes['prefix_' . $i] = (isset($args[$i]['prefix']) && !empty($args[$i]['prefix'])) ? '/' . trim($args[$i]['prefix'], '/') : '';
97
+                if (!isset($this->routes['routes_' . $i]))$this->routes['routes_' . $i] = [];
98 98
             }
99 99
         }
100
-        if($this->countRoutes == 0)$this->countRoutes++;
100
+        if ($this->countRoutes == 0)$this->countRoutes++;
101 101
     }
102 102
 
103 103
     /**
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
      */
120 120
     public function generateRoutesPath()
121 121
     {
122
-        $root = 'http://' . (isset($_SERVER['HTTP_HOST'])?$_SERVER['HTTP_HOST']:$_SERVER['SERVER_NAME']) . str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']);
122
+        $root = 'http://' . (isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : $_SERVER['SERVER_NAME']) . str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']);
123 123
         $count = 0;
124 124
         for ($i = 0; $i < $this->countRoutes; ++$i) {
125 125
             $prefix = (isset($this->routes['prefix_' . $i])) ? $this->routes['prefix_' . $i] : '';
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
                 foreach ($this->routes['routes_' . $i] as $route => $dependencies) {
128 128
                     if (is_array($dependencies) && isset($dependencies['use']))
129 129
                         $use = (is_callable($dependencies['use'])) ? 'closure-' . $count : trim($dependencies['use'], '/');
130
-                    elseif(!is_array($dependencies))
130
+                    elseif (!is_array($dependencies))
131 131
                         $use = (is_callable($dependencies)) ? 'closure-' . $count : trim($dependencies, '/');
132 132
                     else
133 133
                         $use = $route;
Please login to merge, or discard this patch.