Fallo en informe de profesor si no hay estudiantes en un curso

UNIMOODLE / moodle-mod_certifygen

GNU General Public License v3.0

1 stars 1 forks source link

Fallo en informe de profesor si no hay estudiantes en un curso #69

Closed juacas closed 1 month ago

juacas commented 1 month ago

El informe de profesor falla con el error "no students in course"

y la solicitud se queda en un estado [[status_9]]

Al Borrar la petición aparece el error:

Y no se borra la petición por lo que no se puede pedir una nueva.

Comportamiento esperado:

La intención de borrado del administrador debe llevarse a cabo tras una confirmación.
El mensaje de error debe aportar indicaciones para tratar el problema.
En caso de ausencia de estudiantes en un grupo debe poder ejecutarse el informe. No se entiende cual es la lógica del comportamiento actual.

juacas commented 1 month ago

Otro aspecto de esta pantalla.

El botón de "Crear petición" no debe desaparecer tras hacer la petición. Sobre todo si la opción de emisión reiterativa está activa en el modelo.

elena3ip commented 1 month ago

Resumo los puntos a tratar:

en el buscador no deben aparecer cursos sin alumnos ya que es necesario para poder crear el informe (Para sacar los datos en muchas operaciones se divide entre el numero de estudiantes, por lo que no se puede dividir entre 0) (Se sube la corrección en breve a develop)
Poder eliminar una petición (Lo que sucede es que para eliminar la petición no tenia en cuenta el estado y siempre se intentaba primero eliminar desde el método de validación, en este caso no existía tal petición por lo que daba error.) (Se sube la corrección en breves a develop )
La desaparición del botón de crear petición

tmas0 commented 1 month ago

Buenas @elena3ip,

He intentado generar un certificado usando el idioma Español, pero el certificado se emite en Inglés. Lo comento aquí por no generar otro ticket cuando creo que esto está relacionado con lo que se habla en esta incidencia. Si no es así, obro otro, sin problema.

Un saludo

juacas commented 1 month ago

En los casos en que el denominador sea el número de estudiantes y éste sea 0, también el numerador debería ser 0, el indicador debe tomar el valor 0.

El objetivo de dividir por el número de estudiantes es obtener una media normalizada. Añadir simplemente la comprobación: $numstudents==0?0:$valor/$numstudents como salvaguarda.

Es importante que en el caso de que entre la petición del informe de un curso sin estudiantes (por ejemplo mediante WS) se obtenga un resultado del modelo.

Un saludo.

elena3ip commented 1 month ago

Ya está el cambio en develop

tmas0 commented 1 month ago

La parte que me corresponde, está comprobada.

tmas0 commented 1 month ago

También he comprobado la parte que comenta @juacas, viendo que ya va bien.