prinick96 / Ocrend-Framework

app/models/Users.php

Spacing +15 added lines, -15 removed lines

@@ -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);
@@ -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());
         }
     }
@@ -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();
             }
 

Ocrend/Kernel/Models/Traits/DBModel.php

Spacing +6 added lines, -6 removed lines

@@ -55,16 +55,16 @@
         $this->databaseConfig['new_instance'] = false;
 
         # Añadir según lo pasado por $databaseConfig
-        if(is_array($databaseConfig)) {
-            if(array_key_exists('name',$databaseConfig)) {
-                $this->databaseConfig['name'] =  $databaseConfig['name'];
+        if (is_array($databaseConfig)) {
+            if (array_key_exists('name', $databaseConfig)) {
+                $this->databaseConfig['name'] = $databaseConfig['name'];
             } 
 
-            if(array_key_exists('motor',$databaseConfig)) {
-                $this->databaseConfig['motor'] =  $databaseConfig['motor'];
+            if (array_key_exists('motor', $databaseConfig)) {
+                $this->databaseConfig['motor'] = $databaseConfig['motor'];
             } 
 
-            if(array_key_exists('new_instance',$databaseConfig)) {
+            if (array_key_exists('new_instance', $databaseConfig)) {
                 $this->databaseConfig['new_instance'] = (bool) $databaseConfig['new_instance'];
             }
         }

Ocrend/Kernel/Models/Models.php

Spacing +1 added lines, -1 removed lines

@@ -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');
         }
     }

Ocrend/Kernel/Helpers/Strings.php

Spacing +5 added lines, -5 removed lines

@@ -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);
   }
   //------------------------------------------------
   /**
@@ -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()));
   }
   //------------------------------------------------
   /**
@@ -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 +21 added lines, -21 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;
   }
 
    //------------------------------------------------
@@ -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',
@@ -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;
     }
 
@@ -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());
@@ -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:

Ocrend/Kernel/Helpers/Files.php

Spacing +8 added lines, -8 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);
   }
 
@@ -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);
   }
 
   //------------------------------------------------
@@ -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);
       }
     }

Ocrend/Kernel/Database/Database.php

Spacing +10 added lines, -10 removed lines

@@ -72,49 +72,49 @@
     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