Issue Details (XML | Word | Printable)

Key: MDL-14054
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Pierre Pichet
Reporter: Matt Oquist
Votes: 0
Watchers: 3
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

CLONE -Cloze format not displaying the correct answers

Created: 26/Mar/08 04:50 AM   Updated: 11/Dec/08 05:22 AM
Return to search
Component/s: Quiz
Affects Version/s: None
Fix Version/s: 1.9.4, 2.0

File Attachments: 1. File cloze-correctionanswer-patches.tgz (1 kB)
2. File cloze-correctionanswer-patches.tgz (1 kB)

Image Attachments:

1. coorect.jpg
(16 kB)

2. correctandfeedback.jpg
(17 kB)

3. screenshot-1.jpg
(129 kB)
Environment: Linux
Issue Links:
Cloners
 
Dependency
 
Relates
 

Database: MySQL
Participants: Matt Oquist and Pierre Pichet
Security Level: None
Fixed Branches: MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
Cloze format is not displaying correct answers when that option is chosen in the quiz.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Matt Oquist added a comment - 26/Mar/08 04:53 AM - edited
Definitely still an issue in $release = '1.9 + (Build: 20080325)', and I stumbled upon the problem in our production server, which is $release = '1.8.3 +'. Students were submitting answers that looked right, but they were missing essential punctuation or something, and the teachers couldn't tell why Moodle was marking the answer wrong.

I have a patch that takes care of it, at least for shortanswer-type Cloze questions. I'll attach the patch in a few minutes.

I'm not really worried about this problem for any question types other than shortanswer, since that's all we're using in production. But the patch will also take care of NUMERICAL (though I haven't tested that, it's the same case). Multiple choice doesn't need it so much, because the teacher can probably figure out what Moodle was expecting even if it isn't specified.


Matt Oquist added a comment - 26/Mar/08 05:11 AM
This tarball includes patches for 1.6.6, 1.7.4, 1.8.4, and 1.9 build 20080325.

I've tested in 1.8.4 and 1.9, and the patches look correct for 1.6.6 and 1.7.4 on a casual visual inspection.


Matt Oquist added a comment - 26/Mar/08 05:27 AM - edited
Forgot to include the issue key last time; these patches note MDL-14054.

edit: It just occurred to me that I should've used cvs to produce these patches. I dunno why I didn't...anyhoo, the patches are for question/type/multianswer/questiontype.php, which is not obvious by looking at them.


Pierre Pichet added a comment - 26/Mar/08 09:22 AM
I will work on this as this should be synchronized with MDL-13419 which is already CVS to HEAD and 19_STABLE.

Pierre Pichet added a comment - 26/Mar/08 11:54 AM
Comparison of the proposed dispaly with mulchoice and match question types

Pierre Pichet added a comment - 26/Mar/08 11:59 AM
the proposed patch use
echo "<em><small>$correctanswers_str</small></em>";
to display. I think the styles options should be used and I will study this further unless somebody have a proposal.
Should we dispaly the selcect element like the the match question type even if in this case there is no correct answer available.
The in-line restriction for Cloze proscrib the multiple choice radio button style that allows correct answers display.

give your comments


Matt Oquist added a comment - 29/Mar/08 01:06 AM
You're right to worry about these styles being hard-coded. I did that just to visually distinguish the correct answer text from the surrounding question text. Perhaps this isn't even necessary, but in any case I'd hate to see this fix held up by a styles issue. I'd rather see the patch applied without the "<em>" and "<small>" tags, and styles applied later after we come up with an acceptable way to do it.

Pierre Pichet added a comment - 29/Mar/08 02:48 AM
As this is not a major bug but a useful feature, I prefer to do it correctly.
MDL-7494 also address the Cloze question display.
In the mean time users can apply locally your patch or put the correct answer in the feedback field...

Pierre Pichet added a comment - 15/May/08 11:22 PM
Use the popup to show the correct answer.
Here an example with feedback.

Pierre Pichet added a comment - 15/May/08 11:23 PM
Here an example with only correct answer.

Pierre Pichet added a comment - 15/May/08 11:27 PM
Using the popup does not disturb the actual display.
I am planning to offer some display options to cloze question.see MDL-14763.
Need to complete tests before CVS.

Pierre Pichet added a comment - 11/Dec/08 05:22 AM
Was fixed as MDL-14763 is fixed and closed