Passed
Branch beta (bd6a7b)
by Brayan
02:19
created
Ocrend/Kernel/Config/Config.php 1 patch
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -23,17 +23,17 @@
 block discarded – undo
23 23
 final class Config {
24 24
 
25 25
     /**
26
-      * Ruta con parámetros de configuración
27
-      *
28
-      * @var string   
29
-    */
26
+     * Ruta con parámetros de configuración
27
+     *
28
+     * @var string   
29
+     */
30 30
     const FILE_CONFIG_ROUTE = API_INTERFACE . 'Ocrend/Kernel/Config/Ocrend.ini.yml';
31 31
 
32 32
     /**
33
-      * Lee la configuración del archivo self::FILE_CONFIG_ROUTE 
34
-      *
35
-      * @return array : Arreglo con la configuración en el archivo .yml
36
-    */
33
+     * Lee la configuración del archivo self::FILE_CONFIG_ROUTE 
34
+     *
35
+     * @return array : Arreglo con la configuración en el archivo .yml
36
+     */
37 37
     final public function readConfig() : array {
38 38
         return Yaml::parse(file_get_contents(self::FILE_CONFIG_ROUTE));
39 39
     }
Please login to merge, or discard this patch.
Ocrend/Kernel/Config/Start.php 2 patches
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 //---------------------------------------------------------------------------------------------
28 28
 
29 29
 /**
30
-  * Manejador de excepciones y errores
30
+ * Manejador de excepciones y errores
31 31
 */
32 32
 ErrorHandler::register();
33 33
 ExceptionHandler::register();  
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 //---------------------------------------------------------------------------------------------
36 36
 
37 37
 /**
38
-  * Versión mínima de PHP
38
+ * Versión mínima de PHP
39 39
 */
40 40
 if (version_compare(phpversion(), '7.0.0', '<')) {
41 41
   throw new \RuntimeException('La versión actual de PHP es ' . phpversion() . ' y como mínimo se require la versión 7.0.0');
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 //---------------------------------------------------------------------------------------------
45 45
 
46 46
 /**
47
-  * Lectura y escritura de caché para Twig
47
+ * Lectura y escritura de caché para Twig
48 48
 */
49 49
 if(!is_writable(API_INTERFACE . 'app/templates/.cache/') || !is_readable(API_INTERFACE . 'app/templates/.cache/')) {
50 50
   throw new \RuntimeException('Debe conceder permisos de escritura y lectura a la ruta ' . API_INTERFACE . 'app/templates/.cache/ ó crearla si no existe.');
@@ -53,15 +53,15 @@  discard block
 block discarded – undo
53 53
 //---------------------------------------------------------------------------------------------
54 54
 
55 55
 /**
56
-  * Obtiene la configuración inicial del sistema, conexión a la base de datos,
57
-  * constantes de phpmailer, credenciales de la api de paypal, etc.
56
+ * Obtiene la configuración inicial del sistema, conexión a la base de datos,
57
+ * constantes de phpmailer, credenciales de la api de paypal, etc.
58 58
 */
59 59
 $config = (new Config)->readConfig();
60 60
 
61 61
 //---------------------------------------------------------------------------------------------
62 62
 
63 63
 /**
64
-  * Capa orientada a objetos para el uso de sesiones más seguras en PHP
64
+ * Capa orientada a objetos para el uso de sesiones más seguras en PHP
65 65
 */
66 66
 $session = new Session(new NativeSessionStorage(
67 67
   array(
@@ -73,14 +73,14 @@  discard block
 block discarded – undo
73 73
 //---------------------------------------------------------------------------------------------
74 74
 
75 75
 /**
76
-  * Capa orientada a objetos de HTTPFoundation
76
+ * Capa orientada a objetos de HTTPFoundation
77 77
 */
78 78
 $http = Request::createFromGlobals();
79 79
 
80 80
 //---------------------------------------------------------------------------------------------
81 81
 
82 82
 /**
83
-  * Estatus del DEBUG
83
+ * Estatus del DEBUG
84 84
 */
85 85
 if($config['framework']['debug']) {
86 86
   Debug::enable();
@@ -89,6 +89,6 @@  discard block
 block discarded – undo
89 89
 //---------------------------------------------------------------------------------------------
90 90
 
91 91
 /**
92
-  * Define el timezone actual
92
+ * Define el timezone actual
93 93
 */
94 94
 date_default_timezone_set($config['site']['timezone']);
95 95
\ No newline at end of file
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 /**
47 47
   * Lectura y escritura de caché para Twig
48 48
 */
49
-if(!is_writable(API_INTERFACE . 'app/templates/.cache/') || !is_readable(API_INTERFACE . 'app/templates/.cache/')) {
49
+if (!is_writable(API_INTERFACE . 'app/templates/.cache/') || !is_readable(API_INTERFACE . 'app/templates/.cache/')) {
50 50
   throw new \RuntimeException('Debe conceder permisos de escritura y lectura a la ruta ' . API_INTERFACE . 'app/templates/.cache/ ó crearla si no existe.');
51 51
 }
52 52
 
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 /**
83 83
   * Estatus del DEBUG
84 84
 */
85
-if($config['framework']['debug']) {
85
+if ($config['framework']['debug']) {
86 86
   Debug::enable();
87 87
 }
88 88
 
Please login to merge, or discard this patch.
Ocrend/Kernel/Controllers/Controllers.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         $this->template->addGlobal('is_logged', $this->is_logged);
115 115
 
116 116
         # Datos del usuario actual
117
-        if($this->is_logged) {
117
+        if ($this->is_logged) {
118 118
           $this->user = (new Model\Users)->getOwnerUser();
119 119
           $this->template->addGlobal('owner_user', $this->user);
120 120
         }
@@ -145,17 +145,17 @@  discard block
 block discarded – undo
145 145
       $this->controllerConfig['users_not_logged'] = false;
146 146
 
147 147
       # Establecer las configuraciones pasadas
148
-      if(null != $config) {
148
+      if (null != $config) {
149 149
         # Configura el autoreload del caché de twig
150
-        if(array_key_exists('twig_cache_reload',$config)) {
150
+        if (array_key_exists('twig_cache_reload', $config)) {
151 151
           $this->controllerConfig['twig_cache_reload'] = (bool) $config['twig_cache_reload'];
152 152
         }
153 153
         # Configura el controlador para solo ser visto por usuarios logeados
154
-        if(array_key_exists('users_logged',$config)) {
154
+        if (array_key_exists('users_logged', $config)) {
155 155
           $this->controllerConfig['users_logged'] = (bool) $config['users_logged'];
156 156
         }
157 157
         # Configura el controlador para solo ser visto por usuario no logeados
158
-        if(array_key_exists('users_not_logged',$config)) {
158
+        if (array_key_exists('users_not_logged', $config)) {
159 159
           $this->controllerConfig['users_not_logged'] = (bool) $config['users_not_logged'];
160 160
         }
161 161
       }
@@ -168,12 +168,12 @@  discard block
 block discarded – undo
168 168
     */
169 169
     private function knowVisitorPermissions() {
170 170
       # Sólamente usuarios logeados
171
-      if($this->controllerConfig['users_logged'] && !$this->is_logged) {
171
+      if ($this->controllerConfig['users_logged'] && !$this->is_logged) {
172 172
         $this->functions->redir();
173 173
       }
174 174
 
175 175
       # Sólamente usuarios no logeados
176
-      if($this->controllerConfig['users_not_logged'] && $this->is_logged) {
176
+      if ($this->controllerConfig['users_not_logged'] && $this->is_logged) {
177 177
         $this->functions->redir();
178 178
       }
179 179
     }
Please login to merge, or discard this patch.
Indentation   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -25,65 +25,65 @@  discard block
 block discarded – undo
25 25
 abstract class Controllers {
26 26
     
27 27
     /**
28
-      * Obtiene el objeto del template 
29
-      *
30
-      * @var \Twig_Environment
31
-    */
28
+     * Obtiene el objeto del template 
29
+     *
30
+     * @var \Twig_Environment
31
+     */
32 32
     protected $template;
33 33
 
34 34
     /**
35
-      * Verifica si está definida la ruta /id como un integer >= 1
36
-      *
37
-      * @var bool
38
-    */
35
+     * Verifica si está definida la ruta /id como un integer >= 1
36
+     *
37
+     * @var bool
38
+     */
39 39
     protected $isset_id = false;
40 40
 
41 41
     /**
42
-      * Tiene el valor de la ruta /método
43
-      *
44
-      * @var string|null
45
-    */
42
+     * Tiene el valor de la ruta /método
43
+     *
44
+     * @var string|null
45
+     */
46 46
     protected $method;
47 47
 
48 48
     /**
49
-      * Contiene una instancia del helper para funciones
50
-      *
51
-      * @var \Ocrend\Kernel\Helpers\Functions
52
-    */
49
+     * Contiene una instancia del helper para funciones
50
+     *
51
+     * @var \Ocrend\Kernel\Helpers\Functions
52
+     */
53 53
     protected $functions;
54 54
     
55 55
     /**
56
-      * Arreglo con la información del usuario conectado actualmente.
57
-      *
58
-      * @var array 
59
-    */
56
+     * Arreglo con la información del usuario conectado actualmente.
57
+     *
58
+     * @var array 
59
+     */
60 60
     protected $user = array();
61 61
 
62 62
     /**
63
-      * Contiene información sobre el estado del usuario, si está o no conectado.
64
-      *
65
-      * @var bool
66
-    */
63
+     * Contiene información sobre el estado del usuario, si está o no conectado.
64
+     *
65
+     * @var bool
66
+     */
67 67
     private $is_logged = false;
68 68
 
69 69
     /** 
70
-      * Parámetros de configuración para el controlador con la forma:
71
-      * 'parmáetro' => (bool) valor
72
-      *
73
-      * @var array
74
-    */
70
+     * Parámetros de configuración para el controlador con la forma:
71
+     * 'parmáetro' => (bool) valor
72
+     *
73
+     * @var array
74
+     */
75 75
     private $controllerConfig;
76 76
 
77 77
     /**
78
-      * Configuración inicial de cualquier controlador
79
-      *
80
-      * @param IRouter $router: Instancia de un Router
81
-      * @param array|null $configController: Arreglo de configuración con la forma  
82
-      *     'twig_cache_reload' => bool, # Configura el autoreload del caché de twig
83
-      *     'users_logged' => bool, # Configura el controlador para solo ser visto por usuarios logeados
84
-      *     'users_not_logged' => bool, # Configura el controlador para solo ser visto por !(usuarios logeados)
85
-      *
86
-    */
78
+     * Configuración inicial de cualquier controlador
79
+     *
80
+     * @param IRouter $router: Instancia de un Router
81
+     * @param array|null $configController: Arreglo de configuración con la forma  
82
+     *     'twig_cache_reload' => bool, # Configura el autoreload del caché de twig
83
+     *     'users_logged' => bool, # Configura el controlador para solo ser visto por usuarios logeados
84
+     *     'users_not_logged' => bool, # Configura el controlador para solo ser visto por !(usuarios logeados)
85
+     *
86
+     */
87 87
     protected function __construct(IRouter $router, $configController = null) {
88 88
         global $config, $http, $session;
89 89
 
@@ -131,13 +131,13 @@  discard block
 block discarded – undo
131 131
     }
132 132
 
133 133
     /**
134
-      * Establece los parámetros de configuración de un controlador
135
-      *
136
-      * @param IRouter $router: Instancia de un Router
137
-      * @param array|null $config: Arreglo de configuración   
138
-      *
139
-      * @return void
140
-    */
134
+     * Establece los parámetros de configuración de un controlador
135
+     *
136
+     * @param IRouter $router: Instancia de un Router
137
+     * @param array|null $config: Arreglo de configuración   
138
+     *
139
+     * @return void
140
+     */
141 141
     private function setControllerConfig($config) {
142 142
       # Configuración por defecto
143 143
       $this->controllerConfig['twig_cache_reload'] = true;
@@ -162,10 +162,10 @@  discard block
 block discarded – undo
162 162
     }
163 163
     
164 164
     /**
165
-      * Acción que regula quién entra o no al controlador según la configuración
166
-      *
167
-      * @return void
168
-    */
165
+     * Acción que regula quién entra o no al controlador según la configuración
166
+     *
167
+     * @return void
168
+     */
169 169
     private function knowVisitorPermissions() {
170 170
       # Sólamente usuarios logeados
171 171
       if($this->controllerConfig['users_logged'] && !$this->is_logged) {
Please login to merge, or discard this patch.
Ocrend/Kernel/Helpers/Emails.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
     $mail->CharSet = "UTF-8";
36 36
     $mail->Encoding = "quoted-printable";
37 37
 
38
-    if($is_smtp) {
38
+    if ($is_smtp) {
39 39
       $mail->isSMTP();
40 40
       $mail->SMTPAuth = true;
41 41
       $mail->Host = $config['phpmailer']['host'];
@@ -88,13 +88,13 @@  discard block
 block discarded – undo
88 88
     $mail->Subject = $titulo;
89 89
     $mail->Body    = $HTML;
90 90
 
91
-    if(sizeof($adj)) {
91
+    if (sizeof($adj)) {
92 92
       foreach ($adj as $ruta) {
93 93
         $mail->AddAttachment($ruta);
94 94
       }
95 95
     }
96 96
 
97
-    if(!$mail->send()) {
97
+    if (!$mail->send()) {
98 98
       return $mail->ErrorInfo;
99 99
     }
100 100
 
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
     </head>
119 119
     <body style="font-family: Verdana;">
120 120
       <section>
121
-        '.$content.'
121
+        '.$content . '
122 122
       </section>
123 123
     </body>
124 124
     </html>';
Please login to merge, or discard this patch.
Indentation   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -20,14 +20,14 @@  discard block
 block discarded – undo
20 20
 final class Emails {
21 21
 
22 22
   /**
23
-    * FUNCIÓN NO ACCESIBLE, USO ESTRICTO PARA UNA FUNCIÓN INTERNA DEL HELPER
24
-    * Inicializa la clase PHPMailer y las configuraciones necesarias
25
-    * Método privado utilizado en todo el Helper
26
-    *
27
-    * @param bool $is_smtp: Define si se hará la conexión a través de SMTP o no
28
-    *
29
-    * @return \PHPMailer un objeto de la clase PHPMailer
30
-  */
23
+   * FUNCIÓN NO ACCESIBLE, USO ESTRICTO PARA UNA FUNCIÓN INTERNA DEL HELPER
24
+   * Inicializa la clase PHPMailer y las configuraciones necesarias
25
+   * Método privado utilizado en todo el Helper
26
+   *
27
+   * @param bool $is_smtp: Define si se hará la conexión a través de SMTP o no
28
+   *
29
+   * @return \PHPMailer un objeto de la clase PHPMailer
30
+   */
31 31
   final private static function init(bool $is_smtp = true) : \PHPMailer {
32 32
     global $config;
33 33
 
@@ -60,22 +60,22 @@  discard block
 block discarded – undo
60 60
   //------------------------------------------------
61 61
 
62 62
   /**
63
-    * Envía un correo electrónico utilizando PHPMailer
64
-    *
65
-    * @param array $dest: Arreglo con la forma array(
66
-    *                                           'email destinatario 1' => 'nombre destinatario 1',
67
-    *                                           'email destinatario 2' => 'nombre destinatario 2'
68
-    *                                            )
69
-    * @param string $HTML: Contenido en HTML del email
70
-    * @param string $titulo: Asunto del email
71
-    * @param bool $is_smtp: Define si se hará la conexión a través de SMTP o no
72
-    * @param array $adj: Arreglo con direccion local de los adjuntos a enviar, con la forma array(
73
-    *                                                                                       'ruta archivo 1',
74
-    *                                                                                       'ruta archivo 2'
75
-    *                                                                                       )
76
-    *
77
-    * @return string|bool true si fue enviado correctamente, string con el Error descrito por PHPMailer
78
-  */
63
+   * Envía un correo electrónico utilizando PHPMailer
64
+   *
65
+   * @param array $dest: Arreglo con la forma array(
66
+   *                                           'email destinatario 1' => 'nombre destinatario 1',
67
+   *                                           'email destinatario 2' => 'nombre destinatario 2'
68
+   *                                            )
69
+   * @param string $HTML: Contenido en HTML del email
70
+   * @param string $titulo: Asunto del email
71
+   * @param bool $is_smtp: Define si se hará la conexión a través de SMTP o no
72
+   * @param array $adj: Arreglo con direccion local de los adjuntos a enviar, con la forma array(
73
+   *                                                                                       'ruta archivo 1',
74
+   *                                                                                       'ruta archivo 2'
75
+   *                                                                                       )
76
+   *
77
+   * @return string|bool true si fue enviado correctamente, string con el Error descrito por PHPMailer
78
+   */
79 79
   final public static function send_mail(array $dest, string $HTML, string $titulo, bool $is_smtp = true, array $adj = []) {
80 80
     global $config;
81 81
 
@@ -104,12 +104,12 @@  discard block
 block discarded – undo
104 104
   //------------------------------------------------
105 105
 
106 106
   /**
107
-    * Plantilla estándar que muestra de forma amigable el texto, utiliza bootstrap
108
-    *
109
-    * @param string $content: Cadena de texto en HTML, puede ser en bootstrap
110
-    *
111
-    * @return string con el HTML para enviar
112
-  */
107
+   * Plantilla estándar que muestra de forma amigable el texto, utiliza bootstrap
108
+   *
109
+   * @param string $content: Cadena de texto en HTML, puede ser en bootstrap
110
+   *
111
+   * @return string con el HTML para enviar
112
+   */
113 113
   final public static function plantilla(string $content) : string {
114 114
     return '
115 115
     <html>
Please login to merge, or discard this patch.
Ocrend/Kernel/Models/ModelsException.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -20,15 +20,15 @@
 block discarded – undo
20 20
 class ModelsException extends \Exception {
21 21
 
22 22
     /**
23
-      * __construct()
24
-    */
23
+     * __construct()
24
+     */
25 25
     public function __construct($message = null, $code = 1, \Exception $previous = null) {
26 26
         parent::__construct($message, $code, $previous);
27 27
     }
28 28
 
29 29
     /**
30
-      * Muestra el error con un formato u otro dependiendo desde donde se hace la petición.
31
-    */
30
+     * Muestra el error con un formato u otro dependiendo desde donde se hace la petición.
31
+     */
32 32
     public function errorResponse() {
33 33
         throw new \RuntimeException($this->getMessage());
34 34
     }
Please login to merge, or discard this patch.
Ocrend/Kernel/Models/IModels.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  * @author Brayan Narváez <[email protected]>
20 20
  */
21 21
 
22
- interface IModels {
23
-     public function __construct(IRouter $router = null);
24
-     public function __destruct();
25
- }
26 22
\ No newline at end of file
23
+  interface IModels {
24
+      public function __construct(IRouter $router = null);
25
+      public function __destruct();
26
+  }
27 27
\ No newline at end of file
Please login to merge, or discard this patch.
Ocrend/Kernel/Models/Traits/DBModel.php 2 patches
Indentation   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -9,43 +9,43 @@  discard block
 block discarded – undo
9 9
  * file that was distributed with this source code.
10 10
  */
11 11
 
12
- namespace Ocrend\Kernel\Models\Traits;
12
+  namespace Ocrend\Kernel\Models\Traits;
13 13
 
14
- use Ocrend\Kernel\Database\Database;
14
+  use Ocrend\Kernel\Database\Database;
15 15
  
16 16
 /**
17
-  * Añade características a un modelo para que pueda conectarse a una base de datos.
18
-  *
19
-  * @author Brayan Narváez <[email protected]>
17
+ * Añade características a un modelo para que pueda conectarse a una base de datos.
18
+ *
19
+ * @author Brayan Narváez <[email protected]>
20 20
 */
21 21
 
22 22
 trait DBModel {
23 23
 
24 24
     /**
25
-      * Tiene la instancia de la base de datos actual
26
-      *
27
-      * @var null|Database
28
-    */
25
+     * Tiene la instancia de la base de datos actual
26
+     *
27
+     * @var null|Database
28
+     */
29 29
     protected $db = null;
30 30
 
31 31
     /**
32
-      * Contiene la información que se pasa al manejador de la base de datos. 
33
-      * - Nombre de base de datos
34
-      * - Motor de base de datos 
35
-      * - Valor de nueva instancia
36
-      *
37
-      * @var array
38
-    */
32
+     * Contiene la información que se pasa al manejador de la base de datos. 
33
+     * - Nombre de base de datos
34
+     * - Motor de base de datos 
35
+     * - Valor de nueva instancia
36
+     *
37
+     * @var array
38
+     */
39 39
     private $databaseConfig = array();
40 40
 
41 41
     /**
42
-      * Establece la configuración de la base de datos
43
-      *
44
-      * @param array|null $databaseConfig: Configuración de conexión con base de datos con la forma
45
-      *     'name' => string, # Nombre de la base de datos
46
-      *     'motor' => string, # Motor de la base de datos
47
-      *     'new_instance' => bool, # Establecer nueva instancia distinta a alguna ya existente
48
-    */
42
+     * Establece la configuración de la base de datos
43
+     *
44
+     * @param array|null $databaseConfig: Configuración de conexión con base de datos con la forma
45
+     *     'name' => string, # Nombre de la base de datos
46
+     *     'motor' => string, # Motor de la base de datos
47
+     *     'new_instance' => bool, # Establecer nueva instancia distinta a alguna ya existente
48
+     */
49 49
     private function setDatabaseConfig($databaseConfig) {
50 50
         global $config;
51 51
 
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         # Añadir según lo pasado por $databaseConfig
58 58
         if(is_array($databaseConfig)) {
59 59
             if(array_key_exists('name',$databaseConfig)) {
60
-               $this->databaseConfig['name'] =  $databaseConfig['name'];
60
+                $this->databaseConfig['name'] =  $databaseConfig['name'];
61 61
             } 
62 62
 
63 63
             if(array_key_exists('motor',$databaseConfig)) {
@@ -71,13 +71,13 @@  discard block
 block discarded – undo
71 71
     }
72 72
 
73 73
     /**
74
-      * Constructor inicial del modelo.
75
-      *
76
-      * @param array|null $databaseConfig: Configuración de conexión con base de datos con la forma
77
-      *     'name' => string, # Nombre de la base de datos
78
-      *     'motor' => string, # Motor de la base de datos
79
-      *     'new_instance' => bool, # Establecer nueva instancia distinta a alguna ya existente
80
-    */
74
+     * Constructor inicial del modelo.
75
+     *
76
+     * @param array|null $databaseConfig: Configuración de conexión con base de datos con la forma
77
+     *     'name' => string, # Nombre de la base de datos
78
+     *     'motor' => string, # Motor de la base de datos
79
+     *     'new_instance' => bool, # Establecer nueva instancia distinta a alguna ya existente
80
+     */
81 81
     protected function startDBConexion($databaseConfig = null) {
82 82
         # Llenar la configuración a la base de datos
83 83
         $this->setDatabaseConfig($databaseConfig);
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
     }
92 92
 
93 93
     /**
94
-      * Finaliza la conexión con la base de datos.
95
-    */
94
+     * Finaliza la conexión con la base de datos.
95
+     */
96 96
     protected function endDBConexion() {
97 97
         $this->db = null;
98 98
     }
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -55,16 +55,16 @@
 block discarded – undo
55 55
         $this->databaseConfig['new_instance'] = false;
56 56
 
57 57
         # Añadir según lo pasado por $databaseConfig
58
-        if(is_array($databaseConfig)) {
59
-            if(array_key_exists('name',$databaseConfig)) {
60
-               $this->databaseConfig['name'] =  $databaseConfig['name'];
58
+        if (is_array($databaseConfig)) {
59
+            if (array_key_exists('name', $databaseConfig)) {
60
+               $this->databaseConfig['name'] = $databaseConfig['name'];
61 61
             } 
62 62
 
63
-            if(array_key_exists('motor',$databaseConfig)) {
64
-                $this->databaseConfig['motor'] =  $databaseConfig['motor'];
63
+            if (array_key_exists('motor', $databaseConfig)) {
64
+                $this->databaseConfig['motor'] = $databaseConfig['motor'];
65 65
             } 
66 66
 
67
-            if(array_key_exists('new_instance',$databaseConfig)) {
67
+            if (array_key_exists('new_instance', $databaseConfig)) {
68 68
                 $this->databaseConfig['new_instance'] = (bool) $databaseConfig['new_instance'];
69 69
             }
70 70
         }
Please login to merge, or discard this patch.
app/models/Users.php 2 patches
Indentation   +127 added lines, -127 removed lines patch added patch discarded remove patch
@@ -28,36 +28,36 @@  discard block
 block discarded – undo
28 28
 
29 29
 class Users extends Models implements IModels {
30 30
     /**
31
-      * Característica para establecer conexión con base de datos. 
32
-    */
31
+     * Característica para establecer conexión con base de datos. 
32
+     */
33 33
     use DBModel;
34 34
 
35 35
     /**
36
-      * Máximos intentos de inincio de sesión de un usuario
37
-      *
38
-      * @var int
39
-    */
36
+     * Máximos intentos de inincio de sesión de un usuario
37
+     *
38
+     * @var int
39
+     */
40 40
     const MAX_ATTEMPTS = 5;
41 41
 
42 42
     /**
43
-      * Tiempo entre máximos intentos en segundos
44
-      *
45
-      * @var int
46
-    */
43
+     * Tiempo entre máximos intentos en segundos
44
+     *
45
+     * @var int
46
+     */
47 47
     const MAX_ATTEMPTS_TIME = 120; # (dos minutos)
48 48
 
49 49
     /**
50
-      * Log de intentos recientes con la forma 'email' => (int) intentos
51
-      *
52
-      * @var array
53
-    */
50
+     * Log de intentos recientes con la forma 'email' => (int) intentos
51
+     *
52
+     * @var array
53
+     */
54 54
     private $recentAttempts = array();
55 55
 
56
-     /**
57
-      * Hace un set() a la sesión login_user_recentAttempts con el valor actualizado.
58
-      *
59
-      * @return void
60
-    */
56
+      /**
57
+       * Hace un set() a la sesión login_user_recentAttempts con el valor actualizado.
58
+       *
59
+       * @return void
60
+       */
61 61
     private function updateSessionAttempts() {
62 62
         global $session;
63 63
 
@@ -65,13 +65,13 @@  discard block
 block discarded – undo
65 65
     }
66 66
 
67 67
     /**
68
-      * Genera la sesión con el id del usuario que ha iniciado
69
-      *
70
-      * @param string $pass : Contraseña sin encriptar
71
-      * @param string $pass_repeat : Contraseña repetida sin encriptar
72
-      *
73
-      * @throws ModelsException cuando las contraseñas no coinciden
74
-    */
68
+     * Genera la sesión con el id del usuario que ha iniciado
69
+     *
70
+     * @param string $pass : Contraseña sin encriptar
71
+     * @param string $pass_repeat : Contraseña repetida sin encriptar
72
+     *
73
+     * @throws ModelsException cuando las contraseñas no coinciden
74
+     */
75 75
     private function checkPassMatch(string $pass, string $pass_repeat) {
76 76
         if($pass != $pass_repeat) {
77 77
             throw new ModelsException('Las contraseñas no coinciden.');
@@ -79,12 +79,12 @@  discard block
 block discarded – undo
79 79
     }
80 80
 
81 81
     /**
82
-      * Verifica el email introducido, tanto el formato como su existencia en el sistema
83
-      *
84
-      * @param string $email: Email del usuario
85
-      *
86
-      * @throws ModelsException en caso de que no tenga formato válido o ya exista
87
-    */
82
+     * Verifica el email introducido, tanto el formato como su existencia en el sistema
83
+     *
84
+     * @param string $email: Email del usuario
85
+     *
86
+     * @throws ModelsException en caso de que no tenga formato válido o ya exista
87
+     */
88 88
     private function checkEmail(string $email) {
89 89
         # Formato de email
90 90
         if(!Strings::is_email($email)) {
@@ -99,13 +99,13 @@  discard block
 block discarded – undo
99 99
     }
100 100
 
101 101
     /**
102
-      * Restaura los intentos de un usuario al iniciar sesión
103
-      *
104
-      * @param string $email: Email del usuario a restaurar
105
-      *
106
-      * @throws ModelsException cuando hay un error de lógica utilizando este método
107
-      * @return void
108
-    */
102
+     * Restaura los intentos de un usuario al iniciar sesión
103
+     *
104
+     * @param string $email: Email del usuario a restaurar
105
+     *
106
+     * @throws ModelsException cuando hay un error de lógica utilizando este método
107
+     * @return void
108
+     */
109 109
     private function restoreAttempts(string $email) {       
110 110
         if(array_key_exists($email,$this->recentAttempts)) {
111 111
             $this->recentAttempts[$email]['attempts'] = 0;
@@ -118,12 +118,12 @@  discard block
 block discarded – undo
118 118
     }
119 119
 
120 120
     /**
121
-      * Genera la sesión con el id del usuario que ha iniciado
122
-      *
123
-      * @param array $user_data: Arreglo con información de la base de datos, del usuario
124
-      *
125
-      * @return void
126
-    */
121
+     * Genera la sesión con el id del usuario que ha iniciado
122
+     *
123
+     * @param array $user_data: Arreglo con información de la base de datos, del usuario
124
+     *
125
+     * @return void
126
+     */
127 127
     private function generateSession(array $user_data) {
128 128
         global $session, $config;
129 129
 
@@ -132,14 +132,14 @@  discard block
 block discarded – undo
132 132
     }
133 133
 
134 134
     /**
135
-      * Verifica en la base de datos, el email y contraseña ingresados por el usuario
136
-      *
137
-      * @param string $email: Email del usuario que intenta el login
138
-      * @param string $pass: Contraseña sin encriptar del usuario que intenta el login
139
-      *
140
-      * @return bool true: Cuando el inicio de sesión es correcto 
141
-      *              false: Cuando el inicio de sesión no es correcto
142
-    */
135
+     * Verifica en la base de datos, el email y contraseña ingresados por el usuario
136
+     *
137
+     * @param string $email: Email del usuario que intenta el login
138
+     * @param string $pass: Contraseña sin encriptar del usuario que intenta el login
139
+     *
140
+     * @return bool true: Cuando el inicio de sesión es correcto 
141
+     *              false: Cuando el inicio de sesión no es correcto
142
+     */
143 143
     private function authentication(string $email,string $pass) : bool {
144 144
         $email = $this->db->scape($email);
145 145
         $query = $this->db->select('id_user,pass','users',"email='$email'",'LIMIT 1');
@@ -159,10 +159,10 @@  discard block
 block discarded – undo
159 159
     }
160 160
 
161 161
     /**
162
-      * Establece los intentos recientes desde la variable de sesión acumulativa
163
-      *
164
-      * @return void
165
-    */
162
+     * Establece los intentos recientes desde la variable de sesión acumulativa
163
+     *
164
+     * @return void
165
+     */
166 166
     private function setDefaultAttempts() {
167 167
         global $session;
168 168
 
@@ -172,12 +172,12 @@  discard block
 block discarded – undo
172 172
     }
173 173
     
174 174
     /**
175
-      * Establece el intento del usuario actual o incrementa su cantidad si ya existe
176
-      *
177
-      * @param string $email: Email del usuario
178
-      *
179
-      * @return void
180
-    */
175
+     * Establece el intento del usuario actual o incrementa su cantidad si ya existe
176
+     *
177
+     * @param string $email: Email del usuario
178
+     *
179
+     * @return void
180
+     */
181 181
     private function setNewAttempt(string $email) {
182 182
         if(!array_key_exists($email,$this->recentAttempts)) {
183 183
             $this->recentAttempts[$email] = array(
@@ -191,14 +191,14 @@  discard block
 block discarded – undo
191 191
     }
192 192
 
193 193
     /**
194
-      * Controla la cantidad de intentos permitidos máximos por usuario, si llega al límite,
195
-      * el usuario podrá seguir intentando en self::MAX_ATTEMPTS_TIME segundos.
196
-      *
197
-      * @param string $email: Email del usuario
198
-      *
199
-      * @throws ModelsException cuando ya ha excedido self::MAX_ATTEMPTS
200
-      * @return void
201
-    */
194
+     * Controla la cantidad de intentos permitidos máximos por usuario, si llega al límite,
195
+     * el usuario podrá seguir intentando en self::MAX_ATTEMPTS_TIME segundos.
196
+     *
197
+     * @param string $email: Email del usuario
198
+     *
199
+     * @throws ModelsException cuando ya ha excedido self::MAX_ATTEMPTS
200
+     * @return void
201
+     */
202 202
     private function maximumAttempts(string $email) {
203 203
         if($this->recentAttempts[$email]['attempts'] >= self::MAX_ATTEMPTS) {
204 204
             
@@ -219,10 +219,10 @@  discard block
 block discarded – undo
219 219
     }
220 220
 
221 221
     /**
222
-      * Realiza la acción de login dentro del sistema
223
-      *
224
-      * @return array : Con información de éxito/falla al inicio de sesión.
225
-    */
222
+     * Realiza la acción de login dentro del sistema
223
+     *
224
+     * @return array : Con información de éxito/falla al inicio de sesión.
225
+     */
226 226
     public function login() : array {
227 227
         try {
228 228
             global $http;
@@ -258,10 +258,10 @@  discard block
 block discarded – undo
258 258
     }
259 259
 
260 260
     /**
261
-      * Realiza la acción de registro dentro del sistema
262
-      *
263
-      * @return array : Con información de éxito/falla al registrar el usuario nuevo.
264
-    */
261
+     * Realiza la acción de registro dentro del sistema
262
+     *
263
+     * @return array : Con información de éxito/falla al registrar el usuario nuevo.
264
+     */
265 265
     public function register() : array {
266 266
         try {
267 267
             global $http;
@@ -302,11 +302,11 @@  discard block
 block discarded – undo
302 302
     }
303 303
     
304 304
     /**
305
-      * Envía un correo electrónico al usuario que quiere recuperar la contraseña, con un token y una nueva contraseña.
306
-      * Si el usuario no visita el enlace, el sistema no cambiará la contraseña.
307
-      *
308
-      * @return array<string,integer|string>
309
-    */  
305
+     * Envía un correo electrónico al usuario que quiere recuperar la contraseña, con un token y una nueva contraseña.
306
+     * Si el usuario no visita el enlace, el sistema no cambiará la contraseña.
307
+     *
308
+     * @return array<string,integer|string>
309
+     */  
310 310
     public function lostpass() {
311 311
         try {
312 312
             global $http, $config;
@@ -342,8 +342,8 @@  discard block
 block discarded – undo
342 342
 
343 343
             # Enviar el correo electrónico
344 344
             $dest = array();
345
-			$dest[$email] = $user_data[0]['name'];
346
-			$email = Emails::send_mail($dest,Emails::plantilla($HTML),'Recuperar contraseña perdida');
345
+      $dest[$email] = $user_data[0]['name'];
346
+      $email = Emails::send_mail($dest,Emails::plantilla($HTML),'Recuperar contraseña perdida');
347 347
 
348 348
             # Verificar si hubo algún problema con el envío del correo
349 349
             if(false === $email) {
@@ -364,13 +364,13 @@  discard block
 block discarded – undo
364 364
     }
365 365
 
366 366
     /**
367
-      * Cambia la contraseña de un usuario en el sistema, luego de que éste haya solicitado cambiarla.
368
-      * Luego retorna al sitio de inicio con la variable GET success=(bool)
369
-      *
370
-      * La URL debe tener la forma URL/lostpass/cambiar/&token=TOKEN&user=ID
371
-      *
372
-      * @return void
373
-    */  
367
+     * Cambia la contraseña de un usuario en el sistema, luego de que éste haya solicitado cambiarla.
368
+     * Luego retorna al sitio de inicio con la variable GET success=(bool)
369
+     *
370
+     * La URL debe tener la forma URL/lostpass/cambiar/&token=TOKEN&user=ID
371
+     *
372
+     * @return void
373
+     */  
374 374
     public function changeTemporalPass() {
375 375
         global $config, $http;
376 376
         
@@ -394,10 +394,10 @@  discard block
 block discarded – undo
394 394
     }
395 395
 
396 396
     /**
397
-      * Desconecta a un usuario si éste está conectado, y lo devuelve al inicio
398
-      *
399
-      * @return void
400
-    */    
397
+     * Desconecta a un usuario si éste está conectado, y lo devuelve al inicio
398
+     *
399
+     * @return void
400
+     */    
401 401
     public function logout() {
402 402
         global $session;
403 403
 
@@ -409,36 +409,36 @@  discard block
 block discarded – undo
409 409
     }
410 410
 
411 411
     /**
412
-      * Obtiene datos de un usuario según su id en la base de datos
413
-      *    
414
-      * @param int $id: Id del usuario a obtener
415
-      * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios 
416
-      *
417
-      * @return false|array con información del usuario
418
-    */   
412
+     * Obtiene datos de un usuario según su id en la base de datos
413
+     *    
414
+     * @param int $id: Id del usuario a obtener
415
+     * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios 
416
+     *
417
+     * @return false|array con información del usuario
418
+     */   
419 419
     public function getUserById(int $id, string $select = '*') {
420
-       return $this->db->select($select,'users',"id_user='$id'",'LIMIT 1');
420
+        return $this->db->select($select,'users',"id_user='$id'",'LIMIT 1');
421 421
     }
422 422
     
423 423
     /**
424
-      * Obtiene a todos los usuarios
425
-      *    
426
-      * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios 
427
-      *
428
-      * @return false|array con información de los usuarios
429
-    */  
424
+     * Obtiene a todos los usuarios
425
+     *    
426
+     * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios 
427
+     *
428
+     * @return false|array con información de los usuarios
429
+     */  
430 430
     public function getUsers(string $select = '*') {
431
-       return $this->db->select($select,'users');
431
+        return $this->db->select($select,'users');
432 432
     }
433 433
 
434 434
     /**
435
-      * Obtiene datos del usuario conectado actualmente
436
-      *
437
-      * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios
438
-      *
439
-      * @throws ModelsException si el usuario no está logeado
440
-      * @return array con datos del usuario conectado
441
-    */
435
+     * Obtiene datos del usuario conectado actualmente
436
+     *
437
+     * @param string $select : Por defecto es *, se usa para obtener sólo los parámetros necesarios
438
+     *
439
+     * @throws ModelsException si el usuario no está logeado
440
+     * @return array con datos del usuario conectado
441
+     */
442 442
     public function getOwnerUser(string $select = '*') : array {
443 443
         if(null !== $this->id_user) {    
444 444
                
@@ -456,10 +456,10 @@  discard block
 block discarded – undo
456 456
     }
457 457
 
458 458
     /**
459
-      * Instala el módulo de usuarios en la base de datos para que pueda funcionar correctamete.
460
-      *
461
-      * @throws \RuntimeException si no se puede realizar la query
462
-    */
459
+     * Instala el módulo de usuarios en la base de datos para que pueda funcionar correctamete.
460
+     *
461
+     * @throws \RuntimeException si no se puede realizar la query
462
+     */
463 463
     public function install() {
464 464
         if(!$this->db->query("
465 465
             CREATE TABLE IF NOT EXISTS `users` (
@@ -480,16 +480,16 @@  discard block
 block discarded – undo
480 480
     }
481 481
 
482 482
     /**
483
-      * __construct()
484
-    */
483
+     * __construct()
484
+     */
485 485
     public function __construct(IRouter $router = null) {
486 486
         parent::__construct($router);
487 487
         $this->startDBConexion();
488 488
     }
489 489
 
490 490
     /**
491
-      * __destruct()
492
-    */ 
491
+     * __destruct()
492
+     */ 
493 493
     public function __destruct() {
494 494
         parent::__destruct();
495 495
         $this->endDBConexion();
Please login to merge, or discard this patch.
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
       * @throws ModelsException cuando las contraseñas no coinciden
74 74
     */
75 75
     private function checkPassMatch(string $pass, string $pass_repeat) {
76
-        if($pass != $pass_repeat) {
76
+        if ($pass != $pass_repeat) {
77 77
             throw new ModelsException('Las contraseñas no coinciden.');
78 78
         }
79 79
     }
@@ -87,13 +87,13 @@  discard block
 block discarded – undo
87 87
     */
88 88
     private function checkEmail(string $email) {
89 89
         # Formato de email
90
-        if(!Strings::is_email($email)) {
90
+        if (!Strings::is_email($email)) {
91 91
             throw new ModelsException('El email no tiene un formato válido.');
92 92
         }
93 93
         # Existencia de email
94 94
         $email = $this->db->scape($email);
95
-        $query = $this->db->select('id_user','users',"email='$email'",'LIMIT 1');
96
-        if(false !== $query) {
95
+        $query = $this->db->select('id_user', 'users', "email='$email'", 'LIMIT 1');
96
+        if (false !== $query) {
97 97
             throw new ModelsException('El email introducido ya existe.');
98 98
         }
99 99
     }
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
       * @return void
108 108
     */
109 109
     private function restoreAttempts(string $email) {       
110
-        if(array_key_exists($email,$this->recentAttempts)) {
110
+        if (array_key_exists($email, $this->recentAttempts)) {
111 111
             $this->recentAttempts[$email]['attempts'] = 0;
112 112
             $this->recentAttempts[$email]['time'] = null;
113 113
             $this->updateSessionAttempts();
@@ -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('user_id',(int) $user_data['id_user']);
130
+        $session->set('user_id', (int) $user_data['id_user']);
131 131
         $session->set('unique_session', $config['sessions']['unique']);
132 132
     }
133 133
 
@@ -140,12 +140,12 @@  discard block
 block discarded – undo
140 140
       * @return bool true: Cuando el inicio de sesión es correcto 
141 141
       *              false: Cuando el inicio de sesión no es correcto
142 142
     */
143
-    private function authentication(string $email,string $pass) : bool {
143
+    private function authentication(string $email, string $pass) : bool {
144 144
         $email = $this->db->scape($email);
145
-        $query = $this->db->select('id_user,pass','users',"email='$email'",'LIMIT 1');
145
+        $query = $this->db->select('id_user,pass', 'users', "email='$email'", 'LIMIT 1');
146 146
         
147 147
         # Incio de sesión con éxito
148
-        if(false !== $query && Strings::chash($query[0]['pass'],$pass)) {
148
+        if (false !== $query && Strings::chash($query[0]['pass'], $pass)) {
149 149
 
150 150
             # Restaurar intentos
151 151
             $this->restoreAttempts($email);
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
     private function setDefaultAttempts() {
167 167
         global $session;
168 168
 
169
-        if(null != $session->get('login_user_recentAttempts')) {
169
+        if (null != $session->get('login_user_recentAttempts')) {
170 170
             $this->recentAttempts = $session->get('login_user_recentAttempts');
171 171
         }
172 172
     }
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
       * @return void
180 180
     */
181 181
     private function setNewAttempt(string $email) {
182
-        if(!array_key_exists($email,$this->recentAttempts)) {
182
+        if (!array_key_exists($email, $this->recentAttempts)) {
183 183
             $this->recentAttempts[$email] = array(
184 184
                 'attempts' => 0, # Intentos
185 185
                 'time' => null # Tiempo 
@@ -200,14 +200,14 @@  discard block
 block discarded – undo
200 200
       * @return void
201 201
     */
202 202
     private function maximumAttempts(string $email) {
203
-        if($this->recentAttempts[$email]['attempts'] >= self::MAX_ATTEMPTS) {
203
+        if ($this->recentAttempts[$email]['attempts'] >= self::MAX_ATTEMPTS) {
204 204
             
205 205
             # Colocar timestamp para recuperar más adelante la posibilidad de acceso
206
-            if(null == $this->recentAttempts[$email]['time']) {
206
+            if (null == $this->recentAttempts[$email]['time']) {
207 207
                 $this->recentAttempts[$email]['time'] = time() + self::MAX_ATTEMPTS_TIME;
208 208
             }
209 209
             
210
-            if(time() < $this->recentAttempts[$email]['time']) {
210
+            if (time() < $this->recentAttempts[$email]['time']) {
211 211
                 # Setear sesión
212 212
                 $this->updateSessionAttempts();
213 213
                 # Lanzar excepción
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
             $pass = $http->request->get('pass');
236 236
 
237 237
             # Verificar que no están vacíos
238
-            if($this->functions->e($email,$pass)) {
238
+            if ($this->functions->e($email, $pass)) {
239 239
                 throw new ModelsException('Credenciales incompletas.');
240 240
             }
241 241
             
@@ -246,13 +246,13 @@  discard block
 block discarded – undo
246 246
             $this->maximumAttempts($email);
247 247
 
248 248
             # Autentificar
249
-            if($this->authentication($email,$pass)) {
249
+            if ($this->authentication($email, $pass)) {
250 250
                 return array('success' => 1, 'message' => 'Conectado con éxito.');
251 251
             }
252 252
             
253 253
             throw new ModelsException('Credenciales incorrectas.');
254 254
 
255
-        } catch(ModelsException $e) {
255
+        } catch (ModelsException $e) {
256 256
             return array('success' => 0, 'message' => $e->getMessage());
257 257
         }        
258 258
     }
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
             $pass_repeat = $http->request->get('pass_repeat');
274 274
 
275 275
             # Verificar que no están vacíos
276
-            if($this->functions->e($name,$email,$pass,$pass_repeat)) {
276
+            if ($this->functions->e($name, $email, $pass, $pass_repeat)) {
277 277
                 throw new ModelsException('Todos los datos son necesarios');
278 278
             }
279 279
 
@@ -281,10 +281,10 @@  discard block
 block discarded – undo
281 281
             $this->checkEmail($email);
282 282
 
283 283
             # Veriricar contraseñas
284
-            $this->checkPassMatch($pass,$pass_repeat);
284
+            $this->checkPassMatch($pass, $pass_repeat);
285 285
 
286 286
             # Registrar al usuario
287
-            $this->db->insert('users',array(
287
+            $this->db->insert('users', array(
288 288
                 'name' => $name,
289 289
                 'email' => $email,
290 290
                 'pass' => Strings::hash($pass)
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
             ));
297 297
 
298 298
             return array('success' => 1, 'message' => 'Registrado con éxito.');
299
-        } catch(ModelsException $e) {
299
+        } catch (ModelsException $e) {
300 300
             return array('success' => 0, 'message' => $e->getMessage());
301 301
         }        
302 302
     }
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
             $email = $http->request->get('email');
316 316
             
317 317
             # Campo lleno
318
-            if($this->functions->emp($email)) {
318
+            if ($this->functions->emp($email)) {
319 319
                 throw new ModelsException('El campo email debe estar lleno.');
320 320
             }
321 321
 
@@ -323,10 +323,10 @@  discard block
 block discarded – undo
323 323
             $email = $this->db->scape($email);
324 324
 
325 325
             # Obtener información del usuario 
326
-            $user_data = $this->db->select('id_user,name','users',"email='$email'",'LIMIT 1');
326
+            $user_data = $this->db->select('id_user,name', 'users', "email='$email'", 'LIMIT 1');
327 327
 
328 328
             # Verificar correo en base de datos 
329
-            if(false === $user_data) {
329
+            if (false === $user_data) {
330 330
                 throw new ModelsException('El email no está registrado en el sistema.');
331 331
             }
332 332
 
@@ -335,30 +335,30 @@  discard block
 block discarded – undo
335 335
             $pass = uniqid();
336 336
 
337 337
             # Construir mensaje y enviar mensaje
338
-            $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
+            $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.
339 339
 					<br />
340 340
 					<br />
341
-					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
+					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>.';
342 342
 
343 343
             # Enviar el correo electrónico
344 344
             $dest = array();
345 345
 			$dest[$email] = $user_data[0]['name'];
346
-			$email = Emails::send_mail($dest,Emails::plantilla($HTML),'Recuperar contraseña perdida');
346
+			$email = Emails::send_mail($dest, Emails::plantilla($HTML), 'Recuperar contraseña perdida');
347 347
 
348 348
             # Verificar si hubo algún problema con el envío del correo
349
-            if(false === $email) {
349
+            if (false === $email) {
350 350
                 throw new ModelsException('No se ha podido enviar el correo electrónico.');
351 351
             }
352 352
 
353 353
             # Actualizar datos 
354 354
             $id_user = $user_data[0]['id_user'];
355
-            $this->db->update('users',array(
355
+            $this->db->update('users', array(
356 356
                 'tmp_pass' => Strings::hash($pass),
357 357
                 'token' => $token
358
-            ),"id_user='$id_user'",'LIMIT 1');
358
+            ), "id_user='$id_user'", 'LIMIT 1');
359 359
 
360 360
             return array('success' => 1, 'message' => 'Se ha enviado un enlace a su correo electrónico.');
361
-        } catch(ModelsException $e) {
361
+        } catch (ModelsException $e) {
362 362
             return array('success' => 0, 'message' => $e->getMessage());
363 363
         }
364 364
     }
@@ -378,7 +378,7 @@  discard block
 block discarded – undo
378 378
         $id_user = $http->query->get('user');
379 379
         $token = $http->query->get('token');
380 380
 
381
-        if(!$this->functions->emp($token) && is_numeric($id_user) && $id_user >= 1) {
381
+        if (!$this->functions->emp($token) && is_numeric($id_user) && $id_user >= 1) {
382 382
             # Filtros a los datos
383 383
             $id_user = $this->db->scape($id_user);
384 384
             $token = $this->db->scape($token);
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
     public function logout() {
402 402
         global $session;
403 403
 
404
-        if(null != $session->get('user_id')) {
404
+        if (null != $session->get('user_id')) {
405 405
             $session->remove('user_id');
406 406
         }
407 407
 
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
       * @return false|array con información del usuario
418 418
     */   
419 419
     public function getUserById(int $id, string $select = '*') {
420
-       return $this->db->select($select,'users',"id_user='$id'",'LIMIT 1');
420
+       return $this->db->select($select, 'users', "id_user='$id'", 'LIMIT 1');
421 421
     }
422 422
     
423 423
     /**
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
       * @return false|array con información de los usuarios
429 429
     */  
430 430
     public function getUsers(string $select = '*') {
431
-       return $this->db->select($select,'users');
431
+       return $this->db->select($select, 'users');
432 432
     }
433 433
 
434 434
     /**
@@ -440,12 +440,12 @@  discard block
 block discarded – undo
440 440
       * @return array con datos del usuario conectado
441 441
     */
442 442
     public function getOwnerUser(string $select = '*') : array {
443
-        if(null !== $this->id_user) {    
443
+        if (null !== $this->id_user) {    
444 444
                
445
-            $user = $this->db->select($select,'users',"id_user='$this->id_user'",'LIMIT 1');
445
+            $user = $this->db->select($select, 'users', "id_user='$this->id_user'", 'LIMIT 1');
446 446
 
447 447
             # Si se borra al usuario desde la base de datos y sigue con la sesión activa
448
-            if(false === $user) {
448
+            if (false === $user) {
449 449
                 $this->logout();
450 450
             }
451 451
 
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
       * @throws \RuntimeException si no se puede realizar la query
462 462
     */
463 463
     public function install() {
464
-        if(!$this->db->query("
464
+        if (!$this->db->query("
465 465
             CREATE TABLE IF NOT EXISTS `users` (
466 466
                 `id_user` int(11) UNSIGNED NOT NULL AUTO_INCREMENT,
467 467
                 `name` varchar(100) NOT NULL,
Please login to merge, or discard this patch.
Ocrend/Kernel/Helpers/Arrays.php 2 patches
Indentation   +61 added lines, -61 removed lines patch added patch discarded remove patch
@@ -19,28 +19,28 @@  discard block
 block discarded – undo
19 19
 
20 20
 final class Arrays extends \Twig_Extension {
21 21
 
22
-   //------------------------------------------------
22
+    //------------------------------------------------
23 23
 
24 24
   /**
25
-    * Suma los contenidos del segundo arreglo en el primer arreglo, en las coincidencias de llaves.
26
-    *
27
-    * Si el primer arreglo está vacío, copiará todo el segundo arreglo en el primero. 
28
-    *
29
-    * Si el segundo arreglo está vacío, copiará todo el primer arreglo en el segundo.
30
-    *
31
-    * Si el arreglo $a tiene una estructura distinta a $b, la estructura de $a queda intacta y sólamente,
32
-    * se hará sumas en las posiciones donde haya coincidencias de llaves.
33
-    *
34
-    * El valor null será tomado como cero.
35
-    *
36
-    * @param array $a: Primer arreglo
37
-    * @param array $b: Segundo arreglo
38
-    *
39
-    * @throws RuntimeException si los arreglos son de distinto nivel
40
-    * @throws RuntimeException si en la posición de coincidencia no se puede realizar una operación de suma
41
-    *
42
-    * @return array fusión de los dos arreglos, dejando intactal estructura de $a
43
-  */
25
+   * Suma los contenidos del segundo arreglo en el primer arreglo, en las coincidencias de llaves.
26
+   *
27
+   * Si el primer arreglo está vacío, copiará todo el segundo arreglo en el primero. 
28
+   *
29
+   * Si el segundo arreglo está vacío, copiará todo el primer arreglo en el segundo.
30
+   *
31
+   * Si el arreglo $a tiene una estructura distinta a $b, la estructura de $a queda intacta y sólamente,
32
+   * se hará sumas en las posiciones donde haya coincidencias de llaves.
33
+   *
34
+   * El valor null será tomado como cero.
35
+   *
36
+   * @param array $a: Primer arreglo
37
+   * @param array $b: Segundo arreglo
38
+   *
39
+   * @throws RuntimeException si los arreglos son de distinto nivel
40
+   * @throws RuntimeException si en la posición de coincidencia no se puede realizar una operación de suma
41
+   *
42
+   * @return array fusión de los dos arreglos, dejando intactal estructura de $a
43
+   */
44 44
   final public static function arrays_sum(array $a, array $b) : array {
45 45
     # Si alguno está vacío
46 46
     if(sizeof($a) == 0) {
@@ -84,13 +84,13 @@  discard block
 block discarded – undo
84 84
   //------------------------------------------------
85 85
 
86 86
   /**
87
-    * Dado un índice asociativo y un arreglo, devuelve el índice numérico correspondiente al asociativo
88
-    *
89
-    * @param string $index: Índice asociativo del arreglo
90
-    * @param array $a: Arreglo a evaluar
91
-    *
92
-    * @return int el índice correspondiente, -1 si no existe el indice
93
-  */
87
+   * Dado un índice asociativo y un arreglo, devuelve el índice numérico correspondiente al asociativo
88
+   *
89
+   * @param string $index: Índice asociativo del arreglo
90
+   * @param array $a: Arreglo a evaluar
91
+   *
92
+   * @return int el índice correspondiente, -1 si no existe el indice
93
+   */
94 94
   final public static function get_key_by_index(string $index, array $a) : int {
95 95
     $i = 0;
96 96
     foreach ($a as $key => $val) {
@@ -105,14 +105,14 @@  discard block
 block discarded – undo
105 105
   //------------------------------------------------
106 106
 
107 107
   /**
108
-    * Elimina todos los elementos repetidos de un array
109
-    * (string) '1' se considera igual a (int) 1
110
-    *
111
-    * @param array $a: Arreglo a evaluar
112
-    *
113
-    * @return array devuelve un arreglo sin elementos repetidos
114
-    * http://stackoverflow.com/questions/8321620/array-unique-vs-array-flip
115
-  */
108
+   * Elimina todos los elementos repetidos de un array
109
+   * (string) '1' se considera igual a (int) 1
110
+   *
111
+   * @param array $a: Arreglo a evaluar
112
+   *
113
+   * @return array devuelve un arreglo sin elementos repetidos
114
+   * http://stackoverflow.com/questions/8321620/array-unique-vs-array-flip
115
+   */
116 116
   final public static function unique_array(array $a) : array {
117 117
     return array_keys(array_flip($a));
118 118
   }
@@ -120,12 +120,12 @@  discard block
 block discarded – undo
120 120
   //------------------------------------------------
121 121
 
122 122
   /**
123
-    * Evalúa si un arreglo es de tipo asociativo o no
124
-    *
125
-    * @param array $a: Arreglo a evaluar
126
-    *
127
-    * @return bool false si no lo es, true si lo es
128
-  */
123
+   * Evalúa si un arreglo es de tipo asociativo o no
124
+   *
125
+   * @param array $a: Arreglo a evaluar
126
+   *
127
+   * @return bool false si no lo es, true si lo es
128
+   */
129 129
   final public static function is_assoc(array $a) : bool {
130 130
     if(sizeof($a) === 0) {
131 131
       return false;
@@ -137,12 +137,12 @@  discard block
 block discarded – undo
137 137
   //------------------------------------------------
138 138
 
139 139
   /**
140
-    * Evalúa si un arreglo es secuencial (de índices numéricos)
141
-    *
142
-    * @param array $a: Arreglo a evaluar
143
-    *
144
-    * @return bool false si no lo es, true si lo es
145
-  */
140
+   * Evalúa si un arreglo es secuencial (de índices numéricos)
141
+   *
142
+   * @param array $a: Arreglo a evaluar
143
+   *
144
+   * @return bool false si no lo es, true si lo es
145
+   */
146 146
   final public static function is_numeric_array(array $a) : bool {
147 147
     return !self::is_assoc($a);
148 148
   }
@@ -150,12 +150,12 @@  discard block
 block discarded – undo
150 150
   //------------------------------------------------
151 151
 
152 152
   /**
153
-    * Obtiene de forma random un elemento de un arreglo
154
-    *
155
-    * @param array $a: Arreglo a evaluar
156
-    *
157
-    * @return mixed elemento random dentro del arreglo
158
-  */
153
+   * Obtiene de forma random un elemento de un arreglo
154
+   *
155
+   * @param array $a: Arreglo a evaluar
156
+   *
157
+   * @return mixed elemento random dentro del arreglo
158
+   */
159 159
   final public static function array_random_element(array $a) {
160 160
     return $a[array_rand($a)];
161 161
   }
@@ -163,10 +163,10 @@  discard block
 block discarded – undo
163 163
   //------------------------------------------------
164 164
 
165 165
   /**
166
-    * Se obtiene de Twig_Extension y sirve para que cada función esté disponible como etiqueta en twig
167
-    *
168
-    * @return array con todas las funciones con sus respectivos nombres de acceso en plantillas twig
169
-  */
166
+   * Se obtiene de Twig_Extension y sirve para que cada función esté disponible como etiqueta en twig
167
+   *
168
+   * @return array con todas las funciones con sus respectivos nombres de acceso en plantillas twig
169
+   */
170 170
   public function getFunctions() : array {
171 171
     return array(
172 172
       new \Twig_Function('get_key_by_index', array($this, 'get_key_by_index')),
@@ -181,10 +181,10 @@  discard block
 block discarded – undo
181 181
   //------------------------------------------------
182 182
 
183 183
   /**
184
-    * Identificador único para la extensión de twig
185
-    *
186
-    * @return string con el nombre de la extensión
187
-  */
184
+   * Identificador único para la extensión de twig
185
+   *
186
+   * @return string con el nombre de la extensión
187
+   */
188 188
   public function getName() : string {
189 189
     return 'ocrend_framework_helper_arrays';
190 190
   }
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -43,19 +43,19 @@  discard block
 block discarded – undo
43 43
   */
44 44
   final public static function arrays_sum(array $a, array $b) : array {
45 45
     # Si alguno está vacío
46
-    if(sizeof($a) == 0) {
46
+    if (sizeof($a) == 0) {
47 47
       return $b;
48
-    } else if(sizeof($b) == 0) {
48
+    } else if (sizeof($b) == 0) {
49 49
       return $a;
50 50
     }
51 51
 
52 52
     # Recorrer el segundo arreglo
53
-    foreach($b as $llave => $contenido) {
53
+    foreach ($b as $llave => $contenido) {
54 54
       # Verificar que no hay desnivel
55
-      if(!is_array($a[$llave]) && !is_array($b[$llave])) {
55
+      if (!is_array($a[$llave]) && !is_array($b[$llave])) {
56 56
         $a[$llave] += $b[$llave];
57 57
       } else {
58
-        throw new \RuntimeException('Existe un problema para operar en la llave '.$llave.'.');
58
+        throw new \RuntimeException('Existe un problema para operar en la llave ' . $llave . '.');
59 59
         break;
60 60
       }
61 61
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
   final public static function get_key_by_index(string $index, array $a) : int {
77 77
     $i = 0;
78 78
     foreach ($a as $key => $val) {
79
-      if($key == $index) {
79
+      if ($key == $index) {
80 80
         return (int) $i;
81 81
       }
82 82
       $i++;
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
     * @return bool false si no lo es, true si lo es
110 110
   */
111 111
   final public static function is_assoc(array $a) : bool {
112
-    if(sizeof($a) === 0) {
112
+    if (sizeof($a) === 0) {
113 113
       return false;
114 114
     }
115 115
 
Please login to merge, or discard this patch.