From: mjollnir_ <mjollnir_>
Date: Tue, 18 Apr 2006 22:57:28 +0000 (+0000)
Subject: Avoiding warning in last commit:
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=eb55a1bfb9939ad6fc956b4b6d6358e7c2e00eeb;p=moodle.git

Avoiding warning in last commit:
mod/quiz/index changes to only display quiz grades when teacher allows
Changes to index.php to only allow student to view overall quiz scores when
teacher has not set options forbidding this, and it can't give away what
scores were for attempts where results should still be hidden.
Credit: Peter Bulmer peterbulmer@catalyst.net.nz
---

diff --git a/mod/quiz/locallib.php b/mod/quiz/locallib.php
index afd7670858..439aa899e2 100644
--- a/mod/quiz/locallib.php
+++ b/mod/quiz/locallib.php
@@ -565,13 +565,14 @@ function all_attempt_results_visible($quiz, $user) {
     $sql = 'SELECT timefinish, preview FROM '.$CFG->prefix.'quiz_attempts qa'.
         ' WHERE qa.quiz='.$quiz->id.' AND qa.userid='.$user->id.
         ' ORDER BY id DESC';
-    $attempts = get_records_sql($sql);
-    foreach ($attempts as $attempt) {            
-        $attemptoptions = quiz_get_reviewoptions($quiz, $attempt);
-        //if any attempt has scores option not set, not all attempt results are
-        //visible
-        if (!$attemptoptions->scores) {
-           return false;
+    if ($attempts = get_records_sql($sql)) {
+        foreach ($attempts as $attempt) {            
+            $attemptoptions = quiz_get_reviewoptions($quiz, $attempt);
+            //if any attempt has scores option not set, not all attempt results are
+            //visible
+            if (!$attemptoptions->scores) {
+                return false;
+            }
         }
     }
     return true;