MDL-6353 Quiz questions per page setting is confusing. People expect that changing this setting will repaginate the quiz, which is not the case, so change the wording of this setting and its choices.

This commit is contained in:
tjhunt 2008-08-22 02:13:00 +00:00
parent 44b78d2fec
commit 4c07690efc
4 changed files with 22 additions and 14 deletions

View File

@ -1,4 +1,7 @@
<h1>Questions per page</h1>
<h1>Automatically start a new page</h1>
<p>For longer quizzes it makes sense to stretch the quiz over several pages by limiting the number of questions per page. When adding questions to the quiz page breaks will automatically be inserted according to the setting you choose here. However you will also be able to move page breaks around by hand later on the editing page.
</p>
<p>For longer quizzes it makes sense to stretch the quiz over several pages by
limiting the number of questions per page. When adding questions to the quiz
page breaks will automatically be inserted according to the setting you choose
here. However you will also be able to move page breaks around by hand later on
the editing page.</p>

View File

@ -32,6 +32,8 @@ $string['addrandom2'] = 'random questions';
$string['addselectedtoquiz'] = 'Add selected to quiz';
$string['addtoquiz'] = 'Add to quiz';
$string['affectedstudents'] = 'Affected $a';
$string['aftereachquestion'] = 'After adding each question';
$string['afternquestions'] = 'After adding $a questions';
$string['aiken'] = 'Aiken format';
$string['allattempts'] = 'All attempts';
$string['allinone'] = 'Unlimited';
@ -368,6 +370,7 @@ $string['multipleanswers'] = 'Choose at least one answer.';
$string['multiplier'] = 'Multiplier';
$string['name'] = 'Name';
$string['newattemptfail'] = 'Error: Could not start a new attempt at the quiz';
$string['newpageevery'] = 'Automatically start a new page';
$string['noanswers'] = 'No answers were selected!';
$string['noattempts'] = 'No attempts have been made on this quiz';
$string['noattemptstoshow'] = 'There are no attempts to show';

View File

@ -139,17 +139,18 @@
</tr>
<tr valign="top">
<td align="right"><b><?php print_string('questionsperpage', 'quiz') ?>:</b></td>
<td align="right"><b><?php print_string('newpageevery', 'quiz') ?>:</b></td>
<td>
<?php
$perpage= array();
for ($i=0; $i<=50; ++$i) {
$perpage[$i] = $i;
$perpage = array();
$perpage[0] = get_string('never');
$perpage[1] = get_string('aftereachquestion', 'quiz');
for ($i = 2; $i <= 50; ++$i) {
$perpage[$i] = get_string('afternquestions', 'quiz', $i);
}
$perpage[0] = get_string('allinone', 'quiz');
choose_from_menu($perpage, 'questionsperpage', $form->questionsperpage, '');
helpbutton('questionsperpage', get_string('questionsperpage', 'quiz'), 'quiz');
helpbutton('questionsperpage', get_string('newpageevery', 'quiz'), 'quiz');
?>
</td>
<td align="center">

View File

@ -77,12 +77,13 @@ class mod_quiz_mod_form extends moodleform_mod {
//-------------------------------------------------------------------------------
$mform->addElement('header', 'displayhdr', get_string('display', 'form'));
$perpage = array();
for ($i = 0; $i <= 50; ++$i) {
$perpage[$i] = $i;
$perpage[0] = get_string('never');
$perpage[1] = get_string('aftereachquestion', 'quiz');
for ($i = 2; $i <= 50; ++$i) {
$perpage[$i] = get_string('afternquestions', 'quiz', $i);
}
$perpage[0] = get_string('allinone', 'quiz');
$mform->addElement('select', 'questionsperpage', get_string('questionsperpage', 'quiz'), $perpage);
$mform->setHelpButton('questionsperpage', array('questionsperpage', get_string('questionsperpage', 'quiz'), 'quiz'));
$mform->addElement('select', 'questionsperpage', get_string('newpageevery', 'quiz'), $perpage);
$mform->setHelpButton('questionsperpage', array('questionsperpage', get_string('newpageevery', 'quiz'), 'quiz'));
$mform->setAdvanced('questionsperpage', $CFG->quiz_fix_questionsperpage);
$mform->setDefault('questionsperpage', $CFG->quiz_questionsperpage);