Revert "MDL-46978 gradebook: Edit grade item changes visibility of category"
authorSam Hemelryk <sam@moodle.com>
Tue, 7 Oct 2014 21:37:12 +0000 (10:37 +1300)
committerSam Hemelryk <sam@moodle.com>
Tue, 7 Oct 2014 21:37:12 +0000 (10:37 +1300)
lib/grade/grade_category.php
lib/grade/grade_item.php

index e89b7ea..f26f821 100644 (file)
@@ -2249,11 +2249,14 @@ class grade_category extends grade_object {
         }
 
         //if marking category visible make sure parent category is visible MDL-21367
-        if ($this->is_hidden() != $hidden) {
+        if( !$hidden ) {
             $category_array = grade_category::fetch_all(array('id'=>$this->parent));
             if ($category_array && array_key_exists($this->parent, $category_array)) {
                 $category = $category_array[$this->parent];
-                $category->set_hidden($hidden, false);
+                //call set_hidden on the category regardless of whether it is hidden as its parent might be hidden
+                //if($category->is_hidden()) {
+                    $category->set_hidden($hidden, false);
+                //}
             }
         }
     }
index d29761b..6fc05cf 100644 (file)
@@ -608,11 +608,14 @@ class grade_item extends grade_object {
         }
 
         //if marking item visible make sure category is visible MDL-21367
-        if ($this->is_hidden() != $hidden) {
+        if( !$hidden ) {
             $category_array = grade_category::fetch_all(array('id'=>$this->categoryid));
             if ($category_array && array_key_exists($this->categoryid, $category_array)) {
                 $category = $category_array[$this->categoryid];
-                $category->set_hidden($hidden, false);
+                //call set_hidden on the category regardless of whether it is hidden as its parent might be hidden
+                //if($category->is_hidden()) {
+                    $category->set_hidden($hidden, false);
+                //}
             }
         }
     }