From: vyshane <vyshane>
Date: Tue, 11 Apr 2006 08:30:10 +0000 (+0000)
Subject: Fix for Bug #5021 - Unhiding an entire Module unhides all it's child
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=7477f2505baf63d64fe3d2f7fd396d7aff2b25dc;p=moodle.git

Fix for Bug #5021 - Unhiding an entire Module unhides all it's child
resources, even in hidden sections
---

diff --git a/admin/modules.php b/admin/modules.php
index 461548bd5a..2783f80922 100644
--- a/admin/modules.php
+++ b/admin/modules.php
@@ -46,16 +46,18 @@
         if (!$module = get_record("modules", "name", $hide)) {
             error("Module doesn't exist!");
         }
-        set_field("modules", "visible", "0", "id", $module->id);            // Hide main module
-        set_field("course_modules", "visible", "0", "module", $module->id); // Hide all related activity modules
+        set_field("modules", "visible", "0", "id", $module->id);                               // Hide main module
+        set_field('course_modules', 'visibleold', '1', 'visible' ,'1', 'module', $module->id); // Remember the previous visible state so we can toggle this back if the module is unhidden.
+        set_field('course_modules', 'visibleold', '0', 'visible' ,'0', 'module', $module->id);
+        set_field("course_modules", "visible", "0", "module", $module->id);                    // Hide all related activity modules
     }
 
     if (!empty($show) and confirm_sesskey()) {
         if (!$module = get_record("modules", "name", $show)) {
             error("Module doesn't exist!");
         }
-        set_field("modules", "visible", "1", "id", $module->id);            // Show main module
-        set_field("course_modules", "visible", "1", "module", $module->id); // Show all related activity modules
+        set_field("modules", "visible", "1", "id", $module->id);                               // Show main module
+        set_field('course_modules', 'visible', '1', 'visibleold', '1', 'module', $module->id); // Get the previous saved visible state for the course module.
     }
 
     if (!empty($delete) and confirm_sesskey()) {