From 9988d1b7a7d2a1273cbe8eb3f5fbf7a6f5cc3db7 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Wed, 1 Aug 2007 04:50:39 +0000
Subject: [PATCH] When editing old outcomes when outcomes are switched off,
 just treat them like an ordinary scale item

---
 grade/edit/tree/item.php | 2 +-
 grade/lib.php            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

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;
-- 
2.39.5