From: moodler <moodler>
Date: Wed, 1 Aug 2007 04:50:39 +0000 (+0000)
Subject: When editing old outcomes when outcomes are switched off, just treat them like an... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=9988d1b7a7d2a1273cbe8eb3f5fbf7a6f5cc3db7;p=moodle.git

When editing old outcomes when outcomes are switched off, just treat them like an ordinary scale item
---

diff --git a/grade/edit/tree/item.php b/grade/edit/tree/item.php
index 08c38cd8e0..24675d9b9b 100644
--- a/grade/edit/tree/item.php
+++ b/grade/edit/tree/item.php
@@ -28,7 +28,7 @@ if ($mform->is_cancelled()) {
 
 if ($item = get_record('grade_items', 'id', $id, 'courseid', $course->id)) {
     // redirect if outcomeid present
-    if (!empty($item->outcomeid)) {
+    if (!empty($item->outcomeid) && !empty($CFG->enableoutcomes)) {
         $url = $CFG->wwwroot.'/grade/edit/tree/outcome.php?id='.$id.'&amp;courseid='.$courseid;
         redirect($gpr->add_url_params($url));
     }
diff --git a/grade/lib.php b/grade/lib.php
index a63e3a34d7..d44a65442b 100644
--- a/grade/lib.php
+++ b/grade/lib.php
@@ -601,7 +601,7 @@ class grade_tree {
             case 'item':
             case 'categoryitem':
             case 'courseitem':
-                if (empty($object->outcomeid)) {
+                if (empty($object->outcomeid) || empty($CFG->enableoutcomes)) {
                     $url = $CFG->wwwroot.'/grade/edit/tree/item.php?courseid='.$this->courseid.'&amp;id='.$object->id;
                 } else {
                     $url = $CFG->wwwroot.'/grade/edit/tree/outcomeitem.php?courseid='.$this->courseid.'&amp;id='.$object->id;