Completed
Push — master ( c2c9db...effdc3 )
by Sinnarasa
04:18 queued 13s
created
src/Routing/RouteCollection.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -47,12 +47,12 @@  discard block
 block discarded – undo
47 47
     public function addRoutes($routes = null, $options = [])
48 48
     {
49 49
         if (!is_null($routes) && !is_array($routes)) {
50
-            if (strpos($routes, '.php') === false) $routes = trim($routes, '/') . '/';
51
-            if (is_file($routes . '/routes.php') && is_array($routesFile = include $routes . '/routes.php')) $routes = $routesFile;
50
+            if (strpos($routes, '.php') === false) $routes = trim($routes, '/').'/';
51
+            if (is_file($routes.'/routes.php') && is_array($routesFile = include $routes.'/routes.php')) $routes = $routesFile;
52 52
             elseif (is_file($routes) && is_array($routesFile = include $routes)) $routes = $routesFile;
53
-            else throw new \InvalidArgumentException('Argument for "' . get_called_class() . '" constructor is not recognized. Expected argument array or file containing array but "' . $routes . '" given');
53
+            else throw new \InvalidArgumentException('Argument for "'.get_called_class().'" constructor is not recognized. Expected argument array or file containing array but "'.$routes.'" given');
54 54
         }
55
-        $this->routes['routes_' . $this->countRoutes] = is_array($routes) ? $routes : [];
55
+        $this->routes['routes_'.$this->countRoutes] = is_array($routes) ? $routes : [];
56 56
         $this->setRoutes($options, $this->countRoutes);
57 57
         $this->countRoutes++;
58 58
     }
@@ -76,10 +76,10 @@  discard block
 block discarded – undo
76 76
         if (is_array($args)) {
77 77
             $nbrArgs = count($args);
78 78
             for ($i = 0; $i < $nbrArgs; ++$i)
79
-                $this->routes['prefix_' . $i] = '/' . trim($args[$i], '/');
79
+                $this->routes['prefix_'.$i] = '/'.trim($args[$i], '/');
80 80
         } elseif (is_string($args))
81 81
             for ($i = 0; $i < $this->countRoutes; ++$i)
82
-                $this->routes['prefix_' . $i] = '/' . trim($args, '/');
82
+                $this->routes['prefix_'.$i] = '/'.trim($args, '/');
83 83
         if ($this->countRoutes == 0) $this->countRoutes++;
84 84
     }
85 85
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         for ($i = 0; $i < $nbrArgs; ++$i) {
93 93
             if (is_array($args[$i])) {
94 94
                 $this->setRoutes($args[$i], $i);
95
-                if (!isset($this->routes['routes_' . $i])) $this->routes['routes_' . $i] = [];
95
+                if (!isset($this->routes['routes_'.$i])) $this->routes['routes_'.$i] = [];
96 96
             }
97 97
         }
98 98
         if ($this->countRoutes == 0) $this->countRoutes++;
@@ -104,11 +104,11 @@  discard block
 block discarded – undo
104 104
      */
105 105
     private function setRoutes($args = [], $i)
106 106
     {
107
-        $this->routes['block_' . $i] = (isset($args['block']) && !empty($args['block'])) ? rtrim($args['block'], '/') . '/' : '';
108
-        $this->routes['view_dir_' . $i] = (isset($args['view_dir']) && !empty($args['view_dir'])) ? rtrim($args['view_dir'], '/') . '/' : '';
109
-        $this->routes['ctrl_namespace_' . $i] = (isset($args['ctrl_namespace']) && !empty($args['ctrl_namespace'])) ? trim($args['ctrl_namespace'], '\\') . '\\' : '';
110
-        $this->routes['prefix_' . $i] = (isset($args['prefix']) && !empty($args['prefix'])) ? '/' . trim($args['prefix'], '/') : '';
111
-        $this->routes['subdomain_' . $i] = (isset($args['subdomain'])) ? $args['subdomain'] : '';
107
+        $this->routes['block_'.$i] = (isset($args['block']) && !empty($args['block'])) ? rtrim($args['block'], '/').'/' : '';
108
+        $this->routes['view_dir_'.$i] = (isset($args['view_dir']) && !empty($args['view_dir'])) ? rtrim($args['view_dir'], '/').'/' : '';
109
+        $this->routes['ctrl_namespace_'.$i] = (isset($args['ctrl_namespace']) && !empty($args['ctrl_namespace'])) ? trim($args['ctrl_namespace'], '\\').'\\' : '';
110
+        $this->routes['prefix_'.$i] = (isset($args['prefix']) && !empty($args['prefix'])) ? '/'.trim($args['prefix'], '/') : '';
111
+        $this->routes['subdomain_'.$i] = (isset($args['subdomain'])) ? $args['subdomain'] : '';
112 112
     }
113 113
 
114 114
     /**
@@ -135,32 +135,32 @@  discard block
 block discarded – undo
135 135
         $protocol = isset($_SERVER['REQUEST_SCHEME']) ? $_SERVER['REQUEST_SCHEME'] : 'http';
136 136
         $domain = isset($_SERVER['SERVER_NAME']) ? $_SERVER['SERVER_NAME'] : null;
137 137
         $root = (is_null($root))
138
-            ? $protocol . '://' . $domain . ((!empty($_SERVER['SERVER_PORT']) && $_SERVER['SERVER_PORT'] != 80) ? ':' . $_SERVER['SERVER_PORT'] : '') . str_replace('/' . $script_file, '', $_SERVER['SCRIPT_NAME'])
138
+            ? $protocol.'://'.$domain.((!empty($_SERVER['SERVER_PORT']) && $_SERVER['SERVER_PORT'] != 80) ? ':'.$_SERVER['SERVER_PORT'] : '').str_replace('/'.$script_file, '', $_SERVER['SCRIPT_NAME'])
139 139
             : $root;
140 140
         $new_domain = $this->getDomain($root);
141 141
         if (!is_null($domain) && strpos($domain, $new_domain) !== false)
142 142
             $root = str_replace($domain, $new_domain, $root);
143 143
         $count = 0;
144 144
         for ($i = 0; $i < $this->countRoutes; ++$i) {
145
-            $prefix = (isset($this->routes['prefix_' . $i])) ? $this->routes['prefix_' . $i] : '';
146
-            $subdomain = (isset($this->routes['subdomain_' . $i])) ? $this->routes['subdomain_' . $i] : '';
147
-            $url = (!empty($subdomain)) ? str_replace($protocol.'://',$protocol.'://'.$subdomain.'.' ,$root) : $root;
148
-            if (isset($this->routes['routes_' . $i]))
149
-                foreach ($this->routes['routes_' . $i] as $route => $dependencies) {
145
+            $prefix = (isset($this->routes['prefix_'.$i])) ? $this->routes['prefix_'.$i] : '';
146
+            $subdomain = (isset($this->routes['subdomain_'.$i])) ? $this->routes['subdomain_'.$i] : '';
147
+            $url = (!empty($subdomain)) ? str_replace($protocol.'://', $protocol.'://'.$subdomain.'.', $root) : $root;
148
+            if (isset($this->routes['routes_'.$i]))
149
+                foreach ($this->routes['routes_'.$i] as $route => $dependencies) {
150 150
                     if (is_array($dependencies) && isset($dependencies['use']))
151
-                        $use = (is_callable($dependencies['use'])) ? 'closure-' . $count : trim($dependencies['use'], '/');
151
+                        $use = (is_callable($dependencies['use'])) ? 'closure-'.$count : trim($dependencies['use'], '/');
152 152
                     elseif (!is_array($dependencies))
153
-                        $use = (is_callable($dependencies)) ? 'closure-' . $count : trim($dependencies, '/');
153
+                        $use = (is_callable($dependencies)) ? 'closure-'.$count : trim($dependencies, '/');
154 154
                     else
155 155
                         $use = $route;
156 156
                     if (isset($route[0]) && $route[0] == '/') {
157 157
                         (!is_callable($dependencies) && isset($dependencies['name']))
158
-                            ? $this->routesByName[$use . '#' . $dependencies['name']] = $url . $prefix . $route
159
-                            : $this->routesByName[$use] = $url . $prefix . $route;
158
+                            ? $this->routesByName[$use.'#'.$dependencies['name']] = $url.$prefix.$route
159
+                            : $this->routesByName[$use] = $url.$prefix.$route;
160 160
                     } else {
161 161
                         (!is_callable($dependencies) && isset($dependencies['name']))
162
-                            ? $this->routesByName[$use . '#' . $dependencies['name']] = $protocol . '://' . str_replace('{host}', $new_domain, $route) . $prefix
163
-                            : $this->routesByName[$use] = $protocol . '://' . str_replace('{host}', $new_domain, $route) . $prefix;
162
+                            ? $this->routesByName[$use.'#'.$dependencies['name']] = $protocol.'://'.str_replace('{host}', $new_domain, $route).$prefix
163
+                            : $this->routesByName[$use] = $protocol.'://'.str_replace('{host}', $new_domain, $route).$prefix;
164 164
                     }
165 165
                     $count++;
166 166
                 }
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
         foreach ($this->routesByName as $key => $route) {
194 194
             $param = explode('#', $key);
195 195
             $route = str_replace('{subdomain}', $subdomain, $route);
196
-            foreach ($params as $key2 => $value) $route = str_replace(':' . $key2, $value, $route);
196
+            foreach ($params as $key2 => $value) $route = str_replace(':'.$key2, $value, $route);
197 197
             if ($param[0] == trim($name, '/')) return $route;
198 198
             else if (isset($param[1]) && $param[1] == $name) return $route;
199 199
         }
Please login to merge, or discard this patch.