Merge pull request #3 from arojunior/scrutinizer-patch-1

Scrutinizer Auto-Fixes

Author: arojunior

core/helper/Helper.php

@@ -8,7 +8,7 @@ public function toJson($dados, $charset = 'UTF-8')
         header('Content-Type: application/json');
 
         if ($charset <> 'UFT-8') {
-            array_walk_recursive($dados, function (&$value, $key) {
+            array_walk_recursive($dados, function(&$value, $key) {
                 if (is_string($value)) {
                     $value = utf8_encode(self::clean($value));
                 }
@@ -21,23 +21,23 @@ public function toJson($dados, $charset = 'UTF-8')
     public function clean($string)
     {
         $table = array(
-             '?' => 'S', '?' => 's', '?' => 'Dj', '?' => 'dj', '?' => 'Z',
-             '?' => 'z', '?' => 'C', '?' => 'c', '?' => 'C', '?' => 'c',
-             'À' => 'A', 'Á' => 'A', 'Â' => 'A', 'Ã' => 'A', 'Ä' => 'A',
-             'Å' => 'A', 'Æ' => 'A', 'Ç' => 'C', 'È' => 'E', 'É' => 'E',
-             'Ê' => 'E', 'Ë' => 'E', 'Ì' => 'I', 'Í' => 'I', 'Î' => 'I',
-             'Ï' => 'I', 'Ñ' => 'N', 'Ò' => 'O', 'Ó' => 'O', 'Ô' => 'O',
-             'Õ' => 'O', 'Ö' => 'O', 'Ø' => 'O', 'Ù' => 'U', 'Ú' => 'U',
-             'Û' => 'U', 'Ü' => 'U', 'Ý' => 'Y', 'Þ' => 'B', 'ß' => 'Ss',
-             'à' => 'a', 'á' => 'a', 'â' => 'a', 'ã' => 'a', 'ä' => 'a',
-             'å' => 'a', 'æ' => 'a', 'ç' => 'c', 'è' => 'e', 'é' => 'e',
-             'ê' => 'e', 'ë' => 'e', 'ì' => 'i', 'í' => 'i', 'î' => 'i',
-             'ï' => 'i', 'ð' => 'o', 'ñ' => 'n', 'ò' => 'o', 'ó' => 'o',
-             'ô' => 'o', 'õ' => 'o', 'ö' => 'o', 'ø' => 'o', 'ù' => 'u',
-             'ú' => 'u', 'û' => 'u', 'ý' => 'y', 'ý' => 'y', 'þ' => 'b',
-             'ÿ' => 'y', '?' => 'R', '?' => 'r', 'ü' => 'u', 'º' => '',
-             'ª' => '',
-         );
+                '?' => 'S', '?' => 's', '?' => 'Dj', '?' => 'dj', '?' => 'Z',
+                '?' => 'z', '?' => 'C', '?' => 'c', '?' => 'C', '?' => 'c',
+                'À' => 'A', 'Á' => 'A', 'Â' => 'A', 'Ã' => 'A', 'Ä' => 'A',
+                'Å' => 'A', 'Æ' => 'A', 'Ç' => 'C', 'È' => 'E', 'É' => 'E',
+                'Ê' => 'E', 'Ë' => 'E', 'Ì' => 'I', 'Í' => 'I', 'Î' => 'I',
+                'Ï' => 'I', 'Ñ' => 'N', 'Ò' => 'O', 'Ó' => 'O', 'Ô' => 'O',
+                'Õ' => 'O', 'Ö' => 'O', 'Ø' => 'O', 'Ù' => 'U', 'Ú' => 'U',
+                'Û' => 'U', 'Ü' => 'U', 'Ý' => 'Y', 'Þ' => 'B', 'ß' => 'Ss',
+                'à' => 'a', 'á' => 'a', 'â' => 'a', 'ã' => 'a', 'ä' => 'a',
+                'å' => 'a', 'æ' => 'a', 'ç' => 'c', 'è' => 'e', 'é' => 'e',
+                'ê' => 'e', 'ë' => 'e', 'ì' => 'i', 'í' => 'i', 'î' => 'i',
+                'ï' => 'i', 'ð' => 'o', 'ñ' => 'n', 'ò' => 'o', 'ó' => 'o',
+                'ô' => 'o', 'õ' => 'o', 'ö' => 'o', 'ø' => 'o', 'ù' => 'u',
+                'ú' => 'u', 'û' => 'u', 'ý' => 'y', 'ý' => 'y', 'þ' => 'b',
+                'ÿ' => 'y', '?' => 'R', '?' => 'r', 'ü' => 'u', 'º' => '',
+                'ª' => '',
+            );
 
         $string = strtr($string, $table);
 

core/model/Database.php

@@ -16,7 +16,7 @@ public function __construct()
             $this->db_config = parse_ini_file('config.ini');
         }
 
-        $this->conn = new \PDO("mysql:host={$this->db_config['db_host']};".
+        $this->conn = new \PDO("mysql:host={$this->db_config['db_host']};" .
                                 "dbname={$this->db_config['db_name']}",
                                 $this->db_config['db_user'],
                                 $this->db_config['db_pass']

core/model/Model.php

@@ -85,7 +85,7 @@ private function where()
 
     private function find()
     {
-        $sql = "SELECT ".self::fields()." FROM {$this->table} ".self::where();
+        $sql = "SELECT " . self::fields() . " FROM {$this->table} " . self::where();
 
         $this->stmt = $this->conn->prepare($sql);
 
@@ -117,7 +117,7 @@ private function insertQueryString()
     private function updateWhere($data)
     {
         $this->data['conditions'] = [$this->pk => $data[$this->pk]];
-        $where = 'WHERE '.self::conditions('');
+        $where = 'WHERE ' . self::conditions('');
         unset($data[$this->pk]);
 
         return $where;
@@ -142,7 +142,7 @@ public function findOne($data)
     {
         $this->data['conditions'] = $data;
         return $this->fetch = $this->find()
-                                   ->stmt->fetch(PDO::FETCH_ASSOC);
+                                    ->stmt->fetch(PDO::FETCH_ASSOC);
     }
 
     public function findById($id)
@@ -153,7 +153,7 @@ public function findById($id)
     public function exists($id)
     {
         if (is_array($id)) {
-          return (self::findOne($id));
+            return (self::findOne($id));
         }
 
         return (self::findById($id));
@@ -187,7 +187,7 @@ public function save($data)
             $this->lastId = $data[$this->pk];
         }
 
-        if (! empty($this->count)) {
+        if ( ! empty($this->count)) {
             return $this->update($data);
         }
 
@@ -223,7 +223,7 @@ public function delete($data)
     {
         $this->data['conditions'] = $data;
 
-        $sql = "DELETE FROM {$this->table} ".self::where();
+        $sql = "DELETE FROM {$this->table} " . self::where();
         $this->stmt = $this->conn->prepare($sql);
 
         if ( ! empty($this->where)) {

index.php

@@ -27,7 +27,7 @@
 
 $src        = explode('/', $uri);
 $model      = ucfirst($src[1]);
-$controller = $model.'Controller';
+$controller = $model . 'Controller';
 $method     = (isset($src[2])) ? $src[2] : 'index';
 
 if (isset($src[3]) && empty($the_request)) {
@@ -41,14 +41,14 @@
     $load_class = 'SimpleORM\app\controller\\' . $controller;
     $class      = new $load_class();
     $set        = $class->$method($the_request);
-} catch(Exception $e) {
-    echo 'No '.$controller.' found for this route',  $e->getMessage(), "\n";
+} catch (Exception $e) {
+    echo 'No ' . $controller . ' found for this route', $e->getMessage(), "\n";
 }
 
 /*
 * Declare all variables if passed in return
 */
-if (!empty($set) && is_array($set)) {
+if ( ! empty($set) && is_array($set)) {
     foreach ($set as $k => $v) {
         ${$k} = $v;
     }