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/Modules/Localize/Localize.php

Spacing +7 added lines, -7 removed lines

@@ -60,7 +60,7 @@ 
         if (is_array($config)) {
             $config = new \Tight\Modules\Localize\LocalizeConfig($config);
         } else if (!$config instanceof \Tight\Modules\Localize\LocalizeConfig) {
-            throw new \InvalidArgumentException("Argument 1 passed to " . get_class($this) . " must be an array or an instance of Tight\Modules\Localize\LocalizeConfig");
+            throw new \InvalidArgumentException("Argument 1 passed to ".get_class($this)." must be an array or an instance of Tight\Modules\Localize\LocalizeConfig");
         }
         parent::__construct("LocalizeModule");
         $this->setConfig($config);
@@ -123,17 +123,17 @@ 
     public function setLocale($locale) {
         $this->locale = $locale;
         $folder = \Tight\Utils::addTrailingSlash($this->getConfig()->resourceFolder);
-        $fileName = $this->getConfig()->resourceFileName . $this->getConfig()->langSeparator . $locale . "." . $this->getConfig()->resourceFileType;
-        $file = $folder . $fileName;
+        $fileName = $this->getConfig()->resourceFileName.$this->getConfig()->langSeparator.$locale.".".$this->getConfig()->resourceFileType;
+        $file = $folder.$fileName;
         if (is_file($file)) {
             $this->values = json_decode(file_get_contents($file), JSON_FORCE_OBJECT);
         } else {
-            $fileName = $this->getConfig()->resourceFileName . "." . $this->getConfig()->resourceFileType;
-            $file = $folder . $fileName;
+            $fileName = $this->getConfig()->resourceFileName.".".$this->getConfig()->resourceFileType;
+            $file = $folder.$fileName;
             if (is_file($file)) {
                 $this->values = json_decode(file_get_contents($file), JSON_FORCE_OBJECT);
             } else {
-                throw new \Tight\Modules\ModuleException("Resource file <strong>" . $file . "</strong> not found");
+                throw new \Tight\Modules\ModuleException("Resource file <strong>".$file."</strong> not found");
             }
         }
     }
@@ -154,7 +154,7 @@ 
             }
         }
         foreach ($files as $element) {
-            $file = \Tight\Utils::getSlicedFile($directory . $element);
+            $file = \Tight\Utils::getSlicedFile($directory.$element);
             //Removes extension
             $name = $file["name"];
             $explode = explode($this->getConfig()->langSeparator, $name);

src/Modules/ModuleLoader.php

Spacing +1 added lines, -1 removed lines

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