Apply fixes from StyleCI

* release/1.17:
  Eliminados los mensajes de aviso de php en todos los archivos php, actualizada la versión y el changelog
  Modificado el README para eliminar los saltos de línea introducidos por el editor online
  Modificado el README para eliminar los saltos de línea introducidos por el editor online
  Modificado el README para eliminar los saltos de línea introducidos por el editor online
  Modificados los programas para eliminar todos los "warnings" de php

Ajax.php

@@ -1,6 +1,5 @@
 <?php
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -16,7 +15,6 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
 //Clase encargada de procesar las peticiones ajax
 require_once 'inc/configuracion.inc';
@@ -25,7 +23,8 @@ require_once 'Sql.php';
 $ajax = new Ajax();
 echo $ajax->procesa();
 
-class Ajax {
+class Ajax
+{
     private $sql;
     private $tabla;
 
@@ -34,55 +33,61 @@ class Ajax {
         $this->sql = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
         if ($this->sql->error()) {
             return $this->respuesta($this->mensaje(false, 'Error conectando con la Base de Datos'));
-        };
+        }
         $this->tabla = $_GET['tabla'];
     }
+
     private function respuesta($mensaje)
     {
         header('Content-Type: application/json', true, 200);
+
         return $mensaje;
     }
+
     public function procesa()
     {
         $opc = $_GET['opc'];
         switch ($opc) {
-            case "get": return $this->obtiene();
-            case "put": return $this->actualiza();
+            case 'get': return $this->obtiene();
+            case 'put': return $this->actualiza();
         }
     }
+
     private function mensaje($exito, $texto)
     {
-        return json_encode(array("success" => $exito, "msj" => $texto));
+        return json_encode(['success' => $exito, 'msj' => $texto]);
     }
+
     private function actualiza()
     {
-        if(!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
-            $comando = "update " . mysql_escape_string($this->tabla) . " set " . mysql_escape_string($_POST['name']) . " = '" . mysql_escape_string($_POST['value']) . "' where id = '" . mysql_escape_string($_POST['pk']). "';";
+        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
+            $comando = 'update '.mysql_escape_string($this->tabla).' set '.mysql_escape_string($_POST['name'])." = '".mysql_escape_string($_POST['value'])."' where id = '".mysql_escape_string($_POST['pk'])."';";
             $this->sql->ejecuta($comando);
             $exito = !$this->sql->error();
             $mensaje = $this->sql->mensajeError();
             $resp = $this->mensaje($exito, $mensaje);
+
             return $this->respuesta($resp);
         }
     }
+
     private function obtiene()
     {
-        if(!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
-            $comando = "select id, descripcion from " . $this->tabla . " order by descripcion asc;";
+        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
+            $comando = 'select id, descripcion from '.$this->tabla.' order by descripcion asc;';
             $this->sql->ejecuta($comando);
             $exito = !$this->sql->error();
             $mensaje = $this->sql->mensajeError();
             if (!$exito) {
                 return $this->respuesta($this->mensaje($exito, $mensaje));
             }
-            $filas = array();
-            while($r = $this->sql->procesaResultado()) {
-                $filas[] = array($r['id'] => $r['descripcion']);
+            $filas = [];
+            while ($r = $this->sql->procesaResultado()) {
+                $filas[] = [$r['id'] => $r['descripcion']];
             }
             $resp = json_encode($filas);
+
             return $this->respuesta($resp);
         }
     }
 }
-
-?>

AportaContenido.php

@@ -1,7 +1,5 @@
 <?php
-
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,18 +15,17 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-define('PIE', '<center><a target="_blank" href="http://www.gnu.org/licenses/gpl-3.0-standalone.html"><img src="img/gplv3.png" alt="GPL v3"/></a>' .
-        '<a target="_blank" href="http://www.apache.org"><img src="img/apache.gif" alt="Sitio web creado con Apache" /></a>' .
-        '<a target="_blank" href="http://www.mysql.org"><img src="img/mysql.png" width=125 height=47 alt="Gestor de bases de datos mySQL" /></a>' .
+define('PIE', '<center><a target="_blank" href="http://www.gnu.org/licenses/gpl-3.0-standalone.html"><img src="img/gplv3.png" alt="GPL v3"/></a>'.
+        '<a target="_blank" href="http://www.apache.org"><img src="img/apache.gif" alt="Sitio web creado con Apache" /></a>'.
+        '<a target="_blank" href="http://www.mysql.org"><img src="img/mysql.png" width=125 height=47 alt="Gestor de bases de datos mySQL" /></a>'.
         '<a target="_blank" href="http://www.php.net"><img src="img/php.gif" alt="PHP Language" /></a> </center>');
-define('FORMULARIO_ACCESO', '<form name="formulario_acceso" action="index.php?registrarse" method="POST">' .
-        'Usuario<br><input type="text" name="usuario" value="" size="8" /><br><br>Clave<br><input type="password" name="clave" value="" size="8" />' .
+define('FORMULARIO_ACCESO', '<form name="formulario_acceso" action="index.php?registrarse" method="POST">'.
+        'Usuario<br><input type="text" name="usuario" value="" size="8" /><br><br>Clave<br><input type="password" name="clave" value="" size="8" />'.
         '<br><br><button type="submit" name="iniciar" class="btn btn-primary">Iniciar <span class="glyphicon glyphicon-log-in"></span></button></form>');
 define('MENSAJE_DEMO', 'Puede Iniciar sesi&oacute;n con<br>usuario <i><b>demo</b></i><br>contrase&ntilde;a <i>demo</i><br>');
 define('USUARIO_INCORRECTO', '<label class="bg-danger">Usuario y clave incorrectos!</label><br><br>');
-define('CREDITOS', '<div class="modal fade" tabindex="-1" id="creditos" role="dialog" aria-labelledby="modalCreditos" aria-hidden="true">
+define('CREDITOS_CABECERA', '<div class="modal fade" tabindex="-1" id="creditos" role="dialog" aria-labelledby="modalCreditos" aria-hidden="true">
                       <div class="modal-dialog modal-lg">
                         <div class="modal-content">
                             <div class="modal-header">
@@ -37,26 +34,12 @@ define('CREDITOS', '<div class="modal fade" tabindex="-1" id="creditos" role="di
                             </div>
                             <div class="modal-body">
                                 <div class="jumbotron">
-                                    <img src="img/logo.png" class="img-responsive img-rounded" style="float:left"> 
+                                    <img src="img/qrlogo.png" class="img-responsive img-rounded" style="float:left"> 
                                     <h1>Inventario2</h1>
                                     <p> Aplicación para controlar el inventario de un centro educativo.</p>
-                                    <p>En la aplicación se hace uso de los siguientes módulos y/o bibliotecas</p>
-                                    <table class="table table-condensed">
-                                     <thead><tr><th>Biblioteca/Módulo</th><th>Licencia</th></tr></thead>
-                                     <tbody>
-                                      <tr><td><a href="http://getbootstrap.com/" target="_blank">Twitter Bootstrap</a></td><td><a target="_blank" href="https://github.com/twbs/bootstrap/blob/master/LICENSE">MIT</a></td>
-                                      <tr><td><a href="http://www.fpdf.org/" target="_blank">FPDF</a></td><td>Libre</td>
-                                      <tr><td><a href="http://phpqrcode.sourceforge.net/" target="_blank">PHP QR Code Enconder</a></td><td><a target="_blank" href="http://www.gnu.org/licenses/lgpl-3.0.txt">LGPL</a></td>
-                                      <tr><td><a href="http://stefangabos.ro/php-libraries/zebra-image/" target="_blank">Zebra_Image</a></td><td><a target="_blank" href="http://www.gnu.org/licenses/lgpl-3.0.txt">LGPL</a></td>
-                                      <tr><td><a href="http://jasny.github.io/bootstrap/" target="_blank">Jasny Bootstrap</a></td><td><a target="_blank" href="http://www.apache.org/licenses/LICENSE-2.0">Apache 2.0</a></td>
-                                      <tr><td><a href="http://1000hz.github.io/bootstrap-validator/" target="_blank">Bootstrap Validator</a></td><td><a target="_blank" href="https://github.com/1000hz/bootstrap-validator/blob/master/LICENSE">MIT</a></td>
-                                      <tr><td><a href="https://github.com/tkrotoff/jquery-simplecolorpicker" target="_blank">jquery-simplecolorpicker</a></td><td><a target="_blank" href="https://github.com/tkrotoff/jquery-simplecolorpicker/blob/master/LICENSE.txt">MIT</a></td>
-                                      <tr><td><a href="http://eonasdan.github.io/bootstrap-datetimepicker/" target="_blank">Bootstrap datetimepicker</a></td><td><a target="_blank" href="https://github.com/Eonasdan/bootstrap-datetimepicker/blob/master/src/js/bootstrap-datetimepicker.js">MIT</a></td>
-                                      <tr><td><a href="http://silviomoreto.github.io/bootstrap-select/" target="_blank">Bootstrap-select</a></td><td><a target="_blank" href="https://github.com/silviomoreto/bootstrap-select">MIT</a></td>
-                                      <tr><td><a href="https://github.com/vitalets/x-editable" target="_blank">X-editable</a></td><td><a target="_blank" href="https://github.com/vitalets/x-editable/blob/master/LICENSE-MIT">MIT</a></td>
-                                     </tbody>
-                                    </table>
-                                    <p><h5>Copyright &copy; 2008-2014 Ricardo Montañana Gómez</h4>
+                                    <p>En la aplicación se hace uso de los siguientes módulos y/o bibliotecas</p>');
+
+define('CREDITOS_PIE', '                <p><h5>Copyright &copy; 2008-2014 Ricardo Montañana Gómez</h5>
                                     <h5><small>Esta aplicación se distribuye con licencia <a target="_blank" href="http://www.gnu.org/licenses/gpl-3.0.html">GPLv3 </a></small></h5></p>
                                 </div>
                             </div>
@@ -65,18 +48,17 @@ define('CREDITOS', '<div class="modal fade" tabindex="-1" id="creditos" role="di
                     </div>');
 
 // Esta clase aportará el contenido a la plantilla
-class AportaContenido {
-
+class AportaContenido
+{
     /**
-     * 
-     * @var boolean Aporta información sobre si el usuario está registrado o no.
+     * @var bool Aporta información sobre si el usuario está registrado o no.
      */
     private $registrado;
 
     /**
      * @var string Nombre del usuario
      */
-    private $usuario = NULL;
+    private $usuario = null;
 
     /**
      * @var Menu Menú de la página.
@@ -94,7 +76,7 @@ class AportaContenido {
     private $opcionActual;
 
     /**
-     * @var boolean Usuario y clave incorrectos?
+     * @var bool Usuario y clave incorrectos?
      */
     private $usuario_inc = false;
 
@@ -104,19 +86,20 @@ class AportaContenido {
     private $perfil;
 
     /**
-     *
      * @var array Datos pasados en la URL
      */
-    private $datosURL = array();
+    private $datosURL = [];
 
     // El constructor necesita saber cuál es la opción actual
+
     /**
      * Constructor de la clase.
+     *
      * @param BaseDatos $baseDatos  Manejador de la base de datos
-     * @param boolean $registrado usuario registrado si/no
-     * @param String $usuario Nombre del usuario
+     * @param bool      $registrado usuario registrado si/no
+     * @param string    $usuario    Nombre del usuario
      * @param array     $perfil     Permisos de acceso del usuario
-     * @param String $opcion Opción elegida por el usuario
+     * @param string    $opcion     Opción elegida por el usuario
      */
     public function __construct($baseDatos, $registrado, $usuario, $perfil, $opcion)
     {
@@ -129,21 +112,54 @@ class AportaContenido {
     }
 
     /**
-     * Devuelve la fecha actual
+     * Devuelve una tabla HTML con el contenido de las bibliotecas/módulos utilizadas en la aplicación
+     * Si el perfil del usuario es de Configuración devuelve también las versiones de las bibliotecas.
+     *
+     * @return string
+     */
+    public function creaTablaAcercaDe()
+    {
+        $poner = $this->perfil['Config'];
+        $tabla = '<table class="table table-condensed">';
+        $tabla .= '<thead><tr><th>Biblioteca/Módulo</th>'.($poner ? '<th>Versión</th>' : '').'<th>Licencia</th></tr></thead>';
+        $tabla .= '<tbody>';
+        $tabla .= '<tr><td><a href="http://jquery.com/" target="_blank">jquery</a></td>'.($poner ? '<td>2.1.0</td>' : '').'<td><a target="_blank" href="https://jquery.org/license/">MIT</a></td>';
+        $tabla .= '<tr><td><a href="http://getbootstrap.com/" target="_blank">Twitter Bootstrap</a></td>'.($poner ? '<td>3.1.1</td>' : '').'<td><a target="_blank" href="https://github.com/twbs/bootstrap/blob/master/LICENSE">MIT</a></td>';
+        $tabla .= '<tr><td><a href="http://www.fpdf.org/" target="_blank">FPDF</a></td>'.($poner ? '<td>1.7</td>' : '').'<td>Libre</td>';
+        $tabla .= '<tr><td><a href="http://phpqrcode.sourceforge.net/" target="_blank">PHP QR Code Enconder</a></td>'.($poner ? '<td>1.1.4</td>' : '').'<td><a target="_blank" href="http://www.gnu.org/licenses/lgpl-3.0.txt">LGPL</a></td>';
+        $tabla .= '<tr><td><a href="http://stefangabos.ro/php-libraries/zebra-image/" target="_blank">Zebra_Image</a></td>'.($poner ? '<td>2.2.3</td>' : '').'<td><a target="_blank" href="http://www.gnu.org/licenses/lgpl-3.0.txt">LGPL</a></td>';
+        $tabla .= '<tr><td><a href="http://jasny.github.io/bootstrap/" target="_blank">Jasny Bootstrap</a></td>'.($poner ? '<td>3.1.0</td>' : '').'<td><a target="_blank" href="http://www.apache.org/licenses/LICENSE-2.0">Apache 2.0</a></td>';
+        $tabla .= '<tr><td><a href="http://1000hz.github.io/bootstrap-validator/" target="_blank">Bootstrap Validator</a></td>'.($poner ? '<td>0.2.1</td>' : '').'<td><a target="_blank" href="https://github.com/1000hz/bootstrap-validator/blob/master/LICENSE">MIT</a></td>';
+        $tabla .= '<tr><td><a href="https://github.com/tkrotoff/jquery-simplecolorpicker" target="_blank">jquery-simplecolorpicker</a></td>'.($poner ? '<td>0.3.0</td>' : '').'<td><a target="_blank" href="https://github.com/tkrotoff/jquery-simplecolorpicker/blob/master/LICENSE.txt">MIT</a></td>';
+        $tabla .= '<tr><td><a href="http://eonasdan.github.io/bootstrap-datetimepicker/" target="_blank">Bootstrap datetimepicker</a></td>'.($poner ? '<td>2.1.32</td>' : '').'<td><a target="_blank" href="https://github.com/Eonasdan/bootstrap-datetimepicker/blob/master/src/js/bootstrap-datetimepicker.js">MIT</a></td>';
+        $tabla .= '<tr><td><a href="http://silviomoreto.github.io/bootstrap-select/" target="_blank">Bootstrap-select</a></td>'.($poner ? '<td>1.5.4</td>' : '').'<td><a target="_blank" href="https://github.com/silviomoreto/bootstrap-select">MIT</a></td>';
+        $tabla .= '<tr><td><a href="https://github.com/vitalets/x-editable" target="_blank">X-editable</a></td>'.($poner ? '<td>1.5.1</td>' : '').'<td><a target="_blank" href="https://github.com/vitalets/x-editable/blob/master/LICENSE-MIT">MIT</a></td>';
+        $tabla .= '<tr><td><a href="http://momentjs.com/" target="_blank">Moment.js</a></td>'.($poner ? '<td>2.5.1</td>' : '').'<td><a target="_blank" href="https://github.com/moment/moment/blob/develop/LICENSE">MIT</a></td>';
+        $tabla .= '</tbody>';
+        $tabla .= '</table>';
+
+        return $tabla;
+    }
+
+    /**
+     * Devuelve la fecha actual.
+     *
      * @param string $formato formato de devolución de la fecha
      * @param string $idioma  idioma para formatear la fecha, p.ej. es_ES
+     *
      * @return string
      */
     public function fechaActual($formato = '', $idioma = 'es_ES')
     {
-        if ($formato == '')
-            $formato = "%d-%b-%y";
+        if ($formato == '') {
+            $formato = '%d-%b-%y';
+        }
         setlocale(LC_TIME, $idioma);
+
         return strftime($formato);
     }
 
     /**
-     * 
      * @return string Mensaje el usuario debe registrarse.
      */
     private function mensajeRegistro()
@@ -153,73 +169,87 @@ class AportaContenido {
 
     // Procesaremos todas las invocaciones a métodos en
     // la función __call()
+
     /**
      * Procesa las peticiones de contenido de la plantilla.
+     *
      * @param string $metodo     Método a ejecutar
      * @param string $parametros Parámetros del método
+     *
      * @return string Contenido devuelto por el método
      */
     public function __call($metodo, $parametros)
     {
         switch ($metodo) { // Dependiendo del método invocado
-            case 'titulo': // devolvemos el título
-                return PROGRAMA . " v" . VERSION;
             case 'usuario':
-                if ($this->registrado)
+                if ($this->registrado) {
                     return "Usuario=$this->usuario";
-                else
+                } else {
                     return '';
+                }
             case 'fecha':
                 $script = '<script type="text/javascript">
                                 $(function () {
-                                    $(' . "'#fechaCabecera'" . ").datetimepicker({
+                                    $('."'#fechaCabecera'".").datetimepicker({
                                         pick12HourFormat: false,
                                         language: 'es',
                                         pickTime: false
                                         });
                                 });
                                 </script>";
-                $campo = '<input type="hidden" name="fechaCabecera" id="fechaCabecera" value="' . $this->fechaActual("%d/%m/%Y") . '">';
-                $etiqueta = '<label for="fechaCabecera" onClick="$(' . "'#fechaCabecera'" . ").data('DateTimePicker').show();" . '">' . $this->fechaActual() . '</label>';
-                return $etiqueta . $campo . $script;
-            case 'aplicacion': return PROGRAMA . " v" . VERSION;
+                $campo = '<input type="hidden" name="fechaCabecera" id="fechaCabecera" value="'.$this->fechaActual('%d/%m/%Y').'">';
+                $etiqueta = '<label for="fechaCabecera" onClick="$('."'#fechaCabecera'".").data('DateTimePicker').show();".'">'.$this->fechaActual().'</label>';
+
+                return $etiqueta.$campo.$script;
+            case 'aplicacion':
+                $nombre = explode(' ', PROGRAMA);
+                $nombre = $nombre[2];
+
+                return $nombre.' v'.VERSION;
             case 'menu': // el menú
                 if ($this->registrado) {
                     return $this->miMenu->insertaMenu();
                 } else {
                     $salida = FORMULARIO_ACCESO;
                     if ($this->usuario_inc) {
-                        $salida.=USUARIO_INCORRECTO;
+                        $salida .= USUARIO_INCORRECTO;
                     }
                     //$salida.=MENSAJE_DEMO;
                     return $salida;
                 }
             case 'opcion':
-                list($opcion, $parametro) = explode("&", $this->opcionActual);
+                if (strstr($this->opcionActual, '&')) {
+                    list($opcion, $parametro) = explode('&', $this->opcionActual);
+                } else {
+                    $opcion = $this->opcionActual;
+                    $parametro = '';
+                }
                 switch ($opcion) {
                     case 'bienvenido':
-                        return "Men&uacute; Principal";
+                        return 'Men&uacute; Principal';
                     case 'principal':
-                        return "Pantalla Inicial";
-                    case 'articulos': $opcion = "art&iacute;culos";
+                        return 'Pantalla Inicial';
+                    case 'articulos': $opcion = 'art&iacute;culos';
                     case 'elementos':
                     case 'ubicaciones':
                     case 'usuarios':
                     case 'test':
-                        return "Mantenimiento " . ucfirst($opcion);
+                        return 'Mantenimiento '.ucfirst($opcion);
                     case 'configuracion':
                         return 'Configuraci&oacute;n y Preferencias';
-                    case 'informeInventario':return "Informe de Inventario";
+                    case 'informeInventario':return 'Informe de Inventario';
                     case 'descuadres':return 'Informe de descuadres';
                     case 'importacion': return 'Importaci&oacute;n de datos';
                     case 'copiaseg': return 'Copia de seguridad de datos';
                 }
+
                 return '';
             case 'control':
-                if ($this->registrado)
+                if ($this->registrado) {
                     return '<a href="index.php?cerrarSesion">Cerrar Sesi&oacute;n <span class="glyphicon glyphicon-log-out"></span></a>';
-                else
+                } else {
                     return '';
+                }
             // Para incluir el contenido central de la página
             case 'contenido':
                 // tendremos en cuenta cuál es la opción actual
@@ -229,45 +259,56 @@ class AportaContenido {
 //                if (!$this->registrado) {
 //                    return $this->mensajeRegistro();
 //                }
-                list($opcion, $parametro) = explode("&", $this->opcionActual);
+                if (strstr($this->opcionActual, '&')) {
+                    list($opcion, $parametro) = explode('&', $this->opcionActual);
+                } else {
+                    $opcion = $this->opcionActual;
+                    $parametro = '';
+                }
                 switch ($opcion) {
                     case 'bienvenido':
                         $mensaje = '<div class="alert alert-success">';
-                        $mensaje .= 'Bienvenid@ ' . $this->usuario . '</div>';
+                        $mensaje .= 'Bienvenid@ '.$this->usuario.'</div>';
                     case 'principal': // contenido inicial
-
+                        $mensaje = '';
                         $creditos = "$('#creditos').modal({keyboard: false});";
-                        $centro = '<div class="well well-sm">' . CENTRO . '</div>';
-                        return $mensaje . '<br><br><center><img src="img/logo.png" alt="' . PROGRAMA . '" onClick="' . $creditos . '" >' .
-                                '<br><br><label onClick="' . $creditos . '">' . $centro . '</label></center><br><br>' . CREDITOS;
+                        $centro = '<div class="well well-sm">'.CENTRO.'</div>';
+                        $tabla = $this->creaTablaAcercaDe();
+                        $rama_texto = trim(substr(file_get_contents('.git/HEAD'), 16));
+                        $rama = ($rama_texto != 'master' ? '<br><button class="btn btn-warning btn-xs" type="button"onClick="'.$creditos.'"><span class="glyphicon glyphicon-cog"></span> '.$rama_texto.'</button></center>' : '');
+
+                        return $mensaje.'<br><br><center><img src="img/qrlogo.png" alt="'.PROGRAMA.'" onClick="'.$creditos.'" >'.
+                                '<br><br><label onClick="'.$creditos.'">'.$centro.'</label>'.$rama.'</center>'.CREDITOS_CABECERA.$tabla.CREDITOS_PIE;
                     case 'articulos':
                     case 'ubicaciones':
                     case 'test':
                     case 'elementos':
                         $this->cargaDatosURL();
-                        if ($this->datosURL['opc'] == "informe") {
-                            if (!$this->pefil['Informe']) {
+                        if ($this->datosURL['opc'] == 'informe') {
+                            if ($this->perfil['Informe']) {
                                 $this->procesaURL();
-                                $fichero = 'xml/informe' . ucfirst($opcion) . '.xml';
-                                $salida = 'tmp/informe' . ucfirst($opcion) . '.xml';
+                                $fichero = 'xml/informe'.ucfirst($opcion).'.xml';
+                                $salida = TMP.'/informe'.ucfirst($opcion).'.xml';
                                 //Establece los posibles parámetros del listado.
                                 $orden = $this->datosURL['orden'];
-                                $sentido = $this->datosURL['sentido'] == "asc" ? ' ' : ' desc ';
+                                $sentido = $this->datosURL['sentido'] == 'asc' ? ' ' : ' desc ';
                                 $filtro = isset($this->datosURL['buscar']) ? $this->bdd->filtra($this->datosURL['buscar']) : '';
-                                $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla ' . $fichero);
-                                $plantilla = str_replace("{filtro}", $filtro, $plantilla);
-                                $plantilla = str_replace("{orden}", $orden . $sentido, $plantilla);
-                                file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla ' . $salida);
+                                $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla '.$fichero);
+                                $plantilla = str_replace('{filtro}', $filtro, $plantilla);
+                                $plantilla = str_replace('{orden}', $orden.$sentido, $plantilla);
+                                file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla '.$salida);
                                 $informe = new InformePDF($this->bdd, $salida, $this->registrado);
                                 $informe->crea($salida);
                                 $informe->cierraPDF();
+
                                 return $this->devuelveInforme($informe);
                             } else {
-                                return $this->mensajePermisos("Informes");
+                                return $this->mensajePermisos('Informes');
                             }
                         }
                         if ($this->perfil['Consulta']) {
                             $ele = new Mantenimiento($this->bdd, $this->perfil, $opcion);
+
                             return $ele->ejecuta();
                         } else {
                             return $this->mensajePermisos(ucfirst($opcion));
@@ -275,28 +316,30 @@ class AportaContenido {
                     case 'usuarios':
                         if ($this->perfil['Usuarios']) {
                             $this->cargaDatosURL();
-                            if ($this->datosURL['opc'] == "informe") {
+                            if ($this->datosURL['opc'] == 'informe') {
                                 if (!$this->pefil['Informe']) {
                                     $this->procesaURL();
-                                    $fichero = 'xml/informe' . ucfirst($opcion) . '.xml';
-                                    $salida = 'tmp/informe' . ucfirst($opcion) . '.xml';
+                                    $fichero = 'xml/informe'.ucfirst($opcion).'.xml';
+                                    $salida = TMP.'/informe'.ucfirst($opcion).'.xml';
                                     //Establece los posibles parámetros del listado.
                                     $orden = $this->datosURL['orden'];
-                                    $sentido = $this->datosURL['sentido'] == "asc" ? ' ' : ' desc ';
+                                    $sentido = $this->datosURL['sentido'] == 'asc' ? ' ' : ' desc ';
                                     $filtro = isset($this->datosURL['buscar']) ? $this->bdd->filtra($this->datosURL['buscar']) : '';
-                                    $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla ' . $fichero);
-                                    $plantilla = str_replace("{filtro}", $filtro, $plantilla);
-                                    $plantilla = str_replace("{orden}", $orden . $sentido, $plantilla);
-                                    file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla ' . $salida);
+                                    $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla '.$fichero);
+                                    $plantilla = str_replace('{filtro}', $filtro, $plantilla);
+                                    $plantilla = str_replace('{orden}', $orden.$sentido, $plantilla);
+                                    file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla '.$salida);
                                     $informe = new InformePDF($this->bdd, $salida, $this->registrado);
                                     $informe->crea($salida);
                                     $informe->cierraPDF();
+
                                     return $this->devuelveInforme($informe);
                                 } else {
-                                    return $this->mensajePermisos("Informes");
+                                    return $this->mensajePermisos('Informes');
                                 }
                             }
                             $ele = new Mantenimiento($this->bdd, $this->perfil, $opcion);
+
                             return $ele->ejecuta();
                         } else {
                             return $this->mensajePermisos('Usuarios');
@@ -304,6 +347,7 @@ class AportaContenido {
                     case 'configuracion':
                         if ($this->perfil['Config']) {
                             $conf = new Configuracion();
+
                             return $conf->ejecuta();
                         } else {
                             return $this->mensajePermisos('Configuraci&oacute;n');
@@ -311,6 +355,7 @@ class AportaContenido {
                     case 'informeInventario':
                         if ($this->perfil['Informe']) {
                             $info = new InformeInventario($this->bdd);
+
                             return $info->ejecuta();
                         } else {
                             return $this->mensajePermisos('Informes');
@@ -318,36 +363,39 @@ class AportaContenido {
                     case 'importacion':
                         if ($this->perfil['Modificacion'] && $this->perfil['Borrado']) {
                             $import = new Importacion($this->bdd, $this->registrado);
+
                             return $import->ejecuta();
                         } else {
-                            return $this->mensajePermisos("Actualizaci&oacute;n, creaci&oacute;n y borrado de elementos");
+                            return $this->mensajePermisos('Actualizaci&oacute;n, creaci&oacute;n y borrado de elementos');
                         }
                     case 'copiaseg':
                         if ($this->perfil['Config']) {
                             $copia = new CopiaSeguridad();
-                            if ($_GET['confirmado'] == "1") {
+                            if (isset($_GET['confirmado']) && $_GET['confirmado'] == '1') {
                                 if (!$copia->creaCopia()) {
-                                    $tipo = "danger";
-                                    $cabecera = "ERROR";
+                                    $tipo = 'danger';
+                                    $cabecera = 'ERROR';
                                 } else {
-                                    $tipo = "info";
-                                    $cabecera = "INFORMACIÓN";
+                                    $tipo = 'info';
+                                    $cabecera = 'INFORMACIÓN';
                                 }
+
                                 return $this->panel($cabecera, $copia->mensaje(), $tipo);
                             } else {
                                 return $copia->dialogo();
                             }
                         } else {
-                            return $this->mensajePermisos("Copias de seguridad");
+                            return $this->mensajePermisos('Copias de seguridad');
                         }
                 } // Fin del contenido
             case 'usuario_incorrecto':
                 $this->usuario_inc = true;
+
                 return;
             case 'registro': // Si está registrado mostrar bienvenida
                 // si no, un enlace
                 if ($this->bEstaRegistrado) {
-                    return "Bienvenid@ <b>$this->sUsuario</b><hr />" .
+                    return "Bienvenid@ <b>$this->sUsuario</b><hr />".
                             '<a href="index.php?cerrarSesion">Cerrar sesi&oacute;n</a>';
                 } else {
                     return '';
@@ -368,36 +416,36 @@ class AportaContenido {
     }
 
     /**
-     *
      * @param string $tipo
+     *
      * @return string
      */
     public function mensajePermisos($tipo)
     {
-        return $this->panel("ERROR", "No tiene permiso para acceder a $tipo", "danger");
+        return $this->panel('ERROR', "No tiene permiso para acceder a $tipo", 'danger');
     }
 
     private function devuelveInforme($informe)
     {
-        $letras = "abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
-        $nombre = "tmp/informe" . substr(str_shuffle($letras), 0, 10) . ".pdf";
+        $letras = 'abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
+        $nombre = TMP.'/informe'.substr(str_shuffle($letras), 0, 10).'.pdf';
         $informe->guardaArchivo($nombre);
+
         return '<div class="container">
-                    <!--<a href="' . $nombre . '" target="_blank"><span class="glyphicon glyphicon-cloud-download" style="font-size:1.5em;"></span>Descargar Informe</a>--> 
-                    <object data="' . $nombre . '" type="application/pdf" width="100%" height="700" style="float:left;">
+                    <!--<a href="'.$nombre.'" target="_blank"><span class="glyphicon glyphicon-cloud-download" style="font-size:1.5em;"></span>Descargar Informe</a>--> 
+                    <object data="'.$nombre.'" type="application/pdf" width="100%" height="700" style="float:left;">
                     </object>
                 </div>';
     }
 
     public function panel($cabecera, $mensaje, $tipo)
     {
-        $panel = '<div class="panel panel-' . $tipo . '"><div class="panel-heading">';
-        $panel .= '<h3 class="panel-title">' . $cabecera . '</h3></div>';
+        $panel = '<div class="panel panel-'.$tipo.'"><div class="panel-heading">';
+        $panel .= '<h3 class="panel-title">'.$cabecera.'</h3></div>';
         $panel .= '<div class="panel-body">';
         $panel .= $mensaje;
         $panel .= '</div>';
+
         return $panel;
     }
 }
-
-?>

CHANGELOG

@@ -1,3 +1,42 @@
+Version 1.17 29-07-2014
+-Eliminados los mensajes de aviso de php en todos los archivos php
+
+Versión 1.16 28-07-2014
+-Fix #41. Arregla las llamadas a Instalar.php que se hacían desde Inventario.php y desde Instalar.php
+
+Versión 1.15 29-06-2014
+-Crear la opción de clonar registro en Mantenimiento.
+-Crear iconos de clonado en todos los estilos.
+-Corregido determinaAccion en Imagen para aceptar el clonado
+-Corregido codificación/decodificación de la cadena de búsqueda en la URL
+
+Versión 1.14.1 02-06-2014
+-Añadidos enlaces a manual y a aplicación de ejemplo en readme.md
+-Arreglado que los créditos salgan centrados en lugar de alineados a la derecha
+-Añadido la 'v' para indicar la versión en la cabecera de los informes
+-Acortado el nombre de la aplicación en la página principal para que en dispositivos pequeños se visualice bien con el icono nuevo del manual
+
+Versión 1.14 01-06-2014
+-Añadido icono de acceso a la documentación en la barra superior
+-Corregido cierre etiqueta h5 en pie de créditos
+-Añade la rama git en la pantalla de inicio en caso de que no sea la rama master, con un botón que al pulsarlo salen los créditos
+-Fix #39. Devuelve la versión de las bibliotecas/módulos utilizados en la aplicación si el usuario tiene perfil de Configuración, en caso contrario tan sólo el listado de bibliotecas y su licencia.
+-Corregido el nombre de la clase base en Pdf_mysql para que no haya problemas en sistemas que diferencian mayúsculas/minúsculas
+-Añadido el botón volver en Configuración
+-Fix #36. Añadido límite caracteres en los campos de texto a 35 caracteres en Configuración. Añadido el número de columnas en resoluciones pequeñas y grandes en Configuración.
+
+Versión 1.12 06-05-2014
+-Mantenimiento muestra el 'Titulo' del campo tanto en Consulta como en el formulario de edición
+-Muestra cuadro de búsqueda y mensaje correcto cuando no se encuentra la cadena de búsqueda en Mantenimiento
+-Añadidos mensajes de ayuda en los campos de configuración
+-Añadida la columna de cantidad ubicada en mantenimiento de artículos
+-Fix #35. Cambiados los informes de Articulos y Ubicaciones para recoger el campo Nº de elementos
+-Fix #35. Añadido el campo Nº de elementos en el mantenimiento de Articulos y Ubicaciones a través de archivo xml
+-Fix #31. Añadido el directorio temporal en el archivo de configuración
+
+Versión 1.11b 26-04-2014
+-Cambiada la referencia de etiquetas Apli
+
 Versión 1.11 26-04-2014
 -Fix #34 Corregido que salga una etiqueta por cada elemento reflejado en cantidad.
 -Fix #33 Hay que hacer doble click para editar ajax

Configuracion.php

@@ -1,7 +1,5 @@
 <?php
-
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,20 +15,20 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
-class Configuracion {
-    private $configuracion = "inc/configuracion.inc";
-    private $confNueva = "inc/configuracion.new";
-    private $confAnterior = "inc/configuracion.ant";
+class Configuracion
+{
+    private $configuracion = 'inc/configuracion.inc';
+    private $confNueva = 'inc/configuracion.new';
+    private $confAnterior = 'inc/configuracion.ant';
     private $datosConf;
     //Campos del fichero de configuración que se van a editar.
-    private $lista = array('SERVIDOR', 'PUERTO', 'BASEDATOS', 'BASEDATOSTEST', 'USUARIO', 'CLAVE', 'CENTRO', 'NUMFILAS', 'ESTILO', 'PLANTILLA', 'COLORLAT', 'COLORFON', 'MYSQLDUMP', 'GZIP');
+    private $lista = ['SERVIDOR', 'PUERTO', 'BASEDATOS', 'BASEDATOSTEST', 'USUARIO', 'CLAVE', 'CENTRO', 'NUMFILAS', 'ESTILO', 'PLANTILLA', 'COLORLAT', 'COLORFON', 'MYSQLDUMP', 'GZIP', 'TMP'];
     private $campos;
 
     public function __construct()
     {
-        $this->campos = implode(",", $this->lista);
+        $this->campos = implode(',', $this->lista);
     }
 
     //Hecho público para poder efectuar los tests correspondientes.
@@ -46,23 +44,29 @@ class Configuracion {
 
     public function obtieneDatos($linea, &$clave, &$valor)
     {
-        $filtro = str_replace("'", "", $linea);
-        list($clave, $valor) = explode(",", $filtro);
-        list($resto, $campo) = explode("(", $clave);
-        list($valor, $resto) = explode(")", $valor);
-        list($resto, $clave) = explode("(", $clave);
+        $filtro = str_replace("'", '', $linea);
+        list($clave, $valor) = explode(',', $filtro);
+        list($resto, $campo) = explode('(', $clave);
+        list($valor, $resto) = explode(')', $valor);
+        list($resto, $clave) = explode('(', $clave);
         $valor = trim($valor);
     }
 
-    public function ejecuta() {      
+    private function creaTitulo($titulo, $ayuda)
+    {
+        return '<td style="vertical-align:middle"><a class="dato" href="#" data-placement="right" data-content="'.$ayuda.'">'.$titulo.'</a></td>';
+    }
+
+    public function ejecuta()
+    {
         $fichero = $this->obtieneFichero();
         $datos = explode("\n", $fichero);
         $grabar = isset($_POST['SERVIDOR']);
         if ($grabar) {
-            $fsalida = @fopen($this->confNueva, "wb");
+            $fsalida = @fopen($this->confNueva, 'wb');
         }
         foreach ($datos as $linea) {
-            if (stripos($linea, "DEFINE") !== false) {
+            if (stripos($linea, 'DEFINE') !== false) {
                 //Comprueba que tenga una definición correcta
                 $this->obtieneDatos($linea, $clave, $valor);
                 $this->datosConf[$clave] = $valor;
@@ -74,94 +78,98 @@ class Configuracion {
                 //$salida .= "Post=" . var_export($_POST, true);
             }
             if ($grabar) {
-                $registro = substr($linea, 0, 2) == "?>" ? $linea : $linea . "\n";
+                $registro = substr($linea, 0, 2) == '?>' ? $linea : $linea."\n";
                 fwrite($fsalida, $registro);
             }
         }
-        $salida.=$this->formulario();
+        $salida = $this->formulario();
         if ($grabar) {
-            $salida.='<div class="alert alert-success">Configuraci&oacute;n guardada correctamente</div>';
+            $salida .= '<div class="alert alert-success">Configuraci&oacute;n guardada correctamente</div>';
             fclose($fsalida);
             //unlink($this->confAnterior);
             rename($this->configuracion, $this->confAnterior);
             rename($this->confNueva, $this->configuracion);
             unlink($this->confAnterior);
         }
+
         return $salida;
     }
 
-    private function formulario() {
-        $coloresLateral = array("Original" => "#C4FAEC", "Verde" => "#7bd148", "Azul marino" => "#5484ed", "Azul" => "#a4bdfc", "Turquesa" => "#46d6db",
-            "Verde claro" => "#7ae7bf", "Verde oscuro" => "#51b749", "Amarillo" => "#fbd75b", "Naranja" => "#ffb878", "Morado" => "#6633FF",
-            "Rojo oscuro" => "#dc2127", "P&uacute;rpura" => "#dbadff", "Gris" => "#e1e1e1");
-        $coloresFondo = array("Verde" => "#7bd148", "Azul marino" => "#5484ed", "Azul" => "#a4bdfc", "Turquesa" => "#46d6db",
-            "Verde claro" => "#7ae7bf", "Verde oscuro" => "#51b749", "Amarillo" => "#fbd75b", "Naranja" => "#ffb878", "Rojo" => "#ff887c",
-            "Rojo oscuro" => "#dc2127", "P&uacute;rpura" => "#dbadff", "Gris" => "#e1e1e1", "Original" => '#F3FEC8');
-        $personal = $this->datosConf['ESTILO'] == "personal" ? 'selected' : ' ';
-        $bluecurve = $this->datosConf['ESTILO'] == "bluecurve" ? 'selected' : ' ';
-        $cristal = $this->datosConf['ESTILO'] == "cristal" ? 'selected' : ' ';
-        $bootst = $this->datosConf['ESTILO'] == "bootstrap" ? 'selected' : ' ';
-        $normal = $this->datosConf['PLANTILLA'] == "normal" ? 'selected' : ' ';
-        $bootstrap = $this->datosConf['PLANTILLA'] == "bootstrap" ? 'selected' : ' ';
-        $salida = '<center><div class="col-sm-4 col-md-6"><form name="configura" method="post">';
+    private function formulario()
+    {
+        $coloresLateral = ['Original' => '#C4FAEC', 'Verde' => '#7bd148', 'Azul marino' => '#5484ed', 'Azul' => '#a4bdfc', 'Turquesa' => '#46d6db',
+            'Verde claro'             => '#7ae7bf', 'Verde oscuro' => '#51b749', 'Amarillo' => '#fbd75b', 'Naranja' => '#ffb878', 'Morado' => '#6633FF',
+            'Rojo oscuro'             => '#dc2127', 'P&uacute;rpura' => '#dbadff', 'Gris' => '#e1e1e1'];
+        $coloresFondo = ['Verde' => '#7bd148', 'Azul marino' => '#5484ed', 'Azul' => '#a4bdfc', 'Turquesa' => '#46d6db',
+            'Verde claro'        => '#7ae7bf', 'Verde oscuro' => '#51b749', 'Amarillo' => '#fbd75b', 'Naranja' => '#ffb878', 'Rojo' => '#ff887c',
+            'Rojo oscuro'        => '#dc2127', 'P&uacute;rpura' => '#dbadff', 'Gris' => '#e1e1e1', 'Original' => '#F3FEC8'];
+        $personal = $this->datosConf['ESTILO'] == 'personal' ? 'selected' : ' ';
+        $bluecurve = $this->datosConf['ESTILO'] == 'bluecurve' ? 'selected' : ' ';
+        $cristal = $this->datosConf['ESTILO'] == 'cristal' ? 'selected' : ' ';
+        $bootst = $this->datosConf['ESTILO'] == 'bootstrap' ? 'selected' : ' ';
+        $normal = $this->datosConf['PLANTILLA'] == 'normal' ? 'selected' : ' ';
+        $bootstrap = $this->datosConf['PLANTILLA'] == 'bootstrap' ? 'selected' : ' ';
+        $salida = '<center><div class="col-sm-4 col-md-8"><form name="configura" method="post">';
         //$salida.='<p align="center"><table border=1 class="tablaDatos"><tbody>';
-        $salida.='<p align="center"><table border=2 class="table table-hover"><tbody>';
-        $salida.='<th colspan=2 class="info"><center><b>Preferencias</b></center></th>';
-        $salida.='<tr><td>Nombre del Centro</td><td><input type="text" name="CENTRO" value="' . $this->datosConf['CENTRO'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>N&uacute;mero de filas</td><td><input type="text" name="NUMFILAS" value="' . $this->datosConf['NUMFILAS'] . '" size="3" /></td></tr>';
-        $salida.='<tr><td  style="vertical-align:middle">Plantilla</td><td><select name="PLANTILLA" class="form-control">';
-        $salida.='<option value="normal" ' . $normal . '>normal</option>';
-        $salida.='<option ' . $bootstrap . '>bootstrap</option></select></td></tr>';
-        $salida.='<tr><td  style="vertical-align:middle">Estilo</td><td><select name="ESTILO" class="form-control">';
-        $salida.='<option value="personal" ' . $personal . '>personal</option>';
-        $salida.='<option ' . $bluecurve . '>bluecurve</option>';
-        $salida.='<option ' . $bootst . '>bootstrap</option>';
-        $salida.='<option ' . $cristal . '>cristal</option></select></td></tr>';
-        $salida.='<tr><td style="vertical-align:middle">Color Lateral (bootstrap)</td><td style="vertical-align:middle"><select name="COLORLAT" id="COLORLAT" class="form-control">';
+        $salida .= '<p align="center"><table border=2 class="table table-hover"><tbody>';
+        $salida .= '<th colspan=2 class="info"><center><b>Preferencias</b></center></th>';
+        $salida .= '<tr>'.$this->creaTitulo('Nombre del Centro', 'Nombre que aparecerá en los informes y en la página principal de la aplicación').'<td><input type="text" name="CENTRO" value="'.$this->datosConf['CENTRO'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Número de filas', 'Número de filas que aparecerán en la pantalla de consulta de los maestros. Valor entre 10 y 25.').'<td><input type="number" max="25" min="10" name="NUMFILAS" value="'.$this->datosConf['NUMFILAS'].'" size="3" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Plantilla', 'Plantilla html utilizada para mostrar el contenido de la aplicación.').'<td><select name="PLANTILLA" class="form-control">';
+        $salida .= '<option value="normal" '.$normal.'>normal</option>';
+        $salida .= '<option '.$bootstrap.'>bootstrap</option></select></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Estilo', 'Estilo de los botones de control en los mantenimientos de los maestros').'<td><select name="ESTILO" class="form-control">';
+        $salida .= '<option value="personal" '.$personal.'>personal</option>';
+        $salida .= '<option '.$bluecurve.'>bluecurve</option>';
+        $salida .= '<option '.$bootst.'>bootstrap</option>';
+        $salida .= '<option '.$cristal.'>cristal</option></select></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Color Lateral', 'Color que se aplicará a la parte izquierda de la aplicación donde aparece el menú').'<td style="vertical-align:middle"><select name="COLORLAT" id="COLORLAT" class="form-control">';
         foreach ($coloresLateral as $color => $codigo) {
-            $selec = "";
+            $selec = '';
             if (trim($this->datosConf['COLORLAT']) == $codigo) {
-                $selec = "selected";
+                $selec = 'selected';
             }
-            $salida.='<option value="' . $codigo . '" ' . $selec . ' >' . $color . '</option>';
+            $salida .= '<option value="'.$codigo.'" '.$selec.' >'.$color.'</option>';
         }
-        $salida.='</select></td></tr>';
-        $salida.='<tr><td style="vertical-align:middle">Color Fondo (bootstrap)</td><td style="vertical-align:middle"><select name="COLORFON" id="COLORFON" class="form-control">';
+        $salida .= '</select></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Color Fondo', 'Color que aparecerá como fondo en todas las pantallas de la aplicación').'<td style="vertical-align:middle"><select name="COLORFON" id="COLORFON" class="form-control">';
         foreach ($coloresFondo as $color => $codigo) {
-            $selec = "";
+            $selec = '';
             if (trim($this->datosConf['COLORFON']) == $codigo) {
-                $selec = "selected";
+                $selec = 'selected';
             }
-            $salida.='<option value="' . $codigo . '" ' . $selec . ' >' . $color . '</option>';
+            $salida .= '<option value="'.$codigo.'" '.$selec.' >'.$color.'</option>';
         }
-        $salida.='</select></td></tr>';
-        $salida.='<th colspan=2 class="danger"><center><b>Base de datos</b></center></th>';
-        $salida.='<tr><td>Servidor</td><td><input type="text" name="SERVIDOR" value="' . $this->datosConf['SERVIDOR'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>Puerto</td><td><input type="text" name="PUERTO" value="' . $this->datosConf['PUERTO'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>Base de datos</td><td><input type="text" name="BASEDATOS" value="' . $this->datosConf['BASEDATOS'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>Base de datos Tests</td><td><input type="text" name="BASEDATOSTEST" value="' . $this->datosConf['BASEDATOSTEST'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>Usuario</td><td><input type="text" name="USUARIO" value="' . $this->datosConf['USUARIO'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>Clave</td><td><input type="text" name="CLAVE" value="' . $this->datosConf['CLAVE'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>mysqldump</td><td><input type="text" name="MYSQLDUMP" value="' . $this->datosConf['MYSQLDUMP'] . '" size="30" /></td></tr>';
-        $salida.='<tr><td>gzip</td><td><input type="text" name="GZIP" value="' . $this->datosConf['GZIP'] . '" size="30" /></td></tr>';
-        $salida.='<tr align=center><td colspan=2><button type="submit" class="btn btn-primary" name="aceptar"><span class="glyphicon glyphicon-ok"></span> Aceptar</td></tr></p>';
-        $salida.='</form></div></center>';
-        $salida.="<script>
+        $salida .= '</select></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Directorio tmp', 'Directorio donde se almacenarán los archivos temporales de la aplicación y también los archivos e informes que genera').'<td><input type="text" name="TMP" value="'.$this->datosConf['TMP'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<th colspan=2 class="danger"><center><b>Base de datos</b></center></th>';
+        $salida .= '<tr>'.$this->creaTitulo('Servidor', 'Nombre o dirección IP del servidor MySQL. Normalmente localhost').'<td><input type="text" name="SERVIDOR" value="'.$this->datosConf['SERVIDOR'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Puerto', 'Número de puerto donde el servidor admite conexiones MySQL. Normalmente 3306').'<td><input type="text" name="PUERTO" value="'.$this->datosConf['PUERTO'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Base de datos', 'Nombre de la base de datos donde se almacenarán los datos de la aplicación').'<td><input type="text" name="BASEDATOS" value="'.$this->datosConf['BASEDATOS'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Base de datos Tests', 'Nombre de la base de datos donde se almacenarán los datos de prueba de la aplicación').'<td><input type="text" name="BASEDATOSTEST" value="'.$this->datosConf['BASEDATOSTEST'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Usuario', 'Usuario con permisos de lectura/escritura en la base de datos').'<td><input type="text" name="USUARIO" value="'.$this->datosConf['USUARIO'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('Clave', 'Contraseña del usuario con permisos sobre la base de datos').'<td><input type="text" name="CLAVE" value="'.$this->datosConf['CLAVE'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('mysqldump', 'Ruta completa a la utilidad mysqldump. Este programa es necesario para que se puedan hacer las copias de seguridad de la aplicación').'<td><input type="text" name="MYSQLDUMP" value="'.$this->datosConf['MYSQLDUMP'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr>'.$this->creaTitulo('gzip', 'Ruta completa a la utilidad gzip. Este programa es necesario para que se puedan comprimir las copias de seguridad de la aplicación').'<td><input type="text" name="GZIP" value="'.$this->datosConf['GZIP'].'" maxlength="35" size="35" /></td></tr>';
+        $salida .= '<tr align=center><td colspan=2>
+            <a class="btn btn-info" role="button" onClick="location.href='."'index.php'".'"><span class="glyphicon glyphicon-arrow-left"></span> Volver</a>
+            <button type="submit" class="btn btn-primary" name="aceptar"><span class="glyphicon glyphicon-ok"></span> Aceptar</td></tr></p>';
+        $salida .= '</form></div></center>';
+        $salida .= "<script>
                         $(document).ready(function() {
-                            $('select[name=" . '"COLORFON"' . "]').on('change', function() {
-                               $(document.body).css('background-color', $('select[name=" . '"COLORFON"' . "]').val());
-                               $('.main').css('background-color', $('select[name=" . '"COLORFON"' . "]').val());    
+                            $('select[name=".'"COLORFON"'."]').on('change', function() {
+                               $(document.body).css('background-color', $('select[name=".'"COLORFON"'."]').val());
+                               $('.main').css('background-color', $('select[name=".'"COLORFON"'."]').val());    
                             });
-                            $('select[name=" . '"COLORLAT"' . "]').on('change', function() {
-                               $('.sidebar').css('background-color', $('select[name=" . '"COLORLAT"' . "]').val());
+                            $('select[name=".'"COLORLAT"'."]').on('change', function() {
+                               $('.sidebar').css('background-color', $('select[name=".'"COLORLAT"'."]').val());
                             });
-                            $('select[name=" . '"COLORLAT"' . "]').simplecolorpicker({theme: 'glyphicons'});
-                            $('select[name=" . '"COLORFON"' . "]').simplecolorpicker({theme: 'glyphicons'});
+                            $('select[name=".'"COLORLAT"'."]').simplecolorpicker({theme: 'glyphicons'});
+                            $('select[name=".'"COLORFON"'."]').simplecolorpicker({theme: 'glyphicons'});
+                            $('.dato').popover({trigger: 'hover'});
                         });
                     </script>";
+
         return $salida;
     }
-
 }
-
-?>

CopiaSeguridad.php

@@ -1,6 +1,5 @@
 <?php
 /**
- * @package Inventario
  * @copyright Copyright (c) 2014, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -16,17 +15,13 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-class CopiaSeguridad {
+class CopiaSeguridad
+{
     private $mensaje;
     private $baseDatos;
     private $imagenes;
 
-    public function __construct()
-    {
-        $opcion = $_GET['opc'];
-    }
     public function creaCopia()
     {
         if (!$this->copiaBaseDatos()) {
@@ -38,95 +33,106 @@ class CopiaSeguridad {
         if (!$this->empaqueta()) {
             return false;
         }
+
         return true;
     }
+
     public function dialogo()
     {
         $dialogo = '<div class="container col-5"><div class="jumbotron">
                       <h1>Copia de Seguridad</h1>
                       <p>¿Desea realizar una copia de seguridad de todos los datos de la Base de Datos y de todas las Imágenes?</p>
-                      <p><a class="btn btn-primary btn-lg" role="button" onClick="location.href=' . "'index.php'" . '"><span class="glyphicon glyphicon-arrow-left"></span> Volver</a>
-                         <a class="btn btn-success btn-lg" role="button" onClick="location.href=' . "'index.php?copiaseg&confirmado=1'" . '">
+                      <p><a class="btn btn-primary btn-lg" role="button" onClick="location.href='."'index.php'".'"><span class="glyphicon glyphicon-arrow-left"></span> Volver</a>
+                         <a class="btn btn-success btn-lg" role="button" onClick="location.href='."'index.php?copiaseg&confirmado=1'".'">
                              <span class="glyphicon glyphicon-cloud-download"></span> Continuar</a></p>
                     </div></div>';
+
         return $dialogo;
     }
+
     private function copiaBaseDatos()
     {
-        $archivo_sql = "tmp/baseDatos" . BASEDATOS . ".sql";
-        $baseDatosComprimida = $archivo_sql . ".gz";
+        $archivo_sql = TMP.'/baseDatos'.BASEDATOS.'.sql';
+        $baseDatosComprimida = $archivo_sql.'.gz';
         $this->baseDatos = $baseDatosComprimida;
         if (file_exists($baseDatosComprimida)) {
             unlink($baseDatosComprimida);
         }
-        $comando = escapeshellcmd(MYSQLDUMP . ' -h ' . SERVIDOR . ' -P ' . PUERTO . ' -u ' . USUARIO . ' --password=' . CLAVE . ' --result-file=' . $archivo_sql . ' ' . BASEDATOS);                                                       
-        $comando2 = escapeshellcmd(GZIP . ' -9f ' . $archivo_sql);                                    
+        $comando = escapeshellcmd(MYSQLDUMP.' -h '.SERVIDOR.' -P '.PUERTO.' -u '.USUARIO.' --password='.CLAVE.' --result-file='.$archivo_sql.' '.BASEDATOS);
+        $comando2 = escapeshellcmd(GZIP.' -9f '.$archivo_sql);
         exec($comando);
         exec($comando2);
         if (filesize($baseDatosComprimida) < 1024) {
             //No se ha realizado la copia de seguridad
-            $mensaje = "La copia de seguridad no se ha realizado correctamente.<br><br>";
-            $mensaje .= "Compruebe que las rutas a los programas mysqldump y gzip en configuraci&oacute;n est&aacute;n correctamente establecidas ";
-            $mensaje .= "y que los datos de acceso a la base de datos sean correctos.<br>";
-            $mensaje .= "mysqldump=[" . MYSQLDUMP . "]<br>";
-            $mensaje .= "gzip=[" . GZIP . "]";
+            $mensaje = 'La copia de seguridad no se ha realizado correctamente.<br><br>';
+            $mensaje .= 'Compruebe que las rutas a los programas mysqldump y gzip en configuraci&oacute;n est&aacute;n correctamente establecidas ';
+            $mensaje .= 'y que los datos de acceso a la base de datos sean correctos.<br>';
+            $mensaje .= 'mysqldump=['.MYSQLDUMP.']<br>';
+            $mensaje .= 'gzip=['.GZIP.']';
             $this->mensaje = $mensaje;
             $this->error = true;
+
             return false;
         }
+
         return true;
     }
+
     private function copiaImagenes()
     {
-        $copiaImagenes = "tmp/Imagenes.tbz";
+        $copiaImagenes = TMP.'/Imagenes.tbz';
         $this->imagenes = $copiaImagenes;
         if (file_exists($copiaImagenes)) {
             unlink($copiaImagenes);
         }
-        $comando = escapeshellcmd('tar cf ' . $copiaImagenes . ' ' . IMAGEDATA);
+        $comando = escapeshellcmd('tar cf '.$copiaImagenes.' '.IMAGEDATA);
         exec($comando);
 
         if (filesize($copiaImagenes) == 0) {
             $this->error = true;
-            $mensaje = "No se ha podido comprimir el directorio de las imágenes " . IMAGEDATA . "<br>";
-            $mensaje .= "Compruebe que la ruta de acceso al programa tar en configuraci&oacute;n est&aacute; correctamente establecida";
+            $mensaje = 'No se ha podido comprimir el directorio de las imágenes '.IMAGEDATA.'<br>';
+            $mensaje .= 'Compruebe que la ruta de acceso al programa tar en configuraci&oacute;n est&aacute; correctamente establecida';
             $this->mensaje = $mensaje;
+
             return false;
         }
+
         return true;
     }
+
     private function empaqueta()
     {
         // Empaqueta los dos archivos en el que va a devolver
-        $nombreCopia = "tmp/Copia" . BASEDATOS . strftime("%Y%m%d%H%M") . ".tar";
+        $nombreCopia = TMP.'/Copia'.BASEDATOS.strftime('%Y%m%d%H%M').'.tar';
         if (file_exists($nombreCopia)) {
             unlink($nombreCopia);
         }
-        $comando = escapeshellcmd('tar cf ' . $nombreCopia . ' ' . $this->baseDatos . ' ' . $this->imagenes);
+        $comando = escapeshellcmd('tar cf '.$nombreCopia.' '.$this->baseDatos.' '.$this->imagenes);
         exec($comando);
-        if (filesize($nombreCopia) ==0 || !file_exists($nombreCopia)) {
+        if (filesize($nombreCopia) == 0 || !file_exists($nombreCopia)) {
             $this->error = true;
-            $mensaje = "No se ha creado el paquete con los archivos de imágenes en [<b>" . $this->imagenes . "</b>] y <br>";
-            $mensaje .= " con el archivo de Base de Datos [<b>" . $this->baseDatos . "</b>]<br><br>";
-            $mensaje .= "Compruebe que los datos de configuración están correctamente establecidos <br>";
-            $mensaje .= "El comando de copia fue [" . $comando . "]<br>";
-            $mensaje .= "gzip=[" . GZIP . "]";
+            $mensaje = 'No se ha creado el paquete con los archivos de imágenes en [<b>'.$this->imagenes.'</b>] y <br>';
+            $mensaje .= ' con el archivo de Base de Datos [<b>'.$this->baseDatos.'</b>]<br><br>';
+            $mensaje .= 'Compruebe que los datos de configuración están correctamente establecidos <br>';
+            $mensaje .= 'El comando de copia fue ['.$comando.']<br>';
+            $mensaje .= 'gzip=['.GZIP.']';
             $this->mensaje = $mensaje;
+
             return false;
         }
         $this->error = false;
-        unlink ($this->baseDatos);
-        unlink ($this->imagenes);
+        unlink($this->baseDatos);
+        unlink($this->imagenes);
         $mensaje = 'Copia de seguridad realizada con &eacute;xito.<br><br>Pulse sobre el siguiente enlace para descargar:<br><br>';
-        $mensaje .= '<a href="' . $nombreCopia . '">Descargar Copia de Seguridad de Datos</a><br><br>';
+        $mensaje .= '<a href="'.$nombreCopia.'">Descargar Copia de Seguridad de Datos</a><br><br>';
         $mensaje .= 'El paquete de copia contiene un archivo con la copia de la información de la base de datos y un archivo que contiene el directorio de las fotografías e imágenes asociadas a los datos';
         $this->mensaje = $mensaje;
+
         return true;
     }
-    public function mensaje ()
+
+    public function mensaje()
     {
         return $this->mensaje;
     }
 }
-
-?>

Csv.php

@@ -1,7 +1,6 @@
 <?php
 
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,20 +16,18 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-class Csv {
-
+class Csv
+{
     /**
-     * 
-     * @var String Nombre del fichero csv
+     * @var string Nombre del fichero csv
      */
     private $nombre;
 
     /**
      * @var FILE manejador del fichero
      */
-    private $fichero = NULL;
+    private $fichero = null;
 
     /**
      * @var xml conulta asociada a este fichero
@@ -43,7 +40,6 @@ class Csv {
     private $bdd;
 
     /**
-     *
      * @var int Número de registros en el fichero csv
      */
     private $numRegistros;
@@ -59,8 +55,7 @@ class Csv {
     private $datosFichero;
 
     /**
-     * Indices a los campos correspondientes
-     * 
+     * Indices a los campos correspondientes.
      */
     private $idElemento;
     private $idArticulo;
@@ -71,48 +66,54 @@ class Csv {
     private $nSerie;
 
     /**
-      // El constructor necesita saber cuál es la opción actual
-      /**
+     * // El constructor necesita saber cuál es la opción actual.
+     * /**
      * Constructor de la clase.
+     *
      * @param BaseDatos $baseDatos Manejador de la base de datos
      */
-    public function __construct($baseDatos) {
+    public function __construct($baseDatos)
+    {
         $this->bdd = $baseDatos;
     }
 
     /**
-     * Crea un fichero csv con el nombre especificado
-     * @param String $fichero Nombre del fichero
+     * Crea un fichero csv con el nombre especificado.
+     *
+     * @param string $fichero Nombre del fichero
      */
-    public function crea($fichero) {
+    public function crea($fichero)
+    {
         $this->nombre = $fichero;
-        $this->fichero = fopen($this->nombre, "w") or die("No puedo abrir " . $this->nombre . " para escritura.");
+        $this->fichero = fopen($this->nombre, 'w') or die('No puedo abrir '.$this->nombre.' para escritura.');
     }
 
     /**
-     * 
      * @param array $datos escribe la línea en el archivo
      */
-    public function escribeLinea($datos) {
-        fputcsv($this->fichero, $datos, ',', '"') or die("No puedo escribir en el fichero csv");
+    public function escribeLinea($datos)
+    {
+        fputcsv($this->fichero, $datos, ',', '"') or die('No puedo escribir en el fichero csv');
     }
 
-    public function __destruct() {
+    public function __destruct()
+    {
         $this->cierra();
     }
 
-    public function cierra() {
-        fclose($this->fichero) or die("No puedo cerrar el archivo csv");
+    public function cierra()
+    {
+        fclose($this->fichero) or die('No puedo cerrar el archivo csv');
     }
 
     /**
-     * 
-     * @param String $fichero Archivo xml que contiene la definición de la consulta
+     * @param string $fichero Archivo xml que contiene la definición de la consulta
      */
-    public function ejecutaConsulta($fichero) {
-        $consulta = simplexml_load_file($fichero) or die("No puedo cargar el fichero xml " . $fichero . " al csv");
+    public function ejecutaConsulta($fichero)
+    {
+        $consulta = simplexml_load_file($fichero) or die('No puedo cargar el fichero xml '.$fichero.' al csv');
         // Escribe la cabecera del fichero
-        $this->escribeLinea(array($consulta->Pagina->Cabecera, $consulta->Titulo['id'], $consulta->Titulo['Texto']));
+        $this->escribeLinea([$consulta->Pagina->Cabecera, $consulta->Titulo['id'], $consulta->Titulo['Texto']]);
         foreach ($consulta->Pagina->Cuerpo->Col as $campo) {
             $campos[] = $campo['Titulo'];
         }
@@ -120,7 +121,7 @@ class Csv {
         // Escribe los datos de los campos
         $this->bdd->ejecuta($consulta->Datos->Consulta);
         while ($fila = $this->bdd->procesaResultado()) {
-            $campos = array();
+            $campos = [];
             foreach ($consulta->Pagina->Cuerpo->Col as $campo) {
                 $campos[] = $fila[(string) $campo['Nombre']];
             }
@@ -129,12 +130,12 @@ class Csv {
     }
 
     /**
-     * 
-     * @param String $ficheroCSV Nombre del archivo csv
+     * @param string $ficheroCSV Nombre del archivo csv
      */
-    public function cargaCSV($ficheroCSV) {
+    public function cargaCSV($ficheroCSV)
+    {
         $this->nombre = $ficheroCSV;
-        $this->fichero = fopen($this->nombre, "r") or die('No puedo abrir el archivo ' . $this->nombre . " para lectura.");
+        $this->fichero = fopen($this->nombre, 'r') or die('No puedo abrir el archivo '.$this->nombre.' para lectura.');
         list($archivo, $idCabecera, $cabecera) = fgetcsv($this->fichero);
         while ($linea = fgetcsv($this->fichero)) {
             $datosFichero[] = $linea;
@@ -146,113 +147,120 @@ class Csv {
     }
 
     /**
-     * Muestra un resumen de los datos del fichero csv cargado por pantalla
-     * 
+     * Muestra un resumen de los datos del fichero csv cargado por pantalla.
      */
-    public function resumen() {
+    public function resumen()
+    {
         //$mensaje .=
-        $mensaje = "<center><h1>Archivo [inventario" . $this->cabecera[0] . "]</h1>";
-        $mensaje .= "<h2>id=[" . $this->cabecera[1] . "] Descripci&oacute;n=[" . $this->cabecera[2] . "]</h2><br>";
+        $mensaje = '<center><h1>Archivo [inventario'.$this->cabecera[0].']</h1>';
+        $mensaje .= '<h2>id=['.$this->cabecera[1].'] Descripci&oacute;n=['.$this->cabecera[2].']</h2><br>';
         $mensaje .= '<table border=1 class="table table-striped table-bordered table-condensed table-hover"><theader>';
         foreach ($this->datosFichero[0] as $campo) {
             $dato = $campo;
             $mensaje .= "<th><b>$dato</b></th>";
         }
-        $mensaje .= "<th><b>Acci&oacute;n</b></th>";
-        $mensaje .="</theader><tbody>";
+        $mensaje .= '<th><b>Acci&oacute;n</b></th>';
+        $mensaje .= '</theader><tbody>';
         $this->cargaIndices($this->datosFichero[0]);
         //echo "$mensaje contar Datosfichero=[".count($datosFichero)."]";
         for ($i = 1; $i < count($this->datosFichero); $i++) {
-            $mensaje .= "<tr>";
+            $mensaje .= '<tr>';
             $primero = true;
             foreach ($this->datosFichero[$i] as $dato) {
                 if ($primero) {
                     $primero = false;
                     switch ($dato) {
-                        case 'S': $estado = "-Baja-";
-                            $color = "danger";
+                        case 'S': $estado = '-Baja-';
+                            $color = 'danger';
                             break;
-                        case 'Alta': $estado = "-Alta-";
-                            $color = "primary";
+                        case 'Alta': $estado = '-Alta-';
+                            $color = 'primary';
                             break;
-                        case "N" : $estado = $this->compruebaCantidades($i);
+                        case 'N': $estado = $this->compruebaCantidades($i);
                             if ($estado != 0) {
-                                $color = "warning";
+                                $color = 'warning';
                                 if ($estado > 0) {
-                                    $estado = "+" . $estado;
+                                    $estado = '+'.$estado;
                                 }
                             } else {
-                                $estado = "igual";
-                                $color = "info";
+                                $estado = 'igual';
+                                $color = 'info';
                             }
                             break;
                         default: throw new Exception("El archivo csv tiene un formato incorrecto.<br>Bajas=[$dato]");
                     }
                 }
-                $mensaje .= "<td>" . $dato . "</td>";
+                $mensaje .= '<td>'.$dato.'</td>';
             }
-            $mensaje .= '<td align="center"><label class="label label-' . $color . '">' . $estado . '</label></td>';
-            $mensaje .= "</tr>";
+            $mensaje .= '<td align="center"><label class="label label-'.$color.'">'.$estado.'</label></td>';
+            $mensaje .= '</tr>';
         }
-        $mensaje .= "</tbody></table></p><br>";
+        $mensaje .= '</tbody></table></p><br>';
         $mensaje .= $this->panelMensaje('Si se produce cualquier error en el procesamiento del fichero, no se aplicar&aacute; ning&uacute;n cambio en la base de datos.');
 
         $mensaje .= '<form method="post" name="Aceptar" action="index.php?importacion&opc=ejecutar">
-                <button type="button" name="Cancelar" value="Cancelar" onClick="location.href=' . "'index.php'" . '" class="btn btn-danger"><span class="glyphicon glyphicon-remove"></span> Cancelar</button>
+                <button type="button" name="Cancelar" value="Cancelar" onClick="location.href='."'index.php'".'" class="btn btn-danger"><span class="glyphicon glyphicon-remove"></span> Cancelar</button>
                 <button type="submit" name="Aceptar" class="btn btn-primary"><span class="glyphicon glyphicon-ok"></span> Aceptar</button>
-                <input type="hidden" name="ficheroCSV" value="' . $this->nombre . '">
+                <input type="hidden" name="ficheroCSV" value="'.$this->nombre.'">
                 </form></center>';
 
         return $mensaje;
     }
 
     /**
-     * 
      * @param $array línea de datos del fichero csv para comprobar las cantidades si se han modificado o no
+     *
      * @return string
      */
-    private function compruebaCantidades($i) {
-        $ultimo = count($datos);
+    private function compruebaCantidades($i)
+    {
+        //$ultimo = count($datos);
         return $this->datosFichero[$i][$this->cantidadReal] - $this->datosFichero[$i][$this->cantidad];
     }
 
-    private function panelMensaje($info, $tipo = "danger", $cabecera = "&iexcl;Atenci&oacute;n!") {
-        $mensaje = '<div class="panel panel-' . $tipo . '"><div class="panel-heading">';
-        $mensaje .= '<h3 class="panel-title">' . $cabecera . '</h3></div>';
+    private function panelMensaje($info, $tipo = 'danger', $cabecera = '&iexcl;Atenci&oacute;n!')
+    {
+        $mensaje = '<div class="panel panel-'.$tipo.'"><div class="panel-heading">';
+        $mensaje .= '<h3 class="panel-title">'.$cabecera.'</h3></div>';
         $mensaje .= '<div class="panel-body">';
         $mensaje .= $info;
         $mensaje .= '</div>';
         $mensaje .= '</div>';
+
         return $mensaje;
     }
 
-    private function escribeLog($comando) {
-        $fp = fopen($this->nombre.".log", "a");
-        $linea = strftime("%Y/%m/%d")."|".$this->nombre."|".$comando;
-        fputs($fp, $linea . "\n");
+    private function escribeLog($comando)
+    {
+        $fp = fopen($this->nombre.'.log', 'a');
+        $linea = strftime('%Y/%m/%d').'|'.$this->nombre.'|'.$comando;
+        fwrite($fp, $linea."\n");
         fclose($fp);
     }
 
-    private function bajaElemento($i) {
+    private function bajaElemento($i)
+    {
         $id = $this->datosFichero[$i][$this->idElemento];
-        $comando = 'delete from Elementos where id="' . $id . '";';
+        $comando = 'delete from Elementos where id="'.$id.'";';
         $this->escribeLog($comando);
         if (!$this->bdd->ejecuta($comando)) {
-            throw new Exception("Baja-".$this->bdd->mensajeError, $this->bdd->error);
+            throw new Exception('Baja-'.$this->bdd->mensajeError, $this->bdd->error);
         }
     }
 
-    private function modificaElemento($i) {
+    private function modificaElemento($i)
+    {
         $id = $this->datosFichero[$i][$this->idElemento];
-        $comando = 'update Elementos set Cantidad=' . $this->datosFichero[$i][$this->cantidadReal] . ' where id="' . $id . '";';
+        $comando = 'update Elementos set Cantidad='.$this->datosFichero[$i][$this->cantidadReal].' where id="'.$id.'";';
         $this->escribeLog($comando);
         if (!$this->bdd->ejecuta($comando)) {
-            throw new Exception("Modifica-".$this->bdd->mensajeError, $this->bdd->error);
+            throw new Exception('Modifica-'.$this->bdd->mensajeError, $this->bdd->error);
         }
     }
 
-    private function altaElemento($i) {
-        if ($this->cabecera[0] == "Articulo") {
+    private function altaElemento($i)
+    {
+        if ($this->cabecera[0] == 'Articulo') {
             $idUbicacion = $this->datosFichero[$i][$this->idUbicacion];
             $idArticulo = $this->cabecera[1];
             $comando = 'select id from Ubicaciones where Descripcion="'.$this->datosFichero[$i][$this->desUbicacion].'";';
@@ -261,30 +269,31 @@ class Csv {
             $idArticulo = $this->datosFichero[$i][$this->idArticulo];
         }
         $idArt = $datosFichero[$i];
-        $comando = 'insert into Elementos () values (null,' . $idArticulo . ',' . $idUbicacion . ',"' . $this->datosFichero[$i][$this->nSerie];
-        $comando .= '",' . $this->datosFichero[$i][$this->cantidadReal] . ',"' . $this->datosFichero[$i][$this->fechaCompra] . '");';
+        $comando = 'insert into Elementos () values (null,'.$idArticulo.','.$idUbicacion.',"'.$this->datosFichero[$i][$this->nSerie];
+        $comando .= '",'.$this->datosFichero[$i][$this->cantidadReal].',"'.$this->datosFichero[$i][$this->fechaCompra].'");';
         $this->escribeLog($comando);
         if (!$this->bdd->ejecuta($comando)) {
-            throw new Exception("Alta-".$this->bdd->mensajeError, $this->bdd->error);
+            throw new Exception('Alta-'.$this->bdd->mensajeError, $this->bdd->error);
         }
     }
 
-    private function cargaIndices($campos) {
+    private function cargaIndices($campos)
+    {
         for ($i = 0; $i < count($campos); $i++) {
             switch ($campos[$i]) {
-                case "Cant. Real": $this->cantidadReal = $i;
+                case 'Cant. Real': $this->cantidadReal = $i;
                     break;
-                case "Fecha C.": $this->fechaCompra = $i;
+                case 'Fecha C.': $this->fechaCompra = $i;
                     break;
-                case "idUbic": $this->idUbicacion = $i;
+                case 'idUbic': $this->idUbicacion = $i;
                     break;
-                case "idArt": $this->idArticulo = $i;
+                case 'idArt': $this->idArticulo = $i;
                     break;
-                case "idElem": $this->idElemento = $i;
+                case 'idElem': $this->idElemento = $i;
                     break;
-                case "Cantidad": $this->cantidad = $i;
+                case 'Cantidad': $this->cantidad = $i;
                     break;
-                case "N Serie": $this->nSerie = $i;
+                case 'N Serie': $this->nSerie = $i;
                     break;
             }
         }
@@ -293,9 +302,10 @@ class Csv {
     }
 
     /**
-     * Procesa contra la base de datos todas las acciones del archivo
+     * Procesa contra la base de datos todas las acciones del archivo.
      */
-    public function ejecutaFichero() {
+    public function ejecutaFichero()
+    {
         $this->cargaIndices($this->datosFichero[0]);
         //Realiza una transacción para que no se ejecute parcialmente una actualización
         try {
@@ -317,31 +327,31 @@ class Csv {
                             $acciones++;
                         }
                         break;
-                    default: throw new Exception("Acci&oacute;n no reconocida en la importacion [" . $this->datosFichero[0] . "]");
+                    default: throw new Exception('Acci&oacute;n no reconocida en la importacion ['.$this->datosFichero[0].']');
                 }
             }
             $mensaje = "Se han procesado correctamente $acciones acciones en la Base de Datos.";
             $this->bdd->confirmaTransaccion();
-            return $this->panelMensaje($mensaje,"success", "Informaci&oacute;n");
+
+            return $this->panelMensaje($mensaje, 'success', 'Informaci&oacute;n');
         } catch (Exception $e) {
             $this->bdd->abortaTransaccion();
-            $mensaje = "Se ha producido el error [" . $e->getMessage() . "]<br>NO se ha realizado ning&uacute;n cambio en la Base de Datos.";
+            $mensaje = 'Se ha producido el error ['.$e->getMessage().']<br>NO se ha realizado ning&uacute;n cambio en la Base de Datos.';
+
             return $this->panelMensaje($mensaje);
         }
     }
 
-    private function ejecutaFichero2() {
+    private function ejecutaFichero2()
+    {
         echo '<script>visualizaProgreso();</script>';
         for ($i = 1; $i < 80; $i++) {
             //sleep(1);
             $progreso = $i;
-            echo '<script>actProgreso(' . $progreso . ');</script>';
+            echo '<script>actProgreso('.$progreso.');</script>';
             //echo str_repeat(' ',1024*64);
             flush();
             //echo '$(".bar").css("width", "'.$progreso.'");';
         }
     }
-
 }
-
-?>

Distribucion.php

@@ -1,6 +1,5 @@
 <?php
-    /**
-     * @package Inventario
+/**
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -16,7 +15,6 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
-     *  
  */
 //
 // Esta clase procesará una página sustituyendo
@@ -27,7 +25,8 @@
 // y una referencia al objeto cuyos métodos deberán
 // aportar los contenidos.
 //
-class Distribucion {
+class Distribucion
+{
     // Variable para conservar la plantilla
     private $plantilla;
     // Matriz que contendrá los nombres de elementos
@@ -35,35 +34,37 @@ class Distribucion {
     // Referencia al objeto cuyos métodos serán
     // invocados para aportar el contenido
     private $objeto;
+
     // Constructor de la clase
     public function __construct($archivo, $objeto)
     {
         // Recuperamos el contenido del archivo
-            $this->plantilla=file_get_contents($archivo)
+        $this->plantilla = file_get_contents($archivo)
                 or die('Fallo en la apertura de la plantilla '.$archivo);
         // y guardamos la referencia al objeto
-            $this->objeto=$objeto;
+        $this->objeto = $objeto;
         // Extraemos todas las marcas de contenido
-            preg_match_all('/\{[A-Za-z]+\}/', $this->plantilla,$el, PREG_PATTERN_ORDER);
+        preg_match_all('/\{[A-Za-z]+\}/', $this->plantilla, $el, PREG_PATTERN_ORDER);
         // Nos quedamos con la matriz de resultados
-            $this->elementos=$el[0];
+        $this->elementos = $el[0];
     }
+
     // Este método es el encargado de procesar la plantilla
     public function procesaPlantilla()
     {
         // Tomamos la plantilla en una variable local, para
         // así no modificar el contenido original
-            $pagina=$this->plantilla;
+        $pagina = $this->plantilla;
         // Recorremos la matriz de marcas de contenido
-            foreach($this->elementos as $el) {
+        foreach ($this->elementos as $el) {
             // Eliminamos los delimitadores { y }
-                $el=substr($el,1,strlen($el)-2);
+            $el = substr($el, 1, strlen($el) - 2);
             // invocamos a la función del objeto
-                $resultado=$this->objeto->$el();
+            $resultado = $this->objeto->$el();
             // e introducimos su contenido en lugar de la marca
-                $pagina=str_replace('{'.$el.'}',$resultado,$pagina);
+            $pagina = str_replace('{'.$el.'}', $resultado, $pagina);
         }
-            /**
+        /*
          *  @todo Tratar de activar la compresión.
          */
         // Si es posible comprimir
@@ -76,4 +77,3 @@ class Distribucion {
             return $pagina; // enviamos sin comprimir
     }
 }
-?>

EtiquetasPDF.php

@@ -1,13 +1,13 @@
-
 <?php
 
 /**
- * genera un documento PDF a partir de una descripción dada en un archivo XML
+ * genera un documento PDF a partir de una descripción dada en un archivo XML.
+ *
  * @author Ricardo Montañana <rmontanana@gmail.com>
+ *
  * @version 1.0
- * @package Inventario
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -23,27 +23,27 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
 require_once 'phpqrcode.php';
 
-class EtiquetasPDF {
-
+class EtiquetasPDF
+{
     /**
-     * 
      * @var basedatos Controlador de la base de datos
      */
     private $bdd;
     private $docu;
     private $pdf;
     private $def;
-    private $nombreFichero = "tmp/informeEtiquetas.pdf";
+    private $nombreFichero;
 
     /**
-     * El constructor recibe como argumento el nombre del archivo XML con la definición, encargándose de recuperarla y guardar toda la información localmente
+     * El constructor recibe como argumento el nombre del archivo XML con la definición, encargándose de recuperarla y guardar toda la información localmente.
+     *
      * @param basedatos $bdd        manejador de la base de datos
      * @param string    $definicion fichero con la definición del informe en XML
-     * @param boolean $registrado usuario registrado si/no
+     * @param bool      $registrado usuario registrado si/no
+     *
      * @return ficheroPDF
      *                    todo: cambiar este comentario
      */
@@ -52,6 +52,7 @@ class EtiquetasPDF {
         if (!$registrado) {
             return 'Debe registrarse para acceder a este apartado';
         }
+        $this->nombreFichero = TMP.'/informeEtiquetas.pdf';
         // Recuperamos la definición del informe
         $this->def = simplexml_load_file($definicion);
         $this->bdd = $bdd;
@@ -61,7 +62,7 @@ class EtiquetasPDF {
         $this->pdf->setAutoPageBreak(false);
         //echo $def->Titulo.$def->Cabecera;
         $this->pdf->setAuthor(AUTOR, true);
-        $creador = CENTRO . " " . PROGRAMA . VERSION;
+        $creador = CENTRO.' '.PROGRAMA.VERSION;
         $this->pdf->setCreator(html_entity_decode($creador), true);
         $this->pdf->setSubject($this->def->Titulo, true);
         //$this->pdf->setAutoPageBreak(true, 10);
@@ -78,12 +79,13 @@ class EtiquetasPDF {
         $this->pdf->AddPage();
         $tamLinea = 5;
         $fila = -1;
-        $primero = true; $i = 0;
-        $url = explode("/", $_SERVER['SCRIPT_NAME']);
+        $primero = true;
+        $i = 0;
+        $url = explode('/', $_SERVER['SCRIPT_NAME']);
         $aplicacion = $url[1];
-        $protocolo = !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off' || $_SERVER['SERVER_PORT'] == 443 ? "https://" : "http://";
-        $enlace = $protocolo . $_SERVER['SERVER_NAME'] . "/" . $aplicacion . "/index.php?elementos&opc=editar&id=";
-        while($tupla = $this->bdd->procesaResultado()) {
+        $protocolo = !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off' || $_SERVER['SERVER_PORT'] == 443 ? 'https://' : 'http://';
+        $enlace = $protocolo.$_SERVER['SERVER_NAME'].'/'.$aplicacion.'/index.php?elementos&opc=editar&id=';
+        while ($tupla = $this->bdd->procesaResultado()) {
             for ($j = 0; $j < $tupla['cantidad']; $j++) {
                 //Hay que generar tantas etiquetas como ponga la cantidad de cada elemento
                 if ($i % 2) {
@@ -104,31 +106,31 @@ class EtiquetasPDF {
                     $primero = false;
                 }
                 $py = 6 + 41 * $fila;
-                $enlace2=$enlace.$tupla['idEl'];
-                $fichero = "tmp/etiq".rand(1000,9999).".png";
+                $enlace2 = $enlace.$tupla['idEl'];
+                $fichero = TMP.'/etiq'.rand(1000, 9999).'.png';
                 QRcode::png($enlace2, $fichero);
                 $this->pdf->image($fichero, $etiq2, $py, 30, 30);
                 unlink($fichero);
                 $this->pdf->setxy($etiq1, $py);
                 $this->pdf->Cell(30, 10, utf8_decode($tupla['articulo']));
-                $py+=$tamLinea;
+                $py += $tamLinea;
                 $this->pdf->setxy($etiq1, $py);
                 $this->pdf->Cell(30, 10, utf8_decode($tupla['marca']));
-                $py+=$tamLinea;
+                $py += $tamLinea;
                 $this->pdf->setxy($etiq1, $py);
                 $this->pdf->Cell(30, 10, utf8_decode($tupla['modelo']));
-                $py+=$tamLinea;
+                $py += $tamLinea;
                 $this->pdf->setxy($etiq1, $py);
                 $this->pdf->Cell(30, 10, utf8_decode($tupla['numserie']));
-                $py+=$tamLinea;
+                $py += $tamLinea;
                 $this->pdf->setxy($etiq1, $py);
                 $this->pdf->Cell(30, 10, $tupla['fechaCompra']);
-                $py+=$tamLinea-1;
+                $py += $tamLinea - 1;
                 $this->pdf->setxy($etiq2, $py);
                 $this->pdf->Cell(30, 10, utf8_decode($tupla['ubicacion']));
-                $py+=$tamLinea-1;
+                $py += $tamLinea - 1;
                 $this->pdf->setxy($etiq2, $py);
-                $cadena = "idElemento=" . $tupla['idEl'] . " / idArticulo=" . $tupla['idArt'] . " / idUbicacion=" . $tupla['idUbic'];           
+                $cadena = 'idElemento='.$tupla['idEl'].' / idArticulo='.$tupla['idArt'].' / idUbicacion='.$tupla['idUbic'];
                 $this->pdf->Cell(30, 10, $cadena);
                 $i++;
             }
@@ -149,15 +151,16 @@ class EtiquetasPDF {
 
     public function getCabecera()
     {
-        $cabecera = "Content-type: application/pdf";
-        $cabecera = $cabecera . "Content-length: " . strlen($this->docu);
-        $cabecera = $cabecera . "Content-Disposition: inline; filename=" . $this->nombreFichero;
+        $cabecera = 'Content-type: application/pdf';
+        $cabecera = $cabecera.'Content-length: '.strlen($this->docu);
+        $cabecera = $cabecera.'Content-Disposition: inline; filename='.$this->nombreFichero;
+
         return $cabecera;
     }
 
-    public function guardaArchivo($nombre = "tmp/Informe.pdf")
+    public function guardaArchivo($nombre)
     {
-        $fichero = fopen($nombre, "w");
+        $fichero = fopen($nombre, 'w');
         fwrite($fichero, $this->getCabecera());
         fwrite($fichero, $this->getContenido(), strlen($this->getContenido()));
         $this->nombreFichero = $nombre;
@@ -166,10 +169,10 @@ class EtiquetasPDF {
 
     public function enviaCabecera()
     {
-        header("Content-type: application/pdf");
+        header('Content-type: application/pdf');
         $longitud = strlen($this->docu);
         header("Content-length: $longitud");
-        header("Content-Disposition: inline; filename=" . $this->nombreFichero);
+        header('Content-Disposition: inline; filename='.$this->nombreFichero);
     }
 
     public function imprimeInforme()
@@ -178,5 +181,3 @@ class EtiquetasPDF {
         echo $this->docu;
     }
 }
-
-?>

Imagen.php

@@ -1,6 +1,5 @@
 <?php
 /**
- * @package Inventario
  * @copyright Copyright (c) 2014, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -16,19 +15,20 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
 //Para comprimir las imágenes
-require_once('Zebra_Image.php');
-define("HAYQUEGRABAR", 1);
-define("HAYQUEBORRAR", 2);
-define("NOHACERNADA", 3);
+require_once 'Zebra_Image.php';
+define('HAYQUEGRABAR', 1);
+define('HAYQUEBORRAR', 2);
+define('NOHACERNADA', 3);
 
-class Imagen {
+class Imagen
+{
     private $archivoSubido;
     public $archivoComprimido;
     private $extension;
     private $dirData;
+    public $archivoCopiado;
 
     public function __construct()
     {
@@ -37,9 +37,9 @@ class Imagen {
 
     public function determinaAccion($campo)
     {
-        if (isset($_POST[$campo]) && $_POST[$campo] == "") {
+        if (isset($_POST[$campo]) && $_POST[$campo] == '') {
             return HAYQUEBORRAR; //Hay que borrar el archivo de imagen
-        } elseif ($_FILES[$campo]['error'] == 0) {
+        } elseif (isset($_FILES[$campo]['error']) && $_FILES[$campo]['error'] == 0) {
             return HAYQUEGRABAR; //Hay que guardar el archivo de imagen enviado
         } else {
             return NOHACERNADA; //No hay que hacer nada
@@ -76,11 +76,11 @@ class Imagen {
             $finfo = new finfo(FILEINFO_MIME_TYPE);
             if (false === $ext = array_search(
                 $finfo->file($_FILES[$campo]['tmp_name']),
-                array(
+                [
                     'jpg' => 'image/jpeg',
                     'png' => 'image/png',
                     'gif' => 'image/gif',
-                ),
+                ],
                 true
             )) {
                 throw new RuntimeException('Formato de imagen inválido, no es {jpg, png, gif}');
@@ -97,24 +97,40 @@ class Imagen {
             return true;
         } catch (RuntimeException $e) {
             $mensaje = $e->getMessage();
+
             return false;
         }
     }
 
     public static function borraImagenId($tabla, $id)
     {
-        $extensiones = array ("png", "gif", "jpg");
+        $extensiones = ['png', 'gif', 'jpg'];
         foreach ($extensiones as $extension) {
-            $archivo = IMAGEDATA . "/" . $tabla . "_" . $id . "." . $extension;
+            $archivo = IMAGEDATA.'/'.$tabla.'_'.$id.'.'.$extension;
             if (file_exists($archivo)) {
-                unlink ($archivo);
+                unlink($archivo);
             }
         }
     }
 
+    public function copiaImagenId($valorImagen, $tabla, $id, &$mensaje)
+    {
+        $extension = strrchr($valorImagen, '.');
+        $nombre = $this->dirData.'/'.$tabla.'_'.$id.$extension;
+        if (!@copy($valorImagen, $nombre)) {
+            $errors = error_get_last();
+            $mensaje = 'No pudo copiar el archivo '.$valorImagen.' en '.$nombre.' Error = ['.$errors['message'].']';
+
+            return false;
+        }
+        $this->archivoCopiado = $nombre;
+
+        return true;
+    }
+
     public function mueveImagenId($tabla, $id, &$mensaje)
     {
-        if (!$this->comprimeArchivo($tabla . "_" . $id, $mensaje)) {
+        if (!$this->comprimeArchivo($tabla.'_'.$id, $mensaje)) {
             return false;
         } else {
             return true;
@@ -125,7 +141,7 @@ class Imagen {
     {
         $zebra = new Zebra_Image();
         $zebra->source_path = $this->archivoSubido;
-        $this->archivoComprimido = $this->dirData . "/" . $id . "." . $this->extension;
+        $this->archivoComprimido = $this->dirData.'/'.$id.'.'.$this->extension;
         $zebra->target_path = $this->archivoComprimido;
         $zebra->jpeg_quality = 100;
 
@@ -143,28 +159,28 @@ class Imagen {
             switch ($zebra->error) {
                 case 1: $mensaje = 'El fichero origen no se ha encontrado!';
                     break;
-                case 2: $mensaje = 'No se puede leer el archivo origen ' . $this->archivoSubido;
+                case 2: $mensaje = 'No se puede leer el archivo origen '.$this->archivoSubido;
                     break;
-                case 3: $mensaje = 'No se pudo escribir el archivo destino ' . $this->archivoComprimido;
+                case 3: $mensaje = 'No se pudo escribir el archivo destino '.$this->archivoComprimido;
                     break;
-                case 4: $mensaje = 'Formato de fichero origen no soportado ' . $this->archivoSubido;
+                case 4: $mensaje = 'Formato de fichero origen no soportado '.$this->archivoSubido;
                     break;
-                case 5: $mensaje = 'Formato de fichero destino no soportado ' . $this->archivoComprimido;
+                case 5: $mensaje = 'Formato de fichero destino no soportado '.$this->archivoComprimido;
                     break;
-                case 6: $mensaje = 'La versión de la biblioteca GD no soporta el formato de destino ' . $this->archivoComprimido;
+                case 6: $mensaje = 'La versión de la biblioteca GD no soporta el formato de destino '.$this->archivoComprimido;
                     break;
                 case 7: $mensaje = 'La biblioteca GD no está instalada';
                     break;
                 case 8: $mensaje = 'el comando "chmod" está deshabilitado por configuración';
                     break;
             }
+
             return false;
         } else {
             //Borra el archivo subido
             unlink($this->archivoSubido);
+
             return true;
         }
     }
 }
-
-?>

Importacion.php

@@ -1,7 +1,6 @@
 <?php
 
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,20 +16,21 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
-class Importacion {
-
+class Importacion
+{
     private $bdd;
 
-    public function __construct($baseDatos, $registrado) {
+    public function __construct($baseDatos, $registrado)
+    {
         if (!$registrado) {
             return 'Debe registrarse para acceder a este apartado';
         }
         $this->bdd = $baseDatos;
     }
 
-    public function ejecuta() {
+    public function ejecuta()
+    {
         $opc = '';
         if (isset($_GET['opc'])) {
             $opc = $_GET['opc'];
@@ -39,25 +39,29 @@ class Importacion {
             case 'form':return $this->formulario();
             case 'importar':return $this->importarFichero();
             case 'ejecutar':return $this->ejecutaFichero();
-            default: return "Importacion: No entiendo qué me has pedido.";
+            default: return 'Importacion: No entiendo qué me has pedido.';
         }
     }
 
-    private function importarFichero() {
-        $uploadfile = "tmp/" . basename($_FILES['fichero']['name']);
+    private function importarFichero()
+    {
+        $uploadfile = TMP.'/'.basename($_FILES['fichero']['name']);
         if (!move_uploaded_file($_FILES['fichero']['tmp_name'], $uploadfile)) {
-            die('No se pudo subir el fichero ' . $_FILES['userfile']['tmp_name']);
+            die('No se pudo subir el fichero '.$_FILES['userfile']['tmp_name']);
         }
         $csv = new Csv($this->bdd);
         $csv->cargaCSV($uploadfile);
+
         return $csv->resumen();
     }
 
-    private function formulario() {
-        $accion = "index.php?importacion&opc=importar";
+    private function formulario()
+    {
+        $accion = 'index.php?importacion&opc=importar';
+        $salida = '';
         //$salida .= '<script type="text/javascript" src="css/bootstrap-filestyle.min.js"> </script>';
-        $salida .='<div class="col-sm-6 col-md-6">';
-        $salida .= '<form  enctype="multipart/form-data" name="importacion.form" method="post" action="' . $accion . '">' . "\n";
+        $salida .= '<div class="col-sm-6 col-md-6">';
+        $salida .= '<form  enctype="multipart/form-data" name="importacion.form" method="post" action="'.$accion.'">'."\n";
         $salida .= "<fieldset style=\"width: 96%;\"><p><legend style=\"color: red;\"><b>Elige Archivo</b></legend></p>\n";
         //$salida .= '<input type="file" name="fichero" id="fichero" onChange="seleccionFichero(this);" class="filestyle" data-classButton="btn btn-primary">';
         //$salida .= '<input type="file" name="fichero" id="fichero" onChange="seleccionFichero(this);">';
@@ -75,7 +79,7 @@ class Importacion {
                             <a href="#" class="input-group-addon btn btn-default fileinput-exists" data-dismiss="fileinput">Eliminar</a>
                         </div>
                     </div></fieldset>';
-        $salida .= '<p align="center"><button class="btn btn-primary" type=submit><span class="glyphicon glyphicon-cloud-upload"></span> Aceptar</button></p><br>' . "\n";
+        $salida .= '<p align="center"><button class="btn btn-primary" type=submit><span class="glyphicon glyphicon-cloud-upload"></span> Aceptar</button></p><br>'."\n";
         $salida .= '</div>';
         $mensaje = 'Sólo se permiten archivos con extensión CSV';
         $salida .= "<script type='text/javascript'>"."
@@ -89,14 +93,16 @@ class Importacion {
                     location.reload();
                 }}
            </script>";
+
         return $salida;
     }
-    private function ejecutaFichero() {
+
+    private function ejecutaFichero()
+    {
         $archivo = $_POST['ficheroCSV'];
         $csv = new Csv($this->bdd);
         $csv->cargaCSV($archivo);
+
         return $csv->ejecutaFichero();
     }
 }
-
-?>

InformeInventario.php

@@ -1,7 +1,6 @@
 <?php
 
 /**
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,10 +16,9 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
-class InformeInventario {
-
+class InformeInventario
+{
     private $bdd;
 
     public function __construct($baseDatos)
@@ -48,17 +46,19 @@ class InformeInventario {
         $informe = new InformePDF($this->bdd, $enlace, true);
         $informe->crea($enlace);
         $informe->cierraPDF();
+
         return $this->devuelveInforme($informe);
     }
 
     private function devuelveInforme($informe)
     {
-        $letras = "abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
-        $nombre = "tmp/informe" . substr(str_shuffle($letras), 0, 10) . ".pdf";
+        $letras = 'abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
+        $nombre = TMP.'/informe'.substr(str_shuffle($letras), 0, 10).'.pdf';
         $informe->guardaArchivo($nombre);
+
         return '<div class="container">
-                    <!--<a href="' . $nombre . '" target="_blank"><span class="glyphicon glyphicon-cloud-download" style="font-size:1.5em;"></span>Descargar Informe</a>--> 
-                    <object data="' . $nombre . '" type="application/pdf" width="100%" height="700" style="float:left;">
+                    <!--<a href="'.$nombre.'" target="_blank"><span class="glyphicon glyphicon-cloud-download" style="font-size:1.5em;"></span>Descargar Informe</a>--> 
+                    <object data="'.$nombre.'" type="application/pdf" width="100%" height="700" style="float:left;">
                     </object>
                 </div>';
     }
@@ -67,48 +67,50 @@ class InformeInventario {
     {
         $salidaInforme = isset($_POST['salida']) ? $_POST['salida'] : 'pantalla';
         switch ($salidaInforme) {
-            case "pantalla":
-                $fichero = "xml/inventarioUbicacion.xml";
-                $salida = "tmp/inventarioUbicacion.xml";
+            case 'pantalla':
+                $fichero = 'xml/inventarioUbicacion.xml';
+                $salida = TMP.'/inventarioUbicacion.xml';
                 break;
-            case "csv":
-                $fichero = "xml/inventarioUbicacionCSV.xml";
-                $salida = "tmp/inventarioUbicacionCSV.xml";
+            case 'csv':
+                $fichero = 'xml/inventarioUbicacionCSV.xml';
+                $salida = TMP.'/inventarioUbicacionCSV.xml';
                 break;
-            case "etiquetas":
-                $fichero = "xml/inventarioUbicacionEtiquetas.xml";
-                $salida = "tmp/inventarioUbicacionEtiquetas.xml";
+            case 'etiquetas':
+                $fichero = 'xml/inventarioUbicacionEtiquetas.xml';
+                $salida = TMP.'/inventarioUbicacionEtiquetas.xml';
                 break;
         }
-        $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla ' . $fichero);
-        $id = $_POST['id'] == NULL ? $_GET['id'] : $_POST['id'];
-        $comando = "select * from Ubicaciones where id='" . $id . "';";
+        $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla '.$fichero);
+        $id = $_POST['id'] == null ? $_GET['id'] : $_POST['id'];
+        $comando = "select * from Ubicaciones where id='".$id."';";
         $resultado = $this->bdd->ejecuta($comando);
         if (!$resultado) {
             return $this->bdd->mensajeError($comando);
         }
         $fila = $this->bdd->procesaResultado();
-        $plantilla = str_replace("{id}", $id, $plantilla);
-        $plantilla = str_replace("{Descripcion}", $fila['Descripcion'], $plantilla);
-        file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla ' . $salida);
+        $plantilla = str_replace('{id}', $id, $plantilla);
+        $plantilla = str_replace('{Descripcion}', $fila['Descripcion'], $plantilla);
+        file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla '.$salida);
         switch ($salidaInforme) {
-            case "pantalla":
+            case 'pantalla':
                 $informe = new InformePDF($this->bdd, $salida, true);
                 $informe->crea($salida);
                 $informe->cierraPDF();
+
                 return $this->devuelveInforme($informe);
-            case "csv":
+            case 'csv':
                 //Genera una hoja de cálculo en formato csv
-                $nombre = "tmp/Ubicacion" . strftime("%Y%m%d") . rand(100, 999) . ".csv";
+                $nombre = TMP.'/Ubicacion'.strftime('%Y%m%d').rand(100, 999).'.csv';
                 $hoja = new Csv($this->bdd);
                 $hoja->crea($nombre);
                 $hoja->ejecutaConsulta($salida);
-                echo '<script type="text/javascript"> window.open( "' . $nombre . '" ) </script>';
+                echo '<script type="text/javascript"> window.open( "'.$nombre.'" ) </script>';
                 break;
-            case "etiquetas":
+            case 'etiquetas':
                 $etiquetas = new EtiquetasPDF($this->bdd, $salida, true);
                 $etiquetas->crea($salida);
                 $etiquetas->cierraPDF();
+
                 return $this->devuelveInforme($etiquetas);
         }
     }
@@ -117,50 +119,52 @@ class InformeInventario {
     {
         $salidaInforme = isset($_POST['salida']) ? $_POST['salida'] : 'pantalla';
         switch ($salidaInforme) {
-            case "pantalla":
-                $fichero = "xml/inventarioArticulo.xml";
-                $salida = "tmp/inventarioArticulo.xml";
+            case 'pantalla':
+                $fichero = 'xml/inventarioArticulo.xml';
+                $salida = TMP.'/inventarioArticulo.xml';
                 break;
-            case "csv":
-                $fichero = "xml/inventarioArticuloCSV.xml";
-                $salida = "tmp/inventarioArticuloCSV.xml";
+            case 'csv':
+                $fichero = 'xml/inventarioArticuloCSV.xml';
+                $salida = TMP.'/inventarioArticuloCSV.xml';
                 break;
-            case "etiquetas":
-                $fichero = "xml/inventarioArticuloEtiquetas.xml";
-                $salida = "tmp/inventarioArticuloEtiquetas.xml";
+            case 'etiquetas':
+                $fichero = 'xml/inventarioArticuloEtiquetas.xml';
+                $salida = TMP.'/inventarioArticuloEtiquetas.xml';
                 break;
         }
-        $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla ' . $fichero);
-        $id = $_POST['id'] == NULL ? $_GET['id'] : $_POST['id'];
-        $comando = "select * from Articulos where id='" . $id . "';";
+        $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla '.$fichero);
+        $id = $_POST['id'] == null ? $_GET['id'] : $_POST['id'];
+        $comando = "select * from Articulos where id='".$id."';";
         $resultado = $this->bdd->ejecuta($comando);
         if (!$resultado) {
             return $this->bdd->mensajeError($comando);
         }
         $fila = $this->bdd->procesaResultado();
-        $plantilla = str_replace("{id}", $id, $plantilla);
-        $plantilla = str_replace("{Descripcion}", $fila['descripcion'], $plantilla);
-        $plantilla = str_replace("{Marca}", $fila['marca'], $plantilla);
-        $plantilla = str_replace("{Modelo}", $fila['modelo'], $plantilla);
-        file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla ' . $salida);
+        $plantilla = str_replace('{id}', $id, $plantilla);
+        $plantilla = str_replace('{Descripcion}', $fila['descripcion'], $plantilla);
+        $plantilla = str_replace('{Marca}', $fila['marca'], $plantilla);
+        $plantilla = str_replace('{Modelo}', $fila['modelo'], $plantilla);
+        file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla '.$salida);
         switch ($salidaInforme) {
-            case "pantalla":
+            case 'pantalla':
                 $informe = new InformePDF($this->bdd, $salida, true);
                 $informe->crea($salida);
                 $informe->cierraPDF();
+
                 return $this->devuelveInforme($informe);
-            case "csv":
+            case 'csv':
                 //Genera una hoja de cálculo en formato csv
-                $nombre = "tmp/Articulo" . strftime("%Y%m%d") . rand(100, 999) . ".csv";
+                $nombre = TMP.'/Articulo'.strftime('%Y%m%d').rand(100, 999).'.csv';
                 $hoja = new Csv($this->bdd);
                 $hoja->crea($nombre);
                 $hoja->ejecutaConsulta($salida);
-                echo '<script type="text/javascript"> window.open( "' . $nombre . '" ) </script>';
+                echo '<script type="text/javascript"> window.open( "'.$nombre.'" ) </script>';
                 break;
-            case "etiquetas":
+            case 'etiquetas':
                 $etiquetas = new EtiquetasPDF($this->bdd, $salida, true);
                 $etiquetas->crea($salida);
                 $etiquetas->cierraPDF();
+
                 return $this->devuelveInforme($etiquetas);
         }
     }
@@ -168,60 +172,65 @@ class InformeInventario {
     private function listaUbicaciones()
     {
         $salida = "<select class=\"selectpicker show-tick\" name=\"id\" data-live-search=\"true\" data-width=\"auto\">\n";
-        $comando = "select * from Ubicaciones order by Descripcion";
+        $comando = 'select * from Ubicaciones order by Descripcion';
         $resultado = $this->bdd->ejecuta($comando);
         if (!$resultado) {
             return $this->bdd->mensajeError($comando);
         }
         while ($fila = $this->bdd->procesaResultado()) {
-            $salida.="<option value=" . $fila['id'] . ">" . $fila['Descripcion'] . "</option><br>\n";
+            $salida .= '<option value='.$fila['id'].'>'.$fila['Descripcion']."</option><br>\n";
         }
-        $salida.="</select>\n";
+        $salida .= "</select>\n";
+
         return $salida;
     }
 
     private function listaArticulos()
     {
         $salida = "<select class=\"selectpicker show-tick\" name=\"id\" data-live-search=\"true\" data-width=\"auto\">\n";
-        $comando = "select * from Articulos order by descripcion, marca, modelo";
+        $comando = 'select * from Articulos order by descripcion, marca, modelo';
         $resultado = $this->bdd->ejecuta($comando);
         if (!$resultado) {
             return $this->bdd->mensajeError($comando);
         }
         while ($fila = $this->bdd->procesaResultado()) {
-            $salida.="<option value=" . $fila['id'] . ">" . $fila['descripcion'] . "-" . $fila['marca'] . "-" . $fila['modelo'] . "</option><br>\n";
+            $salida .= '<option value='.$fila['id'].'>'.$fila['descripcion'].'-'.$fila['marca'].'-'.$fila['modelo']."</option><br>\n";
         }
-        $salida.="</select>\n";
+        $salida .= "</select>\n";
+
         return $salida;
     }
 
     private function formulario($accion, $etiqueta, $lista)
     {
-        $salida = '<div class="col-sm-6 col-md-6"><form name="informeInventario.form" method="post" action="' . $accion . '">' . "\n";
-        $salida.="<fieldset style=\"width: 96%;\"><p><legend style=\"color: red;\"><b>Elige $etiqueta</b></legend>\n";
-        $salida.="<br><br><label>$etiqueta </label>";
-        $salida.=$lista;
-        $salida.="<br><br>
+        $salida = '<div class="col-sm-6 col-md-6"><form name="informeInventario.form" method="post" action="'.$accion.'">'."\n";
+        $salida .= "<fieldset style=\"width: 96%;\"><p><legend style=\"color: red;\"><b>Elige $etiqueta</b></legend>\n";
+        $salida .= "<br><br><label>$etiqueta </label>";
+        $salida .= $lista;
+        $salida .= "<br><br>
             <label for='salida'>Salida del informe por:</label>";
-        $salida.='<div class="radio"><label><input type="radio" name="salida" value="pantalla" checked><span class="glyphicon glyphicon-list-alt"></span> Pantalla</label></div>';
-        $salida.='<div class="radio"><label><input type="radio" name="salida" value="csv"><span class="glyphicon glyphicon-cloud-download"></span> Archivo CSV</label></div>';
-        $salida.='<div class="radio"><label><input type="radio" name="salida" value="etiquetas"><span class="glyphicon glyphicon-qrcode"></span> Etiquetas (<a target="_new" href="http://www.apli.es/producto/ficha_producto.aspx?referencia=01275&stype=referencia&referenciaValue=01275&q=01275">Apli 1725</a>)</label></div>';
-        $salida.="<br><br></fieldset><p>";
-        $salida.='<p align="center"><button type=submit class="btn btn-primary"><span class="glyphicon glyphicon-ok"></span> Aceptar</button></p><br></div>' . "\n";
-        $salida.="<script>$('.selectpicker').selectpicker();</script>";
+        $salida .= '<div class="radio"><label><input type="radio" name="salida" value="pantalla" checked><span class="glyphicon glyphicon-list-alt"></span> Pantalla</label></div>';
+        $salida .= '<div class="radio"><label><input type="radio" name="salida" value="csv"><span class="glyphicon glyphicon-cloud-download"></span> Archivo CSV</label></div>';
+        $salida .= '<div class="radio"><label><input type="radio" name="salida" value="etiquetas"><span class="glyphicon glyphicon-qrcode"></span> Etiquetas (<a target="_new" href="http://www.apli.es/producto/ficha_producto.aspx?referencia=01275&stype=referencia&referenciaValue=01275&q=01275">Apli 1275</a>)</label></div>';
+        $salida .= '<br><br></fieldset><p>';
+        $salida .= '<p align="center"><button type=submit class="btn btn-primary"><span class="glyphicon glyphicon-ok"></span> Aceptar</button></p><br></div>'."\n";
+        $salida .= "<script>$('.selectpicker').selectpicker();</script>";
+
         return $salida;
     }
 
     private function formularioUbicacion()
     {
         //Genera un formulario con las ubicaciones disponibles.
-        $accion = "index.php?informeInventario&opc=listarUbicacion";
+        $accion = 'index.php?informeInventario&opc=listarUbicacion';
+
         return $this->formulario($accion, 'Ubicaci&oacute;n', $this->listaUbicaciones());
     }
 
     private function formularioArticulo()
     {
-        $accion = "index.php?informeInventario&opc=listarArticulo";
+        $accion = 'index.php?informeInventario&opc=listarArticulo';
+
         return $this->formulario($accion, 'Art&iacute;culo', $this->listaArticulos());
     }
 
@@ -235,18 +244,19 @@ class InformeInventario {
         $dialogo = '<div class="container col-5"><div class="jumbotron">
                       <h1>Inventario Total</h1>
                       <p>¿Desea obtener el inventario de todo el centro?</p>
-                      <p><a class="btn btn-primary btn-lg" role="button" onClick="location.href=' . "'index.php'" . '"><span class="glyphicon glyphicon-arrow-left"></span> Volver</a>
-                         <a class="btn btn-success btn-lg" role="button" onClick="location.href=' . "'index.php?informeInventario&opc=listarTotal'" . '">
-                             <span class="glyphicon glyphicon-cloud-download"></span> Continuar</a></p>
+                      <p><a class="btn btn-primary btn-lg" role="button" onClick="location.href='."'index.php'".'"><span class="glyphicon glyphicon-arrow-left"></span> Volver</a>
+                         <a class="btn btn-success btn-lg" role="button" onClick="location.href='."'index.php?informeInventario&opc=listarTotal'".'">
+                             <span class="glyphicon glyphicon-list-alt"></span> Continuar</a></p>
                     </div></div>';
+
         return $dialogo;
     }
 
     private function listarTotal()
     {
-        $fichero = "xml/inventarioUbicacion.xml";
-        $salida = "tmp/inventarioUbicacion.xml";
-        $comando = "select * from Ubicaciones ;";
+        $fichero = 'xml/inventarioUbicacion.xml';
+        $salida = TMP.'/inventarioUbicacion.xml';
+        $comando = 'select * from Ubicaciones ;';
         $resultado = $this->bdd->ejecuta($comando);
         if (!$resultado) {
             return $this->bdd->mensajeError($comando);
@@ -255,10 +265,10 @@ class InformeInventario {
         $bdatos = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
         $primero = true;
         while ($fila = $this->bdd->procesaResultado()) {
-            $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla ' . $fichero);
-            $plantilla = str_replace("{id}", $fila['id'], $plantilla);
-            $plantilla = str_replace("{Descripcion}", $fila['Descripcion'], $plantilla);
-            file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla ' . $salida);
+            $plantilla = file_get_contents($fichero) or die('Fallo en la apertura de la plantilla '.$fichero);
+            $plantilla = str_replace('{id}', $fila['id'], $plantilla);
+            $plantilla = str_replace('{Descripcion}', $fila['Descripcion'], $plantilla);
+            file_put_contents($salida, $plantilla) or die('Fallo en la escritura de la plantilla '.$salida);
             if ($primero) {
                 $primero = false;
                 $informe = new InformePDF($bdatos, $salida, true);
@@ -266,9 +276,7 @@ class InformeInventario {
             $informe->crea($salida);
         }
         $informe->cierraPDF();
+
         return $this->devuelveInforme($informe);
     }
-
 }
-
-?>

InformePDF.php

@@ -1,13 +1,13 @@
-
 <?php
 
 /**
- * genera un documento PDF a partir de una descripción dada en un archivo XML
+ * genera un documento PDF a partir de una descripción dada en un archivo XML.
+ *
  * @author Ricardo Montañana <rmontanana@gmail.com>
+ *
  * @version 1.0
- * @package Inventario
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana
- * @package Inventario
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -23,12 +23,10 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-class InformePDF {
-
+class InformePDF
+{
     /**
-     * 
      * @var basedatos Controlador de la base de datos
      */
     private $bdd;
@@ -37,14 +35,17 @@ class InformePDF {
     private $def;
 
     /**
-     * El constructor recibe como argumento el nombre del archivo XML con la definición, encargándose de recuperarla y guardar toda la información localmente
+     * El constructor recibe como argumento el nombre del archivo XML con la definición, encargándose de recuperarla y guardar toda la información localmente.
+     *
      * @param basedatos $bdd        manejador de la base de datos
      * @param string    $definicion fichero con la definición del informe en XML
-     * @param boolean $registrado usuario registrado si/no
+     * @param bool      $registrado usuario registrado si/no
+     *
      * @return ficheroPDF
      *                    todo: cambiar este comentario
      */
-    public function __construct($bdd, $definicion, $registrado) {
+    public function __construct($bdd, $definicion, $registrado)
+    {
         if (!$registrado) {
             return 'Debe registrarse para acceder a este apartado';
         }
@@ -54,14 +55,15 @@ class InformePDF {
         $this->pdf = new Pdf_mysql_table($this->bdd->obtieneManejador(), (string) $this->def->Pagina['Orientacion'], (string) $this->def->Pagina['Formato'], (string) utf8_decode($this->def->Titulo['Texto']), (string) $this->def->Pagina->Cabecera);
         //echo $def->Titulo.$def->Cabecera;
         $this->pdf->Open();
-        $this->pdf->setAuthor(AUTOR,true);
-        $creador = CENTRO . " " . PROGRAMA.VERSION;
-        $this->pdf->setCreator(html_entity_decode($creador),true);
-        $this->pdf->setSubject($this->def->Titulo,true);
+        $this->pdf->setAuthor(AUTOR, true);
+        $creador = CENTRO.' '.PROGRAMA.' v'.VERSION;
+        $this->pdf->setCreator(html_entity_decode($creador), true);
+        $this->pdf->setSubject($this->def->Titulo, true);
         $this->pdf->setAutoPageBreak(true, 10);
     }
 
-    public function crea($definicion) {
+    public function crea($definicion)
+    {
 
         //print_r($def);echo $bdd;die();
         // Iniciamos la creación del documento
@@ -75,48 +77,55 @@ class InformePDF {
         foreach ($this->def->Pagina->Cuerpo->Col as $columna) {
             $this->pdf->AddCol((string) $columna['Nombre'], (string) $columna['Ancho'], (string) $columna['Titulo'], (string) $columna['Ajuste'], (string) $columna['Total']);
         }
-        $prop = array('HeaderColor' => array(255, 150, 100),
-            'color1' => array(210, 245, 255),
-            'color2' => array(255, 255, 210),
-            'padding' => 2);
+        $prop = ['HeaderColor' => [255, 150, 100],
+            'color1'           => [210, 245, 255],
+            'color2'           => [255, 255, 210],
+            'padding'          => 2];
         $this->pdf->Table($this->def->Datos->Consulta, $prop);
     }
 
-    public function cierraPDF() {
+    public function cierraPDF()
+    {
         $this->pdf->Close();
         $this->docu = $this->pdf->Output('', 'S');
     }
 
-    public function getContenido() {
+    public function getContenido()
+    {
         return $this->docu;
     }
 
-    public function getCabecera() {
-        $cabecera = "Content-type: application/pdf";
-        $cabecera = $cabecera . "Content-length: " . strlen($this->docu);
-        $cabecera = $cabecera . "Content-Disposition: inline; filename=tmp/Informe.pdf";
+    public function getCabecera()
+    {
+        $cabecera = 'Content-type: application/pdf';
+        $cabecera = $cabecera.'Content-length: '.strlen($this->docu);
+        $cabecera = $cabecera.'Content-Disposition: inline; filename='.TMP.'/Informe.pdf';
+
         return $cabecera;
     }
 
-    public function guardaArchivo($nombre = "tmp/Informe.pdf") {
-        $fichero = fopen($nombre, "w");
+    public function guardaArchivo($nombre)
+    {
+        if (!isset($nombre)) {
+            $nombre = TMP.'/Informe.pdf';
+        }
+        $fichero = fopen($nombre, 'w');
         fwrite($fichero, $this->getCabecera());
         fwrite($fichero, $this->getContenido(), strlen($this->getContenido()));
         fclose($fichero);
     }
 
-    public function enviaCabecera() {
-        header("Content-type: application/pdf");
+    public function enviaCabecera()
+    {
+        header('Content-type: application/pdf');
         $longitud = strlen($this->docu);
         header("Content-length: $longitud");
-        header("Content-Disposition: inline; filename=tmp/Informe.pdf");
+        header('Content-Disposition: inline; filename='.TMP.'/Informe.pdf');
     }
 
-    public function imprimeInforme() {
+    public function imprimeInforme()
+    {
         $this->enviaCabecera();
         echo $this->docu;
     }
-
 }
-
-?>

Instalar.php

@@ -1,8 +1,8 @@
 <?php
 /**
  * Programa de instalación que genera el entorno de ejecución
- * tanto el fichero de configuración como la base de datos
- * @package Inventario
+ * tanto el fichero de configuración como la base de datos.
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -18,11 +18,11 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *
  */
 //Se incluyen los módulos necesarios
-function __autoload($class_name) {
-    require_once $class_name . '.php';
+function __autoload($class_name)
+{
+    require_once $class_name.'.php';
 }
 
 require_once 'inc/configuracion.inc';
@@ -31,18 +31,19 @@ define('NUMPASOS', 3);
 define('MINBYTES', 4096000); // post_max_size y max_upload van con esto
 define('CADENAMINBYTES', '4M');
 define('CONFIGURACION', 'inc/configuracion.inc');
-define('CONFIGTMP', 'tmp/config.tmp');
-define('TMP', './tmp');
+define('CONFIGTMP', TMP.'/config.tmp');
 define('INC', './inc');
 
 $instalar = new Instalar();
 if ($instalar->error) {
     echo $instalar->panelError();
+
     return;
 }
 echo $instalar->ejecuta();
 
-class Instalar {
+class Instalar
+{
     private $contenido;
     private $plant;
     public $error;
@@ -51,9 +52,9 @@ class Instalar {
     public function __construct()
     {
         //Selecciona la plantilla a utilizar
-        $this->plant='plant/';
-        $this->plant.=PLANTILLA;
-        $this->plant.='.html';
+        $this->plant = 'plant/';
+        $this->plant .= PLANTILLA;
+        $this->plant .= '.html';
         $this->error = false;
         $this->eror_msj = '';
         if (INSTALADO != 'no') {
@@ -70,11 +71,14 @@ class Instalar {
     {
         //Comprueba si existe la tabla Articulos
         $sql = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
-        if ($sql->error())
+        if ($sql->error()) {
             return false;
+        }
         $sql->ejecuta('select * from Articulos;');
-        if ($sql->error())
+        if ($sql->error()) {
             return false;
+        }
+
         return true;
     }
 
@@ -82,17 +86,18 @@ class Instalar {
     {
         $paso = isset($_GET['paso']) ? $_GET['paso'] : 0;
         $paso = $paso > NUMPASOS ? '0' : $paso;
-        $i=0;
+        $i = 0;
         //Si quiere ir a un determinado paso se asegura que estén completos los anteriores
         for ($i = 0; $i < $paso; $i++) {
-            $funcion = "validaPaso" . $i;
+            $funcion = 'validaPaso'.$i;
             if (!$this->$funcion()) {
                 break;
             }
         }
-        $funcion = "paso" . $i;
+        $funcion = 'paso'.$i;
         $this->contenido = $this->$funcion();
         $salida = new Distribucion($this->plant, $this);
+
         return $salida->procesaPlantilla();
     }
 
@@ -103,100 +108,107 @@ class Instalar {
         $info .= '<li class="list-group-item list-group-item-info">Configuración de PHP (php.ini)</li>';
         // display_errors
         $displayErr = ini_get('display_errors');
-        $displayErr = $displayErr == "1" || $displayErr == "on" ? "on" : "off";
-        $mensaje = $displayErr == "off" ? $this->retornaLabel(false,'Se debe deshabilitar la impresión de errores') :
-                                          $this->retornaLabel(true, 'Se debe deshabilitar la impresión de errores', "warning");
+        $displayErr = $displayErr == '1' || $displayErr == 'on' ? 'on' : 'off';
+        $mensaje = $displayErr == 'off' ? $this->retornaLabel(false, 'Se debe deshabilitar la impresión de errores') :
+                                          $this->retornaLabel(true, 'Se debe deshabilitar la impresión de errores', 'warning');
         $info .= $this->retornaElemento($mensaje, 'display_errors', $displayErr);
         // post_max_size
         $postMax = ini_get('post_max_size');
-        $mensaje = $this->retornaBytes($postMax) >= MINBYTES ? $this->retornaLabel(false, 'Mínimo: ' . CADENAMINBYTES) :
-                                                               $this->retornaLabel(true, 'Mínimo: ' . CADENAMINBYTES);
+        $mensaje = $this->retornaBytes($postMax) >= MINBYTES ? $this->retornaLabel(false, 'Mínimo: '.CADENAMINBYTES) :
+                                                               $this->retornaLabel(true, 'Mínimo: '.CADENAMINBYTES);
         $info .= $this->retornaElemento($mensaje, 'post_max_size', $postMax);
         // upload_max_filesize
         $uploadMax = ini_get('upload_max_filesize');
-        $mensaje = $this->retornaBytes($uploadMax) >= MINBYTES ? $this->retornaLabel(false, 'Mínimo: ' . CADENAMINBYTES) : 
-                                                                 $this->retornaLabel(true, 'Mínimo: ' . CADENAMINBYTES);
+        $mensaje = $this->retornaBytes($uploadMax) >= MINBYTES ? $this->retornaLabel(false, 'Mínimo: '.CADENAMINBYTES) :
+                                                                 $this->retornaLabel(true, 'Mínimo: '.CADENAMINBYTES);
         $info .= $this->retornaElemento($mensaje, 'upload_max_filesize', $uploadMax);
         // mysqli
         $mysql = extension_loaded('mysqli');
-        $mysql = $mysql ? "on" : "off";
+        $mysql = $mysql ? 'on' : 'off';
         $mensaje = $mysql ? $this->retornaLabel(false, 'Tiene que estar cargada la extensión MySQLi para poder funcionar') :
                             $this->retornaLabel(true, 'Tiene que estar cargada la extensión MySQLi para poder funcionar');
         $info .= $this->retornaElemento($mensaje, 'extensión MySQLi', $mysql);
         $info .= '<li class="list-group-item list-group-item-info">Configuración de la Aplicación</li>';
         // img.dat
-        $mensaje = is_writable(IMAGEDATA) ? $this->retornaLabel(false, "Se debe poder escribir en el directorio " . IMAGEDATA) : 
-                                      $this->retornaLabel(true, "Se debe poder escribir en el directorio " . IMAGEDATA);
-        $valor = is_writable(IMAGEDATA) ? "Sí" : "No";
-        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en ' . IMAGEDATA, $valor);      
+        $mensaje = is_writable(IMAGEDATA) ? $this->retornaLabel(false, 'Se debe poder escribir en el directorio '.IMAGEDATA) :
+                                      $this->retornaLabel(true, 'Se debe poder escribir en el directorio '.IMAGEDATA);
+        $valor = is_writable(IMAGEDATA) ? 'Sí' : 'No';
+        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en '.IMAGEDATA, $valor);
 
         // tmp
-        $mensaje = is_writable(TMP) ? $this->retornaLabel(false, "Se debe poder escribir en el directorio " . TMP) : 
-                                      $this->retornaLabel(true, "Se debe poder escribir en el directorio " . TMP);
-        $valor = is_writable(TMP) ? "Sí" : "No";
-        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en ' . TMP, $valor);
+        $mensaje = is_writable(TMP) ? $this->retornaLabel(false, 'Se debe poder escribir en el directorio '.TMP) :
+                                      $this->retornaLabel(true, 'Se debe poder escribir en el directorio '.TMP);
+        $valor = is_writable(TMP) ? 'Sí' : 'No';
+        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en '.TMP, $valor);
 
         // inc
-        $mensaje = is_writable(INC) ? $this->retornaLabel(false, "Se debe poder escribir en el directorio " . INC) : 
-                                      $this->retornaLabel(true, "Se debe poder escribir en el directorio " . INC);
-        $valor = is_writable(INC) ? "Sí" : "No";
-        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en ' . INC, $valor);
+        $mensaje = is_writable(INC) ? $this->retornaLabel(false, 'Se debe poder escribir en el directorio '.INC) :
+                                      $this->retornaLabel(true, 'Se debe poder escribir en el directorio '.INC);
+        $valor = is_writable(INC) ? 'Sí' : 'No';
+        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en '.INC, $valor);
 
         // configuracion.inc
-        $mensaje = is_writable(CONFIGURACION) ? $this->retornaLabel(false, "Se debe poder escribir en el fichero de configuración ". CONFIGURACION) :
-                                                $this->retornaLabel(true, "Se debe poder escribir en el fichero de configuración ". CONFIGURACION);
-        $valor = is_writable(CONFIGURACION) ? "Sí" : "No";
-        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en ' . CONFIGURACION, $valor);
+        $mensaje = is_writable(CONFIGURACION) ? $this->retornaLabel(false, 'Se debe poder escribir en el fichero de configuración '.CONFIGURACION) :
+                                                $this->retornaLabel(true, 'Se debe poder escribir en el fichero de configuración '.CONFIGURACION);
+        $valor = is_writable(CONFIGURACION) ? 'Sí' : 'No';
+        $info .= $this->retornaElemento($mensaje, 'Se puede escribir en '.CONFIGURACION, $valor);
 
         // Final del paso
-        $info .='</ul>';
-        $info .= $this->validaPaso0() ? $this->retornaBoton(false, "instalar.php?paso=1") : $this->retornaBoton(true, "instalar.php");
+        $info .= '</ul>';
+        $info .= $this->validaPaso0() ? $this->retornaBoton(false, 'Instalar.php?paso=1') : $this->retornaBoton(true, 'Instalar.php');
         $panel = $this->panelMensaje($info, 'primary', 'PASO 1: Configuración del servidor y la aplicación');
+
         return $panel;
     }
+
     private function retornaElemento($validacion, $mensaje, $valor)
     {
         $info = '<li class="list-group-item">';
-        $info .= $validacion . ' ' . $mensaje . ': <span class="badge">' . $valor . '</span>';
+        $info .= $validacion.' '.$mensaje.': <span class="badge">'.$valor.'</span>';
         $info .= '</li>';
+
         return $info;
     }
 
     private function retornaBoton($error, $paso, $javascript = true)
     {
-        $anadido = $javascript ? 'onclick="location.href=' . "'" . $paso . "'". '"' : '';
+        $anadido = $javascript ? 'onclick="location.href='."'".$paso."'".'"' : '';
         if (!$error) {
-            return '<button ' . $anadido . ' type="submit" class="btn btn-success btn-lg pull-right">Continuar <span class="glyphicon glyphicon-arrow-right"></span></button>';
+            return '<button '.$anadido.' type="submit" class="btn btn-success btn-lg pull-right">Continuar <span class="glyphicon glyphicon-arrow-right"></span></button>';
         } else {
-            return '<button ' . $anadido . ' type="submit" class="btn btn-danger btn-lg pull-right">Comprobar de nuevo <span class="glyphicon glyphicon-repeat"></span></button>';
+            return '<button '.$anadido.' type="submit" class="btn btn-danger btn-lg pull-right">Comprobar de nuevo <span class="glyphicon glyphicon-repeat"></span></button>';
         }
     }
 
     private function botonVolver($enlace)
     {
-        $boton = '<button type="button" onClick="location.href=' . "'$enlace'" . '" class="btn btn-success btn-lg pull-left">Paso anterior <span class="glyphicon glyphicon-arrow-left"></span></button>';
+        $boton = '<button type="button" onClick="location.href='."'$enlace'".'" class="btn btn-success btn-lg pull-left">Paso anterior <span class="glyphicon glyphicon-arrow-left"></span></button>';
+
         return $boton;
     }
 
-    private function retornaLabel($error, $mensaje, $tipo = "danger")
+    private function retornaLabel($error, $mensaje, $tipo = 'danger')
     {
         if ($error) {
-            $nombre1 = $tipo; $nombre2 = "remove";
+            $nombre1 = $tipo;
+            $nombre2 = 'remove';
         } else {
-            $nombre1 = "success"; $nombre2 = "ok";
+            $nombre1 = 'success';
+            $nombre2 = 'ok';
         }
-        $mensaje = '<a href="#" data-placement="right" data-toggle="popover" data-content="' . $mensaje . 
-                   '"><span class="label label-' . $nombre1 . '"><span class="glyphicon glyphicon-' . $nombre2 . 
+        $mensaje = '<a href="#" data-placement="right" data-toggle="popover" data-content="'.$mensaje.
+                   '"><span class="label label-'.$nombre1.'"><span class="glyphicon glyphicon-'.$nombre2.
                    '"></span></a>';
-        $mensaje .='<script>$(function () { $("[data-toggle=\'popover\']").popover(); });</script>';
+        $mensaje .= '<script>$(function () { $("[data-toggle=\'popover\']").popover(); });</script>';
+
         return $mensaje;
     }
 
     private function retornaBytes($val)
     {
         $val = trim($val);
-        $last = strtolower($val[strlen($val)-1]);
-        switch($last) {
+        $last = strtolower($val[strlen($val) - 1]);
+        switch ($last) {
             // El modificador 'G' está disponble desde PHP 5.1.0
             case 'g':
                 $val *= 1024;
@@ -205,6 +217,7 @@ class Instalar {
             case 'k':
                 $val *= 1024;
         }
+
         return $val;
     }
 
@@ -218,20 +231,28 @@ class Instalar {
         $escInc = is_writable(INC);
         $escTMP = is_writable(TMP);
         $escIMG = is_writable(IMAGEDATA);
-        if ($this->retornaBytes($postMax) < MINBYTES) 
+        if ($this->retornaBytes($postMax) < MINBYTES) {
             $validar = false;
-        if ($this->retornaBytes($uploadMax) < MINBYTES)
+        }
+        if ($this->retornaBytes($uploadMax) < MINBYTES) {
             $validar = false;
-        if (!$mysql) 
+        }
+        if (!$mysql) {
             $validar = false;
-        if (!$escConfig) 
+        }
+        if (!$escConfig) {
             $validar = false;
-        if (!$escTMP)
+        }
+        if (!$escTMP) {
             $validar = false;
-        if (!$escIMG)
+        }
+        if (!$escIMG) {
             $validar = false;
-        if (!$escInc)
+        }
+        if (!$escInc) {
             $validar = false;
+        }
+
         return $validar;
     }
 
@@ -241,10 +262,10 @@ class Instalar {
         $fichero = $conf->obtieneFichero();
         $datosFichero = explode("\n", $fichero);
         if ($grabar) {
-            $fsalida = @fopen(CONFIGTMP, "wb");
+            $fsalida = @fopen(CONFIGTMP, 'wb');
         }
         foreach ($datosFichero as $linea) {
-            if (stripos($linea, "DEFINE") !== false) {
+            if (stripos($linea, 'DEFINE') !== false) {
                 $conf->obtieneDatos($linea, $clave, $valor);
                 if (stripos($campos, $clave) !== false) {
                     if ($grabar) {
@@ -254,7 +275,7 @@ class Instalar {
                 }
                 $datos[$clave] = $valor;
             }
-            $registro = substr($linea, 0, 2) == "?>" ? $linea : $linea . "\n";
+            $registro = substr($linea, 0, 2) == '?>' ? $linea : $linea."\n";
             if ($grabar) {
                 fwrite($fsalida, $registro);
             }
@@ -277,7 +298,7 @@ class Instalar {
                 $datos[$clave] = $valor;
             }
         } else {
-            $datos = array();
+            $datos = [];
         }
         $this->actualizaConfiguracion($grabar, $campos, $datos);
         if ($grabar && $this->validaPaso1()) {
@@ -285,27 +306,29 @@ class Instalar {
             return $this->paso2();
         }
 
-        $info  = '<form method="post" name="conf" action="instalar.php?paso=1">';
+        $info = '<form method="post" name="conf" action="Instalar.php?paso=1">';
         $info .= '<ul class="list-group">';
         $info .= '<li class="list-group-item list-group-item-info">Datos de configuración</li>';
-        $info .= '<li class="list-group-item">Servidor <input type="text" name="SERVIDOR" class="form-control" placeholder="Nombre del servidor o dirección IP" value="'. $datos['SERVIDOR'] .'"></li>';
-        $info .= '<li class="list-group-item">Puerto <input type="text" name="PUERTO" class="form-control" placeholder="Puerto de conexión" value="'. $datos['PUERTO'] .'"></li>';
-        $info .= '<li class="list-group-item">Base de Datos <input type="text" name="BASEDATOS" class="form-control" placeholder="Nombre de la Base de Datos" value="'. $datos['BASEDATOS'] .'"></li>';
-        $info .= '<li class="list-group-item">Usuario <input type="text" name="USUARIO" class="form-control" placeholder="Usuario" value="'. $datos['USUARIO'] .'"></li>';
-        $info .= '<li class="list-group-item">Contraseña <input type="text" name="CLAVE" class="form-control" placeholder="Contraseña" value="'. $datos['CLAVE'] .'"></li>';
+        $info .= '<li class="list-group-item">Servidor <input type="text" name="SERVIDOR" class="form-control" placeholder="Nombre del servidor o dirección IP" value="'.$datos['SERVIDOR'].'"></li>';
+        $info .= '<li class="list-group-item">Puerto <input type="text" name="PUERTO" class="form-control" placeholder="Puerto de conexión" value="'.$datos['PUERTO'].'"></li>';
+        $info .= '<li class="list-group-item">Base de Datos <input type="text" name="BASEDATOS" class="form-control" placeholder="Nombre de la Base de Datos" value="'.$datos['BASEDATOS'].'"></li>';
+        $info .= '<li class="list-group-item">Usuario <input type="text" name="USUARIO" class="form-control" placeholder="Usuario" value="'.$datos['USUARIO'].'"></li>';
+        $info .= '<li class="list-group-item">Contraseña <input type="text" name="CLAVE" class="form-control" placeholder="Contraseña" value="'.$datos['CLAVE'].'"></li>';
         $info .= '</ul>';
-        $info .= $this->botonVolver("instalar.php");
-        $info .= $this->validaPaso1() ? $this->retornaBoton(false, "instalar.php?paso=1", false) : $this->retornaBoton(true, "instalar.php?paso=1", false);
+        $info .= $this->botonVolver('Instalar.php');
+        $info .= $this->validaPaso1() ? $this->retornaBoton(false, 'Instalar.php?paso=1', false) : $this->retornaBoton(true, 'Instalar.php?paso=1', false);
         $info .= '</form>';
         $panel = $this->panelMensaje($info, 'primary', 'PASO 2: Configuración de la Base de Datos.');
+
         return $panel;
     }
 
     private function validaPaso1()
     {
         $sql = new Sql(SERVIDOR, USUARIO, CLAVE, '');
-        if ($sql->error())
+        if ($sql->error()) {
             return false;
+        }
         $sql = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
         if ($sql->error()) {
             return false;
@@ -320,6 +343,7 @@ class Instalar {
         if ($sql->error()) {
             return false;
         }
+
         return true;
     }
 
@@ -329,8 +353,8 @@ class Instalar {
         if (isset($_POST['usuario'])) {
             //ha enviado el formulario.
             //Crea la base de datos
-            $borra_database = "DROP DATABASE " . BASEDATOS . " ;";
-            $database = "CREATE DATABASE " . BASEDATOS . " DEFAULT CHARACTER SET utf8;";
+            $borra_database = 'DROP DATABASE '.BASEDATOS.' ;';
+            $database = 'CREATE DATABASE '.BASEDATOS.' DEFAULT CHARACTER SET utf8;';
             $articulos = "CREATE TABLE `Articulos` (
                               `id` smallint(6) NOT NULL auto_increment COMMENT 'ordenable,link/Articulo',
                               `descripcion` varchar(60) NOT NULL COMMENT 'ordenable,ajax/text',
@@ -380,7 +404,7 @@ class Instalar {
                               KEY `nombre` (`nombre`)
                             ) ENGINE=InnoDB AUTO_INCREMENT=7 DEFAULT CHARSET=utf8;
                             ";
-            $letras = "abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
+            $letras = 'abcefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
             $sesion = substr(str_shuffle($letras), 0, 8);
             $usuario = $_POST['usuario'];
             $clave = $_POST['clave'];
@@ -391,32 +415,33 @@ class Instalar {
             $sql = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
             $sql->ejecuta($ubicaciones);
             if ($sql->error()) {
-                return $this->panelMensaje($sql->mensajeError(), "danger", "ERROR");
+                return $this->panelMensaje($sql->mensajeError(), 'danger', 'ERROR');
             }
             $sql->ejecuta($articulos);
             if ($sql->error()) {
-                return $this->panelMensaje($sql->mensajeError(), "danger", "ERROR");
+                return $this->panelMensaje($sql->mensajeError(), 'danger', 'ERROR');
             }
             $sql->ejecuta($elementos);
             if ($sql->error()) {
-                return $this->panelMensaje($sql->mensajeError(), "danger", "ERROR");
+                return $this->panelMensaje($sql->mensajeError(), 'danger', 'ERROR');
             }
             $sql->ejecuta($usuarios);
             if ($sql->error()) {
-                return $this->panelMensaje($sql->mensajeError(), "danger", "ERROR");
+                return $this->panelMensaje($sql->mensajeError(), 'danger', 'ERROR');
             }
             $sql->ejecuta($administrador);
             if ($sql->error()) {
-                return $this->panelMensaje($sql->mensajeError(), "danger", "ERROR");
+                return $this->panelMensaje($sql->mensajeError(), 'danger', 'ERROR');
             }
-            $campos="INSTALADO";
-            $datos['INSTALADO'] = "sí";
+            $campos = 'INSTALADO';
+            $datos['INSTALADO'] = 'sí';
             $this->actualizaConfiguracion(true, $campos, $datos);
+
             return $this->resumen();
         }
 
         $info = '      
-        <form data-toggle="validator" role="form" class="form-horizontal" method="post" action="instalar.php?paso=2">
+        <form data-toggle="validator" role="form" class="form-horizontal" method="post" action="Instalar.php?paso=2">
                 <div class="form-group">
                     <label for="usuario" class="control-label col-sm-2">Usuario</label>
                     <div class="form-group col-sm-10">
@@ -436,13 +461,14 @@ class Instalar {
                 </div>
 
                 <div class="form-group col-sm-12">
-                ' . $this->botonVolver("instalar.php?paso=1") . '
+                '.$this->botonVolver('Instalar.php?paso=1').'
                     <button type="submit" class="btn btn-primary pull-right btn-lg" disabled="disabled">Crear base de datos y usuario <span class="glyphicon glyphicon-arrow-right"></button>
                 </div>
             </div>
         </form>
         <script type="text/javascript" src="./css/validator.min.js"></script>';
         $panel = $this->panelMensaje($info, 'primary', 'PASO 3: Creación de la base de datos y el usuario administrador.');
+
         return $panel;
     }
 
@@ -452,13 +478,15 @@ class Instalar {
         return true;
     }
 
-    public function panelMensaje($info, $tipo = "info", $cabecera = "&iexcl;Atenci&oacute;n!") {
-        $mensaje = '<div class="panel panel-' . $tipo . ' col-sm-6"><div class="panel-heading">';
-        $mensaje .= '<h3 class="panel-title">' . $cabecera . '</h3></div>';
+    public function panelMensaje($info, $tipo = 'info', $cabecera = '&iexcl;Atenci&oacute;n!')
+    {
+        $mensaje = '<div class="panel panel-'.$tipo.' col-sm-6"><div class="panel-heading">';
+        $mensaje .= '<h3 class="panel-title">'.$cabecera.'</h3></div>';
         $mensaje .= '<div class="panel-body">';
         $mensaje .= $info;
         $mensaje .= '</div>';
         $mensaje .= '</div>';
+
         return $mensaje;
     }
 
@@ -484,7 +512,7 @@ class Instalar {
 
     public function aplicacion()
     {
-        return PROGRAMA . ' v' . VERSION;
+        return PROGRAMA.' v'.VERSION;
     }
 
     public function usuario()
@@ -495,10 +523,12 @@ class Instalar {
     public function fecha()
     {
         $idioma = 'es_ES';
-        $formato = "%d-%b-%y";
+        $formato = '%d-%b-%y';
         setlocale(LC_TIME, $idioma);
+
         return strftime($formato);
     }
+
     public function cabecera()
     {
         return '<!DOCTYPE html>
@@ -532,8 +562,9 @@ class Instalar {
     public function panelError()
     {
         $mensaje = $this->cabecera();
-        $mensaje .= $this->panelMensaje($this->error_msj, "danger", "&iexcl;ERROR!");
-        $mensaje .= "</body></html>";
+        $mensaje .= $this->panelMensaje($this->error_msj, 'danger', '&iexcl;ERROR!');
+        $mensaje .= '</body></html>';
+
         return $mensaje;
     }
 
@@ -541,19 +572,17 @@ class Instalar {
     {
         $info = '<ul class="list-group">';
         $info .= '<li class="list-group-item list-group-item-info">Paso 1</li>';
-        $info .= $this->retornaElemento($this->retornaLabel(false, ""), "Configuración de PHP");
-        $info .= $this->retornaElemento($this->retornaLabel(false, ""), "Configuración de la aplicación");
+        $info .= $this->retornaElemento($this->retornaLabel(false, ''), 'Configuración de PHP');
+        $info .= $this->retornaElemento($this->retornaLabel(false, ''), 'Configuración de la aplicación');
         $info .= '<li class="list-group-item list-group-item-info">Paso 2</li>';
-        $info .= $this->retornaElemento($this->retornaLabel(false, ""), "Configuración de la base de datos");
+        $info .= $this->retornaElemento($this->retornaLabel(false, ''), 'Configuración de la base de datos');
         $info .= '<li class="list-group-item list-group-item-info">Paso 3</li>';
-        $info .= $this->retornaElemento($this->retornaLabel(false, ""), "Creación de Base de datos");
-        $info .= $this->retornaElemento($this->retornaLabel(false, ""), "Creación del usuario administrador"); 
+        $info .= $this->retornaElemento($this->retornaLabel(false, ''), 'Creación de Base de datos');
+        $info .= $this->retornaElemento($this->retornaLabel(false, ''), 'Creación del usuario administrador');
         $info .= '</ul>';
-        $info .= $this->retornaBoton(false, "index.php", true);
+        $info .= $this->retornaBoton(false, 'index.php', true);
         $panel = $this->panelMensaje($info, 'success', 'Instalación finalizada.');
-        return $panel;
 
+        return $panel;
     }
 }
-
-?>

Inventario.php

@@ -2,9 +2,11 @@
 
 /**
  * Clase Inventario que controla la ejecución principal del programa.
+ *
  * @author Ricardo Montañana Gómez <rmontanana@gmail.com>
+ *
  * @version 1.0
- * @package Inventario
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -20,15 +22,14 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
 // Clase del objeto principal de la aplicación
-class Inventario {
-
+class Inventario
+{
     // Declaración de miembros
     private $bdd; // Enlace con el SGBD
     private $registrado; // Usuario registrado s/n
-    private $usuario = NULL; // Nombre del usuario
+    private $usuario = null; // Nombre del usuario
     private $clave; //contraseña del usuario
     private $opcActual; // Opción elegida por el usuario
     private $perfil; //Permisos del usuario.
@@ -36,13 +37,15 @@ class Inventario {
     private $plant;
 
     // Constructor
-    public function __construct() {
+    public function __construct()
+    {
         // Analizamos la cadena de solicitud para saber
         // qué opción es la actual
         $this->opcActual = $_SERVER['QUERY_STRING'] == '' ? 'principal' : $_SERVER['QUERY_STRING'];
         //Si el programa no está instalado, llama al instalador.
-        if (INSTALADO == "no") {
-            header('location: instalar.php');
+        if (INSTALADO == 'no') {
+            header('location: Instalar.php');
+
             return;
         }
         // Iniciamos una sesión
@@ -50,18 +53,18 @@ class Inventario {
         //Conexión con la base de datos.
         $this->bdd = new Sql(SERVIDOR, USUARIO, CLAVE, BASEDATOS);
         if ($this->bdd->error()) {
-            
             echo '<h1>Fallo al conectar con el servidor MySQL.</h1>';
-            echo "Servidor [ " . SERVIDOR . " ] base de datos [" . BASEDATOS . "]";
+            echo 'Servidor [ '.SERVIDOR.' ] base de datos ['.BASEDATOS.']';
             $this->estado = false;
+
             return;
         } else {
             $this->estado = true;
         }
         //Selecciona la plantilla a utilizar
-        $this->plant='plant/';
-        $this->plant.=PLANTILLA;
-        $this->plant.='.html';
+        $this->plant = 'plant/';
+        $this->plant .= PLANTILLA;
+        $this->plant .= '.html';
         // Comprobamos si el usuario ya está registrado en esta sesión
         $this->registrado = isset($_SESSION['Registrado']);
         if ($this->registrado) {// si está...
@@ -77,14 +80,16 @@ class Inventario {
         }
     }
 
-    public function estado() {
+    public function estado()
+    {
         return $this->estado;
     }
 
 //     Esta función pondrá en marcha la aplicación ocupándose
 //     de las acciones que no generan contenido, esto es
 //     iniciar sesión, cerrarla, etc.
-    public function Ejecuta() {
+    public function Ejecuta()
+    {
         // Dependiendo de la opción a procesar
         switch ($this->opcActual) {
             // El usuario quiere cerrar la sesión actual
@@ -118,7 +123,7 @@ class Inventario {
                 header('location:index.php?usuario_incorrecto');
                 exit;
             case 'usuario_incorrecto':
-                $this->opcActual = "principal";
+                $this->opcActual = 'principal';
                 $contenido = $this->creaContenido();
                 $contenido->usuario_incorrecto();
                 $salida = new Distribucion($this->plant, $contenido);
@@ -136,14 +141,16 @@ class Inventario {
         }
     }
 
-    private function creaContenido() {
+    private function creaContenido()
+    {
         return new AportaContenido($this->bdd, $this->registrado, $this->usuario, $this->perfil, $this->opcActual);
     }
 
     // Esta función comprueba si el usuario está o no registrado,
     // devolviendo su IdSesion en caso afirmativo o false
     // en caso contrario
-    private function usuarioRegistrado() {
+    private function usuarioRegistrado()
+    {
         $this->usuario = $_POST['usuario'];
         $this->clave = $_POST['clave'];
         // ejecuta la consulta para buscar el usuario
@@ -167,10 +174,11 @@ class Inventario {
         return false;
     }
 
-    private function creaPerfil($fila) {
-        return array("Consulta" => $fila['consulta'], "Modificacion" => $fila['modificacion'],
-            "Alta" => $fila['alta'], "Borrado" => $fila['borrado'], "Informe" => $fila['informe'],
-            "Usuarios" => $fila['usuarios'], "Config" => $fila['config']);
+    private function creaPerfil($fila)
+    {
+        return ['Consulta' => $fila['consulta'], 'Modificacion' => $fila['modificacion'],
+            'Alta'         => $fila['alta'], 'Borrado' => $fila['borrado'], 'Informe' => $fila['informe'],
+            'Usuarios'     => $fila['usuarios'], 'Config' => $fila['config']];
     }
 
     // Esta función intenta recuperar el nombre del usuario
@@ -178,7 +186,8 @@ class Inventario {
     // dejando las variables Registrado y Usuario con
     // los valores apropiados
     // @param String Identificador de sesión del usuario actual
-    private function recuperaNombreConId($idSesion) {
+    private function recuperaNombreConId($idSesion)
+    {
         // para ejecutar la consulta para buscar el Id de sesión
         $res = $this->bdd->ejecuta("SELECT * FROM Usuarios WHERE idSesion='$idSesion'");
         // Si no hemos encontrado el ID
@@ -204,5 +213,3 @@ class Inventario {
         }
     }
 }
-
-?>

Menu.php

@@ -1,6 +1,5 @@
 <?php
-    /**
-     * @package Inventario
+/**
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -16,39 +15,43 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
-     *  
  */
 //
 // Esta clase generará el menú de la aplicación.
-class Menu {
+class Menu
+{
     private $opciones;
+
     public function __construct($fichero)
     {
-        $contenido=@file_get_contents($fichero) or
+        $contenido = @file_get_contents($fichero) or
             die("<h1>No puedo generar el menú. No puedo acceder al fichero $fichero</h1>");
         // Obtenemos la lista de pares Opción|Enlace
-        $elementos=explode("\n", $contenido);
-        foreach($elementos as $elemento) {
-            list($tipo, $opcion, $enlace, $destino, $titulo)=explode('|', $elemento);
+        $elementos = explode("\n", $contenido);
+        foreach ($elementos as $elemento) {
+            list($tipo, $opcion, $enlace, $destino, $titulo) = explode('|', $elemento);
             // Los guardamos en la matriz de opciones
-            if ($tipo)
-                $this->opciones[]=$tipo.",".$opcion.",".$enlace.",".$destino.",".$titulo;
+            if ($tipo) {
+                $this->opciones[] = $tipo.','.$opcion.','.$enlace.','.$destino.','.$titulo;
             }
         }
+    }
+
     public function insertaMenu()
     {
-        $salida="";
+        $salida = '';
         reset($this->opciones);
-        foreach($this->opciones as $opcion) {
-            list($tipo,$opcion,$enlace,$destino,$titulo)=explode(",",$opcion);
-            if ($tipo==2)
-                $salida.='<li class="active"><a href="'.$enlace.'" target="'.$destino.'" title="'.$titulo.'">'.$opcion.'</a><br /></li>';
-            else
-                $salida.=
+        foreach ($this->opciones as $opcion) {
+            list($tipo, $opcion, $enlace, $destino, $titulo) = explode(',', $opcion);
+            if ($tipo == 2) {
+                $salida .= '<li class="active"><a href="'.$enlace.'" target="'.$destino.'" title="'.$titulo.'">'.$opcion.'</a><br /></li>';
+            } else {
+                $salida .=
                     //'<span class="label label-default">'.$opcion.'</span><br>';
                     '<label class="">'.$opcion.'</label><br/>';
             }
+        }
+
         return $salida;
     }
 }
-?>

Pdf_mysql_table.php

@@ -17,224 +17,251 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-
-class Pdf_mysql_table extends Fpdf
+class Pdf_mysql_table extends FPDF
 {
     /**
-     * Modificado por Ricardo Montañana 05/2008 para añadir la posibilidad de cálculo de totales
-     * @var $totales float[] Vector de totales de las columnas que lo necesiten
-     */
-  private $ProcessingTable=false,$aCols=array(),$TableX,$HeaderColor;
-  private $RowColors,$ColorIndex;
-  private $bdd,$titulo,$cabecera;
-  private $totales=array(),$procesandoTotales=false;
-  /**
+     * Modificado por Ricardo Montañana 05/2008 para añadir la posibilidad de cálculo de totales.
      *
+     * @var float[] Vector de totales de las columnas que lo necesiten
+     */
+    private $ProcessingTable = false;
+    private $aCols = [];
+    private $TableX;
+    private $HeaderColor;
+    private $RowColors;
+    private $ColorIndex;
+    private $bdd;
+    private $titulo;
+    private $cabecera;
+    private $totales = [];
+    private $procesandoTotales = false;
+
+    /**
      * @param mixed  $bdd         Controlador de la base de datos
      * @param string $orientacion Orientación de la página P/L
      * @param string $formato     Tamaño de la página p. ej. A4
      * @param string $titulo      Título del informe
      * @param string $cabecera    Texto para la cabecera
      */
-  
-
-    public function __construct($bdd,$orientacion,$formato,$titulo='',$cabecera='')
+    public function __construct($bdd, $orientacion, $formato, $titulo = '', $cabecera = '')
     {
-      $this->bdd=$bdd;
-      $this->titulo=$titulo;
-      $this->cabecera=$cabecera;
-      parent::__construct($orientacion,'mm',$formato);
+        $this->bdd = $bdd;
+        $this->titulo = $titulo;
+        $this->cabecera = $cabecera;
+        parent::__construct($orientacion, 'mm', $formato);
     }
+
     public function setTitulo($titulo)
     {
-      $this->titulo=$titulo;
+        $this->titulo = $titulo;
     }
+
     public function iniciaTotales()
     {
-        $this->totales = array();
+        $this->totales = [];
     }
-    function Header()
+
+    public function Header()
     {
         //Modficada por Ricardo Montañana
         //Titulo
-        $fecha=strftime("%d-%b-%Y %H:%M");
-        $this->SetFont('Arial','',8);
-        $this->Cell(0,4,html_entity_decode(CENTRO . " " . PROGRAMA . VERSION,ENT_COMPAT | ENT_HTML401,'ISO-8859-1'),0,1,'L');
-        $this->SetFont('Arial','',18);
-        $this->Cell(0,6,utf8_decode($this->titulo),0,1,'C');
-        $this->SetFont('Arial','',8);
-        $this->Cell(0,3,$fecha,0,1,'R');
-        $this->Cell(0,5,utf8_decode($this->cabecera),0,1,'C');
+        $fecha = strftime('%d-%b-%Y %H:%M');
+        $this->SetFont('Arial', '', 8);
+        $this->Cell(0, 4, html_entity_decode(CENTRO.' '.PROGRAMA.' v'.VERSION, ENT_COMPAT | ENT_HTML401, 'ISO-8859-1'), 0, 1, 'L');
+        $this->SetFont('Arial', '', 18);
+        $this->Cell(0, 6, utf8_decode($this->titulo), 0, 1, 'C');
+        $this->SetFont('Arial', '', 8);
+        $this->Cell(0, 3, $fecha, 0, 1, 'R');
+        $this->Cell(0, 5, utf8_decode($this->cabecera), 0, 1, 'C');
         $this->Ln(10);
         //Print the table header if necessary
-        if($this->ProcessingTable)
+        if ($this->ProcessingTable) {
             $this->TableHeader();
+        }
         //Ensure table header is output
         parent::Header();
     }
+
     public function Footer()
     {
-      $this->SetFont('Arial','',8);
-      $this->setY($this->h-10);
-      $this->Cell(0,6,'-'.$this->PageNo().'-',0,1,'C');
+        $this->SetFont('Arial', '', 8);
+        $this->setY($this->h - 10);
+        $this->Cell(0, 6, '-'.$this->PageNo().'-', 0, 1, 'C');
         parent::Footer();
     }
 
-    function TableHeader()
+    public function TableHeader()
     {
-        $this->SetFont('Arial','B',12);
+        $this->SetFont('Arial', 'B', 12);
         $this->SetX($this->TableX);
-        $fill=!empty($this->HeaderColor);
-        if($fill)
-            $this->SetFillColor($this->HeaderColor[0],$this->HeaderColor[1],$this->HeaderColor[2]);
-        foreach($this->aCols as $col)
-            $this->Cell($col['w'],6,utf8_decode($col['c']),1,0,'C',$fill);
+        $fill = !empty($this->HeaderColor);
+        if ($fill) {
+            $this->SetFillColor($this->HeaderColor[0], $this->HeaderColor[1], $this->HeaderColor[2]);
+        }
+        foreach ($this->aCols as $col) {
+            $this->Cell($col['w'], 6, utf8_decode($col['c']), 1, 0, 'C', $fill);
+        }
         $this->Ln();
     }
 
-    function Row($data)
+    public function Row($data)
     {
         $this->SetX($this->TableX);
-        $ci=$this->ColorIndex;
-        $fill=!empty($this->RowColors[$ci]);
-        if($fill)
-            $this->SetFillColor($this->RowColors[$ci][0],$this->RowColors[$ci][1],$this->RowColors[$ci][2]);
-        foreach($this->aCols as $col) {
+        $ci = $this->ColorIndex;
+        $fill = !empty($this->RowColors[$ci]);
+        if ($fill) {
+            $this->SetFillColor($this->RowColors[$ci][0], $this->RowColors[$ci][1], $this->RowColors[$ci][2]);
+        }
+        foreach ($this->aCols as $col) {
             switch ($col['a']) {
-                case 'D':$alin='R';break;
-                case 'I':$alin='L';break;
-                case 'C':$alin='C';break;
-                default:$alin='L';break;
+                case 'D':$alin = 'R'; break;
+                case 'I':$alin = 'L'; break;
+                case 'C':$alin = 'C'; break;
+                default:$alin = 'L'; break;
             }
             if ($this->procesandoTotales) {
-                $this->SetFont('Arial','B',12);
+                $this->SetFont('Arial', 'B', 12);
             }
-            $this->Cell($col['w'],5,utf8_decode($data[$col['f']]),1,0,$alin,$fill);
+            $dato = isset($data[$col['f']]) ? $data[$col['f']] : '';
+            $this->Cell($col['w'], 5, utf8_decode($dato), 1, 0, $alin, $fill);
             //$this->Cell($col['w'],5,utf8_decode($data[$col['f']]),1,0,$alin,$fill);
             //$this->Cell($col['w'],5,utf8_decode($data['proveedor']),1,0,$alin,$fill);
             //$this->Write(5,"nombre=".$col['f'].",titulo=".$col['c'].",ancho=".$col['w'].",alin=".$col['a']);
             //$this->Write(5,$data[$col['f']].$col['f']);
             //print_r($data);
             //print_r($this->aCols);
-            if ($col['t']=='S' && !$this->procesandoTotales) {
-                $this->totales[$col['f']]+=$data[$col['f']];
+            if ($col['t'] == 'S' && !$this->procesandoTotales) {
+                if (isset($this->totales[$col['f']])) {
+                    $this->totales[$col['f']] += $data[$col['f']];
+                } else {
+                    $this->totales[$col['f']] = $data[$col['f']];
+                }
             }
-            
         }
         $this->Ln();
-        $this->ColorIndex=1-$ci;
+        $this->ColorIndex = 1 - $ci;
     }
 
-    function CalcWidths($width,$align)
+    public function CalcWidths($width, $align)
     {
         //Compute the widths of the columns
-        $TableWidth=0;
-        foreach($this->aCols as $i=>$col)
-        {
-            $w=$col['w'];
-            if($w==-1)
-                $w=$width/count($this->aCols);
-            elseif(substr($w,-1)=='%')
-                $w=$w/100*$width;
-            $this->aCols[$i]['w']=$w;
-            $TableWidth+=$w;
+        $TableWidth = 0;
+        foreach ($this->aCols as $i=>$col) {
+            $w = $col['w'];
+            if ($w == -1) {
+                $w = $width / count($this->aCols);
+            } elseif (substr($w, -1) == '%') {
+                $w = $w / 100 * $width;
+            }
+            $this->aCols[$i]['w'] = $w;
+            $TableWidth += $w;
         }
         //Compute the abscissa of the table
-        if($align=='C')
-            $this->TableX=max(($this->w-$TableWidth)/2,0);
-        elseif($align=='R')
-            $this->TableX=max($this->w-$this->rMargin-$TableWidth,0);
-        else
-            $this->TableX=$this->lMargin;
+        if ($align == 'C') {
+            $this->TableX = max(($this->w - $TableWidth) / 2, 0);
+        } elseif ($align == 'R') {
+            $this->TableX = max($this->w - $this->rMargin - $TableWidth, 0);
+        } else {
+            $this->TableX = $this->lMargin;
+        }
     }
 
-    function AddCol($field=-1,$width=-1,$caption='',$align='I',$total='N')
+    public function AddCol($field = -1, $width = -1, $caption = '', $align = 'I', $total = 'N')
     {
         //Add a column to the table
-        if($field==-1)
-            $field=count($this->aCols);
-        $this->aCols[]=array('f'=>$field,'c'=>$caption,'w'=>$width,'a'=>$align,'t'=>$total);
+        if ($field == -1) {
+            $field = count($this->aCols);
+        }
+        $this->aCols[] = ['f'=>$field, 'c'=>$caption, 'w'=>$width, 'a'=>$align, 't'=>$total];
     }
 
-    function Table($query,$prop=array())
+    public function Table($query, $prop = [])
     {
         //Issue query
-        $res=$this->bdd->query($query) or die('Error: '.$this->bdd->error."<BR>Query: $query");
+        $res = $this->bdd->query($query) or die('Error: '.$this->bdd->error."<BR>Query: $query");
         //Add all columns if none was specified
-        if(count($this->aCols)==0)
-        {
-            $nb=$res->field_count;
-            for($i=0;$i<$nb;$i++)
+        if (count($this->aCols) == 0) {
+            $nb = $res->field_count;
+            for ($i = 0; $i < $nb; $i++) {
                 $this->AddCol();
             }
+        }
         //Retrieve column names when not specified
-        $i=0;
-        foreach($this->aCols as $i=>$col)
-        {
-            if($col['c']=='')
-            {
-                if(is_string($col['f']))
-                    $this->aCols[$i]['c']=ucfirst($col['f']);
-                else
-                    $this->aCols[$i]['c']=ucfirst($res->field_seek($i));
+        $i = 0;
+        foreach ($this->aCols as $i=>$col) {
+            if ($col['c'] == '') {
+                if (is_string($col['f'])) {
+                    $this->aCols[$i]['c'] = ucfirst($col['f']);
+                } else {
+                    $this->aCols[$i]['c'] = ucfirst($res->field_seek($i));
+                }
             }
             $i++;
         }
         //Handle properties
-        if(!isset($prop['width']))
-            $prop['width']=0;
-        if($prop['width']==0)
-            $prop['width']=$this->w-$this->lMargin-$this->rMargin;
-        if(!isset($prop['align']))
-            $prop['align']='C';
-        if(!isset($prop['padding']))
-            $prop['padding']=$this->cMargin;
-        $cMargin=$this->cMargin;
-        $this->cMargin=$prop['padding'];
-        if(!isset($prop['HeaderColor']))
-            $prop['HeaderColor']=array();
-        $this->HeaderColor=$prop['HeaderColor'];
-        if(!isset($prop['color1']))
-            $prop['color1']=array();
-        if(!isset($prop['color2']))
-            $prop['color2']=array();
-        $this->RowColors=array($prop['color1'],$prop['color2']);
+        if (!isset($prop['width'])) {
+            $prop['width'] = 0;
+        }
+        if ($prop['width'] == 0) {
+            $prop['width'] = $this->w - $this->lMargin - $this->rMargin;
+        }
+        if (!isset($prop['align'])) {
+            $prop['align'] = 'C';
+        }
+        if (!isset($prop['padding'])) {
+            $prop['padding'] = $this->cMargin;
+        }
+        $cMargin = $this->cMargin;
+        $this->cMargin = $prop['padding'];
+        if (!isset($prop['HeaderColor'])) {
+            $prop['HeaderColor'] = [];
+        }
+        $this->HeaderColor = $prop['HeaderColor'];
+        if (!isset($prop['color1'])) {
+            $prop['color1'] = [];
+        }
+        if (!isset($prop['color2'])) {
+            $prop['color2'] = [];
+        }
+        $this->RowColors = [$prop['color1'], $prop['color2']];
         //Compute column widths
-        $this->CalcWidths($prop['width'],$prop['align']);
+        $this->CalcWidths($prop['width'], $prop['align']);
         //Print header
         $this->TableHeader();
         //Print rows
-        $this->SetFont('Arial','',11);
-        $this->ColorIndex=0;
-        $this->ProcessingTable=true;
-        $this->procesandoTotales=false;
-        while($row=$res->fetch_assoc()) {
+        $this->SetFont('Arial', '', 11);
+        $this->ColorIndex = 0;
+        $this->ProcessingTable = true;
+        $this->procesandoTotales = false;
+        while ($row = $res->fetch_assoc()) {
             $this->Row($row);
         }
-        $this->procesandoTotales=true;
+        $this->procesandoTotales = true;
         // Procesa los totales
         if ($this->procesaTotales()) {
             $this->Row($this->totales);
         }
-        $this->ProcessingTable=false;
-        $this->cMargin=$cMargin;
-        $this->aCols=array();
+        $this->ProcessingTable = false;
+        $this->cMargin = $cMargin;
+        $this->aCols = [];
     }
+
     /**
-     * Se encarga de generar una línea de totalización si es necesario
+     * Se encarga de generar una línea de totalización si es necesario.
+     *
      * @param array $datos Línea con los totales a imprimir o NULL
-     * @return boolean Si hay que generar la línea o no
+     *
+     * @return bool Si hay que generar la línea o no
      */
     private function procesaTotales()
     {
-        foreach($this->aCols as $col) {
-            if ($col['t']=='S') {
+        foreach ($this->aCols as $col) {
+            if ($col['t'] == 'S') {
                 return true;
             }
         }
+
         return false;
     }
 }
-?>

README.md

@@ -1,4 +1,4 @@
-# Inventario de Centro Educativo
+# Inventario de Centro Educativo [![Project Stats](https://www.ohloh.net/p/inventario2/widgets/project_thin_badge.gif)](https://www.ohloh.net/p/inventario2)
 Copyright (c) 2008-2014, Ricardo Montañana Gómez
 
 Inventario2 is free software: you can redistribute it and/or modify 
@@ -13,6 +13,12 @@ Utiliza:
 *MySQL v. 5.1.x
 *Apache
 
+[Manual de Usuario](http://rmontanana.gitbooks.io/inventario2/)
+
+[Instalación de ejemplo](https://inventario.rmontanana.es)
+
+[Estadísticas del proyecto](https://www.ohloh.net/p/inventario2)
+
 ##Instalación
 Para instalar la aplicación basta con seguir estos pasos:
 ###1. Copiar los archivos en una ubicación a la que tenga acceso el usuario con el que se ejecuta el servidor Apache (apache, _www, etc.).
@@ -26,7 +32,8 @@ Para instalar la aplicación basta con seguir estos pasos:
 ###2. Crear un directorio temporal y dar derechos de escritura a los ficheros de configuración.
     
     mkdir tmp
-    chown apache tmp
+    mkdir img.data
+    chown apache tmp img.data
     chown apache inc/configuracion.inc
     chown apache inc
     

Sql.php

@@ -1,9 +1,11 @@
 <?php
-    /** 
-     * Gestión de una base de datos MySQL
+/**
+ * Gestión de una base de datos MySQL.
+ *
  * @author Ricardo Montañana <rmontanana@gmail.com>
+ *
  * @version 1.0
-     * @package Inventario
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -19,64 +21,69 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
-     *  
  */
-class Sql {
+class Sql
+{
     /**
      * @var mixed Manejador de la base de datos.
      */
-    private $bdd=NULL;
+    private $bdd = null;
     /**
      * @var string Mensaje del último mensaje de error generado
      */
-    private $mensajeError='';
+    private $mensajeError = '';
     /**
-     * @var boolean Almacena el estado de error o no de la última acción.
+     * @var bool Almacena el estado de error o no de la última acción.
      */
-    private $error=false;
+    private $error = false;
     /**
-     * @var boolean Estado de la conexión con la base de datos.
+     * @var bool Estado de la conexión con la base de datos.
      */
-    private $estado=false;
+    private $estado = false;
     /**
      * @var mixed Objeto que alberga la última consulta ejecutada.
      */
-    private $peticion=NULL;
+    private $peticion = null;
     /**
-     * @var integer Número de tuplas afectadas en la última consulta.
+     * @var int Número de tuplas afectadas en la última consulta.
      */
-    private $numero=0;
+    private $numero = 0;
     /**
      * @var string vector de cadenas con los resultados de la petición.
      */
-    private $datos=array();
+    private $datos = [];
     /**
-     * Id del último registro insertado
-     * @var integer mysql_
+     * Id del último registro insertado.
+     *
+     * @var int mysql_
      */
     private $id;
+
     /**
      * Crea un objeto Sql y conecta con la Base de Datos.
+     *
      * @param string $servidor
      * @param string $usuario
      * @param string $baseDatos
      */
-    public function __construct($servidor,$usuario,$clave,$baseDatos)
+    public function __construct($servidor, $usuario, $clave, $baseDatos)
     {
-        $this->bdd = @new mysqli($servidor,$usuario,$clave,$baseDatos);
+        $this->bdd = @new mysqli($servidor, $usuario, $clave, $baseDatos);
         if (mysqli_connect_errno()) {
-            $this->mensajeError='<h1>Fallo al conectar con el servidor MySQL.</h1>';
-            $this->mensajeError.="Servidor [".$servidor ."] base de datos [".$baseDatos."]";
-            $this->error=true;
-            $this->estado=false;
+            $this->mensajeError = '<h1>Fallo al conectar con el servidor MySQL.</h1>';
+            $this->mensajeError .= 'Servidor ['.$servidor.'] base de datos ['.$baseDatos.']';
+            $this->error = true;
+            $this->estado = false;
         } else {
-            $this->mensajeError='';
-            $this->error=false;
-            $this->estado=true;
+            $this->mensajeError = '';
+            $this->error = false;
+            $this->estado = true;
         }
-        $this->peticion=NULL;
+        $this->peticion = null;
+
         return $this;
     }
+
     public function __destruct()
     {
         //Libera la memoria de una posible consulta.
@@ -88,103 +95,132 @@ class Sql {
             $this->bdd->close();
         }
     }
+
     public function filtra($cadena)
     {
         return $this->bdd->real_escape_string($cadena);
     }
+
     public function ejecuta($comando)
     {
         if (!$this->estado) {
-            $this->error=true;
-            $this->mensajeError='No est&aacute; conectado';
+            $this->error = true;
+            $this->mensajeError = 'No est&aacute; conectado';
+
             return false;
         }
 
-        if (!$this->peticion=$this->bdd->query($comando)) {
-            $this->error=true;
-            $this->mensajeError='No pudo ejecutar la petici&oacute;n: '.$comando;
+        if (!$this->peticion = $this->bdd->query($comando)) {
+            $this->error = true;
+            $this->mensajeError = 'No pudo ejecutar la petici&oacute;n: '.$comando;
+
             return false;
         }
-        $this->numero=$this->bdd->affected_rows;
-        $this->id=$this->bdd->insert_id;
-        $this->error=false;
-        $this->mensajeError='';
+        $this->numero = $this->bdd->affected_rows;
+        $this->id = $this->bdd->insert_id;
+        $this->error = false;
+        $this->mensajeError = '';
+
         return true;
     }
+
     public function procesaResultado()
     {
         if (!$this->estado) {
-            $this->error=true;
-            $this->mensajeError='No está conectado a una base de datos';
-            return NULL;
+            $this->error = true;
+            $this->mensajeError = 'No está conectado a una base de datos';
+
+            return;
         }
         if (!$this->peticion) {
-            $this->error=true;
-            $this->mensajeError='No hay un resultado disponible';
-            return NULL;
+            $this->error = true;
+            $this->mensajeError = 'No hay un resultado disponible';
+
+            return;
         }
-        $datos=$this->peticion->fetch_assoc();
-        $this->error=false;
-        $this->mensajeError='';
-        return ($datos);
+        $datos = $this->peticion->fetch_assoc();
+        $this->error = false;
+        $this->mensajeError = '';
+
+        return $datos;
     }
+
     public function camposResultado()
     {
         if (!$this->estado) {
-            $this->error=true;
-            $this->mensajeError='No está conectado a una base de datos';
-            return NULL;
+            $this->error = true;
+            $this->mensajeError = 'No está conectado a una base de datos';
+
+            return;
         }
         if (!$this->peticion) {
-            $this->error=true;
-            $this->mensajeError='No hay un resultado disponible';
-            return NULL;
+            $this->error = true;
+            $this->mensajeError = 'No hay un resultado disponible';
+
+            return;
         }
-        $datos=$this->peticion->fetch_field();
-        $this->error=false;
-        $this->mensajeError='';
-        return ($datos);
+        $datos = $this->peticion->fetch_field();
+        $this->error = false;
+        $this->mensajeError = '';
+
+        return $datos;
     }
+
     /**
      * Devuelve el número de tuplas afectadas en la última petición.
-     * @return integer Número de tuplas.
+     *
+     * @return int Número de tuplas.
      */
-    public function numeroTuplas() {
+    public function numeroTuplas()
+    {
         return $this->numero;
     }
+
     /**
      * Devuelve el número de tuplas total si se ha hecho una consulta select
      * con SELECT SQL_CALC_FOUND_ROWS * ...
-     * @return integer Número de tuplas.
+     *
+     * @return int Número de tuplas.
      */
     public function numeroTotalTuplas()
     {
-        $comando = "select found_rows();";
-        if (!$peticion=$this->bdd->query($comando)) {
-            $this->error=true;
-            $this->mensajeError='No pudo ejecutar la petici&oacute;n: '.$comando;
+        $comando = 'select found_rows();';
+        if (!$peticion = $this->bdd->query($comando)) {
+            $this->error = true;
+            $this->mensajeError = 'No pudo ejecutar la petici&oacute;n: '.$comando;
+
             return false;
         }
         $numero = $peticion->fetch_row();
-        return $numero[0] ;
+
+        return $numero[0];
     }
+
     /**
-     * Devuelve la condición de error de la última petición
-     * @return boolean condición de error.
+     * Devuelve la condición de error de la última petición.
+     *
+     * @return bool condición de error.
      */
-    public function error() {
+    public function error()
+    {
         return $this->error;
     }
+
     /**
-     * Devuelve el mensaje de error de la última petición
+     * Devuelve el mensaje de error de la última petición.
+     *
      * @return <type>
      */
-    public function mensajeError() {
+    public function mensajeError()
+    {
         return $this->mensajeError.$this->bdd->error;
     }
+
     /**
      * Devuelve la estructura de campos de una tabla.
+     *
      * @param string $tabla Nombre de la tabla.
+     *
      * @return string vector asociativo con la descripción de la tabla [campo]->valor
      */
     public function estructura($tabla)
@@ -192,40 +228,46 @@ class Sql {
         if ($this->peticion) {
             $this->peticion->free_result();
         }
-        $comando="show full columns from $tabla";
+        $comando = "show full columns from $tabla";
         if (!$this->ejecuta($comando)) {
             return false;
         }
-        while ($dato=$this->procesaResultado()) {
-            $salida[]=$dato;
+        while ($dato = $this->procesaResultado()) {
+            $salida[] = $dato;
         }
+
         return $salida;
     }
+
     public function ultimoId()
     {
         return $this->id;
     }
+
     public function obtieneManejador()
     {
         return $this->bdd;
     }
+
     public function comienzaTransaccion()
     {
         return $this->bdd->autocommit(false);
     }
+
     public function abortaTransaccion()
     {
         $codigo = $this->bdd->rollback();
         $this->bdd->autocommit(true);
+
         return $codigo;
     }
+
     public function confirmaTransaccion()
     {
         $codigo = $this->bdd->commit();
         $this->bdd->autocommit(true);
         $this->peticion = null;
+
         return $codigo;
     }
 }
-  
-?>

Zebra_Image.php

@@ -37,14 +37,14 @@ ini_set('gd.jpeg_ignore_warning', true);
  *  For more resources visit {@link http://stefangabos.ro/}
  *
  *  @author     Stefan Gabos <contact@stefangabos.ro>
+ *
  *  @version    2.2.3 (last revision: July 14, 2013)
+ *
  *  @copyright  (c) 2006 - 2013 Stefan Gabos
  *  @license    http://www.gnu.org/licenses/lgpl-3.0.txt GNU LESSER GENERAL PUBLIC LICENSE
- *  @package    Zebra_Image
  */
 class Zebra_Image
 {
-
     /**
      *  Indicates the file system permissions to be set for newly created images.
      *
@@ -64,9 +64,9 @@ class Zebra_Image
      *
      *  Default is 0755
      *
-     *  @var integer
+     *  @var int
      */
-    var $chmod_value;
+    public $chmod_value;
 
     /**
      *  If set to FALSE, images having both width and height smaller than the required width and height, will be left
@@ -76,9 +76,9 @@ class Zebra_Image
      *
      *  Default is TRUE
      *
-     *  @var boolean
+     *  @var bool
      */
-    var $enlarge_smaller_images;
+    public $enlarge_smaller_images;
 
     /**
      *  In case of an error read this property's value to see the error's code.
@@ -96,9 +96,9 @@ class Zebra_Image
      *
      *  Default is 0 (no error).
      *
-     *  @var integer
+     *  @var int
      */
-    var $error;
+    public $error;
 
     /**
      *  Indicates the quality of the output image (better quality means bigger file size).
@@ -109,9 +109,9 @@ class Zebra_Image
      *
      *  Default is 85
      *
-     *  @var integer
+     *  @var int
      */
-    var $jpeg_quality;
+    public $jpeg_quality;
 
     /**
      *  Indicates the compression level of the output image (lower compression means bigger file size).
@@ -125,9 +125,9 @@ class Zebra_Image
      *
      *  @since 2.2
      *
-     *  @var integer
+     *  @var int
      */
-    var $png_compression;
+    public $png_compression;
 
     /**
      *  Specifies whether, upon resizing, images should preserve their aspect ratio.
@@ -136,9 +136,9 @@ class Zebra_Image
      *
      *  Default is TRUE
      *
-     *  @var boolean
+     *  @var bool
      */
-    var $preserve_aspect_ratio;
+    public $preserve_aspect_ratio;
 
     /**
      *  Indicates whether a target files should preserve the source file's date/time.
@@ -147,9 +147,9 @@ class Zebra_Image
      *
      *  @since 1.0.4
      *
-     *  @var boolean
+     *  @var bool
      */
-    var $preserve_time;
+    public $preserve_time;
 
     /**
      *  Indicates whether the target image should have a "sharpen" filter applied to it.
@@ -163,9 +163,9 @@ class Zebra_Image
      *
      *  @since 2.2
      *
-     *  @var boolean
+     *  @var bool
      */
-    var $sharpen_images;
+    public $sharpen_images;
 
     /**
      *  Path to an image file to apply the transformations to.
@@ -174,7 +174,7 @@ class Zebra_Image
      *
      *  @var    string
      */
-    var $source_path;
+    public $source_path;
 
     /**
      *  Path (including file name) to where to save the transformed image.
@@ -184,7 +184,7 @@ class Zebra_Image
      *
      *  @var    string
      */
-    var $target_path;
+    public $target_path;
 
     /**
      *  Constructor of the class.
@@ -193,7 +193,7 @@ class Zebra_Image
      *
      *  @return void
      */
-    function Zebra_Image()
+    public function Zebra_Image()
     {
 
         // set default values for properties
@@ -210,7 +210,6 @@ class Zebra_Image
         $this->sharpen_images = false;
 
         $this->source_path = $this->target_path = '';
-
     }
 
     /**
@@ -292,21 +291,17 @@ class Zebra_Image
      *                                                              others weight of 1.0. the result is normalized by dividing
      *                                                              the sum with <b>arg1</b> + 8.0 (sum of the matrix).
      *                                                              any float is accepted;
-     *
      *  @param  mixed   $arg1       Used by the following filters:
      *                              -   <b>brightness</b>       -   sets the brightness level (-255 to 255)
      *                              -   <b>contrast</b>         -   sets the contrast level (-100 to 100)
      *                              -   <b>colorize</b>         -   sets the value of the red component (-255 to 255)
      *                              -   <b>smooth</b>           -   sets the smoothness level
      *                              -   <b>pixelate</b>         -   sets the block size, in pixels
-     *
      *  @param  mixed   $arg2       Used by the following filters:
      *                              -   <b>colorize</b>         -   sets the value of the green component (-255 to 255)
      *                              -   <b>pixelate</b>         -   whether to use advanced pixelation effect or not (defaults to FALSE).
-     *
      *  @param  mixed   $arg3       Used by the following filters:
      *                              -   <b>colorize</b>         -   sets the value of the blue component (-255 to 255)
-     *
      *  @param  mixed   $arg4       Used by the following filters:
      *                              -   <b>colorize</b>         -   alpha channel; a value between 0 and 127. 0 indicates
      *                                                              completely opaque while 127 indicates completely
@@ -314,7 +309,7 @@ class Zebra_Image
      *
      *  @since 2.2.2
      *
-     *  @return boolean             Returns TRUE on success or FALSE on error.
+     *  @return bool             Returns TRUE on success or FALSE on error.
      *
      *                              If {@link http://php.net/manual/en/function.imagefilter.php imagefilter} is not
      *                              available the method will return FALSE without setting an {@link error} code.
@@ -326,11 +321,11 @@ class Zebra_Image
      *                              {@link http://php.net/manual/en/function.imagefilter.php imagefilter} exists and that
      *                              the requested filter is valid, check the {@link error} property to see the error code.
      */
-    function apply_filter($filter, $arg1 = '', $arg2 = '', $arg3 = '', $arg4 = '')
+    public function apply_filter($filter, $arg1 = '', $arg2 = '', $arg3 = '', $arg4 = '')
     {
 
         // if "imagefilter" function exists and the requested filter exists
-        if (function_exists('imagefilter'))
+        if (function_exists('imagefilter')) {
 
             // if image resource was successfully created
             if ($this->_create_from_source()) {
@@ -358,45 +353,51 @@ class Zebra_Image
                 if (is_array($filter)) {
 
                     // iterate through the filters
-                    foreach ($filter as $arguments)
+                    foreach ($filter as $arguments) {
 
                         // if filter exists
-                        if (defined('IMG_FILTER_' . strtoupper($arguments[0]))) {
+                        if (defined('IMG_FILTER_'.strtoupper($arguments[0]))) {
 
                             // try to apply the filter...
-                            if (!@call_user_func_array('imagefilter', array_merge(array($target_identifier, constant('IMG_FILTER_' . strtoupper($arguments[0]))), array_slice($arguments, 1))))
+                            if (!@call_user_func_array('imagefilter', array_merge([$target_identifier, constant('IMG_FILTER_'.strtoupper($arguments[0]))], array_slice($arguments, 1)))) {
 
                                 // ...and trigger an error if the filter could not be applied
-                                trigger_error('Invalid arguments used for "' . strtoupper($arguments[0]) . '" filter', E_USER_WARNING);
+                                trigger_error('Invalid arguments used for "'.strtoupper($arguments[0]).'" filter', E_USER_WARNING);
+                            }
 
                             // if filter doesn't exists, trigger an error
-                        } else trigger_error('Filter "' . strtoupper($arguments[0]) . '" is not available', E_USER_WARNING);
+                        } else {
+                            trigger_error('Filter "'.strtoupper($arguments[0]).'" is not available', E_USER_WARNING);
+                        }
+                    }
 
                     // if a single filter is to be applied and it is available
-                } elseif (defined('IMG_FILTER_' . strtoupper($filter))) {
+                } elseif (defined('IMG_FILTER_'.strtoupper($filter))) {
 
                     // get all the arguments passed to the method
                     $arguments = func_get_args();
 
                     // try to apply the filter...
-                    if (!@call_user_func_array('imagefilter', array_merge(array($target_identifier, constant('IMG_FILTER_' . strtoupper($filter))), array_slice($arguments, 1))))
+                    if (!@call_user_func_array('imagefilter', array_merge([$target_identifier, constant('IMG_FILTER_'.strtoupper($filter))], array_slice($arguments, 1)))) {
 
                         // ...and trigger an error if the filter could not be applied
-                        trigger_error('Invalid arguments used for "' . strtoupper($arguments[0]) . '" filter', E_USER_WARNING);
+                        trigger_error('Invalid arguments used for "'.strtoupper($arguments[0]).'" filter', E_USER_WARNING);
+                    }
 
                     // if filter doesn't exists, trigger an error
-                } else trigger_error('Filter "' . strtoupper($arguments[0]) . '" is not available', E_USER_WARNING);
+                } else {
+                    trigger_error('Filter "'.strtoupper($arguments[0]).'" is not available', E_USER_WARNING);
+                }
 
                 // write image
                 return $this->_write_image($target_identifier);
-
+            }
         }
 
         // if script gets this far, return false
         // note that we do not set the error level as it has been already set
         // by the _create_from_source() method earlier, if the case
         return false;
-
     }
 
     /**
@@ -421,21 +422,18 @@ class Zebra_Image
      *  $img->crop(0, 0, 100, 100);
      *  </code>
      *
-     *  @param  integer     $start_x    x coordinate to start cropping from
-     *
-     *  @param  integer     $start_y    y coordinate to start cropping from
-     *
-     *  @param  integer     $end_x      x coordinate where to end the cropping
-     *
-     *  @param  integer     $end_y      y coordinate where to end the cropping
+     *  @param  int     $start_x    x coordinate to start cropping from
+     *  @param  int     $start_y    y coordinate to start cropping from
+     *  @param  int     $end_x      x coordinate where to end the cropping
+     *  @param  int     $end_y      y coordinate where to end the cropping
      *
      *  @since  1.0.4
      *
-     *  @return boolean     Returns TRUE on success or FALSE on error.
+     *  @return bool     Returns TRUE on success or FALSE on error.
      *
      *                      If FALSE is returned, check the {@link error} property to see the error code.
      */
-    function crop($start_x, $start_y, $end_x, $end_y)
+    public function crop($start_x, $start_y, $end_x, $end_y)
     {
 
         // this method might be also called internally
@@ -453,7 +451,9 @@ class Zebra_Image
 
         // if method is called as usually
         // try to create an image resource from source path
-        } else $result = $this->_create_from_source();
+        } else {
+            $result = $this->_create_from_source();
+        }
 
         // if image resource was successfully created
         if ($result !== false) {
@@ -479,19 +479,17 @@ class Zebra_Image
 
             // write image
             return $this->_write_image($target_identifier);
-
         }
 
         // if script gets this far, return false
         // note that we do not set the error level as it has been already set
         // by the _create_from_source() method earlier
         return false;
-
     }
 
     /**
      *  Flips both horizontally and vertically the image given as {@link source_path} and outputs the resulted image as
-     *  {@link target_path}
+     *  {@link target_path}.
      *
      *  <code>
      *  // include the Zebra_Image library
@@ -514,19 +512,17 @@ class Zebra_Image
      *
      *  @since 2.1
      *
-     *  @return boolean     Returns TRUE on success or FALSE on error.
+     *  @return bool     Returns TRUE on success or FALSE on error.
      *
      *                      If FALSE is returned, check the {@link error} property to see the error code.
      */
-    function flip_both()
+    public function flip_both()
     {
-
         return $this->_flip('both');
-
     }
 
     /**
-     *  Flips horizontally the image given as {@link source_path} and outputs the resulted image as {@link target_path}
+     *  Flips horizontally the image given as {@link source_path} and outputs the resulted image as {@link target_path}.
      *
      *  <code>
      *  // include the Zebra_Image library
@@ -547,19 +543,17 @@ class Zebra_Image
      *  $img->flip_horizontal();
      *  </code>
      *
-     *  @return boolean     Returns TRUE on success or FALSE on error.
+     *  @return bool     Returns TRUE on success or FALSE on error.
      *
      *                      If FALSE is returned, check the {@link error} property to see the error code.
      */
-    function flip_horizontal()
+    public function flip_horizontal()
     {
-
         return $this->_flip('horizontal');
-
     }
 
     /**
-     *  Flips vertically the image given as {@link source_path} and outputs the resulted image as {@link target_path}
+     *  Flips vertically the image given as {@link source_path} and outputs the resulted image as {@link target_path}.
      *
      *  <code>
      *  // include the Zebra_Image library
@@ -580,15 +574,13 @@ class Zebra_Image
      *  $img->flip_vertical();
      *  </code>
      *
-     *  @return boolean     Returns TRUE on success or FALSE on error.
+     *  @return bool     Returns TRUE on success or FALSE on error.
      *
      *                      If FALSE is returned, check the {@link error} property to see the error code.
      */
-    function flip_vertical()
+    public function flip_vertical()
     {
-
         return $this->_flip('vertical');
-
     }
 
     /**
@@ -616,7 +608,7 @@ class Zebra_Image
      *  $img->resize(150, 150, ZEBRA_IMAGE_CROP_CENTER);
      *  </code>
      *
-     *  @param  integer     $width              The width to resize the image to.
+     *  @param  int     $width              The width to resize the image to.
      *
      *                                          If set to <b>0</b>, the width will be automatically adjusted, depending
      *                                          on the value of the <b>height</b> argument so that the image preserves
@@ -638,8 +630,7 @@ class Zebra_Image
      *                                          If either <b>width</b> or <b>height</b> are set to <b>0</b>, the script
      *                                          will consider the value of the {@link preserve_aspect_ratio} to bet set
      *                                          to TRUE regardless of its actual value!
-     *
-     *  @param  integer     $height             The height to resize the image to.
+     *  @param  int     $height             The height to resize the image to.
      *
      *                                          If set to <b>0</b>, the height will be automatically adjusted, depending
      *                                          on the value of the <b>width</b> argument so that the image preserves
@@ -661,7 +652,6 @@ class Zebra_Image
      *                                          If either <b>height</b> or <b>width</b> are set to <b>0</b>, the script
      *                                          will consider the value of the {@link preserve_aspect_ratio} to bet set
      *                                          to TRUE regardless of its actual value!
-     *
      *  @param  int     $method                 (Optional) Method to use when resizing images to exact width and height
      *                                          while preserving aspect ratio.
      *
@@ -698,7 +688,6 @@ class Zebra_Image
      *                                          indicated region of the resulted image.
      *
      *                                          Default is ZEBRA_IMAGE_CROP_CENTER
-     *
      *  @param  hexadecimal $background_color   (Optional) The hexadecimal color (like "#FFFFFF" or "#FFF") of the
      *                                          blank area. See the <b>method</b> argument.
      *
@@ -708,12 +697,12 @@ class Zebra_Image
      *
      *                                          Default is #FFFFFF.
      *
-     *  @return boolean                         Returns TRUE on success or FALSE on error.
+     *  @return bool                         Returns TRUE on success or FALSE on error.
      *
      *                                          If FALSE is returned, check the {@link error} property to see what went
      *                                          wrong
      */
-    function resize($width = 0, $height = 0, $method = ZEBRA_IMAGE_CROP_CENTER, $background_color = '#FFFFFF')
+    public function resize($width = 0, $height = 0, $method = ZEBRA_IMAGE_CROP_CENTER, $background_color = '#FFFFFF')
     {
 
         // if image resource was successfully created
@@ -722,7 +711,9 @@ class Zebra_Image
             // if either width or height is to be adjusted automatically
             // set a flag telling the script that, even if $preserve_aspect_ratio is set to false
             // treat everything as if it was set to true
-            if ($width == 0 || $height == 0) $auto_preserve_aspect_ratio = true;
+            if ($width == 0 || $height == 0) {
+                $auto_preserve_aspect_ratio = true;
+            }
 
             // if aspect ratio needs to be preserved
             if ($this->preserve_aspect_ratio || isset($auto_preserve_aspect_ratio)) {
@@ -775,7 +766,6 @@ class Zebra_Image
 
                         // compute the target image's width so that the image will stay inside the bounding box
                         $target_width = round($vertical_aspect_ratio * $this->source_width);
-
                     }
 
                     // if both width and height are given and image is to be cropped in order to get to the required size
@@ -806,7 +796,6 @@ class Zebra_Image
                     // we will create a copy of the source image
                     $target_width = $this->source_width;
                     $target_height = $this->source_height;
-
                 }
 
                 // if aspect ratio does not need to be preserved
@@ -817,7 +806,6 @@ class Zebra_Image
 
                 // compute the target image's height
                 $target_height = ($height > 0 ? $height : $this->source_height);
-
             }
 
             // if
@@ -1041,21 +1029,20 @@ class Zebra_Image
 
                     // if script gets this far, write the image to disk
                     return $this->_write_image($target_identifier);
-
                 }
 
                 // if we get here it means that
             // smaller images than the given width/height are to be left untouched
             // therefore, we save the image as it is
-            } else return $this->_write_image($this->source_identifier);
-
+            } else {
+                return $this->_write_image($this->source_identifier);
+            }
         }
 
         // if script gets this far return false
         // note that we do not set the error level as it has been already set
         // by the _create_from_source() method earlier
         return false;
-
     }
 
     /**
@@ -1080,10 +1067,9 @@ class Zebra_Image
      *  $img->rotate(45);
      *  </code>
      *
-     *  @param  double  $angle                  Angle by which to rotate the image clockwise.
+     *  @param  float  $angle                  Angle by which to rotate the image clockwise.
      *
      *                                          Between 0 and 360.
-     *
      *  @param  mixed   $background_color       (Optional) The hexadecimal color (like "#FFFFFF" or "#FFF") of the
      *                                          uncovered zone after the rotation.
      *
@@ -1093,12 +1079,12 @@ class Zebra_Image
      *
      *                                          Default is -1.
      *
-     *  @return boolean                         Returns TRUE on success or FALSE on error.
+     *  @return bool                         Returns TRUE on success or FALSE on error.
      *
      *                                          If FALSE is returned, check the {@link error} property to see the error
      *                                          code.
      */
-    function rotate($angle, $background_color = -1)
+    public function rotate($angle, $background_color = -1)
     {
 
         // if image resource was successfully created
@@ -1119,7 +1105,6 @@ class Zebra_Image
 
                     // rotate the image
                     $target_identifier = imagerotate($this->source_identifier, $angle, $background_color);
-
                 }
 
                 // if source image is a transparent GIF
@@ -1172,28 +1157,23 @@ class Zebra_Image
 
                 // rotate the image
                 $target_identifier = imagerotate($this->source_identifier, $angle, $background_color);
-
             }
 
             // write image
             $this->_write_image($target_identifier);
-
         }
 
         // if script gets this far return false
         // note that we do not set the error level as it has been already set
         // by the _create_from_source() method earlier
         return false;
-
     }
 
     /**
      *  Returns an array containing the image identifier representing the image obtained from {@link $source_path}, the
-     *  image's width and height and the image's type
-     *
-     *  @access private
+     *  image's width and height and the image's type.
      */
-    function _create_from_source()
+    public function _create_from_source()
     {
 
         // perform some error checking first
@@ -1255,13 +1235,14 @@ class Zebra_Image
                     $identifier = imagecreatefromgif($this->source_path);
 
                     // get the index of the transparent color (if any)
-                    if (($this->source_transparent_color_index = imagecolortransparent($identifier)) >= 0)
+                    if (($this->source_transparent_color_index = imagecolortransparent($identifier)) >= 0) {
 
                         // get the transparent color's RGB values
                         // we have to mute errors because there are GIF images which *are* transparent and everything
                         // works as expected, but imagecolortransparent() returns a color that is outside the range of
                         // colors in the image's pallette...
                         $this->source_transparent_color = @imagecolorsforindex($identifier, $this->source_transparent_color_index);
+                    }
 
                     break;
 
@@ -1294,18 +1275,18 @@ class Zebra_Image
                     return false;
 
             }
-
         }
 
         // if target file has to have the same timestamp as the source image
         // save it as a global property of the class
-        if ($this->preserve_time) $this->source_image_time = filemtime($this->source_path);
+        if ($this->preserve_time) {
+            $this->source_image_time = filemtime($this->source_path);
+        }
 
         // make available the source image's identifier
         $this->source_identifier = $identifier;
 
         return true;
-
     }
 
     /**
@@ -1314,50 +1295,48 @@ class Zebra_Image
      *  The RGB values will be a value between 0 and 255 each.
      *
      *  @param  string  $color              Hexadecimal representation of a color (i.e. #123456 or #AAA).
-     *
      *  @param  string  $default_on_error   Hexadecimal representation of a color to be used in case $color is not
      *                                      recognized as a hexadecimal color.
      *
      *  @return array                       Returns an associative array with the values of (R)ed, (G)reen and (B)lue
-     *
-     *  @access private
      */
-    function _hex2rgb($color, $default_on_error = '#FFFFFF')
+    public function _hex2rgb($color, $default_on_error = '#FFFFFF')
     {
 
         // if color is not formatted correctly
         // use the default color
-        if (preg_match('/^#?([a-f]|[0-9]){3}(([a-f]|[0-9]){3})?$/i', $color) == 0) $color = $default_on_error;
+        if (preg_match('/^#?([a-f]|[0-9]){3}(([a-f]|[0-9]){3})?$/i', $color) == 0) {
+            $color = $default_on_error;
+        }
 
         // trim off the "#" prefix from $background_color
         $color = ltrim($color, '#');
 
         // if color is given using the shorthand (i.e. "FFF" instead of "FFFFFF")
         if (strlen($color) == 3) {
-
             $tmp = '';
 
             // take each value
             // and duplicate it
-            for ($i = 0; $i < 3; $i++) $tmp .= str_repeat($color[$i], 2);
+            for ($i = 0; $i < 3; $i++) {
+                $tmp .= str_repeat($color[$i], 2);
+            }
 
             // the color in it's full, 6 characters length notation
             $color = $tmp;
-
         }
 
         // decimal representation of the color
         $int = hexdec($color);
 
         // extract and return the RGB values
-        return array(
+        return [
 
             'r' => 0xFF & ($int >> 0x10),
             'g' => 0xFF & ($int >> 0x8),
             'b' => 0xFF & $int
 
-        );
-
+        ];
     }
 
     /**
@@ -1365,13 +1344,11 @@ class Zebra_Image
      *
      *  @since 2.1
      *
-     *  @access private
-     *
-     *  @return boolean     Returns TRUE on success or FALSE on error.
+     *  @return bool     Returns TRUE on success or FALSE on error.
      *
      *                      If FALSE is returned, check the {@link error} property to see the error code.
      */
-    function _flip($orientation)
+    public function _flip($orientation)
     {
 
         // if image resource was successfully created
@@ -1444,23 +1421,19 @@ class Zebra_Image
 
             // write image
             return $this->_write_image($target_identifier);
-
         }
 
         // if script gets this far, return false
         // note that we do not set the error level as it has been already set
         // by the _create_from_source() method earlier
         return false;
-
     }
 
     /**
      *  Creates a blank image of given width, height and background color.
      *
-     *  @param  integer     $width              Width of the new image.
-     *
-     *  @param  integer     $height             Height of the new image.
-     *
+     *  @param  int     $width              Width of the new image.
+     *  @param  int     $height             Height of the new image.
      *  @param  string      $background_color   (Optional) The hexadecimal color of the background.
      *
      *                                          Can also be -1 case in which the script will try to create a transparent
@@ -1469,14 +1442,12 @@ class Zebra_Image
      *                                          Default is "#FFFFFF".
      *
      *  @return                                 Returns the identifier of the newly created image.
-     *
-     *  @access private
      */
-    function _prepare_image($width, $height, $background_color = '#FFFFFF')
+    public function _prepare_image($width, $height, $background_color = '#FFFFFF')
     {
 
         // create a blank image
-        $identifier = imagecreatetruecolor((int)$width <= 0 ? 1 : (int)$width, (int)$height <= 0 ? 1 : (int)$height);
+        $identifier = imagecreatetruecolor((int) $width <= 0 ? 1 : (int) $width, (int) $height <= 0 ? 1 : (int) $height);
 
         // if we are creating a PNG image
         if ($this->target_type == 'png' && $background_color == -1) {
@@ -1514,7 +1485,9 @@ class Zebra_Image
         } else {
 
             // if transparent background color specified, revert to white
-            if ($background_color == -1) $background_color = '#FFFFFF';
+            if ($background_color == -1) {
+                $background_color = '#FFFFFF';
+            }
 
             // convert hex color to rgb
             $background_color = $this->_hex2rgb($background_color);
@@ -1524,12 +1497,10 @@ class Zebra_Image
 
             // fill the image with the background color
             imagefill($identifier, 0, 0, $background_color);
-
         }
 
         // return the image's identifier
         return $identifier;
-
     }
 
     /**
@@ -1541,10 +1512,8 @@ class Zebra_Image
      *  versions!</i>
      *
      *  @param  $identifier identifier  An image identifier
-     *
-     *  @access private
      */
-    function _sharpen_image($image)
+    public function _sharpen_image($image)
     {
 
         // if the "sharpen_images" is set to true and we're running an appropriate version of PHP
@@ -1552,11 +1521,11 @@ class Zebra_Image
         if ($this->sharpen_images && version_compare(PHP_VERSION, '5.1.0') >= 0) {
 
             // the convolution matrix as an array of three arrays of three floats
-            $matrix = array(
-                array(-1.2, -1, -1.2),
-                array(-1, 20, -1),
-                array(-1.2, -1, -1.2),
-            );
+            $matrix = [
+                [-1.2, -1, -1.2],
+                [-1, 20, -1],
+                [-1.2, -1, -1.2],
+            ];
 
             // the divisor of the matrix
             $divisor = array_sum(array_map('array_sum', $matrix));
@@ -1566,12 +1535,10 @@ class Zebra_Image
 
             // sharpen image
             imageconvolution($image, $matrix, $divisor, $offset);
-
         }
 
         // return the image's identifier
         return $image;
-
     }
 
     /**
@@ -1579,13 +1546,11 @@ class Zebra_Image
      *
      *  @param  $identifier identifier  An image identifier
      *
-     *  @return boolean                 Returns TRUE on success or FALSE on error.
+     *  @return bool                 Returns TRUE on success or FALSE on error.
      *
      *                                  If FALSE is returned, check the {@link error} property to see the error code.
-     *
-     *  @access private
      */
-    function _write_image($identifier)
+    public function _write_image($identifier)
     {
 
         // sharpen image if it's required
@@ -1614,7 +1579,6 @@ class Zebra_Image
                     $this->error = 3;
 
                     return false;
-
                 }
 
                 break;
@@ -1638,7 +1602,6 @@ class Zebra_Image
                     $this->error = 3;
 
                     return false;
-
                 }
 
                 break;
@@ -1664,7 +1627,6 @@ class Zebra_Image
                     $this->error = 3;
 
                     return false;
-
                 }
 
                 break;
@@ -1689,19 +1651,18 @@ class Zebra_Image
             chmod($this->target_path, intval($this->chmod_value, 8));
 
         // save the error level
-        } else $this->error = 8;
+        } else {
+            $this->error = 8;
+        }
 
         // if target file has to have the same timestamp as the source image
         if ($this->preserve_time && isset($this->source_image_time)) {
 
             // touch the newly created file
             @touch($this->target_path, $this->source_image_time);
-
         }
 
         // return true
         return true;
-
     }
-
 }

css/dashboard.php

@@ -1,6 +1,6 @@
 <?php
-include('../inc/configuracion.inc');
-header("Content-type: text/css");
+include '../inc/configuracion.inc';
+header('Content-type: text/css');
 ?>
 /*
 * Base structure

inc/configuracion.inc

@@ -33,12 +33,13 @@ define('PROGRAMA', 'Gestión de Inventario.');
 define('CENTRO', 'I.E.S.O. Pascual Serrano');
 define('NUMFILAS', '17'); // Número de registros a mostrar en las pantallas de consulta iniciales
 define('PAUSA', '2'); //Nº segundos de pausa para mostrar mensaje id insertado
-define('ESTILO', 'personal'); //Estilo de los iconos de edición (personal, personal, personal)
+define('ESTILO', 'bootstrap'); //Estilo de los iconos de edición (bootstrap, bootstrap, bootstrap)
 define('PLANTILLA', 'bootstrap'); //Estilo de la plantilla y recursos a utilizar
 define('COLORLAT', '#a4bdfc'); //Color de la barra de menú lateral
 define('COLORFON', '#ffb878'); //Color del fondo de la pantalla
 define('MYSQLDUMP', '/usr/local/bin/mysqldump'); //camino a mysqldump
 define('GZIP', '/usr/bin/gzip'); //Camino a gzip
 define('IMAGEDATA', 'img.data'); //Directorio donde se almacenarán las imágenes
-define('INSTALADO', 'sí') //Indicador que permite ejecutar instalar.php
+define('TMP', './tmp'); //Directorio para archivos temporales
+define('INSTALADO', 'sí'); //Indicador que permite ejecutar instalar.php
 ?>

inc/version.inc

@@ -23,5 +23,5 @@
  */
 
 define('AUTOR', 'Ricardo Montañana Gómez');
-define('VERSION', '1.11');
+define('VERSION', '1.17');
 ?>

index.php

@@ -1,9 +1,11 @@
 <?php
     /**
      * Genera una instancia de la aplicación y la ejecuta.
+     *
      * @author Ricardo Montañana <rmontanana@gmail.com>
+     *
      * @version 1.0
-     * @package Inventario
+     *
      * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
      * @license http://www.gnu.org/licenses/gpl-3.0.txt
      * This file is part of Inventario.
@@ -19,15 +21,15 @@
      *
      * You should have received a copy of the GNU General Public License
      * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
-     *  
      */
     //Se incluyen los módulos necesarios
-    function __autoload($class_name) {
-        require_once $class_name . '.php';
+    function __autoload($class_name)
+    {
+        require_once $class_name.'.php';
     }
-    include('inc/configuracion.inc');
+    include 'inc/configuracion.inc';
 
-    $aplicacion=new Inventario();
-    if ($aplicacion->estado())
+    $aplicacion = new Inventario();
+    if ($aplicacion->estado()) {
         $aplicacion->Ejecuta();
-?>
+    }

plant/bootstrap.html

@@ -34,6 +34,7 @@
                         <span class="icon-bar"></span>
                         <span class="icon-bar"></span>
                     </button>
+                    <a class="navbar-brand" href="http://rmontanana.gitbooks.io/Inventario2/" target="_blank" title="Ayuda"><span class="glyphicon glyphicon-book"></span></a>
                     <a class="navbar-brand" href="index.php">{aplicacion}</a>
                 </div>
                 <div class="navbar-brand">

sql/upgrade.php

@@ -1,7 +1,7 @@
 <?php
 /**
  * Migra los datos de la versión anterior de Inventario a la actual.
- * @package Inventario
+ *
  * @copyright Copyright (c) 2008, Ricardo Montañana Gómez
  * @license http://www.gnu.org/licenses/gpl-3.0.txt
  * This file is part of Inventario.
@@ -17,73 +17,76 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with Inventario.  If not, see <http://www.gnu.org/licenses/>.
- *  
  */
-$host="localhost";
-$baseAnt="Inventario";
-$baseNueva="Inventario2";
-$usuario="test";
-$claveUsuario="tset";
-$probar=false;
-
+$host = 'localhost';
+$baseAnt = 'Inventario';
+$baseNueva = 'Inventario2';
+$usuario = 'test';
+$claveUsuario = 'tset';
+$probar = false;
 
 // No se debería modificar nada después de este comentario
-function creaUbicacion($bd1,$bd2,$clave)
+function creaUbicacion($bd1, $bd2, $clave)
 {
     global $probar;
-    $comando="select nombre from Ubicaciones where codigo=".$clave.";";
-    $resultado=$bd1->query($comando);
-    if ($bd1->affected_rows==0) {
+    $comando = 'select nombre from Ubicaciones where codigo='.$clave.';';
+    $resultado = $bd1->query($comando);
+    if ($bd1->affected_rows == 0) {
         echo $comando;
-        die("No encontró la ubicación ".$clave);
+        die('No encontró la ubicación '.$clave);
     }
-    $dato=$resultado->fetch_assoc();
-    $valor=$dato['nombre'];
-    $comando="insert into Ubicaciones values (NULL,'".$valor."');";
+    $dato = $resultado->fetch_assoc();
+    $valor = $dato['nombre'];
+    $comando = "insert into Ubicaciones values (NULL,'".$valor."');";
     if ($probar) {
         echo $comando;
+
         return 1;
     } else {
-        $test=$bd2->query($comando);
+        $test = $bd2->query($comando);
         if (!$test) {
-            die("**No pudo insertar ubicacion.".$comando);
+            die('**No pudo insertar ubicacion.'.$comando);
         }
+
         return $bd2->insert_id;
     }
 }
-function creaArticulo($bd1,$bd2,$clave)
+function creaArticulo($bd1, $bd2, $clave)
 {
     global $probar;
-    $comando="select * from Articulos where codigo='".$clave."';";
-    $resultado=$bd1->query($comando);
-    if ($bd1->affected_rows==0) {
+    $comando = "select * from Articulos where codigo='".$clave."';";
+    $resultado = $bd1->query($comando);
+    if ($bd1->affected_rows == 0) {
         echo $comando;
-        die("No encontró el artículo ".$clave);
+        die('No encontró el artículo '.$clave);
     }
-    $dato=$resultado->fetch_assoc();
-    $valor1=$dato['descripcion'];
-    $valor2=$dato['marca'];
-    $valor3=$dato['modelo'];
-    $valor4=$dato['cantidad'];
-    $comando="insert into Articulos values (NULL,'".$valor1."','".$valor2."','".$valor3."',".$valor4.");";
+    $dato = $resultado->fetch_assoc();
+    $valor1 = $dato['descripcion'];
+    $valor2 = $dato['marca'];
+    $valor3 = $dato['modelo'];
+    $valor4 = $dato['cantidad'];
+    $comando = "insert into Articulos values (NULL,'".$valor1."','".$valor2."','".$valor3."',".$valor4.');';
     if ($probar) {
         echo $comando;
+
         return 1;
     } else {
-        $test=$bd2->query($comando);
+        $test = $bd2->query($comando);
         if (!$test) {
-            die("**No pudo insertar artículo.".$comando);
+            die('**No pudo insertar artículo.'.$comando);
         }
+
         return $bd2->insert_id;
     }
 }
 function generaSesion()
 {
-    $long=10;
-    $cadena="";
-    for ($i=0;$i<$long;$i++) {
-        $cadena.=chr(rand(40,126));
+    $long = 10;
+    $cadena = '';
+    for ($i = 0; $i < $long; $i++) {
+        $cadena .= chr(rand(40, 126));
     }
+
     return $cadena;
 }
 /*
@@ -91,64 +94,64 @@ function generaSesion()
  * Comienzo del programa principal.
  *
  */
-$bd1=new mysqli($host,$usuario,$claveUsuario,$baseAnt);
-if(mysqli_connect_errno()) {
-  die("**Error conectando a la base de datos antigua.".$bd1->error);
+$bd1 = new mysqli($host, $usuario, $claveUsuario, $baseAnt);
+if (mysqli_connect_errno()) {
+    die('**Error conectando a la base de datos antigua.'.$bd1->error);
 }
-$bd2=new mysqli($host,$usuario,$claveUsuario,$baseNueva);
-if(mysqli_connect_errno()) {
-  die("**Error conectando a la base de datos nueva.".$bd2->error);
+$bd2 = new mysqli($host, $usuario, $claveUsuario, $baseNueva);
+if (mysqli_connect_errno()) {
+    die('**Error conectando a la base de datos nueva.'.$bd2->error);
 }
 $bd2->autocommit(false);
-$datos=$bd1->query("select * from Elementos;");
+$datos = $bd1->query('select * from Elementos;');
 if (!$datos) {
-    die("**No encontró datos en la tabla de elementos.");
+    die('**No encontró datos en la tabla de elementos.');
 }
-$numRegistros=$bd1->affected_rows;
-$contador=0;
-$ubicaciones=array();
-$articulos=array();
-echo "++Comenzando proceso de actualización de Elementos con ".$numRegistros." registros por procesar.<br>\n";
-while($fila=$datos->fetch_assoc()) {
+$numRegistros = $bd1->affected_rows;
+$contador = 0;
+$ubicaciones = [];
+$articulos = [];
+echo '++Comenzando proceso de actualización de Elementos con '.$numRegistros." registros por procesar.<br>\n";
+while ($fila = $datos->fetch_assoc()) {
     $contador++;
-    echo "Procesando registro ".$contador." de ".$numRegistros."<br>\n";
+    echo 'Procesando registro '.$contador.' de '.$numRegistros."<br>\n";
     if (!isset($ubicaciones[$fila['codUbicacion']])) {
-        $ubicaciones[$fila['codUbicacion']]=creaUbicacion($bd1,$bd2,$fila['codUbicacion']);
+        $ubicaciones[$fila['codUbicacion']] = creaUbicacion($bd1, $bd2, $fila['codUbicacion']);
     }
     if (!isset($articulos[$fila['codArticulo']])) {
-        $articulos[$fila['codArticulo']]=creaArticulo($bd1,$bd2,$fila['codArticulo']);
+        $articulos[$fila['codArticulo']] = creaArticulo($bd1, $bd2, $fila['codArticulo']);
     }
-    $comando="insert into Elementos values (NULL,".$articulos[$fila['codArticulo']].",".$ubicaciones[$fila['codUbicacion']];
-    $comando.=",'".$fila['numserie']."',".$fila['cantidad'].",'".$fila['fechaCompra']."');";
+    $comando = 'insert into Elementos values (NULL,'.$articulos[$fila['codArticulo']].','.$ubicaciones[$fila['codUbicacion']];
+    $comando .= ",'".$fila['numserie']."',".$fila['cantidad'].",'".$fila['fechaCompra']."');";
     if ($probar) {
-        echo $comando."<br>";
+        echo $comando.'<br>';
     } else {
-        $res=$bd2->query($comando);
+        $res = $bd2->query($comando);
         if (!$res) {
-            die("**Error ejecutando el comando de actualización. ".$comando." ".$bd2->error);
+            die('**Error ejecutando el comando de actualización. '.$comando.' '.$bd2->error);
         }
     }
 }
 //Traspasa los usuarios
-$datos=$bd1->query("select * from Usuarios;");
+$datos = $bd1->query('select * from Usuarios;');
 if (!$datos) {
-    die("**No encontró datos en la tabla de Usuarios.");
+    die('**No encontró datos en la tabla de Usuarios.');
 }
-$numRegistros=$bd1->affected_rows;
-$contador=0;
-while ($fila=$datos->fetch_assoc()) {
+$numRegistros = $bd1->affected_rows;
+$contador = 0;
+while ($fila = $datos->fetch_assoc()) {
     $contador++;
-    echo "Procesando registro ".$contador." de ".$numRegistros."<br>\n";
-    $sesion=generaSesion();
-    $comando="insert into Usuarios values (NULL,'".$fila['usuario']."','".$fila['usuario']."','".$sesion;
-    $comando.="',".$fila['altas'].",".$fila['modificaciones'].",".$fila['bajas'].",".$fila['consultas'].",";
-    $comando.=$fila['informes'].",".$fila['usuarios'].",1);";
+    echo 'Procesando registro '.$contador.' de '.$numRegistros."<br>\n";
+    $sesion = generaSesion();
+    $comando = "insert into Usuarios values (NULL,'".$fila['usuario']."','".$fila['usuario']."','".$sesion;
+    $comando .= "',".$fila['altas'].','.$fila['modificaciones'].','.$fila['bajas'].','.$fila['consultas'].',';
+    $comando .= $fila['informes'].','.$fila['usuarios'].',1);';
     if ($probar) {
-        echo $comando."<br>";
+        echo $comando.'<br>';
     } else {
-        $res=$bd2->query($comando);
+        $res = $bd2->query($comando);
         if (!$res) {
-            die("**Error ejecutando el comando de actualización. ".$comando." ".$bd2->error);
+            die('**Error ejecutando el comando de actualización. '.$comando.' '.$bd2->error);
         }
     }
 }
@@ -156,4 +159,3 @@ echo "++Fin del proceso.<br>\n";
 $bd2->commit();
 $bd1->close();
 $bd2->close();
-?>

xml/informeArticulos.xml

@@ -2,7 +2,11 @@
 <Informe>
 	<Titulo Texto="Informe de Artículos" />
 	<Datos>
-		<Consulta>select * from Articulos where descripcion like '%{filtro}%' order by {orden};</Consulta>
+		<Consulta>
+                    select A.id as id,A.descripcion as descripcion, A.marca as marca, A.modelo as modelo, A.cantidad as cantidad,
+                    (select count(E.id) from Elementos E where E.id_Articulo=A.id) as Numero from Articulos A
+                    where A.descripcion like '%{filtro}%' order by {orden};
+                </Consulta>
 	</Datos>
 	<Pagina Orientacion="P" Formato="A4">
 		<Cabecera>Relación Artículos</Cabecera>
@@ -10,8 +14,9 @@
 			<Col Nombre="id" Ancho="10" Ajuste="D" Titulo="id"/>
 			<Col Nombre="descripcion" Ancho="70" Ajuste="I" Titulo="Descripción"/>
 			<Col Nombre="marca" Ancho="40" Ajuste="I" Titulo="Marca"/>
-			<Col Nombre="modelo" Ancho="50" Ajuste="I" Titulo="Modelo"/>
-			<Col Nombre="cantidad" Ancho="20" Ajuste="D" Titulo="cantidad"/>
+			<Col Nombre="modelo" Ancho="40" Ajuste="I" Titulo="Modelo"/>
+			<Col Nombre="cantidad" Ancho="20" Ajuste="D" Titulo="Cantidad"/>
+                        <Col Nombre="Numero" Ancho="15" Ajuste="D" Titulo="NºElem"/>
 		</Cuerpo>
 	</Pagina>
 </Informe>

xml/informeUbicaciones.xml

@@ -2,13 +2,16 @@
 <Informe>
 	<Titulo Texto="Informe de Ubicaciones" />
 	<Datos>
-		<Consulta>select * from Ubicaciones where descripcion like '%{filtro}%' order by {orden};</Consulta>
+		<Consulta>select U.id as id, U.Descripcion as Descripcion,(select count(E.id) from Elementos E where E.id_Ubicacion=U.id) as Numero 
+                          from Ubicaciones U where U.descripcion like '%{filtro}%' order by {orden};
+                </Consulta>
 	</Datos>
 	<Pagina Orientacion="P" Formato="A4">
 		<Cabecera>Relación de Ubicaciones</Cabecera>
 		<Cuerpo>
 			<Col Nombre="id" Ancho="10" Ajuste="D" Titulo="id"/>
 			<Col Nombre="Descripcion" Ancho="80" Ajuste="I" Titulo="Descripción"/>
+                        <Col Nombre="Numero" Ancho="15" Ajuste="D" Titulo="Nº Elem"/>
 		</Cuerpo>
 	</Pagina>
 </Informe>

xml/inventarioArticuloEtiquetas.xml

@@ -3,7 +3,7 @@
 	<Titulo Texto="{Descripcion}" id="{id}"/>
 	<Datos>
 		<Consulta>
-			select A.id as id, A.Descripcion as articulo, A.Marca as marca, A.Modelo as modelo, E.id as idEl, U.id as idUbic,U.Descripcion as ubicacion,E.numserie as numserie, 
+			select A.id as idArt, A.Descripcion as articulo, A.Marca as marca, A.Modelo as modelo, E.id as idEl, U.id as idUbic,U.Descripcion as ubicacion,E.numserie as numserie, 
                                 E.fechaCompra as fechaCompra,E.Cantidad as cantidad, E.Cantidad as cantReal, 'N' as Baja
 				from Elementos E, Articulos A, Ubicaciones U where A.id=E.id_Articulo and U.id=E.id_Ubicacion
 				and A.id='{id}' order by U.Descripcion,numserie;

xml/mantenimientoArticulos.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0"?>
+<Mantenimiento>
+    <Titulo>Mantenimiento de Artículos</Titulo>
+    <Consulta>
+        select SQL_CALC_FOUND_ROWS A.id as id,A.descripcion as descripcion, A.marca as marca, A.modelo as modelo, A.cantidad as cantidad,
+         (select sum(E.cantidad) from Elementos E where E.id_Articulo=A.id) as cantubi,
+         (select count(E.id) from Elementos E where E.id_Articulo=A.id) as numero,A.imagen as imagen from Articulos A
+         {buscar} {orden} limit {inferior},{superior};    
+    </Consulta>
+    <Campos>
+        <Col Campo="id" Nombre="id" Ancho="5" Ajuste="D" Titulo="id" Tipo="smallint(6)" Varios="ordenable,link/Articulo" Visible="si"/>
+        <Col Campo="descripcion" Nombre="descripcion" Ancho="50" Ajuste="L" Titulo="Descripción" Visible="si" Varios="ordenable,ajax/text" Editable="si"/>
+        <Col Campo="marca" Nombre="marca" Ancho="40" Ajuste="L" Titulo="Marca" Varios="ordenable,ajax/text" Editable="si" Visible="si"/>
+        <Col Campo="modelo" Nombre="modelo" Ancho="40" Ajuste="L" Titulo="Modelo" Varios="ordenable,ajax/text" Editable="si" Visible="si"/>
+        <Col Campo="cantidad" Nombre="cantidad" Ancho="15" Ajuste="D" Titulo="Cantidad"  Tipo="Int(11)" Visible="si" Editable="si" Varios="ordenable,ajax/number"/>
+        <Col Campo="cantubi" Nombre="cantubi" Ancho="15" Ajuste="D" Titulo="Cant. ubicada" Editable="no" Varios="ordenable" Visible="si"/>
+        <Col Campo="numero" Nombre="numero" Ancho="15" Ajuste="D" Titulo="Nº Elementos" Editable="no" Varios="ordenable" Visible="si"/>
+        <Col Campo="imagen" Nombre="imagen" Ancho="10" Ajuste="C" Titulo="Imagen" Visible="si" Tipo="imagen" Varios="imagen" Editable="si"/>
+    </Campos>
+</Mantenimiento>

xml/mantenimientoUbicaciones.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<Mantenimiento>
+    <Titulo>Mantenimiento de Ubicaciones</Titulo>
+    <Consulta>
+        select SQL_CALC_FOUND_ROWS U.id as id, U.descripcion as Descripcion, U.imagen as imagen, (select count(E.id) from Elementos E where E.id_Ubicacion=U.id) as numero from Ubicaciones U
+        {buscar} {orden} limit {inferior},{superior};    
+    </Consulta>
+    <Campos>
+        <Col Campo="id" Nombre="id" Ancho="5" Ajuste="D" Titulo="id" Tipo="smallint(6)" Varios="ordenable,link/Ubicacion" Visible="si"/>
+        <Col Campo="Descripcion" Nombre="Descripcion" Ancho="50" Ajuste="L" Titulo="Descripción" Visible="si" Varios="ordenable,ajax/text" Editable="si"/>
+        <Col Campo="imagen" Nombre="imagen" Ancho="10" Ajuste="C" Titulo="Imagen" Tipo="imagen" Visible="si" Varios="imagen" Editable="si"/>
+        <Col Campo="numero" Nombre="numero" Ancho="10" Ajuste="D" Titulo="Nº Elementos" Varios="ordenable" Editable="no" Visible="si"/>
+    </Campos>
+</Mantenimiento>