Index: temp_moodle_dev/moodle/mod/quiz/attempt.php =================================================================== RCS file: /usr/local/cvsroot/temp_moodle_dev/moodle/mod/quiz/attempt.php,v diff -u -r1.1 -r1.2 --- temp_moodle_dev/moodle/mod/quiz/attempt.php 14 Nov 2008 05:57:17 -0000 1.1 +++ temp_moodle_dev/moodle/mod/quiz/attempt.php 14 Nov 2008 06:16:59 -0000 1.2 @@ -60,8 +60,8 @@ $ispreviewing = has_capability('mod/quiz:preview', $context); // if no questions have been set up yet redirect to edit.php - if (!$quiz->questions and has_capability('mod/quiz:manage', $context)) { - redirect($CFG->wwwroot . '/mod/quiz/edit.php?cmid=' . $cm->id); + if (!$quiz->questions and has_capability('mod/quiz:manage', $context)) { + redirect($CFG->wwwroot . '/mod/quiz/edit.php?cmid=' . $cm->id); } if (!$ispreviewing) { @@ -85,13 +85,14 @@ /// Check number of attempts $numberofpreviousattempts = count_records_select('quiz_attempts', "quiz = '{$quiz->id}' AND " . "userid = '{$USER->id}' AND timefinish > 0 AND preview != 1"); + if ($quiz->attempts and $numberofpreviousattempts >= $quiz->attempts) { - print_error('nomoreattempts', 'quiz', "view.php?id={$cm->id}"); + print_error('nomoreattempts', 'quiz', "view.php?id={$cm->id}"); } /// Check subnet access if (!$ispreviewing && $quiz->subnet && !address_in_subnet(getremoteaddr(), $quiz->subnet)) { - print_error("subneterror", "quiz", "view.php?id=$cm->id"); + print_error('subneterror', 'quiz', "view.php?id={$cm->id}"); } /// Check password access @@ -102,15 +103,13 @@ if ($quiz->password and empty($SESSION->passwordcheckedquizzes[$quiz->id])) { $enteredpassword = optional_param('quizpassword', '', PARAM_RAW); if (optional_param('cancelpassword', false)) { - // User clicked cancel in the password form. - redirect($CFG->wwwroot . '/mod/quiz/view.php?q=' . $quiz->id); + redirect($CFG->wwwroot . '/mod/quiz/view.php?q=' . $quiz->id); } else if (strcmp($quiz->password, $enteredpassword) === 0) { // User entered the correct password. $SESSION->passwordcheckedquizzes[$quiz->id] = true; } else { - // User entered the wrong password, or has not entered one yet. - $url = $CFG->wwwroot . '/mod/quiz/attempt.php?q=' . $quiz->id; - + + $url = $CFG->wwwroot . '/mod/quiz/attempt.php?q=' . $quiz->id; if (empty($popup)) { print_header('', '', '', 'quizpassword'); } @@ -123,6 +122,7 @@ if (!empty($enteredpassword)) { echo '

', get_string('passworderror', 'quiz'), '

'; } + ?>

@@ -136,7 +136,8 @@ is_lams); } exit; } @@ -156,11 +157,11 @@ } if ($numattempts == 1 && $quiz->delay1) { if ($timenow - $quiz->delay1 < $lastattempt) { - print_error('timedelay', 'quiz', 'view.php?q='.$quiz->id); + print_error('timedelay', 'quiz', 'view.php?q='.$quiz->id); } } else if($numattempts > 1 && $quiz->delay2) { if ($timenow - $quiz->delay2 < $lastattempt) { - print_error('timedelay', 'quiz', 'view.php?q='.$quiz->id); + print_error('timedelay', 'quiz', 'view.php?q='.$quiz->id); } } } @@ -197,19 +198,22 @@ } // make log entries if ($ispreviewing) { + add_to_log($course->id, 'quiz', 'preview', - "attempt.php?id=$cm->id", + 'attempt.php?id='.$cm->id, "$quiz->id", $cm->id); } else { + add_to_log($course->id, 'quiz', 'attempt', - "review.php?attempt=$attempt->id", + 'review.php?attempt='.$attempt->id, "$quiz->id", $cm->id); } } else { // log continuation of attempt only if some time has lapsed - if (($timestamp - $attempt->timemodified) > 600) { // 10 minutes have elapsed + if (($timestamp - $attempt->timemodified) > 600) { // 10 minutes have elapsed + $address5='review.php?attempt='.$attempt->id; add_to_log($course->id, 'quiz', 'continue attemp', // this action used to be called 'continue attempt' but the database field has only 15 characters - "review.php?attempt=$attempt->id", + $address5, "$quiz->id", $cm->id); } } @@ -235,7 +239,7 @@ } if (!$questionlist) { - print_error('noquestionsfound', 'quiz', 'view.php?q='.$quiz->id); + print_error('noquestionsfound', 'quiz', 'view.php?q='.$quiz->id); } $sql = "SELECT q.*, i.grade AS maxgrade, i.id AS instance". @@ -246,7 +250,7 @@ // Load the questions if (!$questions = get_records_sql($sql)) { - print_error('noquestionsfound', 'quiz', 'view.php?q='.$quiz->id); + print_error('noquestionsfound', 'quiz', 'view.php?q='.$quiz->id); } // Load the question type specific information @@ -322,8 +326,9 @@ if ($page) { $pagebit = '&page=' . $page; } + $address6=$CFG->wwwroot . '/mod/quiz/attempt.php?q=' . $quiz->id . $pagebit; print_error('errorprocessingresponses', 'question', - $CFG->wwwroot . '/mod/quiz/attempt.php?q=' . $quiz->id . $pagebit); + $address6); } $attempt->timemodified = $timestamp; @@ -379,9 +384,8 @@ print_error('errorprocessingresponses', 'question', $CFG->wwwroot . '/mod/quiz/attempt.php?q=' . $quiz->id . $pagebit); } - add_to_log($course->id, 'quiz', 'close attempt', - "review.php?attempt=$attempt->id", + 'review.php?attempt='.$attempt->id, "$quiz->id", $cm->id); } @@ -404,12 +408,12 @@ if (!empty($SESSION->passwordcheckedquizzes[$quiz->id])) { unset($SESSION->passwordcheckedquizzes[$quiz->id]); } - redirect($CFG->wwwroot . '/mod/quiz/review.php?attempt='.$attempt->id, 0); + redirect($CFG->wwwroot . '/mod/quiz/review.php?attempt='.$attempt->id, 0); } // Now is the right time to check the open and close times. if (!$ispreviewing && ($timestamp < $quiz->timeopen || ($quiz->timeclose && $timestamp > $quiz->timeclose))) { - print_error('notavailable', 'quiz', "view.php?id={$cm->id}"); + print_error('notavailable', 'quiz', "view.php?id={$cm->id}"); } /// Print the quiz page //////////////////////////////////////////////////////// @@ -427,7 +431,8 @@ ? update_module_button($cm->id, $course->id, get_string('modulename', 'quiz')) : ""; $navigation = build_navigation($strattemptnum, $cm); - print_header_simple(format_string($quiz->name), "", $navigation, "", $headtags, true, $strupdatemodule); + //we pass a new parameter to the function so it won't we printed if is_lams=1 + print_header_simple(format_string($quiz->name), "", $navigation, "", $headtags, true, $strupdatemodule,'',false,'',false,$quiz->is_lams); } echo ''; // for overlib @@ -463,8 +468,7 @@ } // Start the form - echo '', "\n"; if($quiz->timelimit > 0) { // Make sure javascript is enabled for time limited quizzes @@ -506,7 +510,7 @@ if ($quiz->optionflags & QUESTION_ADAPTIVE) { echo "\n"; } - echo "\n"; + echo "\n"; echo ""; @@ -544,6 +548,7 @@ // Finish the page if (empty($popup)) { - print_footer($course); + //we pass a new parameter to the function so it won't we printed if is_lams=1 + print_footer($course,null, false,$cm->is_lams); } ?>