Passed
Branch master (0d61a2)
by Brayan
02:00
created
Ocrend/Kernel/Models/Models.php 2 patches
Indentation   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -23,32 +23,32 @@  discard block
 block discarded – undo
23 23
 abstract class Models {
24 24
 
25 25
     /**
26
-      * Tiene siempre el id pasado por la ruta, en caso de no haber ninguno, será cero.
27
-      *
28
-      * @var int 
29
-    */
26
+     * Tiene siempre el id pasado por la ruta, en caso de no haber ninguno, será cero.
27
+     *
28
+     * @var int 
29
+     */
30 30
     protected $id = 0;
31 31
 
32 32
     /**
33
-      * Contiene una instancia del helper para funciones
34
-      *
35
-      * @var \Ocrend\Kernel\Helpers\Functions
36
-    */
33
+     * Contiene una instancia del helper para funciones
34
+     *
35
+     * @var \Ocrend\Kernel\Helpers\Functions
36
+     */
37 37
     protected $functions;
38 38
 
39 39
     /**
40
-      * Contiene el id del usuario que tiene su sesión iniciada.
41
-      *
42
-      * @var int|null con id del usuario
43
-    */
40
+     * Contiene el id del usuario que tiene su sesión iniciada.
41
+     *
42
+     * @var int|null con id del usuario
43
+     */
44 44
     protected $id_user = null;
45 45
 
46 46
     /**
47
-      * Inicia la configuración inicial de cualquier modelo
48
-      *
49
-      * @param IRouter $router: Instancia de un Router 
50
-      *                                    
51
-    */
47
+     * Inicia la configuración inicial de cualquier modelo
48
+     *
49
+     * @param IRouter $router: Instancia de un Router 
50
+     *                                    
51
+     */
52 52
     protected function __construct(IRouter $router = null) {
53 53
         global $session, $config;
54 54
         
@@ -63,19 +63,19 @@  discard block
 block discarded – undo
63 63
 
64 64
         # Verificar sesión del usuario
65 65
         if(null !== $session->get($config['sessions']['unique'] . '_user_id')) {
66
-           $this->id_user = $session->get($config['sessions']['unique'] . '_user_id');
66
+            $this->id_user = $session->get($config['sessions']['unique'] . '_user_id');
67 67
         }
68 68
     }
69 69
 
70 70
     /**
71
-      * Asigna el id desde un modelo, ideal para cuando queremos darle un valor numérico 
72
-      * que proviene de un formulario y puede ser inseguro.
73
-      *
74
-      * @param mixed $id : Id a asignar en $this->id
75
-      * @param string $default_msg : Mensaje a mostrar en caso de que no se pueda asignar
76
-      *
77
-      * @throws ModelsException
78
-      */
71
+     * Asigna el id desde un modelo, ideal para cuando queremos darle un valor numérico 
72
+     * que proviene de un formulario y puede ser inseguro.
73
+     *
74
+     * @param mixed $id : Id a asignar en $this->id
75
+     * @param string $default_msg : Mensaje a mostrar en caso de que no se pueda asignar
76
+     *
77
+     * @throws ModelsException
78
+     */
79 79
     protected function setId($id, string $default_msg = 'No puedede asignarse el id.') {
80 80
         if (null == $id || !is_numeric($id) || $id <= 0) {
81 81
             throw new ModelsException($default_msg);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@
 block discarded – undo
62 62
         $this->functions = new Functions();
63 63
 
64 64
         # Verificar sesión del usuario
65
-        if(null !== $session->get($config['sessions']['unique'] . '_user_id')) {
65
+        if (null !== $session->get($config['sessions']['unique'] . '_user_id')) {
66 66
            $this->id_user = $session->get($config['sessions']['unique'] . '_user_id');
67 67
         }
68 68
     }
Please login to merge, or discard this patch.
app/models/Users.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
     private function generateSession(array $user_data) {
128 128
         global $session, $config;
129 129
 
130
-        $session->set($config['sessions']['unique'] . '_user_id',(int) $user_data['id_user']);
130
+        $session->set($config['sessions']['unique'] . '_user_id', (int) $user_data['id_user']);
131 131
     }
132 132
 
133 133
     /**
@@ -139,12 +139,12 @@  discard block
 block discarded – undo
139 139
      * @return bool true: Cuando el inicio de sesión es correcto 
140 140
      *              false: Cuando el inicio de sesión no es correcto
141 141
      */
142
-    private function authentication(string $email,string $pass) : bool {
142
+    private function authentication(string $email, string $pass) : bool {
143 143
         $email = $this->db->scape($email);
144
-        $query = $this->db->select('id_user,pass','users',"email='$email'",'LIMIT 1');
144
+        $query = $this->db->select('id_user,pass', 'users', "email='$email'", 'LIMIT 1');
145 145
         
146 146
         # Incio de sesión con éxito
147
-        if(false !== $query && Strings::chash($query[0]['pass'],$pass)) {
147
+        if (false !== $query && Strings::chash($query[0]['pass'], $pass)) {
148 148
 
149 149
             # Restaurar intentos
150 150
             $this->restoreAttempts($email);
@@ -334,30 +334,30 @@  discard block
 block discarded – undo
334 334
             $pass = uniqid();
335 335
 
336 336
             # Construir mensaje y enviar mensaje
337
-            $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.
337
+            $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.
338 338
 					<br />
339 339
 					<br />
340
-					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>.';
340
+					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>.';
341 341
 
342 342
             # Enviar el correo electrónico
343 343
             $dest = array();
344 344
 			$dest[$email] = $user_data[0]['name'];
345
-			$email = Emails::send_mail($dest,Emails::plantilla($HTML),'Recuperar contraseña perdida');
345
+			$email = Emails::send_mail($dest, Emails::plantilla($HTML), 'Recuperar contraseña perdida');
346 346
 
347 347
             # Verificar si hubo algún problema con el envío del correo
348
-            if(false === $email) {
348
+            if (false === $email) {
349 349
                 throw new ModelsException('No se ha podido enviar el correo electrónico.');
350 350
             }
351 351
 
352 352
             # Actualizar datos 
353 353
             $id_user = $user_data[0]['id_user'];
354
-            $this->db->update('users',array(
354
+            $this->db->update('users', array(
355 355
                 'tmp_pass' => Strings::hash($pass),
356 356
                 'token' => $token
357
-            ),"id_user='$id_user'",'LIMIT 1');
357
+            ), "id_user='$id_user'", 'LIMIT 1');
358 358
 
359 359
             return array('success' => 1, 'message' => 'Se ha enviado un enlace a su correo electrónico.');
360
-        } catch(ModelsException $e) {
360
+        } catch (ModelsException $e) {
361 361
             return array('success' => 0, 'message' => $e->getMessage());
362 362
         }
363 363
     }
@@ -400,7 +400,7 @@  discard block
 block discarded – undo
400 400
     public function logout() {
401 401
         global $session, $config;
402 402
 
403
-        if(null != $session->get($config['sessions']['unique'] . '_user_id')) {
403
+        if (null != $session->get($config['sessions']['unique'] . '_user_id')) {
404 404
             $session->remove($config['sessions']['unique'] . '_user_id');
405 405
         }
406 406
 
@@ -416,7 +416,7 @@  discard block
 block discarded – undo
416 416
      * @return false|array con información del usuario
417 417
      */   
418 418
     public function getUserById(int $id, string $select = '*') {
419
-        return $this->db->select($select,'users',"id_user='$id'",'LIMIT 1');
419
+        return $this->db->select($select, 'users', "id_user='$id'", 'LIMIT 1');
420 420
     }
421 421
     
422 422
     /**
@@ -427,7 +427,7 @@  discard block
 block discarded – undo
427 427
      * @return false|array con información de los usuarios
428 428
      */  
429 429
     public function getUsers(string $select = '*') {
430
-        return $this->db->select($select,'users');
430
+        return $this->db->select($select, 'users');
431 431
     }
432 432
 
433 433
     /**
@@ -439,12 +439,12 @@  discard block
 block discarded – undo
439 439
      * @return array con datos del usuario conectado
440 440
      */
441 441
     public function getOwnerUser(string $select = '*') : array {
442
-        if(null !== $this->id_user) {    
442
+        if (null !== $this->id_user) {    
443 443
                
444
-            $user = $this->db->select($select,'users',"id_user='$this->id_user'",'LIMIT 1');
444
+            $user = $this->db->select($select, 'users', "id_user='$this->id_user'", 'LIMIT 1');
445 445
 
446 446
             # Si se borra al usuario desde la base de datos y sigue con la sesión activa
447
-            if(false === $user) {
447
+            if (false === $user) {
448 448
                 $this->logout();
449 449
             }
450 450
 
Please login to merge, or discard this patch.