Completed
Push — master ( 953765...1e20d8 )
by Sinnarasa
02:50
created
src/Routing/Route.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -207,8 +207,8 @@
 block discarded – undo
207 207
     /**
208 208
      * @return array
209 209
      */
210
-    public function getData(){
211
-        return (isset($this->getDetail()['data']) && is_array($this->getDetail()['data']))?$this->getDetail()['data']:[];
210
+    public function getData() {
211
+        return (isset($this->getDetail()['data']) && is_array($this->getDetail()['data'])) ? $this->getDetail()['data'] : [];
212 212
     }
213 213
 
214 214
     /**
Please login to merge, or discard this patch.
src/Routing/RouteCollection.php 1 patch
Spacing   +9 added lines, -9 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['path_' . $this->countRoutes] = (isset($options['path']) && !empty($options['path'])) ? rtrim($options['path'], '/') . '/' : '';
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
      */
62 62
     public function getRoutes($key = null)
63 63
     {
64
-        if(!is_null($key))
65
-            return isset($this->routes[$key])?$this->routes[$key]:'';
64
+        if (!is_null($key))
65
+            return isset($this->routes[$key]) ? $this->routes[$key] : '';
66 66
         return $this->routes;
67 67
     }
68 68
 
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         } elseif (is_string($args))
79 79
             for ($i = 0; $i < $this->countRoutes; ++$i)
80 80
                 $this->routes['prefix_' . $i] = '/' . trim($args, '/');
81
-        if($this->countRoutes == 0)$this->countRoutes++;
81
+        if ($this->countRoutes == 0)$this->countRoutes++;
82 82
     }
83 83
 
84 84
     /**
@@ -88,14 +88,14 @@  discard block
 block discarded – undo
88 88
     {
89 89
         $nbrArgs = count($args);
90 90
         for ($i = 0; $i < $nbrArgs; ++$i) {
91
-            if(is_array($args[$i])){
91
+            if (is_array($args[$i])) {
92 92
                 $this->routes['path_' . $i] = (isset($args[$i]['path']) && !empty($args[$i]['path'])) ? rtrim($args[$i]['path'], '/') . '/' : '';
93 93
                 $this->routes['namespace_' . $i] = (isset($args[$i]['namespace']) && !empty($args[$i]['namespace'])) ? trim($args[$i]['namespace'], '\\') . '\\' : '';
94
-                $this->routes['prefix_' . $i] = (isset($args[$i]['prefix']) && !empty($args[$i]['prefix'])) ? '/'.trim($args[$i]['prefix'], '/') : '';
95
-                if(!isset($this->routes['routes_' . $i]))$this->routes['routes_' . $i] = [];
94
+                $this->routes['prefix_' . $i] = (isset($args[$i]['prefix']) && !empty($args[$i]['prefix'])) ? '/' . trim($args[$i]['prefix'], '/') : '';
95
+                if (!isset($this->routes['routes_' . $i]))$this->routes['routes_' . $i] = [];
96 96
             }
97 97
         }
98
-        if($this->countRoutes == 0)$this->countRoutes++;
98
+        if ($this->countRoutes == 0)$this->countRoutes++;
99 99
     }
100 100
 
101 101
     /**
Please login to merge, or discard this patch.
src/Routing/Dispatcher/ControllerDispatcher.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         foreach ($reflectionMethod->getParameters() as $arg) {
38 38
             if (!is_null($arg->getClass())) {
39 39
                 $class = $arg->getClass()->name;
40
-                array_unshift($dependencies, call_user_func_array($this->route->getTarget('di'),[$class]));
40
+                array_unshift($dependencies, call_user_func_array($this->route->getTarget('di'), [$class]));
41 41
             }
42 42
         }
43 43
         if ($this->route->getResponse('code') == 202)
@@ -58,13 +58,13 @@  discard block
 block discarded – undo
58 58
         $constructor = $reflector->getConstructor();
59 59
         if (is_null($constructor)) {
60 60
             $class = $this->route->getTarget('controller');
61
-            return call_user_func_array($this->route->getTarget('di'),[$class]);
61
+            return call_user_func_array($this->route->getTarget('di'), [$class]);
62 62
         }
63 63
         $dependencies = $constructor->getParameters();
64 64
         $arguments = [];
65 65
         foreach ($dependencies as $dep) {
66 66
             $class = $dep->getClass()->name;
67
-            array_push($arguments, call_user_func_array($this->route->getTarget('di'),[$class]));
67
+            array_push($arguments, call_user_func_array($this->route->getTarget('di'), [$class]));
68 68
         }
69 69
         return $reflector->newInstanceArgs($arguments);
70 70
     }
Please login to merge, or discard this patch.
src/Routing/Match/RoutesMatch.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
     /**
25 25
      * @var array
26 26
      */
27
-    private $matcher = ['matchClosure','matchController','matchTemplate'];
27
+    private $matcher = ['matchClosure', 'matchController', 'matchTemplate'];
28 28
 
29 29
     /**
30 30
      * @param Router $router
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     /**
38 38
      * @param string $matcher
39 39
      */
40
-    public function addMatcher($matcher){
40
+    public function addMatcher($matcher) {
41 41
         $this->matcher[] = $matcher;
42 42
     }
43 43
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
      */
107 107
     private function generateTarget()
108 108
     {
109
-        if(is_callable($this->request['path'])){
109
+        if (is_callable($this->request['path'])) {
110 110
             $this->router->route->setCallback($this->request['path']);
111 111
             $this->router->route->setDetail($this->request);
112 112
             $this->matchClosure();
@@ -119,11 +119,11 @@  discard block
 block discarded – undo
119 119
                     ? $this->router->route->setCallback($this->request['path']['use'])
120 120
                     : $this->router->route->setCallback($this->request['path']);
121 121
             $this->router->route->setDetail($this->request);
122
-            if($this->validMethod()) {
123
-                foreach($this->matcher as $matcher)
124
-                    call_user_func([$this,$matcher]);
122
+            if ($this->validMethod()) {
123
+                foreach ($this->matcher as $matcher)
124
+                    call_user_func([$this, $matcher]);
125 125
                 $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
126
-            }else
126
+            } else
127 127
                 $this->router->route->setResponse(['code' => 405, 'message' => 'Method Not Allowed']);
128 128
         }
129 129
         return $this->router->route->hasTarget();
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
166 166
             $class = (class_exists($routes[0]))
167 167
                 ? $routes[0]
168
-                : $this->router->collection->getRoutes()['namespace_'.$index].$routes[0];
168
+                : $this->router->collection->getRoutes()['namespace_' . $index] . $routes[0];
169 169
             if (!class_exists($class))
170 170
                 throw new \Exception('Class "' . $class . '." is not found');
171 171
             if (method_exists($class, $routes[1])) {
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
             $extension = explode('.', $path);
194 194
             $extension = end($extension);
195 195
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
196
-            $block = $this->router->collection->getRoutes('path_'.$index);
196
+            $block = $this->router->collection->getRoutes('path_' . $index);
197 197
             if (in_array('.' . $extension, $this->router->getConfig()['viewExtension'])) {
198 198
                 if (is_file($block . $path)) {
199 199
                     $target = $block . $path;
@@ -209,13 +209,13 @@  discard block
 block discarded – undo
209 209
                 throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
210 210
             } else {
211 211
                 foreach ($this->router->getConfig()['viewExtension'] as $ext) {
212
-                    if (is_file($block . $path . $ext)){
212
+                    if (is_file($block . $path . $ext)) {
213 213
                         $target = $block . $path . $ext;
214
-                        $this->router->route->setTarget(['dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher', 'template' => $target,'block' => $block,  'extension' => str_replace('.', '', $ext),'callback' => $this->router->getConfig()['viewCallback']]);
214
+                        $this->router->route->setTarget(['dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher', 'template' => $target, 'block' => $block, 'extension' => str_replace('.', '', $ext), 'callback' => $this->router->getConfig()['viewCallback']]);
215 215
                         return true;
216 216
                     }
217 217
                 }
218
-                throw new \Exception('Template file "' . $path . '" is not found in "' .$block . '"');
218
+                throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
219 219
             }
220 220
         }
221 221
         return false;
Please login to merge, or discard this patch.
src/Routing/Match/SmartMatch.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     /**
21 21
      * @var array
22 22
      */
23
-    private $matcher = ['matchTemplate','matchController'];
23
+    private $matcher = ['matchTemplate', 'matchController'];
24 24
 
25 25
     /**
26 26
      * @param Router $router
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
     /**
34 34
      * @param string $matcher
35 35
      */
36
-    public function addMatcher($matcher){
36
+    public function addMatcher($matcher) {
37 37
         $this->matcher[] = $matcher;
38 38
     }
39 39
 
@@ -50,8 +50,8 @@  discard block
 block discarded – undo
50 50
      */
51 51
     public function match()
52 52
     {
53
-        foreach($this->matcher as $matcher){
54
-            if(call_user_func([$this,$matcher])) {
53
+        foreach ($this->matcher as $matcher) {
54
+            if (call_user_func([$this, $matcher])) {
55 55
                 $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
56 56
                 return true;
57 57
             }
@@ -66,9 +66,9 @@  discard block
 block discarded – undo
66 66
     {
67 67
         foreach ($this->router->getConfig()['viewExtension'] as $extension) {
68 68
             for ($i = 0; $i < $this->router->collection->countRoutes; ++$i) {
69
-                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '',$this->router->route->getUrl()));
69
+                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '', $this->router->route->getUrl()));
70 70
                 $end = array_pop($url);
71
-                $url = implode('/', array_map('ucwords', $url)).'/'.$end;
71
+                $url = implode('/', array_map('ucwords', $url)) . '/' . $end;
72 72
                 if (is_file(($template = rtrim($this->router->collection->getRoutes('path_' . $i), '/') . $url . $extension))) {
73 73
                     $this->router->route->setTarget([
74 74
                         'dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher',
@@ -91,12 +91,12 @@  discard block
 block discarded – undo
91 91
     {
92 92
         $routes = array_slice(explode('/', $this->router->route->getUrl()), 1);
93 93
         $i = 0;
94
-        do{
95
-            $route =  ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i))
94
+        do {
95
+            $route = ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i))
96 96
                 ? array_slice($routes, 1) : $routes;
97 97
             if (isset($route[0])) {
98
-                $class =  (class_exists($this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'))
99
-                    ? $this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'
98
+                $class = (class_exists($this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'))
99
+                    ? $this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'
100 100
                     : ucfirst($route[0]) . 'Controller';
101 101
                 if (isset($route[1]) && method_exists($class, $route[1])) {
102 102
                     $this->router->route->setTarget([
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
                 }
111 111
             }
112 112
             ++$i;
113
-        }while($i < $this->router->collection->countRoutes);
113
+        }while ($i < $this->router->collection->countRoutes);
114 114
         return false;
115 115
     }
116 116
 
Please login to merge, or discard this patch.
src/Routing/Match/CustomMatch.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 namespace JetFire\Routing\Match;
10 10
 
11 11
 
12
-class CustomMatch extends RoutesMatch implements MatcherInterface{
12
+class CustomMatch extends RoutesMatch implements MatcherInterface {
13 13
 
14 14
 
15 15
 } 
16 16
\ No newline at end of file
Please login to merge, or discard this patch.
src/Routing/Router.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     {
45 45
         $this->collection = $collection;
46 46
         $this->route = new Route();
47
-        $this->config['di'] = function($class){
47
+        $this->config['di'] = function($class) {
48 48
             return new $class;
49 49
         };
50 50
     }
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
     /**
69 69
      * @param string $matcher
70 70
      */
71
-    public function addMatcher($matcher){
71
+    public function addMatcher($matcher) {
72 72
         $this->config['matcher'][] = $matcher;
73 73
     }
74 74
 
@@ -146,9 +146,9 @@  discard block
 block discarded – undo
146 146
     {
147 147
         if (isset($this->route->getResponse()['templates']) && isset($this->route->getResponse()['templates'][$this->route->getResponse('code')])) {
148 148
             $this->route->setCallback($this->route->getResponse()['templates'][$this->route->getResponse('code')]);
149
-            foreach($this->config['matcherInstance'] as $matcher) {
149
+            foreach ($this->config['matcherInstance'] as $matcher) {
150 150
                 foreach (call_user_func([$matcher, 'getMatcher']) as $match)
151
-                    if (call_user_func([$matcher, $match])){ $this->callTarget(); break; }
151
+                    if (call_user_func([$matcher, $match])) { $this->callTarget(); break; }
152 152
             }
153 153
         }
154 154
         http_response_code($this->route->getResponse('code'));
Please login to merge, or discard this patch.