towards XHTML compliance

UniGe · Sep 12, 2004 · 24f2b65 · 24f2b65
1 parent f13cfd7
commit 24f2b65
Show file tree

Hide file tree

Showing 32 changed files with 397 additions and 352 deletions.
diff --git a/course/category.php b/course/category.php
@@ -1,4 +1,4 @@
-<?PHP // $Id$
+<?php // $Id$
       // Displays the top level category or all courses
       // In editing mode, allows the admin to edit a category, 
       // and rearrange courses
@@ -378,7 +378,7 @@
         }
 
         if ($abletomovecourses) {
-            echo "<tr><td colspan=3 align=right>";
+            echo "<tr><td colspan=\"3\" align=\"right\">";
             echo "<br />";
             unset($displaylist[$category->id]);
             choose_from_menu ($displaylist, "moveto", "", get_string("moveselectedcoursesto"), "javascript:document.movecourses.submit()");

diff --git a/course/delete.php b/course/delete.php
@@ -1,4 +1,4 @@
-<?PHP // $Id$
+<?php // $Id$
       // Admin-only code to delete a course utterly
 
 	require_once("../config.php");
@@ -37,7 +37,7 @@
                      "<a href=\"category.php?id=$course->category\">$category->name</a> -> ".
                      "$strdeletecheck");
 
-        notice_yesno("$strdeletecoursecheck<BR><BR>$course->fullname ($course->shortname)", 
+        notice_yesno("$strdeletecoursecheck<br /><br />$course->fullname ($course->shortname)", 
                      "delete.php?id=$course->id&delete=".md5($course->timemodified), 
                      "category.php?id=$course->category");
         exit;

diff --git a/course/edit.html b/course/edit.html
@@ -1,4 +1,4 @@
-<?PHP
+<?php
     // Make sure all variables are defined
     if (!isset($form->showreports)) {
         $form->showreports = 0;
@@ -16,10 +16,10 @@
         $form->enrolperiod = 0;
     }
 ?>
-<FORM METHOD="post" action="edit.php" NAME="form">
-<table cellpadding=9 cellspacing=0 >
-<tr valign=top>
-    <td align="right"><P><?php  print_string("category") ?>:</td>
+<form method="post" action="edit.php" name="form">
+<table cellpadding="9" cellspacing="0" >
+<tr valign="top">
+    <td align="right"><p><?php  print_string("category") ?>:</td>
     <td><?php
            $displaylist = array();
            $parentlist = array();
@@ -29,53 +29,53 @@
         ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("fullname") ?>:</td>
-    <td><input type="text" name="fullname" maxlength="254" size=50 value="<?php  p($form->fullname) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("fullname") ?>:</td>
+    <td><input type="text" name="fullname" maxlength="254" size="50" value="<?php  p($form->fullname) ?>" />
     <?php  helpbutton("coursefullname", get_string("fullname")) ?>
     <?php  if (isset($err["fullname"])) formerr($err["fullname"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("shortname") ?>:</td>
-    <td><input type="text" name="shortname" maxlength="15"  size="10" value="<?php  p($form->shortname) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("shortname") ?>:</td>
+    <td><input type="text" name="shortname" maxlength="15"  size="10" value="<?php  p($form->shortname) ?>" />
     <?php  helpbutton("courseshortname", get_string("shortname")) ?>
     <?php  if (isset($err["shortname"])) formerr($err["shortname"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("idnumber") ?>:</td>
-    <td><input type="text" name="idnumber" maxlength="100"  size="10" value="<?php  p($form->idnumber) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("idnumber") ?>:</td>
+    <td><input type="text" name="idnumber" maxlength="100"  size="10" value="<?php  p($form->idnumber) ?>" />
     <?php  helpbutton("courseidnumber", get_string("idnumber")) ?>
     <?php  if (isset($err["idnumber"])) formerr($err["idnumber"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("summary") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("summary") ?>:</td>
     <td><?php
         print_textarea($usehtmleditor, 10, 50, 660, 200, "summary", $form->summary);
         helpbutton("text", get_string("helptext"));
         if (isset($err["summary"])) formerr($err["summary"]);
     ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("format") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("format") ?>:</td>
     <td><?php
            choose_from_menu ($form->courseformats, "format", "$form->format", "");
            helpbutton("courseformats", get_string("courseformats"));
         ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("startdate") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("startdate") ?>:</td>
     <td><?php
            print_date_selector("startday", "startmonth", "startyear", $form->startdate);
            helpbutton("coursestartdate", get_string("startdate"));
     ?></td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("enrolperiod") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("enrolperiod") ?>:</td>
     <td>
         <?php
             $periodmenu[0] = get_string('unlimited');
@@ -88,8 +88,8 @@
         ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("numberweeks") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("numberweeks") ?>:</td>
     <td><?php
            for ($i=1; $i<=52; $i++) {
               $sectionmenu[$i] = "$i";
@@ -98,8 +98,8 @@
            helpbutton("coursenumsections", get_string("numberweeks"));
     ?></td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("groupmode") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("groupmode") ?>:</td>
     <td><?php
     unset($choices);
     $choices[NOGROUPS] = get_string("groupsnone");
@@ -118,8 +118,8 @@
     helpbutton("groupmodeforce", get_string("groupmodeforce")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("availability") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("availability") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("courseavailablenot");
@@ -128,15 +128,15 @@
     helpbutton("courseavailability", get_string("availability")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("enrolmentkey") ?>:</td>
-    <td><input type="text" name="password" size=25 value="<?php  p($form->password) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("enrolmentkey") ?>:</td>
+    <td><input type="text" name="password" size="25" value="<?php  p($form->password) ?>" />
     <?php  helpbutton("enrolmentkey", get_string("enrolmentkey")) ?>
     <?php  if (isset($err["password"])) formerr($err["password"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("opentoguests") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("opentoguests") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("guestsno");
@@ -149,16 +149,16 @@
 <?php
     if ($CFG->enrol != 'internal') {
 ?>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("cost") ?>:</td>
-    <td><input type="text" name="cost" maxlength="6" size=6 value="<?php  p($form->cost) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("cost") ?>:</td>
+    <td><input type="text" name="cost" maxlength="6" size="6" value="<?php  p($form->cost) ?>" />
     </td>
 </tr>
 <?php
     }
 ?>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("hiddensections") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("hiddensections") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("hiddensectionscollapsed");
@@ -167,8 +167,8 @@
     helpbutton("coursehiddensections", get_string("hiddensections")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("newsitemsnumber") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("newsitemsnumber") ?>:</td>
     <td><?php
        $newsitem = get_string("newsitem");
        $newsitems = get_string("newsitems");
@@ -189,8 +189,8 @@
     ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("showgrades") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("showgrades") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("no");
@@ -199,8 +199,8 @@
     helpbutton("coursegrades", get_string("grades")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("showreports") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("showreports") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("no");
@@ -209,44 +209,44 @@
     helpbutton("coursereports", get_string("activityreport")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("maximumupload") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("maximumupload") ?>:</td>
     <td><?php
     $choices = get_max_upload_sizes($CFG->maxbytes);
     choose_from_menu ($choices, "maxbytes", $form->maxbytes, "");
     helpbutton("courseuploadsize", get_string("maximumupload")); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("wordforteacher") ?>:</td>
-    <td><input type="text" name="teacher" maxlength="100" size=25 value="<?php  p($form->teacher) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("wordforteacher") ?>:</td>
+    <td><input type="text" name="teacher" maxlength="100" size="25" value="<?php  p($form->teacher) ?>" />
     (<?php  print_string("wordforteachereg") ?>)
     <?php  if (isset($err["teacher"])) formerr($err["teacher"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("wordforteachers") ?>:</td>
-    <td><input type="text" name="teachers" maxlength="100" size=25 value="<?php  p($form->teachers) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("wordforteachers") ?>:</td>
+    <td><input type="text" name="teachers" maxlength="100" size="25" value="<?php  p($form->teachers) ?>" />
     (<?php  print_string("wordforteacherseg") ?>)
     <?php  if (isset($err["teachers"])) formerr($err["teachers"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("wordforstudent") ?>:</td>
-    <td><input type="text" name="student" maxlength="100" size=25 value="<?php  p($form->student) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("wordforstudent") ?>:</td>
+    <td><input type="text" name="student" maxlength="100" size="25" value="<?php  p($form->student) ?>" />
     (<?php  print_string("wordforstudenteg") ?>)
     <?php  if (isset($err["student"])) formerr($err["student"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("wordforstudents") ?>:</td>
-    <td><input type="text" name="students" maxlength="100" size=25 value="<?php  p($form->students) ?>">
+<tr valign="top">
+    <td align="right"><p><?php  print_string("wordforstudents") ?>:</td>
+    <td><input type="text" name="students" maxlength="100" size="25" value="<?php  p($form->students) ?>" />
     (<?php  print_string("wordforstudentseg") ?>)
     <?php  if (isset($err["students"])) formerr($err["students"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("forcelanguage") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("forcelanguage") ?>:</td>
   <td>
     <?php
         $languages[''] = get_string("forceno");
@@ -257,7 +257,7 @@
 </tr>
 <tr>
     <td></td>
-    <td><input type="submit" value="<?php  print_string("savechanges") ?>"></td>
+    <td><input type="submit" value="<?php  print_string("savechanges") ?>" /></td>
 </table>
-<input type="hidden" name="id" value="<?php echo $form->id ?>">
+<input type="hidden" name="id" value="<?php echo $form->id ?>" />
 </form>
diff --git a/course/edit.php b/course/edit.php
@@ -1,4 +1,4 @@
-<?PHP // $Id$
+<?php // $Id$
       // Edit course settings
 
     require_once("../config.php");

diff --git a/course/editsection.html b/course/editsection.html
@@ -1,8 +1,8 @@
 <center>
 <form name="theform" method="post" action="editsection.php">
-<table cellpadding=5>
-<tr valign=top>
-    <td align=right><p><b><?php print_string("summary") ?>:</b></p>
+<table cellpadding="5">
+<tr valign="top">
+    <td align="right"><p><b><?php print_string("summary") ?>:</b></p>
     <br />
     <font size="1">
      <?php helpbutton("summaries", get_string("helpsummaries"), "moodle", true, true);
@@ -21,7 +21,7 @@
     </td>
 </tr>
 </table>
-<input type="hidden" name=id value="<?php echo $form->id ?>">
-<input type="submit" value="<?php print_string("savechanges") ?>">
+<input type="hidden" name="id" value="<?php echo $form->id ?>" />
+<input type="submit" value="<?php print_string("savechanges") ?>" />
 </form>
 </center>
diff --git a/course/editsection.php b/course/editsection.php
@@ -1,4 +1,4 @@
-<?PHP // $Id$
+<?php // $Id$
       // Edit the introduction of a section
 
     require_once("../config.php");
@@ -49,7 +49,7 @@
     $stredit = get_string("edit", "", " $sectionname $section->section");
 
 	print_header("$course->shortname: $stredit", "$course->fullname", 
-                 "<A HREF=\"$CFG->wwwroot/course/view.php?id=$course->id\">$course->shortname</A> 
+                 "<a href=\"$CFG->wwwroot/course/view.php?id=$course->id\">$course->shortname</a> 
                   -> $stredit");
 
     print_heading(get_string("summaryof", "", "$sectionname $form->section"));