*Corregido que no aparecía el mensaje de bienvenida al hacer login en la aplicación

* release/1.17:
  Eliminados los mensajes de aviso de php en todos los archivos php, actualizada la versión y el changelog

AportaContenido.php

@@ -210,7 +210,11 @@ class AportaContenido {
                     return $salida;
                 }
             case 'opcion':
+                if (strstr($this->opcionActual, "&")) {
                     list($opcion, $parametro) = explode("&", $this->opcionActual);
+                } else {
+                    $opcion = $this->opcionActual; $parametro = "";
+                }
                 switch ($opcion) {
                     case 'bienvenido':
                         return "Menú Principal";
@@ -244,13 +248,19 @@ class AportaContenido {
 //                if (!$this->registrado) {
 //                    return $this->mensajeRegistro();
 //                }
+                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>';
                     case 'principal': // contenido inicial
-
+                        if (!isset($mensaje)) {
+                            $mensaje = "";
+                        }
                         $creditos = "$('#creditos').modal({keyboard: false});";
                         $centro = '<div class="well well-sm">' . CENTRO . '</div>';
                         $tabla = $this->creaTablaAcercaDe();
@@ -264,7 +274,7 @@ class AportaContenido {
                     case 'elementos':
                         $this->cargaDatosURL();
                         if ($this->datosURL['opc'] == "informe") {
-                            if (!$this->pefil['Informe']) {
+                            if ($this->perfil['Informe']) {
                                 $this->procesaURL();
                                 $fichero = 'xml/informe' . ucfirst($opcion) . '.xml';
                                 $salida = TMP.'/informe' . ucfirst($opcion) . '.xml';
@@ -343,7 +353,7 @@ class AportaContenido {
                     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";

CHANGELOG

@@ -1,3 +1,6 @@
+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
 

Configuracion.php

@@ -82,7 +82,7 @@ class Configuracion {
                 fwrite($fsalida, $registro);
             }
         }
-        $salida.=$this->formulario();
+        $salida = $this->formulario();
         if ($grabar) {
             $salida.='<div class="alert alert-success">Configuraci&oacute;n guardada correctamente</div>';
             fclose($fsalida);

CopiaSeguridad.php

@@ -23,10 +23,6 @@ class CopiaSeguridad {
     private $baseDatos;
     private $imagenes;
     
-    public function __construct()
-    {
-        $opcion = $_GET['opc'];
-    }
     public function creaCopia()
     {
         if (!$this->copiaBaseDatos()) {

Csv.php

@@ -76,7 +76,8 @@ class Csv {
      * Constructor de la clase. 
      * @param BaseDatos $baseDatos Manejador de la base de datos
      */
-    public function __construct($baseDatos) {
+    public function __construct($baseDatos)
+    {
         $this->bdd = $baseDatos;
     }
 
@@ -84,7 +85,8 @@ class Csv {
      * 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.");
     }
@@ -93,15 +95,18 @@ class Csv {
      * 
      * @param array $datos escribe la línea en el archivo
      */
-    public function escribeLinea($datos) {
+    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() {
+    public function cierra()
+    {
         fclose($this->fichero) or die("No puedo cerrar el archivo csv");
     }
 
@@ -109,7 +114,8 @@ class Csv {
      * 
      * @param String $fichero Archivo xml que contiene la definición de la consulta
      */
-    public function ejecutaConsulta($fichero) {
+    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']));
@@ -132,7 +138,8 @@ class 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.");
         list($archivo, $idCabecera, $cabecera) = fgetcsv($this->fichero);
@@ -149,7 +156,8 @@ class Csv {
      * 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>";
@@ -211,12 +219,14 @@ class Csv {
      * @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) {
+    private function compruebaCantidades($i)
-        $ultimo = count($datos);
+    {
+        //$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!") {
+    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">';
@@ -226,14 +236,16 @@ class Csv {
         return $mensaje;
     }
 
-    private function escribeLog($comando) {
+    private function escribeLog($comando)
+    {
         $fp = fopen($this->nombre . ".log", "a");
         $linea = strftime("%Y/%m/%d") . "|" . $this->nombre . "|" . $comando;
         fputs($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 . '";';
         $this->escribeLog($comando);
@@ -242,7 +254,8 @@ class Csv {
         }
     }
 
-    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 . '";';
         $this->escribeLog($comando);
@@ -251,7 +264,8 @@ class Csv {
         }
     }
 
-    private function altaElemento($i) {
+    private function altaElemento($i)
+    {
         if ($this->cabecera[0] == "Articulo") {
             $idUbicacion = $this->datosFichero[$i][$this->idUbicacion];
             $idArticulo = $this->cabecera[1];
@@ -269,7 +283,8 @@ class Csv {
         }
     }
 
-    private function cargaIndices($campos) {
+    private function cargaIndices($campos)
+    {
         for ($i = 0; $i < count($campos); $i++) {
             switch ($campos[$i]) {
                 case "Cant. Real": $this->cantidadReal = $i;
@@ -295,7 +310,8 @@ class Csv {
     /**
      * 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 {
@@ -330,7 +346,8 @@ class Csv {
         }
     }
 
-    private function ejecutaFichero2() {
+    private function ejecutaFichero2()
+    {
         echo '<script>visualizaProgreso();</script>';
         for ($i = 1; $i < 80; $i++) {
             //sleep(1);

Importacion.php

@@ -55,6 +55,7 @@ class Importacion {
 
     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";

Mantenimiento.php

@@ -200,6 +200,7 @@ class Mantenimiento {
         }
         //$salida.=$comando;
         //$salida.=var_export($this->campos,true);
+        $cant = 0;
         while ($fila = $this->bdd->procesaResultado()) {
             $salida.='<tr bottom="middle">';
             foreach ($fila as $clave => $valor) {
@@ -218,7 +219,7 @@ class Mantenimiento {
                             $datoEnlace = $tmpco[1];
                         }
                     }
-                    $this->campoBusca = $dato[1];
+                    $this->campoBusca = isset($dato[1]) ? $dato[1] : "";
                     $valor = '<a title="Inventario de ' . $valor . '" $target="_blank" href="index.php?informeInventario&opc=listar' . $datoEnlace . '&id=' . $id . '">' . $valor;
                 }
                 if (strstr($this->campos[$clave]['Comment'], "imagen") && isset($valor)) {
@@ -344,6 +345,8 @@ class Mantenimiento {
                 $informe = "";
             }
             $this->restoreURL();
+        } else {
+            $anterior = $rew = $az = $informe = $za = $siguiente = $fwd = "";
         }
         if ($this->perfil['Alta']) {
             $this->datosURL['opc'] = 'nuevo';
@@ -422,14 +425,14 @@ class Mantenimiento {
             } else {
                 $coma = ",";
             }
-            if ($this->campos[$campo]['Type'] == 'Boolean(1)') {
+            if (isset($this->campos[$campo]['Type']) && $this->campos[$campo]['Type'] == 'Boolean(1)') {
                 $valor = "";
                 if (empty($_POST[$campo])) {
                     $valor = "0";
                 }
                 $valor = $_POST[$campo] == "on" ? '1' : $valor;
             } else {
-                if (stristr($this->campos[$campo]['Comment'], "imagen")) {
+                if (isset($this->campos[$campo]['Comment']) && stristr($this->campos[$campo]['Comment'], "imagen")) {
                     //procesa el envío de la imagen
                     $imagen = new Imagen();
                     $accion = $imagen->determinaAccion($campo);
@@ -442,7 +445,11 @@ class Mantenimiento {
                         $campoImagen = $campo;
                     } else {
                         //Comprobamos si hay clonación y hay imagen a clonar.
+                        if (isset($_POST[$campo])) {
                             $valor = $_POST[$campo];
+                        } else {
+                            $valor = "";
+                        }
                         if ($_POST['tipoOperacion'] == CLONAR && file_exists($valor)) {
                             $hayImagen = true;
                             $campoImagen = $campo;
@@ -719,9 +726,10 @@ class Mantenimiento {
                 break;
         }
         $accion = $this->montaURL();
-        $salida.='<div class="col-sm-8"><form name="mantenimiento.form" enctype="multipart/form-data" class="form-horizontal" role="form" method="post" action="' . $accion . '">' . "\n";
+        $salida = '<div class="col-sm-8"><form name="mantenimiento.form" enctype="multipart/form-data" class="form-horizontal" role="form" method="post" action="' . $accion . '">' . "\n";
         $salida .= "<fieldset style=\"width: 96%;\"><p><legend style=\"color: red;\"><b>$tipo</b></legend>\n";
         //$salida.= var_export($datos,true);
+        $campos = "";
         foreach ($this->campos as $clave => $valor) {
             if ($valor["Editable"] == "no") {
                 //Se salta los campos que no deben aparecer
@@ -860,7 +868,7 @@ class Mantenimiento {
     
     private function creaModal($valor, $id)
     {
-        $mensaje .= '
+        $mensaje = '
                 <div id="mensajeModal'.$id.'" class="modal fade " tabindex="-1" role="dialog" aria-labelledby="myLargeModalLabel" aria-hidden="true">
                 <div class="modal-dialog text-center">
                 <div class="modal-content text-center">

Pdf_mysql_table.php

@@ -111,7 +111,8 @@ class Pdf_mysql_table extends FPDF
             if ($this->procesandoTotales) {
                 $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']);
@@ -119,7 +120,10 @@ class Pdf_mysql_table extends FPDF
             //print_r($data);
             //print_r($this->aCols);
             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']];
             }
             
         }

inc/version.inc

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