When Feedback Time is set to After Each question - on the last question, no feedback is shown, but Quiz displays the quiz score page only.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | feedback_last_question_1298044_d6.patch | 1.43 KB | rballou |
| #7 | feedback_last_question-5682986-1298044.patch | 1.63 KB | ipwa |
| #6 | feedback_last_question-5682986-1298044.patch | 1.63 KB | ipwa |
| #2 | feedback_last_question-1298044-2.patch | 1.56 KB | guillaumev |
Comments
Comment #1
guillaumev commentedNote that this bug also appears in the 7.x version...
Comment #2
guillaumev commentedHere is a patch fixing the issue in version 7.x-4.x. It hasn't been tested very well and I haven't seen the code of 6.x-4.2 but I guess/hope it will be easy to adapt...
Comment #3
falcon commentedThe feedback system will be reworked for quiz 5 and we will fix this problem then.
Comment #4
sivaji_ganesh_jojodae commentedJust added a workaround code to quiz_take_quiz() to fix this. Seems to work okay for me.
Comment #6
ipwa commented@falcon its great that the feedback system will be reworked.
However this is a bug and it should be fixed on 7.x-4.x since its the current release and a lot of people are using the module with this bug. If a fix gets committed you will have less support issue requests related to feedback :)
Attempting to re-roll patch since it didn't apply cleanly.
Comment #7
ipwa commented@falcon its great that the feedback system will be reworked.
However this is a bug and it should be fixed on 7.x-4.x since its the current release and a lot of people are using the module with this bug. If a fix gets committed you will have less support issue requests related to feedback :)
Attempting to re-roll patch since it didn't apply cleanly.
Comment #8
ipwa commentedI can confirm this works well.
The patch applies cleanly in dev and it applies in the alpha12 release although you get a couple of offset warnings.
Comment #9
rballou commentedThis used to be related to a 6.x release and the patch above is for 7.x. I have created the following patch for 6.x which also seems to work:
Comment #10
mcpuddin commentedThis is in 7.x-4.x-dev already (comment #4). I'm closing out this issue. If you a d6 patch, I'd say open it up in a different issue.