search

UNIMOODLE / moodle-local_notificationsagent

Unimoodle L5 P4
Other
3 stars 4 forks source link

Etiqueta de regla "Inválida" en regla forzada #111

Closed juacas closed 4 months ago

juacas commented 5 months ago

Al forzar una regla desde administración, aparece correctamente instalada en el curso pero con la etiqueta "Inválida". No sé si es un glich de presentación o está incumpliendo alguna restricción. En el ejemplo, debería autoconfigurarse porque la acción se refiere al foro de "Novedades".

He comprobado que el estado "inválida" sale cuando el curso tiene o no tiene habilitada una fecha de finalización.

image

juacas commented 5 months ago

Si clono la regla forzada (inválida), sin cambiar nada, genero una regla válida.

image

juacas commented 5 months ago

He notado que la regla "inválida" no actualiza el la visualización el nombre del foro si le cambio el nombre. La regla instanciada localmente sí actualiza el nombre. Me temo que no se está haciendo el enlace adecuado al cmid de la actividad "Foro novedades" del curso. La funcionalidad prevista es que el marcador de "Foro novedades" se sustituyera por el cmid por la primera actividad forum tipo "News" del curso.

juacas commented 5 months ago

Veo que la clase notificationsaction_forummessage\forum_message no implementa el método validation(). La implementación por defecto valida el cmid, pero en esta acción el valor "-1" indica que hay que obtener el cmid del foro news. He propuesto un PR para esto.

fpano commented 4 months ago

Corregida en el PR y añadido también test unitario sobre el método.