Comments for test results don't show up!
When I validate a test, the comments that I add for each of the students' answers (the "Edit individual feedback" field under each question) don't show up after saving my review. I don't see them after opening the feedback again, and neither does the student. I've lost quite a few comments, adding them in the certainty that they are being saved and students can see them, before I realized they were not there.
Updated by Torkil Zachariassen over 5 years ago
This might be related to #7448, where educators, in sessions, were able to comment on attempts, but the submit button failed. I had to modidify the code in order to make this work.
first we tried to allow coaches to edit in sessions, but had to turn this off, as this enabled coaches to erase central tests through the excerceses pool.
...torkil... still on 1.9.8 until. next week :-)
Updated by Yannick Warnier over 5 years ago
- Category set to Exercises
- Target version set to 1.10.0
Thanks for reporting. We'll try to verify that on 1.10.x. We don't support earlier versions for small bugs, but hopefully the patch we provide for this one in future versions can be applied to your version easily.
Updated by Torkil Zachariassen about 5 years ago
The comments worked fine in 1.9.8, but broke after an upgrade to 1.9.10 (Note: This is not against 1.10.0, but 1.9.x)
Testing various versions of exercise_show.php from
(exercise_show.php, exercise_show.php-1.9.8-works, exercise_show.php_73e38162, exercise_show.php_fc3508ec, exercise_show.php.000, exercise_show.php_470c12d0, exercise_show.php_9e5c73a8) provided some insight:
Thus the lastest version that works, is 470c12d0
The culprit was found to be 73e38162 ( BT#9247 https://support.chamilo.org/projects/chamilo-18/repository/revisions/73e381620438970423d371e717ad0bb8d5667a29 ), and I did not manage to fix this part.
Using 470c12d0 I managed to incorporate the changes from the latest version (9e5c73a8) by hand. So basicly we can use the latest version, but have to eliminate whatever happend in 73e38162.
The result is attached, and works for me as a workaround for now.
Updated by Julio Montoya about 5 years ago
- Assignee changed from Julio Montoya to Yannick Warnier
I just apply a fix to this issue.
Now it should work.