Project

General

Profile

Bug #5756

Pas de notification envoyée pour l'expression orale

Added by Anonymous almost 9 years ago. Updated almost 9 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
Yoselyn Castillo
Category:
Exercises
Target version:
Start date:
23/11/2012
Due date:
% Done:

100%

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

Description

Quand un apprenant soumet un test d'expression orale, le responsable du cours n'est pas averti par un mail + message sur la plateforme comme c'est le cas pour les questions ouvertes.


Files

question.class.php (59.4 KB) question.class.php Yoselyn Castillo, 24/12/2012 14:21
chamilo-5756.png (17.1 KB) chamilo-5756.png Yannick Warnier, 24/12/2012 18:41
exercise.class.php (153 KB) exercise.class.php Yoselyn Castillo, 25/12/2012 15:40
exercise.lib.php (90.3 KB) exercise.lib.php Yoselyn Castillo, 25/12/2012 15:40
exercise.class.php (153 KB) exercise.class.php Yoselyn Castillo, 28/12/2012 18:16

Associated revisions

Revision 7386207f (diff)
Added by Yoselyn Castillo almost 9 years ago

Fixed review message for oral expression questions - refs #5756

Revision 64f2c5c6 (diff)
Added by Yoselyn Castillo almost 9 years ago

Fixing oral question email bug see #5756

Revision 309eb1af (diff)
Added by Yoselyn Castillo almost 9 years ago

Fixing oral question email bug see #5756

History

#1

Updated by Anonymous almost 9 years ago

De plus, le statut des questions d'expression orale après validation est "Faux" au lieu de "Non révisé".

#2

Updated by Yoselyn Castillo almost 9 years ago

  • Status changed from New to Assigned

verifying this

#3

Updated by Yoselyn Castillo almost 9 years ago

  • Assignee set to Yoselyn Castillo
#4

Updated by Yoselyn Castillo almost 9 years ago

Fixed message when question is not reviewed. Upload this.
Where i can change the setting "email_alert_manager_on_new_quiz" in 1.9.4 in order to test the changes for sending notification for questions.

#5

Updated by Yannick Warnier almost 9 years ago

I just sent your patch. You can change the email_alert_manager_on_new_quiz setting in the course settings (the "Configuración del curso" tool inside the course, in Spanish), in the "Notifications" part (see screenshot).

#6

Updated by Yannick Warnier almost 9 years ago

  • Assignee changed from Julio Montoya to Yoselyn Castillo
  • % Done changed from 0 to 50
#7

Updated by Yoselyn Castillo almost 9 years ago

I have made some fixes in order to send notifications for oral expression tests too. It is necessary to create the lang variables for 'OralQuestionsAttempted' and 'OralQuestionsAttemptedAre' for being used in the body of message.
These are the files

#8

Updated by Julio Montoya almost 9 years ago

  • Assignee changed from Yannick Warnier to Yoselyn Castillo

files were sent

#9

Updated by Yannick Warnier almost 9 years ago

  • % Done changed from 50 to 60

Variables added to translate.chamilo.org.

Yoselyn, cuando generas una nueva variable de idioma, si esta string contiene una parte... variable (como en este caso "Oral questions attempted are [parte variable]), tienes que pensar en otros idiomas en los cuales la gramática no es la misma y el objeto podría ser al inicio (como es el caso muchas veces en Japonés).

Por ello, la forma correcta de diseñar tu variable es así:
$OralQuestionsAttemptedAreX
y nosotros, al ingresarla a translate.chamilo.org para que sea traducida por todos, le ponemos en Inglés el valor:
"Oral questions attempted are %s"
y cuando lo usas en el código, lo usas así:
sprintf(get_lang('OralQuestionsAttemptedAre'),'[parte variable]');

Si no se hace así, genera imposibilidad de traducirlo correctamente en Japonés y otros idiomas del estilo (árabe, chino, etc), así que es muy importante.

En conclusión: por favor, modifica tu parche para que use este sprintf con OralQuestionsAttemptedAreX en lugar de OralQuestionsAttemptedAre directo, por favor. Yo lo puse así (con la X) en translate.chamilo.org.

#10

Updated by Yannick Warnier almost 9 years ago

Ah, y también... dame el texto en Español que quisiste darle con esta variable, para que podamos poner la base en el translate.chamilo.org.

#11

Updated by Yannick Warnier almost 9 years ago

Y también :-) prueba de evitar el código no-necesario bajo ciertas condiciones. En exercise.class.php::send_notification_for_oral_questions(), toda la concatenación de $msg que se hace antes del foreach y del if no es necesaria si nunca vas a entrar al "if", así que debería ser evitada (en programación, concatenaciones de strings son más costosas que operaciones lógicas o aritméticas simples).

#12

Updated by Yannick Warnier almost 9 years ago

  • % Done changed from 60 to 70

Aparte de esto, lo probé y me llegó el correo, así que estoy conforme (tan pronto como se arreglen estos detalles)

#13

Updated by Yoselyn Castillo almost 9 years ago

1. El texto seria:
Las preguntas Orales contestadas son:
The oral questions attempted are:
2. Changed way of use variable OralQuestionsAttemptedAreX
3. Code improved
Upload the patch

#14

Updated by Julio Montoya almost 9 years ago

Yoselyn Castillo wrote:

1. El texto seria:
Las preguntas Orales contestadas son:
The oral questions attempted are:
2. Changed way of use variable OralQuestionsAttemptedAreX
3. Code improved
Upload the patch

File was sent

#15

Updated by Julio Montoya almost 9 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from Yannick Warnier to Yoselyn Castillo
#16

Updated by Yannick Warnier almost 9 years ago

  • Status changed from Needs testing to Feature implemented
  • % Done changed from 70 to 100

Great! Tested and confirmed. Thanks and Happy New Year!

Also available in: Atom PDF