From: rkingdon <rkingdon>
Date: Wed, 29 Sep 2004 21:28:55 +0000 (+0000)
Subject: Minor tidy of "not graded" workshops to supress grades appearing in
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=b514883a042426f611c93c3a183c1c6bf1395163;p=moodle.git

Minor tidy of "not graded" workshops to supress grades appearing in
student pages.
---

diff --git a/mod/workshop/lib.php b/mod/workshop/lib.php
index 3cfaf2d1c1..60fa08914a 100644
--- a/mod/workshop/lib.php
+++ b/mod/workshop/lib.php
@@ -612,11 +612,13 @@ function workshop_delete_instance($id) {
 ///////////////////////////////////////////////////////////////////////////////
 function workshop_grades($workshopid) {
 /// Must return an array of grades, indexed by user, and a max grade.
-/// only retruns grades in phase 2 or greater
-global $CFG;
+/// only returns grades in phase 2 or greater
+/// returns nothing if workshop is not graded
+    global $CFG;
 
+    $return = null;
     if ($workshop = get_record("workshop", "id", $workshopid)) {
-        if ($workshop->phase > 1) {
+        if ($workshop->phase > 1 and $workshop->gradingstrategy) {
             if ($students = get_course_students($workshop->course)) {
                 foreach ($students as $student) {
                     if ($workshop->wtype) {
@@ -641,10 +643,13 @@ global $CFG;
                 }
             }
         }
-        if ($workshop->wtype) {
-            $return->maxgrade = $workshop->grade + $workshop->gradinggrade;
-        } else { // ignore grading grades for simple assignemnts
-            $return->maxgrade = $workshop->grade;
+        // set maximum grade if graded
+        if ($workshop->gradingstrategy) {
+            if ($workshop->wtype) {
+                $return->maxgrade = $workshop->grade + $workshop->gradinggrade;
+            } else { // ignore grading grades for simple assignemnts
+                $return->maxgrade = $workshop->grade;
+            }
         }
     }
     return $return;
diff --git a/mod/workshop/locallib.php b/mod/workshop/locallib.php
index 92045ad6a2..c34034db5f 100644
--- a/mod/workshop/locallib.php
+++ b/mod/workshop/locallib.php
@@ -908,9 +908,11 @@ function workshop_list_assessed_submissions($workshop, $user) {
                 }
                 // has assessment been graded?
                 if ($assessment->timegraded and ($timenow - $assessment->timegraded > $CFG->maxeditingtime)) {
-                    $comment .= "; ".get_string("thegradeforthisassessmentis", "workshop", 
+                    if ($workshop->gradingstrategy) { // supress grading grade if not graded
+                        $comment .= "; ".get_string("thegradeforthisassessmentis", "workshop", 
                             number_format($assessment->gradinggrade * $workshop->gradinggrade / 100, 0)).
                             " / $workshop->gradinggrade";
+                    }
                 }
                 // if peer agreements show whether agreement has been reached
                 if ($workshop->agreeassessments) {
@@ -969,8 +971,10 @@ function workshop_list_peer_assessments($workshop, $user) {
                                 get_string("view", "workshop")."</a>";
                             $comment = get_string("assessedon", "workshop", userdate($assessment->timecreated));
                             $grade = number_format($assessment->grade * $workshop->grade / 100, 1);
-                            $comment .= "; ".get_string("gradeforsubmission", "workshop").
-                                ": $grade / $workshop->grade"; 
+                            if ($workshop->gradingstrategy) { // supress grade if not graded
+                                $comment .= "; ".get_string("gradeforsubmission", "workshop").
+                                    ": $grade / $workshop->grade"; 
+                            }
                             if ($assessment->timegraded) {
                                 if (!$assessment->gradinggrade) {
                                     // it's a bad assessment
@@ -1561,10 +1565,12 @@ function workshop_list_teacher_submissions($workshop, $user) {
                 }
                 // see if the assessment is graded
                 if ($assessment->timegraded) {
-                    // show grading grade
-                    $comment = get_string("thegradeforthisassessmentis", "workshop", 
+                    // show grading grade (supressed if workshop not graded)
+                    if ($workshop->gradingstrategy) {
+                        $comment = get_string("thegradeforthisassessmentis", "workshop", 
                             number_format($assessment->gradinggrade * $workshop->gradinggrade / 100, 1))." / ".
                             $workshop->gradinggrade;
+                    }
                 } elseif ($assessment->timecreated < $timenow) {
                     $comment = get_string("awaitinggradingbyteacher", "workshop", $course->teacher);
                 }
diff --git a/mod/workshop/mod.html b/mod/workshop/mod.html
index 30d176ec9f..9a9c49fd66 100644
--- a/mod/workshop/mod.html
+++ b/mod/workshop/mod.html
@@ -58,6 +58,9 @@
     if (empty($form->anonymous)) {
         $form->anonymous = 0;
     }
+    if (empty($form->usepassword)) {
+        $form->usepassword = 0;
+    }
     if (empty($form->maxbytes)) {
         $form->maxbytes = "";
     }