MDL-9873 questions Set question text field to required.
authorEric Merrill <merrill@oakland.edu>
Wed, 25 Sep 2013 13:07:04 +0000 (09:07 -0400)
committerEric Merrill <merrill@oakland.edu>
Wed, 25 Sep 2013 13:07:04 +0000 (09:07 -0400)
After much discussion, it was decided that the damage done by not
not setting question text (student goes to take test and sees nothing)
is more of a problem then the use cases where teachers want empty
questions (inputing paper tests). In those cases, they will need
to add something like a period to the question text field.

question/type/edit_question_form.php

index 80430b1..a186eba 100644 (file)
@@ -188,6 +188,7 @@ abstract class question_edit_form extends question_wizard_form {
         $mform->addElement('editor', 'questiontext', get_string('questiontext', 'question'),
                 array('rows' => 15), $this->editoroptions);
         $mform->setType('questiontext', PARAM_RAW);
+        $mform->addRule('questiontext', null, 'required', null, 'client');
 
         $mform->addElement('text', 'defaultmark', get_string('defaultmark', 'question'),
                 array('size' => 7));