Permalink
Browse files

MDL-38700 course: only change visibility if a module moves to a diffe…

…rent section
  • Loading branch information...
1 parent ba97fbe commit 914624d51d7383f7682ce445489cbf0f7cc17926 @mackensen mackensen committed with danpoltawski Mar 25, 2013
Showing with 13 additions and 11 deletions.
  1. +13 −11 course/lib.php
View
@@ -3163,17 +3163,19 @@ function moveto_module($mod, $section, $beforemod=NULL) {
}
// if moving to a hidden section then hide module
- if (!$section->visible && $mod->visible) {
- // Set this in the object because it is sent as a response to ajax calls.
- $mod->visible = 0;
- set_coursemodule_visible($mod->id, 0);
- // Set visibleold to 1 so module will be visible when section is made visible.
- $DB->set_field('course_modules', 'visibleold', 1, array('id' => $mod->id));
- }
- if ($section->visible && !$mod->visible) {
- set_coursemodule_visible($mod->id, $mod->visibleold);
- // Set this in the object because it is sent as a response to ajax calls.
- $mod->visible = $mod->visibleold;
+ if ($mod->section != $section->id) {
+ if (!$section->visible && $mod->visible) {
+ // Set this in the object because it is sent as a response to ajax calls.
+ $mod->visible = 0;
+ set_coursemodule_visible($mod->id, 0);
+ // Set visibleold to 1 so module will be visible when section is made visible.
+ $DB->set_field('course_modules', 'visibleold', 1, array('id' => $mod->id));
+ }
+ if ($section->visible && !$mod->visible) {
+ set_coursemodule_visible($mod->id, $mod->visibleold);
+ // Set this in the object because it is sent as a response to ajax calls.
+ $mod->visible = $mod->visibleold;
+ }
}
/// Add the module into the new section

0 comments on commit 914624d

Please sign in to comment.