ctt-gob-es / oaw

Rastreador Observatorio de Accesibilidad Web
European Union Public License 1.2
22 stars 30 forks source link

Falta valor en la tabla complejidades_lista cuando se selecciona "Única" #42

Open Entamut opened 1 year ago

Entamut commented 1 year ago

Hola,

Falta un valor en la tabla "complejidades_lista", y cuando se selecciona "Única" en "Cantidad de páginas" se produce una excepción el proceso nunca termina correctamente.

Pasos para reproducir el error:

  1. Hacer login con un usuario que tenga permisos para acceder a "Servicio Diagnóstico"
  2. Hacer click en "Servicio Diagnóstico"
  3. Introducir correo electrónico
  4. Seleccionar "Sitio Web" en Seleccionar el tipo de análisis
  5. Introducir una URL en "URL del portal o página a analizar"
  6. En "Opciones de rastreo", seleccionar "Única" en "Cantidad de páginas"
  7. Hacer click en "Solicitar informe"

Resultado esperado:

  1. Se completa el proceso correctamente y se envía el informe al email indicado

Resultado actual:

  1. Se produce una excepción en es/inteco/rastreador2/dao/basic/service/DiagnosisDAO.java:118. Al invocar al método ComplejidadDAO.getById(Connection c, String id) ya que el valor id es igual a 4 y dicho valor no existe en la base de datos.

Fix propuesto:

  1. Añadir el valor 4 a la tabla "complejidades_lista" en el script INSERT_TABLES_OAW_5.0.0

Ver PR https://github.com/ctt-gob-es/oaw/pull/41 con un ejemplo del cambio propuesto.

El valor 4 escrito de manera literal se encuentra en src/main/webapp/diagnostico.html:189

<select id="complexity" name="complexity" class="form-control">
    <option selected="selected" value="4">&#218;nica</option>
    <option value="1">Complejidad baja (17 p&aacute;ginas)</option>
    <option value="2">Complejidad media (33 p&aacute;ginas)</option>
    <option value="3">Complejidad alta (51 p&aacute;ginas)</option>
</select>
victor-cima-ctic commented 1 year ago

Hola @Entamut como bien dices tiene que existir una entrada asociada a cada elemento del selector en la tabla complejidades_lista para el correcto funcionamiento del servicio básico de diagnóstico.