From: martin <martin>
Date: Sat, 27 Jul 2002 07:22:20 +0000 (+0000)
Subject: Fix to account for people not in course
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=96e0ad26b6149c4799835829ed9e1bb3f4cb1905;p=moodle.git

Fix to account for people not in course
---

diff --git a/mod/journal/lib.php b/mod/journal/lib.php
index 3cc7b8c1f1..c1fa1fffde 100644
--- a/mod/journal/lib.php
+++ b/mod/journal/lib.php
@@ -80,10 +80,10 @@ function journal_cron () {
 
     $cutofftime = time() - $CFG->maxeditingtime;
 
-    if ($entries = get_records_sql("SELECT e.*, j.course, j.name 
+    if ($entries = get_records_sql("SELECT e.*, j.course, j.name
                                      FROM   journal_entries e, journal j
-                                     WHERE  e.mailed = '0' AND 
-                                     e.timemarked < '$cutofftime' AND e.timemarked > 0
+                                     WHERE  e.mailed = '0' 
+                                     AND e.timemarked < '$cutofftime' AND e.timemarked > 0
                                      AND e.journal = j.id")) {
         $timenow = time();
 
@@ -96,16 +96,21 @@ function journal_cron () {
                 continue;
             }
 
-            if (! $teacher = get_record("user", "id", "$entry->teacher")) {
-                echo "Could not find teacher $entry->teacher\n";
+            if (! $course = get_record("course", "id", "$entry->course")) {
+                echo "Could not find course $entry->course\n";
                 continue;
             }
 
-            if (! $course = get_record("course", "id", "$entry->course")) {
-                echo "Could not find course $entry->course\n";
+            if (! isstudent($course->id, $user->id) and !isteacher($course->id, $user->id)) {
+                continue;  // Not an active participant
+            }
+
+            if (! $teacher = get_record("user", "id", "$entry->teacher")) {
+                echo "Could not find teacher $entry->teacher\n";
                 continue;
             }
 
+
             if (! $mod = get_coursemodule_from_instance("journal", $entry->journal, $course->id)) {
                 echo "Could not find course module for journal id $entry->journal\n";
                 continue;