Project

General

Profile

Actions

Bug #7717

closed

Asunto genérico en el e-mail de anuncios

Added by José Angel Ruiz (Nosolored) almost 8 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Announcements
Target version:
Start date:
22/05/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Al enviar un mensaje desde la herramienta anuncios, ha desaparecido el título del anuncio en el asunto del mensaje, apareciendo en su lugar un asunto genérico del tipo "tiene un nuevo mensaje de..."


Files

Selection_016.png (79.6 KB) Selection_016.png Julio Montoya, 15/07/2015 11:37
Actions #1

Updated by Yannick Warnier almost 8 years ago

  • Target version set to 1.10.0
Actions #2

Updated by Julio Montoya over 7 years ago

  • Status changed from New to Needs more info

Si exacto. Lo que se envía al usuario es una notificación que tiene un nuevo mensaje en la plataforma.

Actions #3

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 1.10.0 to 2.0

Si no es un error, lo paso a la versión siguiente, pero sería bueno ponerle un mensaje más completo, porque no somos como Facebook: a nosotros no nos interesa que regrese al portal solo para ver un mensaje. Nos interesa que lo lean, nada más (para esto quizás podríamos desarrollar un mecanismo de registro de lectura para informar el docente que se ha leido, pero eso será para otras versiones.

Actions #4

Updated by Laura Guirao over 7 years ago

Hola,

Tal como lo vemos nosotros es un error, de hecho lo descubrimos porque un cliente se nos quejó por ello.

En el archivo anouncement_email.class.php hay una función definida llamada subject, que devuelve el título del curso más el título del anuncio, pero más tarde en el envío del e-mail esta función se ignora y se añade un asunto genérico. En versiones anteriores iba bien.
No es que sea un error muy grave, pero quedaba mejor antes. Los estudiantes reciben bastantes mensajes de la plataforma, y creo que es práctico que reciban un asunto que les informe sobre el contenido del mensaje y no que sean todos genéricos.

Jose Angel ha hecho una corrección para un cliente, la publicará aquí.

Actions #5

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 2.0 to 1.10.0

Bien Laura, que no demore mucho Angel entonces, porque espero publicar una alpha antes de irme a Chamilo Con

Actions #6

Updated by José Angel Ruiz (Nosolored) over 7 years ago

Yannick te envío la propuesta que hice.

https://github.com/chamilo/chamilo-lms/pull/757

Simplemente lo que hago es crear un tipo nuevo de notificación

const NOTIFICATION_TYPE_ANNOUNCEMENT = 5;

En la función que da formato al título consulto si es del tipo "NOTIFICATION_TYPE_ANNOUNCEMENT" y si es así mantiene el título del anuncio.

En el resto de las funciones simplemente lo que hago es pasar en los parámetros de la función el tipo del mensaje.

Actions #7

Updated by Julio Montoya over 7 years ago

José Angel Ruiz (Nosolored) wrote:

Yannick te envío la propuesta que hice.

https://github.com/chamilo/chamilo-lms/pull/757

Simplemente lo que hago es crear un tipo nuevo de notificación
[...]
En la función que da formato al título consulto si es del tipo "NOTIFICATION_TYPE_ANNOUNCEMENT" y si es así mantiene el título del anuncio.

En el resto de las funciones simplemente lo que hago es pasar en los parámetros de la función el tipo del mensaje.

En realidad debería de existir un nuevo tipo de mensaje que sea "directo". Es decir, sin el texto "tiene un nuevo mensaje de".

Agregar una opción en send_message(..., $directMessage = false) y luego en save_notification()
agregar una constante:

Notification::NOTIFICATION_TYPE_DIRECT_MESSAGE

De esta manera no se tendrá que usar solamente para anuncios si no para cualquier lugar en Chamilo.

Actions #8

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya

Estoy de acuerdo con el $direct_message (o $directMessage) para 1.10.x. Adelante.

Actions #9

Updated by Julio Montoya over 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)

Acabo de agregar este cambio.

Actions #10

Updated by Laura Guirao over 7 years ago

Acabo de probarlo en unstable.chamilo.org enviando un anuncio y llega correctamente el asunto, incluyendo el nombre del curso y el título del anuncio, genial.

No me ha llegado el anuncio por email aunque le he marcado la casilla de enviar por email, solo lo he podido ver en la bandeja de entrada de la plataforma. ¿Es esto normal al ser una plataforma de pruebas o es un error?

Actions #11

Updated by Julio Montoya over 7 years ago

Laura Guirao wrote:

Acabo de probarlo en unstable.chamilo.org enviando un anuncio y llega correctamente el asunto, incluyendo el nombre del curso y el título del anuncio, genial.

No me ha llegado el anuncio por email aunque le he marcado la casilla de enviar por email, solo lo he podido ver en la bandeja de entrada de la plataforma. ¿Es esto normal al ser una plataforma de pruebas o es un error?

Ayer recibí unas notificaciones a mi correo.

Actions #12

Updated by Laura Guirao over 7 years ago

Hola Julio,

A mi no me llegan por email los anuncios del curso (he probado en un curso llamado Test Julio) y tampoco aparece el icono del sobrecito junto a los anuncios enviados por mail como en anteriores versiones.
Hemos hecho varias pruebas con diferentes usuarios y cuentas de correo.

No entiendo bien lo que quieres decir cuando indicas "Ayer recibí unas notificaciones a mi correo."
¿Quieres decir que tienes una redirección en unstable.chamilo.org y te llegan a ti en lugar de a cada usuario? Si es así, se explicaría.

Actions #13

Updated by Julio Montoya over 7 years ago

Laura Guirao wrote:

Hola Julio,

A mi no me llegan por email los anuncios del curso (he probado en un curso llamado Test Julio) y tampoco aparece el icono del sobrecito junto a los anuncios enviados por mail como en anteriores versiones.
Hemos hecho varias pruebas con diferentes usuarios y cuentas de correo.

No entiendo bien lo que quieres decir cuando indicas "Ayer recibí unas notificaciones a mi correo."
¿Quieres decir que tienes una redirección en unstable.chamilo.org y te llegan a ti en lugar de a cada usuario? Si es así, se explicaría.

Unas preguntas:

Con qué usuario están probando?
Cuál es el curso y los usuarios a quienes se les envían los anuncios?

Acabo de recibir a mi correo:


John Doe to me

10:54 AM
Probando los cambios de Julio para el asunto de los mensajes de anuncios

1 Doe John
test julio

Probando los cambios de Julio para el asunto de los mensajes de anuncios

1 Doe John
test julio

See message
You have received this notification because you are subscribed or involved in it to change your notification preferences please click here: https://unstable.chamilo.org/main/auth/profile.php
Actions #14

Updated by Laura Guirao over 7 years ago

Con el usuario Nosolored Student, en el curso Test Julio
Y también ha probado Jose Angel con el usuario jaruiz nsr, en el curso prueba

Actions #15

Updated by Julio Montoya over 7 years ago

Cuando entro aqui:

https://unstable.chamilo.org/main/announcements/announcements.php?cidReq=TESTJULIO&id_session=0&gidReq=0&gradebook=0&origin=&action=modify&id=5

Yo recibo un correo a mi cuenta gmail. Revisar si el usuario "jaruiz" recibe los correos. Tal vez están en la carpeta SPAM?

Actions #16

Updated by Laura Guirao over 7 years ago

No recibimos nada, ni en cuentas de Nosolored ni en la mia de gmail que he probado a ponérsela al usuario. Tampoco en la bandeja de spam, es muy raro

Actions #17

Updated by Julio Montoya over 7 years ago

Creo que el problema está aqui:

https://unstable.chamilo.org/main/admin/user_edit.php?user_id=87

El usuario tiene que escoger una opción para:

Notify by mail on new invitation received 
Notify by mail on new personal message received 
Notify by mail on new message received in group 

Por defecto debería ser "Upon reception"

Actions #18

Updated by Laura Guirao over 7 years ago

Hola Julio,

Sí, acabo de verlo, y al modificar uno de mis usuarios sí me ha llegado el e-mail.
Por defecto al crear un usuario esos campos aparecen en blanco (Please select an option) y es como los teníamos.

En cualquier caso pienso que esto no debería influir en el envio de un anuncio. Si un profesor marca expresamente que el anuncio se envíe por e-mail, ¿no debería llegar a los usuarios independientemente de lo que tengan en esos campos de su perfil?

Actions #19

Updated by Julio Montoya over 7 years ago

Laura Guirao wrote:

Hola Julio,

Sí, acabo de verlo, y al modificar uno de mis usuarios sí me ha llegado el e-mail.
Por defecto al crear un usuario esos campos aparecen en blanco (Please select an option) y es como los teníamos.

Este es un bug, Por defecto al crear un usuario debería de aparecer "Upon reception". Ahora lo corrigo.

En cualquier caso pienso que esto no debería influir en el envio de un anuncio. Si un profesor marca expresamente que el anuncio se envíe por e-mail, ¿no debería llegar a los usuarios independientemente de lo que tengan en esos campos de su perfil?

Los emails/mensajes depende de esos campos del perfil del usuario.

Actions #20

Updated by Julio Montoya over 7 years ago

Acabo de agregar una correción.

Si el usuario no ha escogido ninguna opción se envia inmediatamente el mensaje/correo.
Por defecto cuando se crea un usuario está marcado como "Upon reception"

Actions #21

Updated by Laura Guirao over 7 years ago

De acuerdo

Entonces este bug del asunto de anuncios se puede cerrar.
Gracias.

Actions #22

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 0 to 100

Perfecto, gracias a todos

Actions

Also available in: Atom PDF