AlejandroPF / tight

src/Route.php

Braces +3 added lines, -2 removed lines

@@ -72,8 +72,9 @@
     }
 
     public function setCallable($callback) {
-        if (is_callable($callback))
-            $this->callable = $callback;
+        if (is_callable($callback)) {
+                    $this->callable = $callback;
+        }
     }
 
     /**

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
     private $argsPattern = "(\\w+)";
 
     public function __construct($pattern, $callable) {
-        $this->argsPattern = self::PARAMETER_CHARACTER . $this->argsPattern;
+        $this->argsPattern = self::PARAMETER_CHARACTER.$this->argsPattern;
         $this->setPattern($pattern);
         $this->setCallable($callable);
         $this->filterPattern();
@@ -71,13 +71,13 @@ 
         $pattern = $this->getPattern();
         $match = [];
         $pattern = str_replace("/", "\\/", $pattern);
-        if (preg_match_all("/" . $this->argsPattern . "/", $pattern, $match)) {
+        if (preg_match_all("/".$this->argsPattern."/", $pattern, $match)) {
             $strings = $match[0];
             $parameters = $match[1];
             $this->params = $parameters;
             $pattern = str_replace($strings, str_replace(self::PARAMETER_CHARACTER, "", $this->argsPattern), $pattern);
         }
-        $this->setPattern("/^" . $pattern . "$/");
+        $this->setPattern("/^".$pattern."$/");
     }
 
     public function setPattern($pattern) {

src/Utils.php

Spacing +3 added lines, -3 removed lines

@@ -53,7 +53,7 @@ 
         // Get the substring after $substrToRemove
         $post = substr($string, $lastIndex);
         // Return $pre and $post
-        return $pre . $post;
+        return $pre.$post;
     }
 
     /**
@@ -77,7 +77,7 @@ 
     public static function addTrailingSlash($path) {
         if (null !== $path && is_string($path) && !empty($path)) {
             if (substr($path, -1) !== "/") {
-                $path .="/";
+                $path .= "/";
             }
         }
         return $path;
@@ -101,7 +101,7 @@ 
      * @return string
      */
     public static function removeDouble($string, $substring) {
-        return str_replace($substring . $substring, $substring, $string);
+        return str_replace($substring.$substring, $substring, $string);
     }
 
 }

tests/UtilsTest.php

Spacing +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
     public function testUtilsAddTrailingSlash() {
         $path = "/etc/php5";
         $this->assertEquals("/etc/php5/", Utils::addTrailingSlash($path));
-        $this->assertEquals("/etc/php5/", Utils::addTrailingSlash($path . "/"));
+        $this->assertEquals("/etc/php5/", Utils::addTrailingSlash($path."/"));
     }
 
     /**
@@ -84,7 +84,7 @@ 
         $pathUnx = "/var/htdocs";
         $this->assertEquals("C:/php/ext/", Utils::filterPath($pathWin));
         $this->assertEquals("/var/htdocs/", Utils::filterPath($pathUnx));
-        $this->assertEquals("/var/htdocs/", Utils::filterPath($pathUnx . "/"));
+        $this->assertEquals("/var/htdocs/", Utils::filterPath($pathUnx."/"));
     }
 
     /**

tests/LocalizeModuleTest.php

Spacing +4 added lines, -4 removed lines

@@ -62,9 +62,9 @@ 
     private $module;
 
     public function setUp() {
-        $valuesFile_en = self::$resourceFolder . "/values.json";
-        $valuesFile_es = self::$resourceFolder . "/values_es.json";
-        $valuesFile_fr = self::$resourceFolder . "/values_fr.json";
+        $valuesFile_en = self::$resourceFolder."/values.json";
+        $valuesFile_es = self::$resourceFolder."/values_es.json";
+        $valuesFile_fr = self::$resourceFolder."/values_fr.json";
         if (!is_dir(self::$resourceFolder)) {
             mkdir(self::$resourceFolder);
         }
@@ -78,7 +78,7 @@ 
             file_put_contents($valuesFile_fr, json_encode($this->locale_fr));
         }
         $this->config = [
-            "resourceFolder" => "./" . self::$resourceFolder . "/"
+            "resourceFolder" => "./".self::$resourceFolder."/"
         ];
         $this->module = new \Tight\Modules\Localize\Localize($this->config);
     }

src/Tight.php

Spacing +5 added lines, -5 removed lines

@@ -180,8 +180,8 @@ 
                     <body>
                         <h1>Tight Framework Exception</h1>
 EXC;
-        $output .= "<p><strong>" . get_class($exception) . ": </strong>" . $exception->getMessage() . "</p>";
-        $output .= "<p class='padding-left'>in <strong>" . $lastTrace['file'] . "</strong> at line <strong>" . $lastTrace['line'] . "</strong></p>";
+        $output .= "<p><strong>".get_class($exception).": </strong>".$exception->getMessage()."</p>";
+        $output .= "<p class='padding-left'>in <strong>".$lastTrace['file']."</strong> at line <strong>".$lastTrace['line']."</strong></p>";
         $output .= "<br/>";
         $output .= "<p>Stack Trace:</p>";
         $trace = $exception->getTrace();
@@ -191,9 +191,9 @@ 
             $class = isset($element["class"]) ? $element["class"] : "";
             $type = isset($element["type"]) ? $element["type"] : "";
             $func = isset($element["function"]) ? $element["function"] : "";
-            $file = isset($element["file"]) ? "at <strong>" . $element["file"] . "</strong>" : "";
-            $line = isset($element["line"]) ? " at line <strong>" . $element["line"] . "</strong>" : "";
-            $output .= "<p>#" . ($index + 1) . ": " . $class . $type . $func . "() " . $file . $line . "</strong></p>";
+            $file = isset($element["file"]) ? "at <strong>".$element["file"]."</strong>" : "";
+            $line = isset($element["line"]) ? " at line <strong>".$element["line"]."</strong>" : "";
+            $output .= "<p>#".($index + 1).": ".$class.$type.$func."() ".$file.$line."</strong></p>";
         }
         echo $output;
     }

tests/RouterTest.php

Spacing +4 added lines, -4 removed lines

@@ -40,10 +40,10 @@ 
      * @covers Tight\Router::__construct
      */
     public function setUp() {
-        $dirs = ["models","views","controllers"];
+        $dirs = ["models", "views", "controllers"];
         $size = count($dirs);
         for ($index = 0; $index < $size; $index++) {
-            if(!is_dir($dirs[$index])){
+            if (!is_dir($dirs[$index])) {
                 mkdir($dirs[$index]);
             }
         }
@@ -52,13 +52,13 @@ 
             return "Hello";
         });
         $this->router->get("/hello/:id", function($id) {
-            return "Hello " . $id;
+            return "Hello ".$id;
         });
         $this->router->post("/world/", function() {
             return "world";
         });
         $this->router->post("/world/:id", function($id) {
-            return $id . " world";
+            return $id." world";
         });
         $this->router->map(["get", "post"], "/map/", function() {
             return "map";

tests/TightTest.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
         unset($app);
         $app = new \Tight\Tight;
         $app->setConfig($this->config);
-        $this->assertEquals($expectedClass,$app->getConfig());
+        $this->assertEquals($expectedClass, $app->getConfig());
     }
 
     /**

src/Router.php

Spacing +16 added lines, -16 removed lines

@@ -171,7 +171,7 @@ 
         }
         $pattern = array_shift($args); // 1st index-> Pattern
         $callable = array_pop($args); // Last index-> callable
-        $route = new Route(Utils::removeDouble($this->basePath . $pattern, "/"), $callable);
+        $route = new Route(Utils::removeDouble($this->basePath.$pattern, "/"), $callable);
         $route->setHttpMethods($methods);
         if (count($args) > 0) {
             // Adds the middleware
@@ -247,7 +247,7 @@ 
      * @return string Request URN
      */
     public function getRequestUrn() {
-        $output = Utils::addTrailingSlash("/" . Utils::removeSubstring($_SERVER['REQUEST_URI'], $this->basePath));
+        $output = Utils::addTrailingSlash("/".Utils::removeSubstring($_SERVER['REQUEST_URI'], $this->basePath));
         return $output;
     }
 
@@ -265,22 +265,22 @@ 
         $viewDir = $config->mvc["view_dir"];
         if (is_dir($controllerDir) && is_dir($modelDir) && is_dir($viewDir)) {
             if (strpos(strtolower($className), "controller") !== FALSE) {
-                if (is_file($controllerDir . $className . ".php")) {
-                    require_once $controllerDir . $className . ".php";
+                if (is_file($controllerDir.$className.".php")) {
+                    require_once $controllerDir.$className.".php";
                 } else {
-                    $fileNotFound = $controllerDir . $className . ".php";
+                    $fileNotFound = $controllerDir.$className.".php";
                 }
             } elseif (strpos(strtolower($className), "model") !== FALSE) {
-                if (is_file($modelDir . $className . ".php")) {
-                    require_once $modelDir . $className . ".php";
+                if (is_file($modelDir.$className.".php")) {
+                    require_once $modelDir.$className.".php";
                 } else {
-                    $fileNotFound = $modelDir . $className . ".php";
+                    $fileNotFound = $modelDir.$className.".php";
                 }
             } elseif (strpos(strtolower($className), "view") !== FALSE) {
-                if (is_file($viewDir . $className . ".php")) {
-                    require_once $viewDir . $className . ".php";
+                if (is_file($viewDir.$className.".php")) {
+                    require_once $viewDir.$className.".php";
                 } else {
-                    $fileNotFound = $viewDir . $className . ".php";
+                    $fileNotFound = $viewDir.$className.".php";
                 }
             }
         } else {
@@ -290,7 +290,7 @@ 
             $err = !is_dir($controllerDir) ? "Controller directory not found" : !is_dir($modelDir) ? "Model directory not found" : "View directory not found";
             throw new \Tight\Exception\NotFoundException($err);
         } else if ($fileNotFound !== false) {           
-            $err = "File <strong>" . $fileNotFound . "</strong> not found";
+            $err = "File <strong>".$fileNotFound."</strong> not found";
             throw new \Tight\Exception\NotFoundException($err);
         }
     }
@@ -316,9 +316,9 @@ 
                 }
             }
             $name = ucwords($name);
-            $contName = $name . "Controller";
-            $viewName = $name . "View";
-            $modName = $name . "Model";
+            $contName = $name."Controller";
+            $viewName = $name."View";
+            $modName = $name."Model";
             $model = new $modName();
             $view = new $viewName();
             $controller = new $contName($model, $view);
@@ -326,7 +326,7 @@ 
                 if (method_exists($controller, $method)) {
                     call_user_method_array($method, $controller, $args);
                 } else {
-                    throw new \Tight\Exception\RouterException("Method <strong>" . $method . "</strong> not defined for <strong>" . $contName . "</strong> class");
+                    throw new \Tight\Exception\RouterException("Method <strong>".$method."</strong> not defined for <strong>".$contName."</strong> class");
                 }
             }
             $controller->render();

src/Modules/ModuleLoader.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
             $this->modules[$module->getModuleName()] = $module;
             $module->onLoad();
         } else {
-            throw new \Tight\Exception\ModuleException("Module <strong>" . $module->getModuleName() . "</strong> already exists");
+            throw new \Tight\Exception\ModuleException("Module <strong>".$module->getModuleName()."</strong> already exists");
         }
     }