prinick96 / Ocrend-Framework

Ocrend/Kernel/Models/Models.php

Indentation +26 added lines, -26 removed lines

@@ -23,32 +23,32 @@ 
 abstract class Models {
 
     /**
-      * Tiene siempre el id pasado por la ruta, en caso de no haber ninguno, será cero.
-      *
-      * @var int 
-    */
+     * Tiene siempre el id pasado por la ruta, en caso de no haber ninguno, será cero.
+     *
+     * @var int 
+     */
     protected $id = 0;
 
     /**
-      * Contiene una instancia del helper para funciones
-      *
-      * @var \Ocrend\Kernel\Helpers\Functions
-    */
+     * Contiene una instancia del helper para funciones
+     *
+     * @var \Ocrend\Kernel\Helpers\Functions
+     */
     protected $functions;
 
     /**
-      * Contiene el id del usuario que tiene su sesión iniciada.
-      *
-      * @var int|null con id del usuario
-    */
+     * Contiene el id del usuario que tiene su sesión iniciada.
+     *
+     * @var int|null con id del usuario
+     */
     protected $id_user = null;
 
     /**
-      * Inicia la configuración inicial de cualquier modelo
-      *
-      * @param IRouter $router: Instancia de un Router 
-      *                                    
-    */
+     * Inicia la configuración inicial de cualquier modelo
+     *
+     * @param IRouter $router: Instancia de un Router 
+     *                                    
+     */
     protected function __construct(IRouter $router = null) {
         global $session, $config;
         
@@ -64,19 +64,19 @@ 
         # Verificar sesión del usuario
         $session_name = $config['sessions']['unique'] . '_user_id';
         if(null !== $session->get($session_name)) {
-           $this->id_user = $session->get($session_name);
+            $this->id_user = $session->get($session_name);
         }
     }
 
     /**
-      * Asigna el id desde un modelo, ideal para cuando queremos darle un valor numérico 
-      * que proviene de un formulario y puede ser inseguro.
-      *
-      * @param mixed $id : Id a asignar en $this->id
-      * @param string $default_msg : Mensaje a mostrar en caso de que no se pueda asignar
-      *
-      * @throws ModelsException
-      */
+     * Asigna el id desde un modelo, ideal para cuando queremos darle un valor numérico 
+     * que proviene de un formulario y puede ser inseguro.
+     *
+     * @param mixed $id : Id a asignar en $this->id
+     * @param string $default_msg : Mensaje a mostrar en caso de que no se pueda asignar
+     *
+     * @throws ModelsException
+     */
     protected function setId($id, string $default_msg = 'No puedede asignarse el id.') {
         if (null == $id || !is_numeric($id) || $id <= 0) {
             throw new ModelsException($default_msg);

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         
         # Verificar sesión del usuario
         $session_name = $config['sessions']['unique'] . '_user_id';
-        if(null !== $session->get($session_name)) {
+        if (null !== $session->get($session_name)) {
            $this->id_user = $session->get($session_name);
         }
     }

app/models/Users.php

Indentation +7 added lines, -7 removed lines

@@ -301,11 +301,11 @@ 
     }
     
     /**
-      * Envía un correo electrónico al usuario que quiere recuperar la contraseña, con un token y una nueva contraseña.
-      * Si el usuario no visita el enlace, el sistema no cambiará la contraseña.
-      *
-      * @return array<string,integer|string>
-    */  
+     * Envía un correo electrónico al usuario que quiere recuperar la contraseña, con un token y una nueva contraseña.
+     * Si el usuario no visita el enlace, el sistema no cambiará la contraseña.
+     *
+     * @return array<string,integer|string>
+     */  
     public function lostpass() {
         try {
             global $http, $config;
@@ -342,7 +342,7 @@ 
 
             # Enviar el correo electrónico
             $dest = array();
-			$dest[$email] = $user_data[0]['name'];
+      $dest[$email] = $user_data[0]['name'];
             $email_send = Emails::send($dest,array(
                 # Título del mensaje
                 '{{title}}' => 'Recuperar contraseña de ' . $config['site']['name'],
@@ -416,7 +416,7 @@ 
     public function logout() {
         global $session, $config;
 	    
-	$session_name = $config['sessions']['unique'] . '_user_id';
+  $session_name = $config['sessions']['unique'] . '_user_id';
 	    
         if(null != $session->get($session_name)) {
             $session->remove($session_name);

Spacing +20 added lines, -20 removed lines

@@ -127,7 +127,7 @@ 
     private function generateSession(array $user_data) {
         global $session, $config;
 
-        $session->set($config['sessions']['unique'] . '_user_id',(int) $user_data['id_user']);
+        $session->set($config['sessions']['unique'] . '_user_id', (int) $user_data['id_user']);
     }
 
     /**
@@ -139,12 +139,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);
@@ -332,18 +332,18 @@ 
             # Generar token y contraseña 
             $token = md5(time());
             $pass = uniqid();
-            $link = $config['site']['url'] . 'lostpass/cambiar/&token='.$token.'&user='.$user_data[0]['id_user'];
+            $link = $config['site']['url'] . 'lostpass/cambiar/&token=' . $token . '&user=' . $user_data[0]['id_user'];
 
             # 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="'. $link .'" target="_blank">clic aquí</a> o en el botón de recuperar.';
+					Para cambiar su contraseña por <b>'. $pass . '</b> haga <a href="' . $link . '" target="_blank">clic aquí</a> o en el botón de recuperar.';
 
             # Enviar el correo electrónico
             $dest = array();
 			$dest[$email] = $user_data[0]['name'];
-            $email_send = Emails::send($dest,array(
+            $email_send = Emails::send($dest, array(
                 # Título del mensaje
                 '{{title}}' => 'Recuperar contraseña de ' . $config['site']['name'],
                 # Url de logo
@@ -357,23 +357,23 @@ 
                 # Texto del boton
                 '{{btn-name}}' => 'Recuperar Contraseña',
                 # Copyright
-                '{{copyright}}' => '&copy; '.date('Y') .' <a href="'.$config['site']['url'].'">'.$config['site']['name'].'</a> - Todos los derechos reservados.'
-              ),0);
+                '{{copyright}}' => '&copy; ' . date('Y') . ' <a href="' . $config['site']['url'] . '">' . $config['site']['name'] . '</a> - Todos los derechos reservados.'
+              ), 0);
 
             # 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());
         }
     }
@@ -418,7 +418,7 @@ 
 	    
 	$session_name = $config['sessions']['unique'] . '_user_id';
 	    
-        if(null != $session->get($session_name)) {
+        if (null != $session->get($session_name)) {
             $session->remove($session_name);
         }
 
@@ -434,7 +434,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');
     }
     
     /**
@@ -445,7 +445,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');
     }
 
     /**
@@ -457,12 +457,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();
             }