prinick96 / Ocrend-Framework

app/models/Users.php

Spacing +39 added lines, -39 removed lines

@@ -73,7 +73,7 @@ 
       * @throws ModelsException cuando las contraseñas no coinciden
     */
     private function checkPassMatch(string $pass, string $pass_repeat) {
-        if($pass != $pass_repeat) {
+        if ($pass != $pass_repeat) {
             throw new ModelsException('Las contraseñas no coinciden.');
         }
     }
@@ -87,13 +87,13 @@ 
     */
     private function checkEmail(string $email) {
         # Formato de email
-        if(!Strings::is_email($email)) {
+        if (!Strings::is_email($email)) {
             throw new ModelsException('El email no tiene un formato válido.');
         }
         # Existencia de email
         $email = $this->db->scape($email);
-        $query = $this->db->select('id_user','users',"email='$email'",'LIMIT 1');
-        if(false !== $query) {
+        $query = $this->db->select('id_user', 'users', "email='$email'", 'LIMIT 1');
+        if (false !== $query) {
             throw new ModelsException('El email introducido ya existe.');
         }
     }
@@ -107,7 +107,7 @@ 
       * @return void
     */
     private function restoreAttempts(string $email) {       
-        if(array_key_exists($email,$this->recentAttempts)) {
+        if (array_key_exists($email, $this->recentAttempts)) {
             $this->recentAttempts[$email]['attempts'] = 0;
             $this->recentAttempts[$email]['time'] = null;
             $this->updateSessionAttempts();
@@ -127,7 +127,7 @@ 
     private function generateSession(array $user_data) {
         global $session, $config;
 
-        $session->set('user_id',(int) $user_data['id_user']);
+        $session->set('user_id', (int) $user_data['id_user']);
         $session->set('unique_session', $config['sessions']['unique']);
     }
 
@@ -140,12 +140,12 @@ 
       * @return bool true: Cuando el inicio de sesión es correcto 
       *              false: Cuando el inicio de sesión no es correcto
     */
-    private function authentication(string $email,string $pass) : bool {
+    private function authentication(string $email, string $pass) : bool {
         $email = $this->db->scape($email);
-        $query = $this->db->select('id_user,pass','users',"email='$email'",'LIMIT 1');
+        $query = $this->db->select('id_user,pass', 'users', "email='$email'", 'LIMIT 1');
         
         # Incio de sesión con éxito
-        if(false !== $query && Strings::chash($query[0]['pass'],$pass)) {
+        if (false !== $query && Strings::chash($query[0]['pass'], $pass)) {
 
             # Restaurar intentos
             $this->restoreAttempts($email);
@@ -166,7 +166,7 @@ 
     private function setDefaultAttempts() {
         global $session;
 
-        if(null != $session->get('login_user_recentAttempts')) {
+        if (null != $session->get('login_user_recentAttempts')) {
             $this->recentAttempts = $session->get('login_user_recentAttempts');
         }
     }
@@ -179,7 +179,7 @@ 
       * @return void
     */
     private function setNewAttempt(string $email) {
-        if(!array_key_exists($email,$this->recentAttempts)) {
+        if (!array_key_exists($email, $this->recentAttempts)) {
             $this->recentAttempts[$email] = array(
                 'attempts' => 0, # Intentos
                 'time' => null # Tiempo 
@@ -200,14 +200,14 @@ 
       * @return void
     */
     private function maximumAttempts(string $email) {
-        if($this->recentAttempts[$email]['attempts'] >= self::MAX_ATTEMPTS) {
+        if ($this->recentAttempts[$email]['attempts'] >= self::MAX_ATTEMPTS) {
             
             # Colocar timestamp para recuperar más adelante la posibilidad de acceso
-            if(null == $this->recentAttempts[$email]['time']) {
+            if (null == $this->recentAttempts[$email]['time']) {
                 $this->recentAttempts[$email]['time'] = time() + self::MAX_ATTEMPTS_TIME;
             }
             
-            if(time() < $this->recentAttempts[$email]['time']) {
+            if (time() < $this->recentAttempts[$email]['time']) {
                 # Setear sesión
                 $this->updateSessionAttempts();
                 # Lanzar excepción
@@ -235,7 +235,7 @@ 
             $pass = $http->request->get('pass');
 
             # Verificar que no están vacíos
-            if($this->functions->e($email,$pass)) {
+            if ($this->functions->e($email, $pass)) {
                 throw new ModelsException('Credenciales incompletas.');
             }
             
@@ -246,13 +246,13 @@ 
             $this->maximumAttempts($email);
 
             # Autentificar
-            if($this->authentication($email,$pass)) {
+            if ($this->authentication($email, $pass)) {
                 return array('success' => 1, 'message' => 'Conectado con éxito.');
             }
             
             throw new ModelsException('Credenciales incorrectas.');
 
-        } catch(ModelsException $e) {
+        } catch (ModelsException $e) {
             return array('success' => 0, 'message' => $e->getMessage());
         }        
     }
@@ -273,7 +273,7 @@ 
             $pass_repeat = $http->request->get('pass_repeat');
 
             # Verificar que no están vacíos
-            if($this->functions->e($name,$email,$pass,$pass_repeat)) {
+            if ($this->functions->e($name, $email, $pass, $pass_repeat)) {
                 throw new ModelsException('Todos los datos son necesarios');
             }
 
@@ -281,10 +281,10 @@ 
             $this->checkEmail($email);
 
             # Veriricar contraseñas
-            $this->checkPassMatch($pass,$pass_repeat);
+            $this->checkPassMatch($pass, $pass_repeat);
 
             # Registrar al usuario
-            $this->db->insert('users',array(
+            $this->db->insert('users', array(
                 'name' => $name,
                 'email' => $email,
                 'pass' => Strings::hash($pass)
@@ -296,7 +296,7 @@ 
             ));
 
             return array('success' => 1, 'message' => 'Registrado con éxito.');
-        } catch(ModelsException $e) {
+        } catch (ModelsException $e) {
             return array('success' => 0, 'message' => $e->getMessage());
         }        
     }
@@ -315,7 +315,7 @@ 
             $email = $http->request->get('email');
             
             # Campo lleno
-            if($this->functions->emp($email)) {
+            if ($this->functions->emp($email)) {
                 throw new ModelsException('El campo email debe estar lleno.');
             }
 
@@ -323,10 +323,10 @@ 
             $email = $this->db->scape($email);
 
             # Obtener información del usuario 
-            $user_data = $this->db->select('id_user,name','users',"email='$email'",'LIMIT 1');
+            $user_data = $this->db->select('id_user,name', 'users', "email='$email'", 'LIMIT 1');
 
             # Verificar correo en base de datos 
-            if(false === $user_data) {
+            if (false === $user_data) {
                 throw new ModelsException('El email no está registrado en el sistema.');
             }
 
@@ -335,30 +335,30 @@ 
             $pass = uniqid();
 
             # Construir mensaje y enviar mensaje
-            $HTML = 'Hola <b>'. $user_data[0]['name'] .'</b>, ha solicitado recuperar su contraseña perdida, si no ha realizado esta acción no necesita hacer nada.
+            $HTML = 'Hola <b>' . $user_data[0]['name'] . '</b>, ha solicitado recuperar su contraseña perdida, si no ha realizado esta acción no necesita hacer nada.
 					<br />
 					<br />
-					Para cambiar su contraseña por <b>'. $pass .'</b> haga <a href="'. $config['site']['url'] . 'lostpass/cambiar/&token='.$token.'&user='.$user_data[0]['id_user'].'" target="_blank">clic aquí</a>.';
+					Para cambiar su contraseña por <b>'. $pass . '</b> haga <a href="' . $config['site']['url'] . 'lostpass/cambiar/&token=' . $token . '&user=' . $user_data[0]['id_user'] . '" target="_blank">clic aquí</a>.';
 
             # Enviar el correo electrónico
             $dest = array();
 			$dest[$email] = $user_data[0]['name'];
-			$email = Emails::send_mail($dest,Emails::plantilla($HTML),'Recuperar contraseña perdida');
+			$email = Emails::send_mail($dest, Emails::plantilla($HTML), 'Recuperar contraseña perdida');
 
             # Verificar si hubo algún problema con el envío del correo
-            if(false === $email) {
+            if (false === $email) {
                 throw new ModelsException('No se ha podido enviar el correo electrónico.');
             }
 
             # Actualizar datos 
             $id_user = $user_data[0]['id_user'];
-            $this->db->update('users',array(
+            $this->db->update('users', array(
                 'tmp_pass' => Strings::hash($pass),
                 'token' => $token
-            ),"id_user='$id_user'",'LIMIT 1');
+            ), "id_user='$id_user'", 'LIMIT 1');
 
             return array('success' => 1, 'message' => 'Se ha enviado un enlace a su correo electrónico.');
-        } catch(ModelsException $e) {
+        } catch (ModelsException $e) {
             return array('success' => 0, 'message' => $e->getMessage());
         }
     }
@@ -378,7 +378,7 @@ 
         $id_user = $http->query->get('user');
         $token = $http->query->get('token');
 
-        if(!$this->functions->emp($token) && is_numeric($id_user) && $id_user >= 1) {
+        if (!$this->functions->emp($token) && is_numeric($id_user) && $id_user >= 1) {
             # Filtros a los datos
             $id_user = $this->db->scape($id_user);
             $token = $this->db->scape($token);
@@ -401,7 +401,7 @@ 
     public function logout() {
         global $session;
 
-        if(null != $session->get('user_id')) {
+        if (null != $session->get('user_id')) {
             $session->remove('user_id');
         }
 
@@ -417,7 +417,7 @@ 
       * @return false|array con información del usuario
     */   
     public function getUserById(int $id, string $select = '*') {
-       return $this->db->select($select,'users',"id_user='$id'",'LIMIT 1');
+       return $this->db->select($select, 'users', "id_user='$id'", 'LIMIT 1');
     }
     
     /**
@@ -428,7 +428,7 @@ 
       * @return false|array con información de los usuarios
     */  
     public function getUsers(string $select = '*') {
-       return $this->db->select($select,'users');
+       return $this->db->select($select, 'users');
     }
 
     /**
@@ -440,12 +440,12 @@ 
       * @return array con datos del usuario conectado
     */
     public function getOwnerUser(string $select = '*') : array {
-        if(null !== $this->id_user) {    
+        if (null !== $this->id_user) {    
                
-            $user = $this->db->select($select,'users',"id_user='$this->id_user'",'LIMIT 1');
+            $user = $this->db->select($select, 'users', "id_user='$this->id_user'", 'LIMIT 1');
 
             # Si se borra al usuario desde la base de datos y sigue con la sesión activa
-            if(false === $user) {
+            if (false === $user) {
                 $this->logout();
             }
 
@@ -461,7 +461,7 @@ 
       * @throws \RuntimeException si no se puede realizar la query
     */
     public function install() {
-        if(!$this->db->query("
+        if (!$this->db->query("
             CREATE TABLE IF NOT EXISTS `users` (
                 `id_user` int(11) UNSIGNED NOT NULL AUTO_INCREMENT,
                 `name` varchar(100) NOT NULL,

Ocrend/Kernel/Database/Database.php

Spacing +21 added lines, -21 removed lines

@@ -39,7 +39,7 @@ 
   final public static function Start($name = null, $motor = null, bool $new_instance = false) : Database {
     global $config;
 
-    if(!self::$inst instanceof self or $new_instance) {
+    if (!self::$inst instanceof self or $new_instance) {
       self::$inst = new self(
           null === $name ? $config['database']['name'] : $name,
           null === $motor ? $config['database']['motor'] : $motor
@@ -72,63 +72,63 @@ 
     try {
       switch ($motor) {
         case 'sqlite':
-          parent::__construct('sqlite:'.$name);
+          parent::__construct('sqlite:' . $name);
         break;
         case 'cubrid':
-          parent::__construct('cubrid:host='.$config['database']['host'].';dbname='.$name.';port='.$config['database']['port'],
+          parent::__construct('cubrid:host=' . $config['database']['host'] . ';dbname=' . $name . ';port=' . $config['database']['port'],
           $config['database']['user'],
           $config['database']['pass'],
           $comun_config);
         break;
         case 'firebird':
-          parent::__construct('firebird:dbname='.$config['database']['host'].':'.$name,
+          parent::__construct('firebird:dbname=' . $config['database']['host'] . ':' . $name,
           $config['database']['user'],
           $config['database']['pass'],
           $comun_config);
         break;
         case 'odbc';
-          parent::__construct('odbc:'.$name,
+          parent::__construct('odbc:' . $name,
           $config['database']['user'],
           $config['database']['pass']);
         break;
         case 'oracle':
           parent::__construct('oci:dbname=(DESCRIPTION =
             (ADDRESS_LIST =
-              (ADDRESS = (PROTOCOL = '.$config['database']['protocol'].')(HOST = '.$motor.')(PORT = '.$config['database']['port'].'))
+              (ADDRESS = (PROTOCOL = '.$config['database']['protocol'] . ')(HOST = ' . $motor . ')(PORT = ' . $config['database']['port'] . '))
             )
             (CONNECT_DATA =
-              (SERVICE_NAME = '.$name.')
+              (SERVICE_NAME = '.$name . ')
             )
           );charset=utf8',
           $config['database']['user'],
           $config['database']['pass'],
-          array_merge(array(\PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC),$comun_config));
+          array_merge(array(\PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC), $comun_config));
         break;
         case 'postgresql':
-          parent::__construct('pgsql:host='.$config['database']['host'].';dbname='.$name.';charset=utf8',
+          parent::__construct('pgsql:host=' . $config['database']['host'] . ';dbname=' . $name . ';charset=utf8',
           $config['database']['user'],
           $config['database']['pass'],
           $comun_config);
         break;
         case 'mysql':
-          parent::__construct('mysql:host='.$config['database']['host'].';dbname='.$name,
+          parent::__construct('mysql:host=' . $config['database']['host'] . ';dbname=' . $name,
           $config['database']['user'],
           $config['database']['pass'],
-          array_merge(array(\PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'),$comun_config));
+          array_merge(array(\PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'), $comun_config));
         break;
         case 'mssql':
           # Añadido por marc2684
           # DOC: https://github.com/prinick96/Ocrend-Framework/issues/7
-          parent::__construct('sqlsrv:Server='.$config['database']['host'].';Database='.$name.';ConnectionPooling=0',
+          parent::__construct('sqlsrv:Server=' . $config['database']['host'] . ';Database=' . $name . ';ConnectionPooling=0',
           $config['database']['user'],
           $config['database']['pass'],
           $comun_config);
         break;
         default:
-          throw new \RuntimeException('Motor '. $motor .' de conexión no identificado.');
+          throw new \RuntimeException('Motor ' . $motor . ' de conexión no identificado.');
         break;
       }
-    } catch(\PDOException $e) {
+    } catch (\PDOException $e) {
       throw new \RuntimeException('Problema al conectar con la base de datos: ' . $e->getMessage());
     } 
   }
@@ -163,18 +163,18 @@ 
     * @return mixed elemento sano
   */
   final public function scape($e) {
-    if(null === $e) {
+    if (null === $e) {
       return '';
     }
 
-    if(is_numeric($e) and $e <= 2147483647) {
-      if(explode('.',$e)[0] != $e) {
+    if (is_numeric($e) and $e <= 2147483647) {
+      if (explode('.', $e)[0] != $e) {
         return (float) $e;
       }
       return (int) $e;
     }
 
-    return (string) trim(str_replace(['\\',"\x00",'\n','\r',"'",'"',"\x1a"],['\\\\','\\0','\\n','\\r',"\'",'\"','\\Z'],$e));
+    return (string) trim(str_replace(['\\', "\x00", '\n', '\r', "'", '"', "\x1a"], ['\\\\', '\\0', '\\n', '\\r', "\'", '\"', '\\Z'], $e));
   }
 
   /**
@@ -202,7 +202,7 @@ 
   */
   final public function insert(string $table, array $e) : \PDOStatement {
       if (sizeof($e) == 0) {
-          throw new \RuntimeException('El arreglo pasado por $this->db->insert(\''.$table.'\',...) está vacío.');
+          throw new \RuntimeException('El arreglo pasado por $this->db->insert(\'' . $table . '\',...) está vacío.');
       }
 
       $query = "INSERT INTO $table (";
@@ -232,7 +232,7 @@ 
   */
   final public function update(string $table, array $e, string $where, string $limit = '') : \PDOStatement {
       if (sizeof($e) == 0) {
-          throw new \RuntimeException('El arreglo pasado por $this->db->update(\''.$table.'\'...) está vacío.');
+          throw new \RuntimeException('El arreglo pasado por $this->db->update(\'' . $table . '\'...) está vacío.');
       }
 
       $query = "UPDATE $table SET ";
@@ -269,7 +269,7 @@ 
     $result = $sql->fetchAll();
     $sql->closeCursor();
 
-    if(sizeof($result) > 0) {
+    if (sizeof($result) > 0) {
       return $result;
     }
 

Ocrend/Kernel/Helpers/Arrays.php

Spacing +14 added lines, -14 removed lines

@@ -43,32 +43,32 @@ 
   */
   final public static function arrays_sum(array $a, array $b) : array {
     # Si alguno está vacío
-    if(sizeof($a) == 0) {
+    if (sizeof($a) == 0) {
       return $b;
-    } else if(sizeof($b) == 0) {
+    } else if (sizeof($b) == 0) {
       return $a;
     }
 
     # Recorrer el segundo arreglo
-    foreach($b as $llave => $contenido) {
+    foreach ($b as $llave => $contenido) {
 
       # Desnivel
-      if(is_array($contenido)) {
-        throw new \RuntimeException('El arreglo $b en la llave '.$llave.' es un arreglo y no puede operar.');
+      if (is_array($contenido)) {
+        throw new \RuntimeException('El arreglo $b en la llave ' . $llave . ' es un arreglo y no puede operar.');
         break;
       }
 
       # Existencia de la llave
-      if(array_key_exists($llave,$a)) {
+      if (array_key_exists($llave, $a)) {
         # Excepciones de uso
-        if(is_array($a[$llave])) {
-          throw new \RuntimeException('El arreglo $a en la llave '.$llave.' es un arreglo y no puede operar.');
+        if (is_array($a[$llave])) {
+          throw new \RuntimeException('El arreglo $a en la llave ' . $llave . ' es un arreglo y no puede operar.');
           break;
-        } else if(!is_numeric($a[$llave]) && null !== $a[$llave]) {
-          throw new \RuntimeException('El arreglo $a en la llave '.$llave.' no es un número y no puede operar.');
+        } else if (!is_numeric($a[$llave]) && null !== $a[$llave]) {
+          throw new \RuntimeException('El arreglo $a en la llave ' . $llave . ' no es un número y no puede operar.');
           break;
-        } else if(!is_numeric($b[$llave]) && null !== $b[$llave]) {
-          throw new \RuntimeException('El arreglo $b en la llave '.$llave.' no es un número y no puede operar.');
+        } else if (!is_numeric($b[$llave]) && null !== $b[$llave]) {
+          throw new \RuntimeException('El arreglo $b en la llave ' . $llave . ' no es un número y no puede operar.');
           break;
         }
 
@@ -94,7 +94,7 @@ 
   final public static function get_key_by_index(string $index, array $a) : int {
     $i = 0;
     foreach ($a as $key => $val) {
-      if($key == $index) {
+      if ($key == $index) {
         return (int) $i;
       }
       $i++;
@@ -127,7 +127,7 @@ 
     * @return bool false si no lo es, true si lo es
   */
   final public static function is_assoc(array $a) : bool {
-    if(sizeof($a) === 0) {
+    if (sizeof($a) === 0) {
       return false;
     }
 

Ocrend/Kernel/Router/Router.php

Spacing +13 added lines, -13 removed lines

@@ -91,7 +91,7 @@ 
     */
     final private function setCollectionRule(string $index, string $rule) {
         # Verificar si la regla existe
-        if(!in_array($rule,self::RULES)) {
+        if (!in_array($rule, self::RULES)) {
             throw new \RuntimeException('La regla ' . $rule . ' no existe.');
         }
         # Definir la regla para la ruta
@@ -103,14 +103,14 @@ 
     */
     final private function checkRequests() {
         # Verificar si existe peticiones
-        if(null !== $this->requestUri) {
-            $this->real_request = explode('/',$this->requestUri);
+        if (null !== $this->requestUri) {
+            $this->real_request = explode('/', $this->requestUri);
             $this->routerCollection['/controller'] = $this->real_request[0];
         }
 
         # Setear las siguientes rutas
-        $this->routerCollection['/method'] = array_key_exists(1,$this->real_request) ? $this->real_request[1] : null;
-        $this->routerCollection['/id'] = array_key_exists(2,$this->real_request) ? $this->real_request[2] : null;
+        $this->routerCollection['/method'] = array_key_exists(1, $this->real_request) ? $this->real_request[1] : null;
+        $this->routerCollection['/id'] = array_key_exists(2, $this->real_request) ? $this->real_request[2] : null;
     }
 
     /**
@@ -123,7 +123,7 @@ 
     */
     final public function setRoute(string $index, string $rule = 'none') {
         # Nombres de rutas no permitidos
-        if(in_array($index,['/controller','/method','/id'])) {
+        if (in_array($index, ['/controller', '/method', '/id'])) {
             throw new \RuntimeException('No puede definirse ' . $index . ' como índice en la ruta.');
         }
 
@@ -135,8 +135,8 @@ 
             
         # Definir la ruta y regla
         $lastRoute = sizeof($this->routerCollection);
-        $this->routerCollection[$index] = array_key_exists($lastRoute,$this->real_request) ? $this->real_request[$lastRoute] : null;
-        $this->setCollectionRule($index,$rule);
+        $this->routerCollection[$index] = array_key_exists($lastRoute, $this->real_request) ? $this->real_request[$lastRoute] : null;
+        $this->setCollectionRule($index, $rule);
     }
     
     /**
@@ -149,7 +149,7 @@ 
     */
     final public function getRoute(string $index) {
           # Verificar existencia de ruta
-        if(!array_key_exists($index,$this->routerCollection)) {
+        if (!array_key_exists($index, $this->routerCollection)) {
             throw new \RuntimeException('La ruta ' . $index . ' no está definida en el controlador.');
         }
 
@@ -157,7 +157,7 @@ 
         $ruta = $this->routerCollection[$index];
 
         # Retornar ruta con la regla definida aplicada
-        switch($this->routerCollectionRules[$index]) {
+        switch ($this->routerCollectionRules[$index]) {
             case 'none':
                 return $ruta;
             case 'letters':
@@ -211,7 +211,7 @@ 
     */
     final public function getId(bool $with_rules = false) {
         $id = $this->routerCollection['/id'];
-        if($with_rules) {
+        if ($with_rules) {
             return (null !== $id && is_numeric($id) && $id > 0) ? (int) $id : null;
         }
 
@@ -225,10 +225,10 @@ 
     */
     final public function executeController() {
         # Definir controlador
-        if(null != ($controller = $this->getController())) {
+        if (null != ($controller = $this->getController())) {
             $controller = $controller . 'Controller';
 
-            if(!is_readable('app/controllers/' . $controller . '.php')) {
+            if (!is_readable('app/controllers/' . $controller . '.php')) {
                 $controller = 'errorController';
             }
 

Ocrend/Kernel/Models/Models.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
  * @author Brayan Narváez <prinick@ocrend.com>
  */
 
-abstract class Models  {
+abstract class Models {
 
     /**
       * Tiene siempre el id pasado por la ruta, en caso de no haber ninguno, será cero.
@@ -53,7 +53,7 @@ 
         global $session, $config;
         
         # Id captado por la ruta
-        if(null != $router) {
+        if (null != $router) {
             $this->id = $router->getId(true);
             $this->id = null == $this->id ? 0 : $this->id; 
         }
@@ -62,7 +62,7 @@ 
         $this->functions = new Functions();
 
         # Verificar sesión del usuario
-        if(null !== $session->get('user_id') && $session->get('unique_session') == $config['sessions']['unique']) {
+        if (null !== $session->get('user_id') && $session->get('unique_session') == $config['sessions']['unique']) {
            $this->id_user = $session->get('user_id');
         }
     }
@@ -77,7 +77,7 @@ 
       * @throws ModelsException
       */
     protected function setId($id, string $default_msg = 'No puedede asignarse el id.') {
-        if(null == $id || !is_numeric($id) || $id <= 0) {
+        if (null == $id || !is_numeric($id) || $id <= 0) {
             throw new ModelsException($default_msg);
         }
 

Ocrend/Kernel/Helpers/Strings.php

Spacing +13 added lines, -13 removed lines

@@ -41,7 +41,7 @@ 
     } elseif ($diff->m == 1) {
         $text = '1 mes';
     } elseif ($diff->d > 7) {
-        $text = ceil($diff->d / 7) . ' semanas';
+        $text = ceil($diff->d/7) . ' semanas';
     } elseif ($diff->d == 7) {
         $text = '1 semana';
     } elseif ($diff->d > 1) {
@@ -86,7 +86,7 @@ 
     * @return string Hash, con la forma $2a$10$87b2b603324793cc37f8dOPFTnHRY0lviq5filK5cN4aMCQDJcC9G
   */
   final public static function hash(string $p) : string {
-    return crypt($p, '$2a$10$' . substr(sha1(mt_rand()),0,22));
+    return crypt($p, '$2a$10$' . substr(sha1(mt_rand()), 0, 22));
   }
   //------------------------------------------------
   /**
@@ -99,9 +99,9 @@ 
     *
   */
   final public static function date_difference(string $ini, string $fin) : int {
-    $ini_i = explode('-',str_replace('/','-',$ini));
-    $fin_i = explode('-',str_replace('/','-',$fin));
-    return (int) floor((mktime(0, 0, 0, $fin_i[1], $fin_i[0], $fin_i[2]) - mktime(0, 0, 0, $ini_i[1], $ini_i[0], $ini_i[2])) / 86400);
+    $ini_i = explode('-', str_replace('/', '-', $ini));
+    $fin_i = explode('-', str_replace('/', '-', $fin));
+    return (int) floor((mktime(0, 0, 0, $fin_i[1], $fin_i[0], $fin_i[2]) - mktime(0, 0, 0, $ini_i[1], $ini_i[0], $ini_i[2]))/86400);
   }
   //------------------------------------------------
   /**
@@ -113,7 +113,7 @@ 
     *
   */
   final public static function calculate_age(string $cumple) : int {
-    $age = explode('.', (string) (self::date_difference($cumple,date('d-m-Y',time())) / 365));
+    $age = explode('.', (string) (self::date_difference($cumple, date('d-m-Y', time()))/365));
     return (int) $age[0];
   }
   //------------------------------------------------
@@ -124,7 +124,7 @@ 
     *
   */
   final public static function days_of_month() : int {
-    return cal_days_in_month(CAL_GREGORIAN, (int) date('m',time()), (int) date('Y',time()));
+    return cal_days_in_month(CAL_GREGORIAN, (int) date('m', time()), (int) date('Y', time()));
   }
   //------------------------------------------------
   /**
@@ -146,7 +146,7 @@ 
     * @return string del texto sin espacios
   */
   final public static function remove_spaces(string $s) : string {
-    return trim(str_replace(' ','',$s));
+    return trim(str_replace(' ', '', $s));
   }
   //------------------------------------------------
   /**
@@ -191,9 +191,9 @@ 
   */
   final public static function url_amigable(string $url) : string {
     $url = strtolower($url);
-    $url = str_replace (['á', 'é', 'í', 'ó', 'ú', 'ñ'],['a', 'e', 'i', 'o', 'u', 'n'], $url);
-    $url = str_replace([' ', '&', '\r\n', '\n', '+', '%'],'-',$url);
-    return preg_replace (['/[^a-z0-9\-<>]/', '/[\-]+/', '/<[^>]*>/'],['', '-', ''], $url);
+    $url = str_replace(['á', 'é', 'í', 'ó', 'ú', 'ñ'], ['a', 'e', 'i', 'o', 'u', 'n'], $url);
+    $url = str_replace([' ', '&', '\r\n', '\n', '+', '%'], '-', $url);
+    return preg_replace(['/[^a-z0-9\-<>]/', '/[\-]+/', '/<[^>]*>/'], ['', '-', ''], $url);
   }
   //------------------------------------------------
   /**
@@ -246,7 +246,7 @@ 
         '<span style="font-size: $1px">$2</span>',
         '<span style="font-family: $1">$2</span>'
     );
-    return nl2br(preg_replace($BBcode,$HTML,$string));
+    return nl2br(preg_replace($BBcode, $HTML, $string));
   }
   //------------------------------------------------
   /**
@@ -294,7 +294,7 @@ 
     * @return int Cantidad de palabras
   */
   final public static function count_words(string $s) : int {
-    return (int) str_word_count($s,0);
+    return (int) str_word_count($s, 0);
   }
   //------------------------------------------------
   /**

Ocrend/Kernel/Helpers/Functions.php

Spacing +38 added lines, -38 removed lines

@@ -29,7 +29,7 @@ 
       * @return bool con el resultado de la comparación
    */
   final private function check_str_to_time(int $index, array $detail, int $max) : bool {
-    return !array_key_exists($index,$detail) || !is_numeric($detail[$index]) || intval($detail[$index]) < $max;
+    return !array_key_exists($index, $detail) || !is_numeric($detail[$index]) || intval($detail[$index]) < $max;
   }
 
    //------------------------------------------------
@@ -45,7 +45,7 @@ 
   final public function redir($url = null) {
     global $config;
     
-    if(null == $url) {
+    if (null == $url) {
       $url = $config['site']['url'];
     }
     
@@ -64,7 +64,7 @@ 
     * @return float con el porcentaje correspondiente
   */
   final public function percent(float $por, float $n) : float {
-    return $n * ($por / 100);
+    return $n*($por/100);
   }
 
   //------------------------------------------------
@@ -77,8 +77,8 @@ 
     * @return string del tamaño $size convertido a la unidad más adecuada
   */
   final public function convert(int $size) : string {
-      $unit = array('bytes','kb','mb','gb','tb','pb');
-      return round($size/pow(1024,($i=floor(log($size,1024)))),2).' '.$unit[$i];
+      $unit = array('bytes', 'kb', 'mb', 'gb', 'tb', 'pb');
+      return round($size/pow(1024, ($i = floor(log($size, 1024)))), 2) . ' ' . $unit[$i];
   }
 
   //------------------------------------------------
@@ -104,7 +104,7 @@ 
      * @return bool con true si está vacío, false si no, un espacio en blanco cuenta como vacío
    */
    final public function emp($var) : bool {
-     return (isset($var) && empty(trim(str_replace(' ','',$var))));
+     return (isset($var) && empty(trim(str_replace(' ', '', $var))));
    }
 
    //------------------------------------------------
@@ -118,8 +118,8 @@ 
      * @return bool con true si están todos llenos, false si al menos uno está vacío
    */
    final public function all_full(array $array) : bool {
-     foreach($array as $e) {
-       if($this->emp($e) and $e != '0') {
+     foreach ($array as $e) {
+       if ($this->emp($e) and $e != '0') {
          return false;
        }
      }
@@ -135,7 +135,7 @@ 
    */
     final public function e() : bool  {
       for ($i = 0, $nargs = func_num_args(); $i < $nargs; $i++) {
-        if(null === func_get_arg($i) || ($this->emp(func_get_arg($i)) && func_get_arg($i) != '0')) {
+        if (null === func_get_arg($i) || ($this->emp(func_get_arg($i)) && func_get_arg($i) != '0')) {
           return true;
         }
       }
@@ -153,7 +153,7 @@ 
       * @return string con la fecha en formato humano (y en español)
     */
     final public function fecha(string $format, int $time = 0) : string  {
-       $date = date($format,$time == 0 ? time() : $time);
+       $date = date($format, $time == 0 ? time() : $time);
        $cambios = array(
          'Monday'=> 'Lunes',
          'Tuesday'=> 'Martes',
@@ -186,7 +186,7 @@ 
          'Apr'=> 'Abr',
          'Dec'=> 'Dic'
        );
-       return str_replace(array_keys($cambios),array_values($cambios),$date);
+       return str_replace(array_keys($cambios), array_values($cambios), $date);
      }
 
    //------------------------------------------------
@@ -206,13 +206,13 @@ 
     # Revisar protocolo
     $base = $config['site']['router']['protocol'] . '://';
 
-    if(strtolower($www) == 'www') {
+    if (strtolower($www) == 'www') {
       $base .= 'www.' . $config['site']['router']['path'];
     } else {
       $base .= $config['site']['router']['path'];
     }
   
-    return '<base href="'.$base.'" />';
+    return '<base href="' . $base . '" />';
   }
   
   //------------------------------------------------
@@ -226,7 +226,7 @@ 
    * @return string con el número del día
   */
   final public function last_day_month(int $mes, int $anio) : string {
-    return date('d', (mktime(0,0,0,$mes + 1, 1, $anio) - 1));
+    return date('d', (mktime(0, 0, 0, $mes + 1, 1, $anio) - 1));
   }
 
   //------------------------------------------------
@@ -239,7 +239,7 @@ 
    * @return string cifra con cero a la izquirda
   */
   final public function cero_izq(int $num) : string {
-    if($num < 10) {
+    if ($num < 10) {
       return '0' . $num;
     }
 
@@ -257,24 +257,24 @@ 
     * @return int|null con el timestamp
   */
   final public function str_to_time($fecha, string $hora = '00:00:00') {
-    if(null == $fecha) {
+    if (null == $fecha) {
       return null;
     }
     
-    $detail = explode('/',$fecha);
+    $detail = explode('/', $fecha);
 
     // Formato de día incorrecto
-    if($this->check_str_to_time(0,$detail,1)) {
+    if ($this->check_str_to_time(0, $detail, 1)) {
       return null;
     }
 
     // Formato de mes incorrecto
-    if($this->check_str_to_time(1,$detail,1) || intval($detail[1]) > 12) {
+    if ($this->check_str_to_time(1, $detail, 1) || intval($detail[1]) > 12) {
       return null;
     }
 
     // Formato del año
-    if($this->check_str_to_time(2,$detail,1970)) {
+    if ($this->check_str_to_time(2, $detail, 1970)) {
       return null;
     }
 
@@ -284,7 +284,7 @@ 
     $year = intval($detail[2]);
 
     // Veriricar dia según mes
-    if($day > $this->last_day_month($month,$year)) {
+    if ($day > $this->last_day_month($month, $year)) {
       return null;
     }
 
@@ -302,11 +302,11 @@ 
   */
   final public function desde_date(int $desde) {
      # Obtener esta fecha
-     $hoy = date('d/m/Y/D',time());
-     $hoy = explode('/',$hoy);
+     $hoy = date('d/m/Y/D', time());
+     $hoy = explode('/', $hoy);
 
 
-    switch($desde) {
+    switch ($desde) {
       # Hoy
       case 1:
         return date('d/m/Y', time());
@@ -318,7 +318,7 @@ 
       # Semana
       case 3:
         # Día de la semana actual
-        switch($hoy[3]) {
+        switch ($hoy[3]) {
           case 'Mon':
             $dia = $hoy[0];
           break;
@@ -343,21 +343,21 @@ 
         }
 
         # Mes anterior y posiblemente, año también.
-        if($dia == 0) {
+        if ($dia == 0) {
           # Verificamos si estamos en enero
-          if($hoy[1] == 1) {
-            return $this->last_day_month($hoy[1],$hoy[2]) .'/'. $this->cero_izq($hoy[1] - 1) .'/' . ($hoy[2] - 1);
+          if ($hoy[1] == 1) {
+            return $this->last_day_month($hoy[1], $hoy[2]) . '/' . $this->cero_izq($hoy[1] - 1) . '/' . ($hoy[2] - 1);
           }
           
           # Si no es enero, es el año actual
-          return $this->last_day_month($hoy[1],$hoy[2]) .'/'. $this->cero_izq($hoy[1] - 1) .'/' . $hoy[2];
+          return $this->last_day_month($hoy[1], $hoy[2]) . '/' . $this->cero_izq($hoy[1] - 1) . '/' . $hoy[2];
         }
         
-        return $this->cero_izq($dia) .'/'. $this->cero_izq($hoy[1]) .'/' . $hoy[2];
+        return $this->cero_izq($dia) . '/' . $this->cero_izq($hoy[1]) . '/' . $hoy[2];
 
       # Mes
       case 4:
-        return '01/'. $this->cero_izq($hoy[1]) .'/' . $hoy[2];
+        return '01/' . $this->cero_izq($hoy[1]) . '/' . $hoy[2];
        
       # Año
       case 5:
@@ -396,13 +396,13 @@ 
        new \Twig_Function('e_dynamic', array($this, 'e')),
        new \Twig_Function('all_full', array($this, 'all_full')),
        new \Twig_Function('fecha', array($this, 'fecha')),
-       new \Twig_Function('base_assets',array($this, 'base_assets')),
-       new \Twig_Function('timestamp',array($this, 'timestamp')),
-       new \Twig_Function('desde_date',array($this, 'desde_date')),
-       new \Twig_Function('cero_izq',array($this, 'cero_izq')),
-       new \Twig_Function('last_day_month',array($this, 'last_day_month')),
-       new \Twig_Function('str_to_time',array($this, 'str_to_time')),
-       new \Twig_Function('desde_date',array($this, 'desde_date'))
+       new \Twig_Function('base_assets', array($this, 'base_assets')),
+       new \Twig_Function('timestamp', array($this, 'timestamp')),
+       new \Twig_Function('desde_date', array($this, 'desde_date')),
+       new \Twig_Function('cero_izq', array($this, 'cero_izq')),
+       new \Twig_Function('last_day_month', array($this, 'last_day_month')),
+       new \Twig_Function('str_to_time', array($this, 'str_to_time')),
+       new \Twig_Function('desde_date', array($this, 'desde_date'))
      );
    }
 

Ocrend/Kernel/Helpers/Files.php

Spacing +16 added lines, -16 removed lines

@@ -48,7 +48,7 @@ 
     * @return int catidad de bytes escritos en el archivo
   */
   final public static function write_file(string $dir, string $content) : int {
-    $f = new \SplFileObject($dir,'w');
+    $f = new \SplFileObject($dir, 'w');
     return (int) $f->fwrite($content);
   }
 
@@ -62,7 +62,7 @@ 
     * @return bool true si borró el fichero, false si no (porque no existía)
   */
   final public static function delete_file(string $route) : bool {
-    if(file_exists($route)) {
+    if (file_exists($route)) {
       unlink($route);
 
       return true;
@@ -94,7 +94,7 @@ 
     * @return bool true si es una imagen, false si no lo es
   */
   final public static function is_image(string $file_name) : bool {
-    return (bool) in_array(self::get_file_ext($file_name),['jpg','png','jpeg','gif','JPG','PNG','JPEG','GIF']);
+    return (bool) in_array(self::get_file_ext($file_name), ['jpg', 'png', 'jpeg', 'gif', 'JPG', 'PNG', 'JPEG', 'GIF']);
   }
 
   //------------------------------------------------
@@ -135,7 +135,7 @@ 
   */
   final public static function get_files_in_dir(string $dir, string $types = '') : array {
     $array = array();
-    if(is_dir($dir)) {
+    if (is_dir($dir)) {
       foreach (glob($dir . '*' . $types) as $file) {
         $array[] = $file;
       }
@@ -154,11 +154,11 @@ 
     * @return bool con true si fue creado con éxito, false si el directorio ya existía o hubo algún error
   */
   final public static function create_dir(string $dir, int $permisos = 0755) : bool {
-    if(is_dir($dir)) {
+    if (is_dir($dir)) {
       return false;
     }
     
-    return (bool) mkdir($dir,$permisos,true);
+    return (bool) mkdir($dir, $permisos, true);
   }
 
   //------------------------------------------------
@@ -175,7 +175,7 @@ 
   final public static function rm_dir(string $dir) {
 
     # Evitar una desgracia
-    if(in_array($dir,[
+    if (in_array($dir, [
       'Ocrend/',
       'Ocrend/Kernel/',
       'Ocrend/vendor/',
@@ -188,10 +188,10 @@ 
       throw new \RuntimeException('No puede eliminar la ruta ' . $dir . ' ya que es crítica.');
     }
 
-    foreach(glob($dir . "/*") as $archivos_carpeta)   { 
-        if (is_dir($archivos_carpeta))   {
+    foreach (glob($dir . "/*") as $archivos_carpeta) { 
+        if (is_dir($archivos_carpeta)) {
             self::rm_dir($archivos_carpeta);
-        } else  {
+        } else {
             unlink($archivos_carpeta);
         }
     }
@@ -210,7 +210,7 @@ 
     * @return int cantidad de  imágenes
   */
   final public static function images_in_dir(string $dir) : int {
-    return sizeof(glob($dir . '{*.jpg,*.gif,*.png,*.gif,*.jpeg,*.JPG,*.GIF,*.PNG,*.JPEG}',GLOB_BRACE));
+    return sizeof(glob($dir . '{*.jpg,*.gif,*.png,*.gif,*.jpeg,*.JPG,*.GIF,*.PNG,*.JPEG}', GLOB_BRACE));
   }
 
   //------------------------------------------------
@@ -230,16 +230,16 @@ 
 
     self::create_dir($new_dir);
 
-    foreach(glob($old_dir . ($only_images ? '{*.jpg,*.gif,*.png,*.gif,*.jpeg,*.JPG,*.GIF,*.PNG,*.JPEG}' : '*'),GLOB_BRACE) as $file) {
-      if(file_exists($file)) {
+    foreach (glob($old_dir . ($only_images ? '{*.jpg,*.gif,*.png,*.gif,*.jpeg,*.JPG,*.GIF,*.PNG,*.JPEG}' : '*'), GLOB_BRACE) as $file) {
+      if (file_exists($file)) {
           unlink($file);
       }
       
-      $name = explode('/',$file);
+      $name = explode('/', $file);
       $name = end($name);
-      copy($file,$new_dir . $name);
+      copy($file, $new_dir . $name);
 
-      if($delete_old) {
+      if ($delete_old) {
         unlink($file);
       }
     }