From: dongsheng <dongsheng>
Date: Tue, 10 Jun 2008 06:27:38 +0000 (+0000)
Subject: "MDL-14129, fix print_error"
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=550f965438903d512f5342afe0b164657bd06679;p=moodle.git

"MDL-14129, fix print_error"
---

diff --git a/mod/quiz/attempt.php b/mod/quiz/attempt.php
index 307e20af77..e2784245bb 100644
--- a/mod/quiz/attempt.php
+++ b/mod/quiz/attempt.php
@@ -28,23 +28,23 @@
 
     if ($id) {
         if (! $cm = get_coursemodule_from_id('quiz', $id)) {
-            print_error("There is no coursemodule with id $id");
+            print_error('invalidcoursemodule');
         }
         if (! $course = $DB->get_record('course', array('id' => $cm->course))) {
-            print_error("Course is misconfigured");
+            print_error("coursemisconf");
         }
         if (! $quiz = $DB->get_record('quiz', array('id' => $cm->instance))) {
-            print_error("The quiz with id $cm->instance corresponding to this coursemodule $id is missing");
+            print_error('invalidcoursemodule');
         }
     } else {
         if (! $quiz = $DB->get_record('quiz', array('id' => $q))) {
-            print_error("There is no quiz with id $q");
+            print_error('invalidcoursemodule');
         }
         if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-            print_error("The course with id $quiz->course that the quiz with id $q belongs to is missing");
+            print_error('invalidcourseid');
         }
         if (! $cm = get_coursemodule_from_instance("quiz", $quiz->id, $course->id)) {
-            print_error("The course module for the quiz with id $q is missing");
+            print_error('invalidcoursemodule');
         }
     }
 
@@ -187,7 +187,7 @@
 
 /// Restore the question sessions to their most recent states creating new sessions where required.
     if (!$states = get_question_states($questions, $quiz, $attempt, $lastattemptid)) {
-        print_error('Could not restore question sessions');
+        print_error('cannotrestore', 'quiz');
     }
 
 /// If we are starting a new attempt, save all the newly created states.
diff --git a/mod/quiz/comment.php b/mod/quiz/comment.php
index 535995d610..b800d7b5fe 100644
--- a/mod/quiz/comment.php
+++ b/mod/quiz/comment.php
@@ -13,18 +13,18 @@
     $questionid =required_param('question', PARAM_INT); // question id
 
     if (! $attempt = $DB->get_record('quiz_attempts', array('uniqueid' => $attemptid))) {
-        print_error('No such attempt ID exists');
+        print_error('invalidattemptid', 'quiz');
     }
     if (! $quiz = $DB->get_record('quiz', array('id' => $attempt->quiz))) {
-        print_error('Course module is incorrect');
+        print_error('invalidcoursemodule');
     }
     if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-        print_error('Course is misconfigured');
+        print_error('coursemisconf');
     }
 
     // Teachers are only allowed to comment and grade on closed attempts
     if (!($attempt->timefinish > 0)) {
-        print_error('Attempt has not closed yet');
+        print_error('attemptclosed', 'quiz');
     }
 
     $cm = get_coursemodule_from_instance('quiz', $quiz->id);
@@ -36,7 +36,7 @@
 
     // Load question
     if (! $question = $DB->get_record('question', array('id' => $questionid))) {
-        print_error('Question for this session is missing');
+        print_error('questionmissing', 'quiz');
     }
     $question->maxgrade = $DB->get_field('quiz_question_instances', 'grade', array('quiz' => $quiz->id, 'question' => $question->id));
     // Some of the questions code is optimised to work with several questions
@@ -45,7 +45,7 @@
     $questions[$key] = &$question;
     // Add additional questiontype specific information to the question objects.
     if (!get_question_options($questions)) {
-        print_error("Unable to load questiontype specific question information");
+        print_error('cannotloadtypeinfo', 'quiz');
     }
 
     // Load state
diff --git a/mod/quiz/editlib.php b/mod/quiz/editlib.php
index 22812863da..aeee04b52e 100644
--- a/mod/quiz/editlib.php
+++ b/mod/quiz/editlib.php
@@ -44,7 +44,7 @@ function quiz_delete_quiz_question($id, &$quiz) {
     $quiz->questions = str_replace(',0,0', ',0', $quiz->questions);
     // save new questionlist in database
     if (!$DB->set_field('quiz', 'questions', $quiz->questions, array('id' => $quiz->instance))) {
-        print_error('Could not save question list');
+        print_error('cannotsavequestion', 'quiz');
     }
     $DB->delete_records('quiz_question_instances', array('quiz' => $quiz->instance, 'question', $question));
     return true;
@@ -88,7 +88,7 @@ function quiz_add_quiz_question($id, &$quiz) {
     // Save new questionslist in database
     $quiz->questions = implode(",", $questions);
     if (!$DB->set_field('quiz', 'questions', $quiz->questions, array('id' => $quiz->id))) {
-        print_error('Could not save question list');
+        print_error('cannotsavequestion', 'quiz');
     }
 
     // update question grades
diff --git a/mod/quiz/grade.php b/mod/quiz/grade.php
index e2751a1f98..4eba8d71dc 100644
--- a/mod/quiz/grade.php
+++ b/mod/quiz/grade.php
@@ -5,15 +5,15 @@
     $id   = required_param('id', PARAM_INT);          // Course module ID
 
     if (! $cm = get_coursemodule_from_id('quiz', $id)) {
-        print_error('Course Module ID was incorrect');
+        print_error('invalidcoursemodule');
     }
 
     if (! $quiz = $DB->get_record('quiz', array('id' => $cm->instance))) {
-        print_error('quiz ID was incorrect');
+        print_error('invalidquizid');
     }
 
     if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-        print_error('Course is misconfigured');
+        print_error('coursemisconf');
     }
 
     require_login($course->id, false, $cm);
diff --git a/mod/quiz/index.php b/mod/quiz/index.php
index 7bafe05f1f..70174227c1 100644
--- a/mod/quiz/index.php
+++ b/mod/quiz/index.php
@@ -11,7 +11,7 @@
 
     $id = required_param('id', PARAM_INT);
     if (!$course = $DB->get_record('course', array('id' => $id))) {
-        print_error("Course ID is incorrect");
+        print_error('invalidcourseid');
     }
     $coursecontext = get_context_instance(CONTEXT_COURSE, $id);
     require_login($course->id);
diff --git a/mod/quiz/pagelib.php b/mod/quiz/pagelib.php
index 698dd34187..fc2c59e372 100644
--- a/mod/quiz/pagelib.php
+++ b/mod/quiz/pagelib.php
@@ -20,7 +20,7 @@ class page_quiz extends page_generic_activity {
 
     function init_quick($data) {
         if(empty($data->pageid)) {
-            print_error('Cannot quickly initialize page: empty course id');
+            print_error('cannotinitpage', '', '', null);
         }
         $this->activityname = 'quiz';
         parent::init_quick($data);
diff --git a/mod/quiz/quizfile.php b/mod/quiz/quizfile.php
index 1296c75fa5..03c0188802 100644
--- a/mod/quiz/quizfile.php
+++ b/mod/quiz/quizfile.php
@@ -20,13 +20,13 @@
     $relativepath = get_file_argument('quizfile.php');
 
     if (!$relativepath) {
-        print_error('No valid arguments supplied or incorrect server configuration');
+        print_error('invalidargorconf');
     }
 
     // extract relative path components
     $args = explode('/', trim($relativepath, '/'));
     if (count($args) < 3) { // always at least category, question and path
-        print_error('No valid arguments supplied');
+        print_error('invalidarguments');
     }
 
     $quizid       = (int)array_shift($args);
@@ -34,11 +34,11 @@
     $relativepath = implode ('/', $args);
 
     if (!($question = $DB->get_record('question', array('id' => $questionid)))) {
-        print_error('No valid arguments supplied');
+        print_error('invalidarguments');
     }
 
     if (!($questioncategory = $DB->get_record('question_categories', array('id' => $question->category)))) {
-        print_error('No valid arguments supplied');
+        print_error('invalidarguments');
     }
 
     /////////////////////////////////////
@@ -48,7 +48,7 @@
         if ($questioncategory->publish) {
             require_login();
             if (!isteacherinanycourse()) {
-              print_error('No valid arguments supplied');
+                print_error('invalidarguments');
             }
         } else {
             require_login($questioncategory->course);
@@ -57,10 +57,10 @@
         }
     } else {
         if (!($quiz = $DB->get_record('quiz', array('id' => $quizid)))) {
-            print_error('No valid arguments supplied');
+            print_error('invalidarguments');
         }
         if (!($course = $DB->get_record('course', array('id' => $quiz->course)))) {
-            print_error('No valid arguments supplied');
+            print_error('invalidarguments');
         }
         require_login($course->id);
 
@@ -71,7 +71,7 @@
         //    and ! ($quiz->review  &&  time() > $quiz->timeclose)
         //        || !quiz_get_user_attempts($quiz->id, $USER->id) )
         //{
-        //    print_error("Logged-in user is not allowed to view this quiz");
+        //    print_error('noview', 'quiz');
         //}
 
         ///////////////////////////////////////////////////
@@ -82,14 +82,14 @@
         // For now, let's not worry about this.  The following check doesn't
         // work for randomly selected questions and it gets complicated
         //if (!in_array($question->id, explode(',', $quiz->questions), FALSE)) {
-        //    print_error("Specified question is not on the specified quiz");
+        //    print_error('specificquestionnotonquiz', 'quiz');
         //}
     }
 
     // Have the question check whether it uses this file or not
     if (!$QTYPES[$question->qtype]->uses_quizfile($question,
                                                        $relativepath)) {
-        print_error("The specified file path is not on the specified question");
+        print_error('specificapathnotonquestion', 'quiz');
     }
 
 
diff --git a/mod/quiz/report.php b/mod/quiz/report.php
index b7f30e66eb..e53d68865f 100644
--- a/mod/quiz/report.php
+++ b/mod/quiz/report.php
@@ -13,26 +13,26 @@
 
     if ($id) {
         if (! $cm = get_coursemodule_from_id('quiz', $id)) {
-            print_error("There is no coursemodule with id $id");
+            print_error('invalidcoursemodule');
         }
 
         if (! $course = $DB->get_record('course', array('id' => $cm->course))) {
-            print_error("Course is misconfigured");
+            print_error('coursemisconf');
         }
 
         if (! $quiz = $DB->get_record('quiz', array('id' => $cm->instance))) {
-            print_error("The quiz with id $cm->instance corresponding to this coursemodule $id is missing");
+            print_error('invalidcoursemodule');
         }
 
     } else {
         if (! $quiz = $DB->get_record('quiz', array('id' => $q))) {
-            print_error("There is no quiz with id $q");
+            print_error('invalidquizid', 'quiz');
         }
         if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-            print_error("The course with id $quiz->course that the quiz with id $q belongs to is missing");
+            print_error('invalidcourseid');
         }
         if (! $cm = get_coursemodule_from_instance("quiz", $quiz->id, $course->id)) {
-            print_error("The course module for the quiz with id $q is missing");
+            print_error('invalidcoursemodule');
         }
     }
 
@@ -62,7 +62,7 @@
     $mode = clean_param($mode, PARAM_SAFEDIR);
 
     if (! is_readable("report/$mode/report.php")) {
-        print_error("Report not known ($mode)");
+        print_error('reportnotfound', 'quiz', '', $mode);
     }
 
     include("report/default.php");  // Parent class
@@ -71,7 +71,7 @@
     $report = new quiz_report();
 
     if (! $report->display($quiz, $cm, $course)) {             // Run the report!
-        print_error("Error occurred during pre-processing!");
+        print_error("preprocesserror", 'quiz');
     }
 
 /// Print footer
diff --git a/mod/quiz/review.php b/mod/quiz/review.php
index 185706f9ee..91b8216f94 100644
--- a/mod/quiz/review.php
+++ b/mod/quiz/review.php
@@ -18,16 +18,16 @@
     $showall = optional_param('showall', 0, PARAM_BOOL);
 
     if (!$attempt = quiz_load_attempt($attemptid)) {
-        print_error("No such attempt ID exists");
+        print_error('invalidattemptid', 'quiz');
     }
     if (! $quiz = $DB->get_record('quiz', array('id' => $attempt->quiz))) {
-        print_error("The quiz with id $attempt->quiz belonging to attempt $attempt is missing");
+        print_error('invalidquizid', 'quiz');
     }
     if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-        print_error("The course with id $quiz->course that the quiz with id $quiz->id belongs to is missing");
+        print_error("invalidcoursemodule");
     }
     if (! $cm = get_coursemodule_from_instance("quiz", $quiz->id, $course->id)) {
-        print_error("The course module for the quiz with id $quiz->id is missing");
+        print_error("invalidcoursemodule");
     }
 
 /// Check login and get contexts.
@@ -81,7 +81,7 @@
 
 /// Restore the question sessions to their most recent states creating new sessions where required.
     if (!$states = get_question_states($questions, $quiz, $attempt)) {
-        print_error('Could not restore question sessions');
+        print_error('cannotrestore', 'quiz');
     }
 
 /// Work out appropriate title.
diff --git a/mod/quiz/reviewquestion.php b/mod/quiz/reviewquestion.php
index 4ba52588d6..bf0342fc2b 100644
--- a/mod/quiz/reviewquestion.php
+++ b/mod/quiz/reviewquestion.php
@@ -21,39 +21,39 @@
 
     if ($stateid) {
         if (! $state = $DB->get_record('question_states', array('id' => $stateid))) {
-            print_error('Invalid state id');
+            print_error('invalidstateid', 'quiz');
         }
         if (! $attempt = $DB->get_record('quiz_attempts', array('uniqueid' => $state->attempt))) {
-            print_error('No such attempt ID exists');
+            print_error('invalidattemptid', 'quiz');
         }
     } elseif ($attemptid) {
         if (! $attempt = $DB->get_record('quiz_attempts', array('id' => $attemptid))) {
-            print_error('No such attempt ID exists');
+            print_error('invalidattemptid', 'quiz');
         }
         if (! $neweststateid = $DB->get_field('question_sessions', 'newest', array('attemptid' => $attempt->uniqueid, 'questionid' => $questionid))) {
             // newest_state not set, probably because this is an old attempt from the old quiz module code
             if (! $state = $DB->get_record('question_states', array('question' => $questionid, 'attempt' => $attempt->uniqueid))) {
-                print_error('Invalid question id');
+                print_error('invalidquestionid', 'quiz');
             }
         } else {
             if (! $state = $DB->get_record('question_states', array('id' => $neweststateid))) {
-                print_error('Invalid state id');
+                print_error('invalidstateid', 'quiz');
             }
         }
     } else {
-        print_error('Parameter missing');
+        print_error('missingparameter');
     }
     if (! $question = $DB->get_record('question', array('id' => $state->question))) {
-        print_error('Question for this state is missing');
+        print_error('questionmissing', 'quiz');
     }
     if (! $quiz = $DB->get_record('quiz', array('id' => $attempt->quiz))) {
-        print_error('Course module is incorrect');
+        print_error('invalidcoursemodule');
     }
     if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-        print_error('Course is misconfigured');
+        print_error('coursemisconf');
     }
     if (! $cm = get_coursemodule_from_instance('quiz', $quiz->id, $course->id)) {
-        print_error('Course Module ID was incorrect');
+        print_error('invalidcoursemodule');
     }
 
     require_login($course->id, false, $cm);
@@ -77,7 +77,7 @@
             }
         }
         if ($attempt->userid != $USER->id) {
-            print_error('This is not your attempt!');
+            print_error('notyourattempt', 'quiz');
         }
     }
 
@@ -101,7 +101,7 @@
     $questions[$key] = &$question;
     // Add additional questiontype specific information to the question objects.
     if (!get_question_options($questions)) {
-        print_error("Unable to load questiontype specific question information");
+        print_error('cannotloadtypeinfo', 'quiz');
     }
 
     $session = $DB->get_record('question_sessions', array('attemptid' => $attempt->uniqueid, 'questionid' => $question->id));
diff --git a/mod/quiz/tabs.php b/mod/quiz/tabs.php
index 096f61c6ba..4b72032f16 100644
--- a/mod/quiz/tabs.php
+++ b/mod/quiz/tabs.php
@@ -8,7 +8,7 @@
  */
 
 if (empty($quiz)) {
-    print_error('You cannot call this script in that way');
+    print_error('cannotcallscript');
 }
 if (!isset($currenttab)) {
     $currenttab = '';
diff --git a/mod/quiz/view.php b/mod/quiz/view.php
index 87a6176723..c1261e2c83 100644
--- a/mod/quiz/view.php
+++ b/mod/quiz/view.php
@@ -13,23 +13,23 @@
 
     if ($id) {
         if (! $cm = get_coursemodule_from_id('quiz', $id)) {
-            print_error("There is no coursemodule with id $id");
+            print_error('invalidcoursemodule');
         }
         if (! $course = $DB->get_record('course', array('id' => $cm->course))) {
-            print_error("Course is misconfigured");
+            print_error('coursemisconf');
         }
         if (! $quiz = $DB->get_record('quiz', array('id' => $cm->instance))) {
-            print_error("The quiz with id $cm->instance corresponding to this coursemodule $id is missing");
+            print_error('invalidcoursemodule');
         }
     } else {
         if (! $quiz = $DB->get_record('quiz', array('id' => $q))) {
-            print_error("There is no quiz with id $q");
+            print_error('invalidquizid', 'quiz');
         }
         if (! $course = $DB->get_record('course', array('id' => $quiz->course))) {
-            print_error("The course with id $quiz->course that the quiz with id $q belongs to is missing");
+            print_error('invalidcourseid');
         }
         if (! $cm = get_coursemodule_from_instance("quiz", $quiz->id, $course->id)) {
-            print_error("The course module for the quiz with id $q is missing");
+            print_error('invalidcoursemodule');
         }
     }