prinick96 / Ocrend-Framework

Ocrend/Kernel/Database/Database.php

Indentation +110 added lines, -110 removed lines

@@ -50,10 +50,10 @@ 
   }
 
   /**
-    * Motor de base de datos MySQL
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos MySQL
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_mysql(array $params) {
     parent::__construct('mysql:host='.$params['host'].';dbname='.$params['name'],
     $params['user'],
@@ -65,19 +65,19 @@ 
   }
   
   /**
-    * Motor de base de datos SQLite
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos SQLite
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_sqlite(array $params) {
     parent::__construct('sqlite:'.$params['name']);
   }
 
   /**
-    * Motor de base de datos Cubrid
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos Cubrid
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_cubrid(array $params) {
     parent::__construct('cubrid:host='.$params['host'].';dbname='.$params['name'].';port='.$params['port'],
     $params['user'],
@@ -88,10 +88,10 @@ 
   }
 
   /**
-    * Motor de base de datos Firebird
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos Firebird
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_firebird(array $params) {
     parent::__construct('firebird:dbname='.$params['host'].':'.$params['name'],
     $params['user'],
@@ -102,10 +102,10 @@ 
   }
 
   /**
-    * Motor de base de datos ODBC
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos ODBC
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_odbc(array $params) {
     parent::__construct('odbc:'.$params['name'],
       $params['user'],
@@ -114,10 +114,10 @@ 
   }
 
   /**
-    * Motor de base de datos Oracle
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos Oracle
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_oracle(array $params) {
     parent::__construct('oci:dbname=(DESCRIPTION =
     (ADDRESS_LIST =
@@ -134,10 +134,10 @@ 
   }
 
   /**
-    * Motor de base de datos PostgreSQL
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos PostgreSQL
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_postgresql(array $params) {
     parent::__construct('pgsql:host='.$params['host'].';dbname='.$params['name'].';charset=utf8',
     $params['user'],
@@ -148,11 +148,11 @@ 
   }
 
   /**
-    * Motor de base de datos MSSQL
-    * Añadido por marc2684 https://github.com/prinick96/Ocrend-Framework/issues/7
-    *
-    * @param array $params: Lista de parámetros de configuración
-  */
+   * Motor de base de datos MSSQL
+   * Añadido por marc2684 https://github.com/prinick96/Ocrend-Framework/issues/7
+   *
+   * @param array $params: Lista de parámetros de configuración
+   */
   final private function motor_mssql(array $params) {
     parent::__construct('sqlsrv:Server='.$params['host'].';Database='.$params['name'].';ConnectionPooling=0',
     $params['user'],
@@ -163,16 +163,16 @@ 
   }
 
   /**
-    * Inicia la conexión con la base de datos seleccionada
-    *
-    * @param string|null $name : Nombre de la base de datos a conectar
-    * @param string|null $motor: Motor de la base de datos a conectar
-    * @param bool $new_instance: true para iniciar una nueva instancia (al querer conectar a una DB distinta)
-    *
-    * @throws \RuntimeException si el motor no existe
-    * @throws \RuntimeException si existe algún problema de conexión con la base de datos
-    * @return Database : Instancia de conexión
-  */
+   * Inicia la conexión con la base de datos seleccionada
+   *
+   * @param string|null $name : Nombre de la base de datos a conectar
+   * @param string|null $motor: Motor de la base de datos a conectar
+   * @param bool $new_instance: true para iniciar una nueva instancia (al querer conectar a una DB distinta)
+   *
+   * @throws \RuntimeException si el motor no existe
+   * @throws \RuntimeException si existe algún problema de conexión con la base de datos
+   * @return Database : Instancia de conexión
+   */
   final public function __construct(string $name, string $motor) {
     global $config;
 
@@ -196,34 +196,34 @@ 
   }
 
   /**
-    * Convierte en arreglo asociativo de todos los resultados arrojados por una query
-    *
-    * @param object \PDOStatement $query, valor devuelto de la query
-    *
-    * @return array arreglo asociativo
-  */
+   * Convierte en arreglo asociativo de todos los resultados arrojados por una query
+   *
+   * @param object \PDOStatement $query, valor devuelto de la query
+   *
+   * @return array arreglo asociativo
+   */
   final public function fetch_array(\PDOStatement $query) : array {
     return $query->fetchAll(\PDO::FETCH_ASSOC);
   }
 
-   /**
-    * Cantidad de filas encontradas después de un SELECT
-    *
-    * @param object \PDOStatement $query, valor devuelto de la query
-    *
-    * @return int numero de filas encontradas
-  */
+    /**
+     * Cantidad de filas encontradas después de un SELECT
+     *
+     * @param object \PDOStatement $query, valor devuelto de la query
+     *
+     * @return int numero de filas encontradas
+     */
   final public function rows(\PDOStatement $query) : int {
     return $query->rowCount();
   }
 
   /**
-    * Sana un valor para posteriormente ser introducido en una query
-    *
-    * @param null/string/int/float a sanar
-    *
-    * @return mixed elemento sano
-  */
+   * Sana un valor para posteriormente ser introducido en una query
+   *
+   * @param null/string/int/float a sanar
+   *
+   * @return mixed elemento sano
+   */
   final public function scape($e) {
     if (null === $e) {
       return '';
@@ -240,28 +240,28 @@ 
   }
 
   /**
-    * Borra una serie de elementos de forma segura de una tabla en la base de datos
-    *
-    * @param string $table: Tabla a la cual se le quiere remover un elemento
-    * @param string $where: Condición de borrado que define quien/quienes son dichos elementos
-    * @param string $limit: Por defecto se limita a borrar un solo elemento que cumpla el $where
-    *
-    * @return \PDOStatement
-  */
+   * Borra una serie de elementos de forma segura de una tabla en la base de datos
+   *
+   * @param string $table: Tabla a la cual se le quiere remover un elemento
+   * @param string $where: Condición de borrado que define quien/quienes son dichos elementos
+   * @param string $limit: Por defecto se limita a borrar un solo elemento que cumpla el $where
+   *
+   * @return \PDOStatement
+   */
   final public function delete(string $table, string $where, string $limit = 'LIMIT 1') : \PDOStatement {
     return $this->query("DELETE FROM $table WHERE $where $limit;");
   }
 
   /**
-    * Inserta una serie de elementos a una tabla en la base de datos
-    *
-    * @param string $table: Tabla a la cual se le va a insertar elementos
-    * @param array $e: Arreglo asociativo de elementos, con la estrctura 'campo_en_la_tabla' => 'valor_a_insertar_en_ese_campo',
-    *                  todos los elementos del arreglo $e, serán sanados por el método sin necesidad de hacerlo manualmente al crear el arreglo
-    *
-    * @throws \RuntimeException si el arreglo está vacío
-    * @return \PDOStatement
-  */
+   * Inserta una serie de elementos a una tabla en la base de datos
+   *
+   * @param string $table: Tabla a la cual se le va a insertar elementos
+   * @param array $e: Arreglo asociativo de elementos, con la estrctura 'campo_en_la_tabla' => 'valor_a_insertar_en_ese_campo',
+   *                  todos los elementos del arreglo $e, serán sanados por el método sin necesidad de hacerlo manualmente al crear el arreglo
+   *
+   * @throws \RuntimeException si el arreglo está vacío
+   * @return \PDOStatement
+   */
   final public function insert(string $table, array $e) : \PDOStatement {
       if (sizeof($e) == 0) {
           throw new \RuntimeException('El arreglo pasado por $this->db->insert(\'' . $table . '\',...) está vacío.');
@@ -281,17 +281,17 @@ 
   }
 
   /**
-    * Actualiza elementos de una tabla en la base de datos según una condición
-    *
-    * @param string $table: Tabla a actualizar
-    * @param array $e: Arreglo asociativo de elementos, con la estrctura 'campo_en_la_tabla' => 'valor_a_insertar_en_ese_campo',
-    *                  todos los elementos del arreglo $e, serán sanados por el método sin necesidad de hacerlo manualmente al crear el arreglo
-    * @param string $where: Condición que indica quienes serán modificados
-    * @param string $limite: Límite de elementos modificados, por defecto los modifica a todos
-    *
-    * @throws \RuntimeException si el arreglo está vacío
-    * @return \PDOStatement
-  */
+   * Actualiza elementos de una tabla en la base de datos según una condición
+   *
+   * @param string $table: Tabla a actualizar
+   * @param array $e: Arreglo asociativo de elementos, con la estrctura 'campo_en_la_tabla' => 'valor_a_insertar_en_ese_campo',
+   *                  todos los elementos del arreglo $e, serán sanados por el método sin necesidad de hacerlo manualmente al crear el arreglo
+   * @param string $where: Condición que indica quienes serán modificados
+   * @param string $limite: Límite de elementos modificados, por defecto los modifica a todos
+   *
+   * @throws \RuntimeException si el arreglo está vacío
+   * @return \PDOStatement
+   */
   final public function update(string $table, array $e, string $where, string $limit = '') : \PDOStatement {
       if (sizeof($e) == 0) {
           throw new \RuntimeException('El arreglo pasado por $this->db->update(\'' . $table . '\'...) está vacío.');
@@ -308,24 +308,24 @@ 
   }
 
   /**
-    * Selecciona y lista en un arreglo asociativo/numérico los resultados de una búsqueda en la base de datos
-    *
-    * @param string $e: Elementos a seleccionar separados por coma
-    * @param string $tbale: Tabla de la cual se quiere extraer los elementos $e
-    * @param string $where: Condición que indica quienes son los que se extraen, si no se coloca extrae todos
-    * @param string $limite: Límite de elemntos a traer, por defecto trae TODOS los que cumplan $where
-    *
-    * @return mixed false si no encuentra ningún resultado, array asociativo/numérico si consigue al menos uno
-  */
+   * Selecciona y lista en un arreglo asociativo/numérico los resultados de una búsqueda en la base de datos
+   *
+   * @param string $e: Elementos a seleccionar separados por coma
+   * @param string $tbale: Tabla de la cual se quiere extraer los elementos $e
+   * @param string $where: Condición que indica quienes son los que se extraen, si no se coloca extrae todos
+   * @param string $limite: Límite de elemntos a traer, por defecto trae TODOS los que cumplan $where
+   *
+   * @return mixed false si no encuentra ningún resultado, array asociativo/numérico si consigue al menos uno
+   */
   final public function select(string $e, string $table, string $where = '1 = 1', string $limit = "") {    
     return $this->query_select("SELECT $e FROM $table WHERE $where $limit;");
   }
 
-   /**
-    * Realiza una query, ideal para trabajar con SELECTS, JOINS, etc
-    *
-    * @return array|false si no encuentra ningún resultado, array asociativo/numérico si consigue al menos uno
-  */
+    /**
+     * Realiza una query, ideal para trabajar con SELECTS, JOINS, etc
+     *
+     * @return array|false si no encuentra ningún resultado, array asociativo/numérico si consigue al menos uno
+     */
   final public function query_select(string $query) {
     $sql = $this->query($query);
     $result = $sql->fetchAll();
@@ -339,13 +339,13 @@ 
   }
 
   /**
-    * Alert para evitar clonaciones
-    *
-    * @throws \RuntimeException si se intenta clonar la conexión
-    * @return void
-  */
+   * Alert para evitar clonaciones
+   *
+   * @throws \RuntimeException si se intenta clonar la conexión
+   * @return void
+   */
   final public function __clone() {
     throw new \RuntimeException('Estás intentando clonar la Conexión');
   }
 
- }
\ No newline at end of file
+  }
\ No newline at end of file

Spacing +16 added lines, -16 removed lines

@@ -55,7 +55,7 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_mysql(array $params) {
-    parent::__construct('mysql:host='.$params['host'].';dbname='.$params['name'],
+    parent::__construct('mysql:host=' . $params['host'] . ';dbname=' . $params['name'],
     $params['user'],
     $params['pass'],
     array(\PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8', 
@@ -70,7 +70,7 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_sqlite(array $params) {
-    parent::__construct('sqlite:'.$params['name']);
+    parent::__construct('sqlite:' . $params['name']);
   }
 
   /**
@@ -79,9 +79,9 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_cubrid(array $params) {
-    parent::__construct('cubrid:host='.$params['host'].';dbname='.$params['name'].';port='.$params['port'],
+    parent::__construct('cubrid:host=' . $params['host'] . ';dbname=' . $params['name'] . ';port=' . $params['port'],
     $params['user'],
-    $params['pass'],array(
+    $params['pass'], array(
       \PDO::ATTR_EMULATE_PREPARES => false,
       \PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION
     ));
@@ -93,9 +93,9 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_firebird(array $params) {
-    parent::__construct('firebird:dbname='.$params['host'].':'.$params['name'],
+    parent::__construct('firebird:dbname=' . $params['host'] . ':' . $params['name'],
     $params['user'],
-    $params['pass'],array(
+    $params['pass'], array(
       \PDO::ATTR_EMULATE_PREPARES => false,
       \PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION
     ));
@@ -107,7 +107,7 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_odbc(array $params) {
-    parent::__construct('odbc:'.$params['name'],
+    parent::__construct('odbc:' . $params['name'],
       $params['user'],
       $params['pass']
     ); 
@@ -121,12 +121,12 @@ 
   final private function motor_oracle(array $params) {
     parent::__construct('oci:dbname=(DESCRIPTION =
     (ADDRESS_LIST =
-      (ADDRESS = (PROTOCOL = '.$params['protocol'].')(HOST = '.$params['host'].')(PORT = '.$params['port'].'))
+      (ADDRESS = (PROTOCOL = '.$params['protocol'] . ')(HOST = ' . $params['host'] . ')(PORT = ' . $params['port'] . '))
     )
     (CONNECT_DATA =
-      (SERVICE_NAME = '.$params['name'].')
+      (SERVICE_NAME = '.$params['name'] . ')
     )
-    );charset=utf8',$params['database']['user'],$params['database']['pass'],
+    );charset=utf8',$params['database']['user'], $params['database']['pass'],
     array(\PDO::ATTR_EMULATE_PREPARES => false,
       \PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION,
       \PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC
@@ -139,9 +139,9 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_postgresql(array $params) {
-    parent::__construct('pgsql:host='.$params['host'].';dbname='.$params['name'].';charset=utf8',
+    parent::__construct('pgsql:host=' . $params['host'] . ';dbname=' . $params['name'] . ';charset=utf8',
     $params['user'],
-    $params['pass'],array(
+    $params['pass'], array(
       \PDO::ATTR_EMULATE_PREPARES => false,
       \PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION
     ));
@@ -154,9 +154,9 @@ 
     * @param array $params: Lista de parámetros de configuración
   */
   final private function motor_mssql(array $params) {
-    parent::__construct('sqlsrv:Server='.$params['host'].';Database='.$params['name'].';ConnectionPooling=0',
+    parent::__construct('sqlsrv:Server=' . $params['host'] . ';Database=' . $params['name'] . ';ConnectionPooling=0',
     $params['user'],
-    $params['pass'],array(\PDO::ATTR_EMULATE_PREPARES => false,
+    $params['pass'], array(\PDO::ATTR_EMULATE_PREPARES => false,
       \PDO::ATTR_ERRMODE =>
       \PDO::ERRMODE_EXCEPTION
     ));
@@ -178,7 +178,7 @@ 
 
     try {
       # Verificar existencia del motor
-      if(method_exists($this,'motor_' . $motor)) {
+      if (method_exists($this, 'motor_' . $motor)) {
         $this->{'motor_' . $motor}(array(
           'host' => $config['database']['host'],
           'name' => $name,
@@ -188,7 +188,7 @@ 
           'protocol' => $config['database']['protocol']
         ));
       } else {
-        throw new \RuntimeException('Motor '. $motor .' de conexión no identificado.');
+        throw new \RuntimeException('Motor ' . $motor . ' de conexión no identificado.');
       }
     } catch (\PDOException $e) {
       throw new \RuntimeException('Problema al conectar con la base de datos: ' . $e->getMessage());

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

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