W3C validation and misc code cleanup.

course/mod.php

@@ -674,5 +674,4 @@
     }
 
     print_footer($course);
-
-?>
+?>

course/report.php

@@ -21,10 +21,14 @@
     $directories = get_list_of_plugins('course/report');
 
     foreach ($directories as $directory) {
-        echo '<div class="plugin '.$directory.'">';
-        include_once($CFG->dirroot.'/course/report/'.$directory.'/mod.php');  // Fragment for listing
-        echo '</div>';
+        $pluginfile = $CFG->dirroot.'/course/report/'.$directory.'/mod.php';
+        if (file_exists($pluginfile)) {
+            echo '<div class="plugin">';
+            //echo $pluginfile;
+            include_once($pluginfile);  // Fragment for listing
+            echo '</div>';
+        }
     }
-
+    
     print_footer();
 ?>

course/report/log/mod.php

@@ -13,10 +13,9 @@
     print_log_selector_form($course); 
 
     echo '<br />';
+    echo '<div style="text-align:center;">';
     print_heading(get_string('chooselivelogs') .':');
-
-    echo '<center><h3>';
     link_to_popup_window('/course/report/log/live.php?id='. $course->id,'livelog', get_string('livelogs'), 500, 800);
-    echo '</h3></center>';
-
-?>
+    echo '</div>';
+    echo '<br />';
+?>

course/report/outline/mod.php

@@ -4,8 +4,9 @@
         die('Direct access to this script is forbidden.');    ///  It must be included from a Moodle page
     }
 
+    echo '<p style="text-align:center;">';
     $activityreport = get_string( 'activityreport' );
-    echo "<center><a href=\"{$CFG->wwwroot}/course/report/outline/index.php?id={$course->id}\">";
-    echo "$activityreport</a></center>\n";
-
-?>
+    echo "<a href=\"{$CFG->wwwroot}/course/report/outline/index.php?id={$course->id}\">";
+    echo "$activityreport</a>\n";
+    echo '</p>';
+?>

course/report/participation/index.php

@@ -103,7 +103,7 @@
     
     
     // print first controls.
-    echo '<form action="index.php" action="get">'."\n".
+    echo '<form action="index.php" method="get">'."\n".
         '<input type="hidden" name="id" value="'.$course->id.'" />'."\n".
         '<input type="hidden" name="oldmod" value="'.$moduleid.'" />'."\n".
         '<input type="hidden" name="instanceid" value="'.$instanceid.'" />'."\n".

course/report/participation/mod.php

@@ -66,7 +66,7 @@
     
     
     // print first controls.
-    echo '<form action="'.$CFG->wwwroot.'/course/report/participation/index.php" action="get">'."\n".
+    echo '<form action="'.$CFG->wwwroot.'/course/report/participation/index.php" method="get">'."\n".
         '<input type="hidden" name="id" value="'.$course->id.'" />'."\n".
         '<table align="center" cellpadding="10"><tr>'."\n".
         "<td>\n".

course/report/stats/mod.php

@@ -3,8 +3,12 @@
     if (!defined('MOODLE_INTERNAL')) {
         die('Direct access to this script is forbidden.');    ///  It must be included from a Moodle page
     }
-
+    
+    echo '<p style="text-align:center;">';
     if (!empty($CFG->enablestats)) {
         echo '<a href="'.$CFG->wwwroot.'/course/report/stats/index.php?course='.$course->id.'">'.get_string('stats').'</a>';
+    } else {
+        echo '<a href="'.$CFG->wwwroot.'/admin/config.php#configsectionstats">'.get_string('statsoff').'</a>';
     }
-?>
+    echo '</p>';
+?>