W3C validation and misc code cleanup.

diff --git a/course/mod.php b/course/mod.php
index 719b63308556d0d4d9cf02c41e1c586b5076bae3..aec1a5e2ae829aca575342e7f607fd0c5ec08771 100644 (file)
--- a/course/mod.php
+++ b/course/mod.php
@@ -674,5 +674,4 @@
     }
 
     print_footer($course);
-
-?>
+?>
\ No newline at end of file

diff --git a/course/report.php b/course/report.php
index b281c66b448929459de4f7cca96d86664fd4a177..408350247de1b1fe86302a363c8962fe86ba808d 100644 (file)
--- a/course/report.php
+++ b/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();
 ?>

diff --git a/course/report/log/mod.php b/course/report/log/mod.php
index 2fffe70cc98718c8563681cad6cfc21f28c30070..081e41eebe1183b9ef997fe4e8e69dc69c5952c5 100644 (file)
--- a/course/report/log/mod.php
+++ b/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 />';
+?>
\ No newline at end of file

diff --git a/course/report/outline/mod.php b/course/report/outline/mod.php
index 25f4cf27af6b9e8bea674a1789585b1b8cf458a4..a662720ea5f36783df7284f9d759ecca02f01d76 100644 (file)
--- a/course/report/outline/mod.php
+++ b/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>';
+?>
\ No newline at end of file

diff --git a/course/report/participation/index.php b/course/report/participation/index.php
index e2bde1284a6f1275eb21730600edd26144b607b1..0a2f55d3e7cfde78fc249385ae68201947062f5d 100644 (file)
--- a/course/report/participation/index.php
+++ b/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".

diff --git a/course/report/participation/mod.php b/course/report/participation/mod.php
index b850a0d003988201833b5dd7c0784ae8ef27f30e..710c0e399e3413969e9d1ab9d256490fd453511d 100644 (file)
--- a/course/report/participation/mod.php
+++ b/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".

diff --git a/course/report/stats/mod.php b/course/report/stats/mod.php
index 7680376a63e3972a1af7726b21918a271e09ef71..5ace60957fa73fd94e379ff87bfec402cff627e2 100644 (file)
--- a/course/report/stats/mod.php
+++ b/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>';
+?>
\ No newline at end of file