Scoring system is buggy (and bonus URL reflections)
| Project: | Quiz |
| Version: | 6.x-4.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | falcon |
| Status: | active |
The new scoring system for long and short answer is buggy, see the two first attached screen shots first showing a long answer then a short one.
It also gives these two error messages:
* 0 cannot be longer than 3 characters but is currently 13 characters long.
* The given score (4) is higher than the maximum allowed score (2)
There's also an issue with different values being reported as total score on different pages, much easier to undestand by viewing the third screen shot :)
Additional URL thoughts..
Don't want to nitpick but at the same time I suspect this is very easy to adjust, and that now could be a good time to do so, so I just toss it out here!
When you're at admin -> quiz -> reports -> results (to see a list over quizes) the URL is:
/admin/quiz/reports/results which makes perfectly sense.
However, if you go further on to see the results on one quiz from that page, that is admin -> quiz -> reports -> results -> [a specific quiz] the URL is:
/admin/quiz/30/view where I believe it would be natural to have it at
/admin/quiz/reports/results/30/
..and if you go further from that page, to see a specific result from a student that URL is:
/admin/quiz/reports/43/results where I believe it would be natural to have it at
/admin/quiz/reports/results/30/43
..I'm quite sure my suggestions here follow general best practices for URL / path, which most Drupal modules follow, and that quiz should too! :)
(note that 30 here is the quiz id, and 43 here is the specific result id)
| Attachment | Size |
|---|---|
| short-answer.png | 22.87 KB |
| long-answer.png | 67.47 KB |
| different-scores.png | 194.05 KB |
#1
#2
I have tried to solve the scoring issues. Please test it.
I agree with you on the urls as well, but the current url structure is old, and we might break existing pages by changing it. We could make the old urls redirect to the new, but I'm not sure if we should do that?
#3
Sorry, but there are still issues here, will poke further down into it to maybe figure out exactly what the issue(s) are.
The long and short answer decimal problem (two first screenshots in original bug report) are still there (see new attachment).
The issue with different values reported on total score is kind of solved. I believe it now shows the same values, but the values are not always correct :)