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 about 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 about 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 almost 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 Yannick Warnier almost 5 years ago
- Assignee changed from Yannick Warnier to Julio Montoya
- % Done changed from 100 to 50
Julio, could you review this? It has to do with your changes to the max number of fckeditors, but note that this is only for 1.9.x (so don't send anything to 1.10.x)
Updated by Julio Montoya almost 5 years ago
- Assignee changed from Julio Montoya to Yannick Warnier
I just apply a fix to this issue.
Now it should work.