Merge branch 'MDL-32988-master' of git://github.com/sammarshallou/moodle
authorAparup Banerjee <aparup@moodle.com>
Wed, 16 May 2012 06:03:56 +0000 (14:03 +0800)
committerAparup Banerjee <aparup@moodle.com>
Wed, 16 May 2012 06:03:56 +0000 (14:03 +0800)
1  2 
lib/moodlelib.php

diff --combined lib/moodlelib.php
@@@ -4528,7 -4528,20 +4528,20 @@@ function remove_course_contents($course
              // Ooops, this module is not properly installed, force-delete it in the next block
          }
      }
      // We have tried to delete everything the nice way - now let's force-delete any remaining module data
+     // Remove all data from availability and completion tables that is associated
+     // with course-modules belonging to this course. Note this is done even if the
+     // features are not enabled now, in case they were enabled previously.
+     $DB->delete_records_select('course_modules_completion',
+            'coursemoduleid IN (SELECT id from {course_modules} WHERE course=?)',
+            array($courseid));
+     $DB->delete_records_select('course_modules_availability',
+            'coursemoduleid IN (SELECT id from {course_modules} WHERE course=?)',
+            array($courseid));
+     // Remove course-module data.
      $cms = $DB->get_records('course_modules', array('course'=>$course->id));
      foreach ($cms as $cm) {
          if ($module = $DB->get_record('modules', array('id'=>$cm->module))) {
          context_helper::delete_instance(CONTEXT_MODULE, $cm->id);
          $DB->delete_records('course_modules', array('id'=>$cm->id));
      }
-     // Remove all data from availability and completion tables that is associated
-     // with course-modules belonging to this course. Note this is done even if the
-     // features are not enabled now, in case they were enabled previously
-     $DB->delete_records_select('course_modules_completion',
-            'coursemoduleid IN (SELECT id from {course_modules} WHERE course=?)',
-            array($courseid));
-     $DB->delete_records_select('course_modules_availability',
-            'coursemoduleid IN (SELECT id from {course_modules} WHERE course=?)',
-            array($courseid));
      if ($showfeedback) {
          echo $OUTPUT->notification($strdeleted.get_string('type_mod_plural', 'plugin'), 'notifysuccess');
      }
@@@ -10096,22 -10101,20 +10101,22 @@@ function get_performance_info() 
               } else {
                   $othercount += 1;
               }
 -             $details .= "<div class='yui-module'><p>$module</p>";
 -             foreach ($backtraces as $backtrace) {
 -                 $details .= "<div class='backtrace'>$backtrace</div>";
 +             if (!empty($CFG->yuimoduledebug)) {
 +                 // hidden feature for developers working on YUI module infrastructure
 +                 $details .= "<div class='yui-module'><p>$module</p>";
 +                 foreach ($backtraces as $backtrace) {
 +                     $details .= "<div class='backtrace'>$backtrace</div>";
 +                 }
 +                 $details .= '</div>';
               }
 -             $details .= '</div>';
           }
           $info['html'] .= "<span class='includedyuimodules'>Included YUI modules: $yuicount</span> ";
           $info['txt'] .= "includedyuimodules: $yuicount ";
           $info['html'] .= "<span class='includedjsmodules'>Other JavaScript modules: $othercount</span> ";
           $info['txt'] .= "includedjsmodules: $othercount ";
 -         // Slightly odd to output the details in a display: none div. The point
 -         // Is that it takes a lot of space, and if you care you can reveal it
 -         // using firebug.
 -         $info['html'] .= '<div id="yui-module-debug" class="notifytiny">'.$details.'</div>';
 +         if ($details) {
 +             $info['html'] .= '<div id="yui-module-debug" class="notifytiny">'.$details.'</div>';
 +         }
       }
  
      if (!empty($PERF->logwrites)) {