fixing hot bugs

src/app.py

@@ -62,6 +62,8 @@ def incidencia(idIncidencia):
     cambioCierre = get_cambio_by_estado(idIncidencia, 3)
     estado = get_estado(incidencia.estado)
     categoria = get_categoria(incidencia.categoria)
+    print(incidencia)
+    print(incidencia.comentario)
     return render_template('info_incidencia.html', incidencia=incidencia, estado=estado, categoria=categoria, cambioApertura=cambioApertura, cambioAsignada=cambioAsignada, cambioCierre=cambioCierre)
 
 @app.route('/index')
@@ -169,8 +171,8 @@ from flask_sqlalchemy import SQLAlchemy
 from sqlalchemy import or_
 from flask_login import UserMixin
 
-app.config['SQLALCHEMY_DATABASE_URI'] = 'mysql://PGPI_grupo02:JEbITzwe@127.0.0.1:3306/PGPI_grupo02'
-#app.config['SQLALCHEMY_DATABASE_URI'] = 'mysql://PGPI_grupo02:JEbITzwe@jair.lab.inf.uva.es:3306/PGPI_grupo02'
+#app.config['SQLALCHEMY_DATABASE_URI'] = 'mysql://PGPI_grupo02:JEbITzwe@127.0.0.1:3306/PGPI_grupo02'
+app.config['SQLALCHEMY_DATABASE_URI'] = 'mysql://PGPI_grupo02:JEbITzwe@jair.lab.inf.uva.es:3306/PGPI_grupo02'
 app.config['SQLALCHEMY_TRACK_MODIFICATIONS'] = False
 db = SQLAlchemy(app)
 

src/schema.sql

@@ -12,7 +12,7 @@ DROP TABLE IF EXISTS cambio;
 
 CREATE TABLE estado(
     id INTEGER PRIMARY KEY,
-    estado VARCHAR (30)
+    estado VARCHAR (50)
 );
 
 
@@ -51,10 +51,10 @@ CREATE TABLE usuario(
 CREATE TABLE incidencia(
     id INTEGER PRIMARY KEY AUTO_INCREMENT,
     titulo VARCHAR(100),
-    comentario VARCHAR(200),
+    comentario VARCHAR(500),
     prioridad INTEGER,
     tiempoEstimado INTEGER,
-    descripcion VARCHAR(200),
+    descripcion VARCHAR(500),
     fecha DATE,
     estado INTEGER REFERENCES estado(id),
     tecnicoAsignado VARCHAR(50) REFERENCES usuario(nick),
@@ -79,8 +79,8 @@ INSERT INTO estado (id, estado) VALUES (0, 'Abierta');
 INSERT INTO estado (id, estado) VALUES (1, 'Asignada a un técnico');
 INSERT INTO estado (id, estado) VALUES (2, 'Solicitado cierre por el cliente');
 INSERT INTO estado (id, estado) VALUES (3, 'Solicitado cierre por el técnico');
-INSERT INTO estado (id, estado) VALUES (4, 'Cerrada sin solucion');
-INSERT INTO estado (id, estado) VALUES (5, 'Cerrada con solusion');
+INSERT INTO estado (id, estado) VALUES (4, 'Cerrada sin solución');
+INSERT INTO estado (id, estado) VALUES (5, 'Cerrada con solución');
 
 /* Poblando la tabla: tipo_usuario */
 INSERT INTO tipo_usuario (id, tipo_usuario) VALUES (0, 'supervisor');

src/static/css/clientView.css

@@ -146,7 +146,7 @@ nav {
 
 .btn {
     margin: 1em;
-    width: 10em;
+    width: 13em;
 }
 
 

src/templates/add_comentario.html

@@ -67,8 +67,8 @@
                 <h4>Identificador: {{ incidencia.id}}</h4>
                 <h4>Título: {{ incidencia.titulo }}</h4>
                 <textarea placeholder="Añadir Comentario" name="comentario">{{ incidencia.comentario }}</textarea>
-                <button type="submit" name="action" value="add_com">Añadir Comentario</button>
-                <button type="submit" name="action" value="cancelar">Cancelar</button>
+                <button class="btn btn-outline-primary" type="submit" name="action" value="add_com">Añadir Comentario</button>
+                <button class="btn btn-outline-primary" type="submit" name="action" value="cancelar">Cancelar</button>
             
             </form>
     

src/templates/add_tiempo.html

@@ -67,9 +67,10 @@
                 <h4>Identificador: {{ incidencia.id}}</h4>
                 <h4>Título: {{ incidencia.titulo }}</h4>
                 <h5>Descripción: {{ incidencia.descripcion }}</h5>
-                <input type="number" pattern="[1-999]"  name="tiempo">
-                <button type="submit" name="action" value="add_tiempo">Añadir tiempo</button>
-                <button type="submit" name="action" value="cancelar">Cancelar</button>
+                <p>Introduzca el número en horas: </p>
+                <input type="number" pattern="\d+"  name="tiempo" min="1" max="72">
+                <button class="btn btn-outline-primary" type="submit" name="action" value="add_tiempo" >Añadir tiempo</button>
+                <button class="btn btn-outline-primary" type="submit" name="action" value="cancelar">Cancelar</button>
             
             </form>
     

src/templates/completar_incidencia.html

@@ -52,9 +52,9 @@
       
       <p> Incidencia {{ incidencia.id }} </p>
       <textarea placeholder="Comentario" name="comentario" required></textarea>
-      <input type="text" placeholder="prioridad" name="prioridad" required/>
+      <input type="number" min="1" max="10" pattern="\d+" placeholder="Prioridad" name="prioridad" required title="Campo obligatorio. Numero entre 1 y 10"/>
 
-      <select name="tecnico" required>
+      <select name="tecnico" required placeholder="Elegir técnico">
           {% for i in range(listaTecnicos|length) %} 
           <option value="{{ listaTecnicos[i].nick }}">{{ listaTecnicos[i].nick }}</option>
           {% endfor %}

src/templates/incidencias_cliente.html

@@ -72,9 +72,9 @@
         </div>
         {% endfor %}
         {% set i=((incidencias|length)/3)|int %}
+
         <div class="card-deck" style="margin: 1em;">
         {% for j in range((incidencias|length)%3) %}
-        
             <div class="card" id="card1">
                 <div class="card-body">
                     <a href="/incidencia/{{ incidencias[3*i+j]['id']  }}">

src/templates/incidencias_supervisor.html

@@ -56,6 +56,8 @@
         </div>
 
         <div style="display: flex;">
+
+            {% if (incidencias_abiertas|length) > 0 %}
             {% if current_user.tipo==0 %}
             <div>
                 <p class="text-center" id="tituloColumna">Pendientes de asignar</p>
@@ -79,8 +81,11 @@
 
             </div>
             {% endif %}
+            {% endif %}
+
+            {% if (incidencias_notif_cierre|length) > 0 %}
             {% if current_user.tipo==0 %}
-            <div style="border-left: 1px solid grey; height: 70%;">
+            <div>
                 <p class="text-center" id="tituloColumna">Pendientes de Cerrar</p>
                 {% for i in range(incidencias_notif_cierre|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -100,8 +105,11 @@
                 {% endfor %}
             </div>
             {% endif %}
+            {% endif %}
+
+            {% if (incidencias_notif_cierre_cliente|length) > 0 %}
             {% if current_user.tipo==0 %}
-            <div style="border-left: 1px solid grey; height: 70%;">
+            <div>
                 <p class="text-center" id="tituloColumna">Pendientes de Cerrar Cliente</p>
                 {% for i in range(incidencias_notif_cierre_cliente|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -121,8 +129,11 @@
                 {% endfor %}
             </div>
             {% endif %}
+            {% endif %}
+
+            {% if (incidencias|length) > 0 %}
             {% if current_user.tipo==2 %}
-            <div style="border-left: 1px solid grey; height: 70%;">
+            <div>
                 <p class="text-center" id="tituloColumna">Incidencias abiertas</p>
                 {% for i in range(incidencias|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -142,8 +153,11 @@
                 {% endfor %}
             </div>
             {% endif %}
+            {% endif %}
+
+            {% if (incidencias_estado|length) > 0 %}
             {% if current_user.tipo==2 %}
-            <div style="border-left: 1px solid grey; height: 70%;">
+            <div>
                 <p class="text-center" id="tituloColumna">Incidencias en Resolucion</p>
                 {% for i in range(incidencias_estado|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -163,8 +177,11 @@
                 {% endfor %}
             </div>
             {% endif %}
+            {% endif %}
+
+            {% if (incidencias_estado_cierre|length) > 0 %}
             {% if current_user.tipo==2 %}
-            <div style="border-left: 1px solid grey; height: 70%;">
+            <div>
                 <p class="text-center" id="tituloColumna">Incidencias Solicitud Cierre</p>
                 {% for i in range(incidencias_estado_cierre|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -184,6 +201,7 @@
                 {% endfor %}
             </div>
             {% endif %}
+            {% endif %}
 
         </div>
     </div>

src/templates/incidencias_tecnico.html

@@ -49,6 +49,7 @@
         </div>
 
         <div style="display: flex;">
+            {% if incidencias_abiertas|length > 0 %}
             <div>
                 <p class="text-center" id="tituloColumna">Abiertas</p>
                 {% for i in range(incidencias_abiertas|length) %}
@@ -59,16 +60,19 @@
                                 <p class="card-title">{{ incidencias_abiertas[i].titulo }}</p>
                                 <hr>
                                 <p class="card-text">Descripción:</p>
-                                <p class="card-text .text-justify" id="card-description">{{ incidencias_abiertas[i].titulo }}</p>
+                                <p class="card-text .text-justify" id="card-description">{{ incidencias_abiertas[i].descripcion }}</p>
                                 <hr>
                                 <p class="card-text">Incidencia pendiente de asignar a un tecnico.</p>
+                            </a>
                         </div>
                     </div>
                 </div>
                 {% endfor %}
             </div>
+            {% endif %}
 
-            <div style="border-left: 1px solid grey; height: 70%;">
+            {% if incidencias_notif_cierre|length > 0 %}
+            <div>
                 <p class="text-center" id="tituloColumna">Notificacion Cierre</p>
                 {% for i in range(incidencias_notif_cierre|length) %}
                 <div class="card-deck" style="margin: 1em;">
@@ -81,31 +85,35 @@
                                 <p class="card-text .text-justify" id="card-description">{{ incidencias_notif_cierre[i].descripcion }}</p>
                                 <hr>
                                 <p class="card-text">Incidencia abierta por {{ incidencias_notif_cierre[i].reportadaPor }}</p>
+                            </a>
                         </div>
                     </div>
                 </div>
                 {% endfor %}
             </div>
+            {% endif %}
 
-            <div style="border-left: 1px solid grey; height: 70%;">
+            {% if incidencias_pendientes_cierre|length > 0 %}
+            <div>
                 <p class="text-center" id="tituloColumna">Pendiente Cierre</p>
                 {% for i in range(incidencias_pendientes_cierre|length) %}
                 <div class="card-deck" style="margin: 1em;">
                     <div class="card" id="card2">
                         <div class="card-body">
-                            <!-- <a href="/informacion_incidencia/{{ incidencias_notif_cierre[i].id  }}"> --> 
+                            <a href="/incidencia/{{ incidencias_pendientes_cierre[i].id  }}"> 
                                 <p class="card-title">{{ incidencias_pendientes_cierre[i].titulo }}</p>
                                 <hr>
                                 <p class="card-text">Descripción:</p>
                                 <p class="card-text .text-justify" id="card-description">{{ incidencias_pendientes_cierre[i].descripcion }}</p>
                                 <hr>
                                 <p class="card-text">Incidencia abierta por {{ incidencias_pendientes_cierre[i].reportadaPor }}</p>
-                            <!-- </a> -->
+                            </a>
                         </div>
                     </div>
                 </div>
                 {% endfor %}
             </div>
+            {% endif %}
 
         </div>
     </div>

src/templates/info_incidencia.html

@@ -57,7 +57,7 @@
                         <p class="card-text text-justify" id="card-description"> {{ incidencia.descripcion}}</p>
                        
                     </div>
-                    {% if incidencia.comentario is defined and incidencia.comentario|length %}
+                    {% if (incidencia.comentario is defined) and (incidencia.comentario|length) %}
                     <div class="card-body">     
                             <h5>Comentario:</h5>
                         <p class="card-text text-justify" id="card-description"> {{ incidencia.comentario}}</p>
@@ -146,7 +146,7 @@
       </div>
 
       <div class="steps-three">
-          {% if incidencia.estado == 4 or incidencia.etado == 5%}
+          {% if incidencia.estado == 4 or incidencia.estado == 5 %}
         <img class="steps-img" src="http://placehold.it/50/3498DB/3498DB" alt="" />
         <h3 class="steps-name">
           Cerrada
@@ -178,12 +178,12 @@
             {% endif %}}
 
             {% if (current_user.tipo == 0) and (incidencia.estado ==  1) %}
-            <button type="button" class="btn btn-outline-primary">Añadir comentario</button>
+            <button type="submit" class="btn btn-outline-primary" name="action" value="add_comentario">Añadir comentario</button>
             {% endif %}}
 
             {% if (current_user.tipo == 0) and (incidencia.estado ==  3 or incidencia.estado==2) %}
-            <button type="submit" class="btn btn-outline-primary" name="action" value="Solucion">Cerrar Incidencia Solucionada</button>
-            <button type="submit" class="btn btn-outline-primary" name="action" value="n-Solucion">Cerrar Incidencia sin Solucion</button>
+            <button type="submit" class="btn btn-outline-primary" name="action" value="Solucion">Cerrar Solucion</button>
+            <button type="submit" class="btn btn-outline-primary" name="action" value="n-Solucion">Cerrar sin Solucion</button>
             {% endif %}}
 
 
@@ -200,7 +200,6 @@
 
 
             {% if (current_user.tipo == 2) and (incidencia.estado!=2) %}
-            <button type="button" class="btn btn-outline-primary">Ver comentario</button>
             <button type="submit" class="btn btn-outline-primary" name="action" value="cierre_cliente">Solicitar Cierre</button>
             {% endif %}
  </form>

src/templates/registrar_incidencia.html

@@ -52,7 +52,7 @@
     <form class="incidentCli" method='POST' action='/registrar_incidencia'>
       
       <input type="text" placeholder="Título" name="titulo" required/>
-      <textarea placeholder="Descripción" name="descripcion" required></textarea>
+      <textarea placeholder="Descripción" name="descripcion" required minlength="100" title="La descripcion ha de tener minimo 100 caracteres"></textarea>
       <input type="number" pattern="[0-9]" placeholder="Id elemento inventario " name="idElementoInventario"/>
       <input type="date" id="fecha" name="fecha" required>
                         <select name="categoria" required>