some fixes for XHTML compliance

diff --git a/admin/auth.php b/admin/auth.php
index fb804b8c3326559b20c6f15dd701cfaf3309139c..b44aa21ef7e4504970a35f40d2ff33189b81ad4f 100644 (file)
--- a/admin/auth.php
+++ b/admin/auth.php
@@ -15,7 +15,7 @@
 
 /// If data submitted, then process and store.
 
-       if ($config = data_submitted()) {
+    if ($config = data_submitted()) {
 
         $config = (array)$config;
         validate_form($config, $err);
@@ -35,7 +35,7 @@
                 $focus = "form.$key";
             }
         }
-       }
+    }
 
 /// Otherwise fill and print the form.
 
@@ -48,11 +48,11 @@
         $options[$module] = get_string("auth_$module"."title", "auth");
     }
     asort($options);
-       if (isset($_GET['auth'])) {
-           $auth = $_GET['auth'];
-       } else {
-        $auth = $config->auth;
-       } 
+    if (isset($_GET['auth'])) {
+        $auth = $_GET['auth'];
+    } else {
+          $auth = $config->auth;
+    } 
     require_once("$CFG->dirroot/auth/$auth/lib.php"); //just to make sure that current authentication functions are loaded
     if (! isset($config->guestloginbutton)) {
         $config->guestloginbutton = 1;
@@ -89,15 +89,15 @@
     $strusers = get_string("users");
 
     print_header("$site->shortname: $strauthenticationoptions", "$site->fullname",
-                  "<A HREF=\"index.php\">$stradministration</A> -> <a href=\"users.php\">$strusers</a> -> $strauthenticationoptions", "$focus");
+                  "<a href=\"index.php\">$stradministration</a> -> <a href=\"users.php\">$strusers</a> -> $strauthenticationoptions", "$focus");
 
-    echo "<CENTER><P><B>";
-    echo "<form TARGET=\"{$CFG->framename}\" NAME=\"authmenu\" method=\"post\" action=\"auth.php\">";
+    echo "<center><p><b>";
+    echo "<form target=\"{$CFG->framename}\" name=\"authmenu\" method=\"post\" action=\"auth.php\">";
     print_string("chooseauthmethod","auth");
 
-       choose_from_menu ($options, "auth", $auth, "","document.location='auth.php?auth='+document.authmenu.auth.options[document.authmenu.auth.selectedIndex].value", "");
+    choose_from_menu ($options, "auth", $auth, "","document.location='auth.php?auth='+document.authmenu.auth.options[document.authmenu.auth.selectedIndex].value", "");
 
-    echo "</B></P></CENTER>";
+    echo "</b></p></center>";
         
     print_simple_box_start("center", "100%", "$THEME->cellheading");
     print_heading($options[$auth]);
@@ -106,7 +106,7 @@
     print_string("auth_$auth"."description", "auth");
     print_simple_box_end();
 
-    echo "<hr>";
+    echo "<hr />";
 
     print_heading($strsettings);
 
@@ -116,11 +116,11 @@
 
     if ($auth != "email" and $auth != "none" and $auth != "manual") {
         echo "<tr valign=\"top\">";
-           echo "<td align=right nowrap><p>";
+        echo "<td align=\"right\" nowrap=\"nowrap\"><p>";
         print_string("changepassword", "auth");
         echo ":</p></td>";
-           echo "<td>";
-        echo "<input type=\"text\" name=\"changepassword\" size=40 value=\"$config->changepassword\">";
+        echo "<td>";
+        echo "<input type=\"text\" name=\"changepassword\" size=\"40\" value=\"$config->changepassword\">";
         echo "</td>";
         echo "<td>";
         print_string("changepasswordhelp","auth");
@@ -129,10 +129,10 @@
     }
 
     echo "<tr valign=\"top\">";
-       echo "<td align=right nowrap><p>";
+    echo "<td align=\"right\" nowrap=\"nowrap\"><p>";
     print_string("guestloginbutton", "auth");
     echo ":</p></td>";
-       echo "<td>";
+    echo "<td>";
     choose_from_menu($guestoptions, "guestloginbutton", $config->guestloginbutton, "");
     echo "</td>";
     echo "<td>";
@@ -140,17 +140,17 @@
     echo "</td></tr>";
 
     if (function_exists('auth_user_create')){    
-           echo "<tr valign=\"top\">";
-               echo "<td align=right nowrap><p>";
-               print_string("auth_user_create", "auth");
-               echo ":</p></td>";
-               echo "<td>";
-               choose_from_menu($createoptions, "auth_user_create", $config->auth_user_create, "");
-               echo "</td>";
-               echo "<td>";
-               print_string("auth_user_creation","auth");
-               echo "</td></tr>";
-       }
+    echo "<tr valign=\"top\">";
+    echo "<td align=\"right\" nowrap=\"nowrap\"><p>";
+    print_string("auth_user_create", "auth");
+    echo ":</p></td>";
+    echo "<td>";
+    choose_from_menu($createoptions, "auth_user_create", $config->auth_user_create, "");
+    echo "</td>";
+    echo "<td>";
+    print_string("auth_user_creation","auth");
+    echo "</td></tr>";
+    }
 
     echo "</table><center><p><input type=\"submit\" value=\"";
     print_string("savechanges");

diff --git a/admin/backup.php b/admin/backup.php
index 608f39e06892c8de96e7ad4a9480841877509099..3f609b235b0d662d7ab150064661255c17061eff 100644 (file)
--- a/admin/backup.php
+++ b/admin/backup.php
@@ -111,7 +111,7 @@
                   "<a href=\"configure.php\">$strconfiguration</a> -> ".
                   $strbackup);
 
-    echo "<p align=right><a href=\"../backup/log.php\">".get_string("logs")."</a></p>";
+    echo "<p align=\"right\"><a href=\"../backup/log.php\">".get_string("logs")."</a></p>";
 
     print_heading($strbackup);
 

diff --git a/admin/blocks.php b/admin/blocks.php
index 95d5947e9b2a691c12cdc223b2c2b7021c4bbe75..098e380cf3e0b10e7a340af5fefc21f5de2cc2c0 100644 (file)
--- a/admin/blocks.php
+++ b/admin/blocks.php
@@ -145,7 +145,7 @@
     foreach ($blockbyname as $blockname => $blockid) {
 
         // [pj] This is not used anywhere, but I'm leaving it in for the future
-        //$icon = "<img src=\"$modpixpath/$block->name/icon.gif\" hspace=10 height=16 width=16 border=0>";
+        //$icon = "<img src=\"$modpixpath/$block->name/icon.gif\" hspace="10" height="16" width="16" border="0">";
         $blockobject = $blockobjects[$blockid];
 
         $delete = '<a href="blocks.php?delete='.$blockid.'">'.$strdelete.'</a>';

diff --git a/admin/config.html b/admin/config.html
index 270f46ae39158fed9306e845afd2a68bea9cc337..f642c1c3f354325b9f4bd82c35a866574d9a8c6c 100644 (file)
--- a/admin/config.html
+++ b/admin/config.html
@@ -1,8 +1,8 @@
 <form method="post" action="config.php" name="form">
 
-<table cellpadding=9 cellspacing=0 >
-<tr valign=top>
-    <td align=right><p>lang:</td>
+<table cellpadding="9" cellspacing="0" >
+<tr valign="top">
+    <td align="right"><p>lang:</td>
     <td>
     <?php choose_from_menu (get_list_of_languages(), "lang", $config->lang, "", "", ""); ?>
     </td>
@@ -10,8 +10,8 @@
     <?php print_string("configlang") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>langmenu:</td>
+<tr valign="top">
+    <td align="right"><p>langmenu:</td>
     <td>
     <?php
        unset($options);
@@ -25,26 +25,26 @@
     <?php print_string("configlangmenu") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>langlist:</td>
+<tr valign="top">
+    <td align="right"><p>langlist:</td>
     <td>
-    <input name=langlist type=text size=40 value="<?php p($config->langlist) ?>">
+    <input name="langlist" type="text" size="40" value="<?php p($config->langlist) ?>">
     </td>
     <td>
     <?php print_string("configlanglist") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>locale:</td>
+<tr valign="top">
+    <td align="right"><p>locale:</td>
     <td>
-    <input name=locale type=text size=10 value="<?php p($config->locale) ?>">
+    <input name="locale" type="text" size="10" value="<?php p($config->locale) ?>">
     </td>
     <td>
     <?php print_string("configlocale") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>timezone:</td>
+<tr valign="top">
+    <td align="right"><p>timezone:</td>
     <td><?php
        if (abs($config->timezone) > 13) {
            $config->timezone = 99;
@@ -71,49 +71,49 @@
     <?php print_string("configtimezone") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>country:</td>
+<tr valign="top">
+    <td align="right"><p>country:</td>
     <td><?php choose_from_menu (get_list_of_countries(), "country", $config->country, get_string("selectacountry"), "") ?>
     </td>
     <td>
     <?php print_string("configcountry") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>smtphosts:</td>
+<tr valign="top">
+    <td align="right"><p>smtphosts:</td>
     <td>
-    <input name=smtphosts type=text size=30 value="<?php p($config->smtphosts)?>">
+    <input name="smtphosts" type="text" size="30" value="<?php p($config->smtphosts)?>">
     </td>
     <td>
     <?php print_string("configsmtphosts") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>smtpuser:</td>
+<tr valign="top">
+    <td align="right"><p>smtpuser:</td>
     <td>
-    <input name=smtpuser type=text size=10 value="<?php p($config->smtpuser)?>">
+    <input name="smtpuser" type="text" size="10" value="<?php p($config->smtpuser)?>">
     </td>
-    <td rowspan=2>
+    <td rowspan="2">
     <?php print_string("configsmtpuser") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>smtppass:</td>
+<tr valign="top">
+    <td align="right"><p>smtppass:</td>
     <td>
-    <input name=smtppass type=text size=10 value="<?php p($config->smtppass)?>">
+    <input name="smtppass" type="text" size="10" value="<?php p($config->smtppass)?>">
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>noreplyaddress:</td>
+<tr valign="top">
+    <td align="right"><p>noreplyaddress:</td>
     <td>
-    <input name=noreplyaddress type=text size=30 value="<?php p($config->noreplyaddress)?>">
+    <input name="noreplyaddress" type="text" size="30" value="<?php p($config->noreplyaddress)?>">
     </td>
     <td>
     <?php print_string("confignoreplyaddress") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>gdversion:</td>
+<tr valign="top">
+    <td align="right"><p>gdversion:</td>
     <td>
     <?php
        unset($options);
@@ -130,8 +130,8 @@
     <?php print_string("configgdversion") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>maxeditingtime:</td>
+<tr valign="top">
+    <td align="right"><p>maxeditingtime:</td>
     <td>
     <?php
        unset($options);
@@ -149,8 +149,8 @@
     <?php print_string("configmaxeditingtime") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>longtimenosee:</td>
+<tr valign="top">
+    <td align="right"><p>longtimenosee:</td>
     <td>
     <?php
        unset($options);
@@ -173,8 +173,8 @@
     <?php print_string("configlongtimenosee") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>deleteunconfirmed:</td>
+<tr valign="top">
+    <td align="right"><p>deleteunconfirmed:</td>
     <td>
     <?php
        unset($options);
@@ -197,8 +197,8 @@
     <?php print_string("configdeleteunconfirmed") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>loglifetime:</td>
+<tr valign="top">
+    <td align="right"><p>loglifetime:</td>
     <td>
     <?php
        unset($options);
@@ -265,8 +265,8 @@
     <?php print_string('confignotifyloginthreshold') ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>sessiontimeout:</td>
+<tr valign="top">
+    <td align="right"><p>sessiontimeout:</td>
     <td>
     <?php
        unset($options);
@@ -287,35 +287,35 @@
     <?php print_string("configsessiontimeout") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>sessioncookie:</td>
+<tr valign="top">
+    <td align="right"><p>sessioncookie:</td>
     <td>
-    <input name=sessioncookie type=text size=10 value="<?php p($config->sessioncookie)?>">
+    <input name="sessioncookie" type="text" size="10" value="<?php p($config->sessioncookie)?>">
     </td>
     <td>
     <?php print_string("configsessioncookie") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>zip:</td>
+<tr valign="top">
+    <td align="right"><p>zip:</td>
     <td>
-    <input name=zip type=text size=30 value="<?php p($config->zip) ?>">
+    <input name="zip" type="text" size="30" value="<?php p($config->zip) ?>">
     </td>
     <td>
     <?php print_string("configzip") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>unzip:</td>
+<tr valign="top">
+    <td align="right"><p>unzip:</td>
     <td>
-    <input name=unzip type=text size=30 value="<?php p($config->unzip) ?>">
+    <input name="unzip" type="text" size="30" value="<?php p($config->unzip) ?>">
     </td>
     <td>
     <?php print_string("configunzip") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>slasharguments:</td>
+<tr valign="top">
+    <td align="right"><p>slasharguments:</td>
     <td>
     <?php unset($options);
        $options[0] = "file.php?file=/pic.jpg";
@@ -328,23 +328,23 @@
     <?php print_string("configslasharguments") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>proxyhost:</td>
+<tr valign="top">
+    <td align="right"><p>proxyhost:</td>
     <td>
-    <input name=proxyhost type=text size=30 value="<?php p($config->proxyhost) ?>">
+    <input name="proxyhost" type="text" size="30" value="<?php p($config->proxyhost) ?>">
     </td>
-    <td rowspan=2>
+    <td rowspan="2">
     <?php print_string("configproxyhost") ?>
     </td>
-</TR>
-<tr valign=top>
-    <td align=right><p>proxyport:</td>
+</tr>
+<tr valign="top">
+    <td align="right"><p>proxyport:</td>
     <td>
-    <input name=proxyport type=text size=5 value="<?php p($config->proxyport) ?>">
+    <input name="proxyport" type="text" size="5" value="<?php p($config->proxyport) ?>">
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>debug:</td>
+<tr valign="top">
+    <td align="right"><p>debug:</td>
     <td>
     <?php
        unset($options);
@@ -358,22 +358,22 @@
     <?php print_string("configdebug") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>framename:</td>
+<tr valign="top">
+    <td align="right"><p>framename:</td>
     <td>
     <?php
        if (empty($config->framename)) {
            $config->framename = "_top";
        }
     ?>
-    <input name=framename type=text size=15 value="<?php p($config->framename)?>">
+    <input name="framename" type="text" size="15" value="<?php p($config->framename)?>">
     </td>
     <td>
     <?php print_string("configframename") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>secureforms:</td>
+<tr valign="top">
+    <td align="right"><p>secureforms:</td>
     <td>
     <?php
        unset($options);
@@ -388,8 +388,8 @@
     </td>
 </tr>
 
-<tr valign=top>
-    <td align=right><p>loginhttps:</td>
+<tr valign="top">
+    <td align="right"><p>loginhttps:</td>
     <td>
     <?php
        unset($options);
@@ -404,8 +404,8 @@
     </td>
 </tr>
 
-<tr valign=top>
-    <td align=right><p>teacherassignteachers:</td>
+<tr valign="top">
+    <td align="right"><p>teacherassignteachers:</td>
     <td>
     <?php
        unset($options);
@@ -419,8 +419,8 @@
     <?php print_string("configteacherassignteachers") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>allusersaresitestudents:</td>
+<tr valign="top">
+    <td align="right"><p>allusersaresitestudents:</td>
     <td>
     <?php
        unset($options);
@@ -434,8 +434,8 @@
     <?php print_string("configallusersaresitestudents") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>showsiteparticipantslist:</td>
+<tr valign="top">
+    <td align="right"><p>showsiteparticipantslist:</td>
     <td>
     <?php
        unset($options);
@@ -449,8 +449,8 @@
     <?php print_string("configshowsiteparticipantslist") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>allowunenroll:</td>
+<tr valign="top">
+    <td align="right"><p>allowunenroll:</td>
     <td>
     <?php
        unset($options);
@@ -464,8 +464,8 @@
     <?php print_string("configallowunenroll") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>maxbytes:</td>
+<tr valign="top">
+    <td align="right"><p>maxbytes:</td>
     <td>
     <?php
        $options = get_max_upload_sizes();
@@ -476,8 +476,8 @@
     <?php print_string("configmaxbytes") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>fullnamedisplay:</td>
+<tr valign="top">
+    <td align="right"><p>fullnamedisplay:</td>
     <td>
     <?php
        unset($options);
@@ -493,8 +493,8 @@
     <?php print_string("configfullnamedisplay") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>extendedusernamechars:</td>
+<tr valign="top">
+    <td align="right"><p>extendedusernamechars:</td>
     <td>
     <?php
        unset($options);
@@ -508,8 +508,8 @@
     <?php print_string("configextendedusernamechars") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>autologinguests:</td>
+<tr valign="top">
+    <td align="right"><p>autologinguests:</td>
     <td>
     <?php
        unset($options);
@@ -523,8 +523,8 @@
     <?php print_string("configautologinguests") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>forcelogin:</td>
+<tr valign="top">
+    <td align="right"><p>forcelogin:</td>
     <td>
     <?php
        unset($options);
@@ -538,8 +538,8 @@
     <?php print_string("configforcelogin") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>forceloginforprofiles:</td>
+<tr valign="top">
+    <td align="right"><p>forceloginforprofiles:</td>
     <td>
     <?php
        unset($options);
@@ -553,8 +553,8 @@
     <?php print_string("configforceloginforprofiles") ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p>opentogoogle:</td>
+<tr valign="top">
+    <td align="right"><p>opentogoogle:</td>
     <td>
     <?php
        unset($options);
@@ -569,8 +569,8 @@
     </td>
 </tr>
 
-<tr valign=top>
-        <td align=right><p>enablerssfeeds:</td>
+<tr valign="top">
+        <td align="right"><p>enablerssfeeds:</td>
         <td>
     <?php
        unset($options);
@@ -591,8 +591,8 @@
     </td>
 </tr>
 
-<tr valign=top>
-        <td align=right><p>digestmailtime:</td>
+<tr valign="top">
+        <td align="right"><p>digestmailtime:</td>
         <td>
     <?php
         $hours = array();
@@ -610,7 +610,7 @@
 </tr>
 
 <tr>
-    <td colspan=3 align=center>
+    <td colspan="3" align="center">
     <input type="submit" value="<?php print_string("savechanges") ?>"></td>
 </tr>
 </table>

diff --git a/admin/delete.php b/admin/delete.php
index ca1988a3f02127703fd716185b9bcf0873d85e77..1404b5ebfa0c6de3e7bd03a975b59df615fc7dba 100644 (file)
--- a/admin/delete.php
+++ b/admin/delete.php
@@ -27,7 +27,7 @@
 
     delete_subdirectories($deletedir);
 
-    echo "<H1 align=center>Done!</H1>";
+    echo "<h1 align="center">Done!</h1>";
     print_continue($CFG->wwwroot);
     exit;
 
@@ -43,16 +43,16 @@ function delete_subdirectories($rootdir) {
                 delete_subdirectories($fullfile);
                 echo "Deleting $fullfile ... ";
                 if (rmdir($fullfile)) {
-                    echo "Done.<BR>";
+                    echo "Done.<br />";
                 } else {
-                    echo "FAILED.<BR>";
+                    echo "FAILED.<br />";
                 }
             } else {
                 echo "Deleting $fullfile ... ";
                 if (unlink("$fullfile")) {
-                    echo "Done.<BR>";
+                    echo "Done.<br />";
                 } else {
-                    echo "FAILED.<BR>";
+                    echo "FAILED.<br />";
                 }
             }
         }

diff --git a/admin/editor.html b/admin/editor.html
index df556f17c4c314aeb055b588e0499d1748dc2039..211e2b6a56c7fb93b36a3e3ce05f34e5321b3a95 100644 (file)
--- a/admin/editor.html
+++ b/admin/editor.html
@@ -1,7 +1,7 @@
 <form method="post" action="<?php print($GLOBALS['ME']);?>">
 <table border="0" cellpadding="4" cellspacing="2">
-<tr valign=top>
-    <td align=right><p>htmleditor:</td>
+<tr valign="top">
+    <td align="right"><p>htmleditor:</td>
     <td>
     <?php
        unset($options);

diff --git a/admin/enrol.php b/admin/enrol.php
index c43986c11045e72164f0e301390fe77efab0b1c4..4312823f644bfcf38325da6d2672c174dd1cada3 100644 (file)
--- a/admin/enrol.php
+++ b/admin/enrol.php
@@ -65,7 +65,7 @@
     print_string("description", "enrol_$enrol");
     print_simple_box_end();
 
-    echo "<hr>";
+    echo "<hr />";
    // print_heading($str->settings);
     
     $enrolment->config_form($frm);

diff --git a/admin/filters.html b/admin/filters.html
index 5f7f10fe5a13d0acd9f25d251fa79251ffec2941..79f0c55f10d6ab4eb3cbaa43b93943f993f6fa55 100644 (file)
--- a/admin/filters.html
+++ b/admin/filters.html
@@ -82,7 +82,7 @@
 <form name="options" id="options" method="post" action="filters.php">
 <input type="hidden" name="options" value="1">
 <table cellpadding="20">
-<tr valign=top>
+<tr valign="top">
        <td nowrap="nowrap" align="right"><?php p($strcachetext) ?>:</td>
        <td>
     <?php 
@@ -113,7 +113,7 @@
        <?php p($strconfigcachetext) ?>
     </td>
 </tr>
-<tr valign=top>
+<tr valign="top">
        <td nowrap="nowrap" align="right"><?php p($strfilteruploadedfiles) ?>:</td>
        <td>
     <?php 
@@ -128,7 +128,7 @@
        <?php p($strconfigfilteruploadedfiles) ?>
     </td>
 </tr>
-<tr valign=top>
+<tr valign="top">
     <td>&nbsp;</td>
     <td><input type="submit" value="<?php print_string("savechanges") ?>"></td>
     <td>&nbsp;</td>

diff --git a/admin/index.php b/admin/index.php
index 72825ab6e5fa67f5e28f99c77426a80346e59f46..1164acf5944d91c4dddca04228390ac6008f9504 100644 (file)
--- a/admin/index.php
+++ b/admin/index.php
@@ -21,7 +21,7 @@
 
 /// Check some PHP server settings
 
-    $documentationlink = "please read the <A HREF=\"../doc/?frame=install.html&sub=webserver\">install documentation</A>";
+    $documentationlink = "please read the <a href=\"../doc/?frame=install.html&sub=webserver\">install documentation</a>";
 
     if (ini_get_bool('session.auto_start')) {
         error("The PHP server variable 'session.auto_start' should be Off - $documentationlink");
@@ -48,10 +48,10 @@
     $dirroot = dirname(realpath("../index.php"));
     if (!empty($dirroot) and $dirroot != $CFG->dirroot) {
         error("Please fix your settings in config.php:
-              <P>You have:
-              <P>\$CFG->dirroot = \"".addslashes($CFG->dirroot)."\";
-              <P>but it should be:
-              <P>\$CFG->dirroot = \"".addslashes($dirroot)."\";",
+              <p>You have:
+              <p>\$CFG->dirroot = \"".addslashes($CFG->dirroot)."\";
+              <p>but it should be:
+              <p>\$CFG->dirroot = \"".addslashes($dirroot)."\";",
               "./");
     }
 
@@ -92,7 +92,7 @@
         if (empty($agreelicence)) {
             $strlicense = get_string("license");
             print_header($strlicense, $strlicense, $strlicense, "", "", false, "&nbsp;", "&nbsp;");
-            print_heading("<A HREF=\"http://moodle.org\">Moodle</A> - Modular Object-Oriented Dynamic Learning Environment");
+            print_heading("<a href=\"http://moodle.org\">Moodle</a> - Modular Object-Oriented Dynamic Learning Environment");
             print_heading(get_string("copyrightnotice"));
             print_simple_box_start("center");
             echo text_to_html(get_string("gpl"));
@@ -285,53 +285,53 @@
     $table->cellspacing = 3;
     $table->width = "40%";
 
-    $configdata  = "<font size=+1>&nbsp;</font><a href=\"config.php\">".get_string("configvariables")."</a> - <font size=1>".
+    $configdata  = "<font size=+1>&nbsp;</font><a href=\"config.php\">".get_string("configvariables")."</a> - <font size=\"1\">".
                     get_string("adminhelpconfigvariables")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"site.php\">".get_string("sitesettings")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"site.php\">".get_string("sitesettings")."</a> - <font size=\"1\">".
                     get_string("adminhelpsitesettings")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"../theme/index.php\">".get_string("themes")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"../theme/index.php\">".get_string("themes")."</a> - <font size=\"1\">".
                     get_string("adminhelpthemes")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"lang.php\">".get_string("language")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"lang.php\">".get_string("language")."</a> - <font size=\"1\">".
                     get_string("adminhelplanguage")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"modules.php\">".get_string("managemodules")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"modules.php\">".get_string("managemodules")."</a> - <font size=\"1\">".
                     get_string("adminhelpmanagemodules")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"blocks.php\">".get_string("manageblocks")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"blocks.php\">".get_string("manageblocks")."</a> - <font size=\"1\">".
                     get_string("adminhelpmanageblocks")."</font><br />";
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"filters.php\">".get_string("managefilters")."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"filters.php\">".get_string("managefilters")."</a> - <font size=\"1\">".
                     get_string("adminhelpmanagefilters")."</font><br />";
     if (!isset($CFG->disablescheduledbackups)) {
-        $configdata .= "<font size=+1>&nbsp;</font><a href=\"backup.php\">".get_string("backup")."</a> - <font size=1>".
+        $configdata .= "<font size=+1>&nbsp;</font><a href=\"backup.php\">".get_string("backup")."</a> - <font size=\"1\">".
                         get_string("adminhelpbackup")."</font><br />";
     }
-    $configdata .= "<font size=+1>&nbsp;</font><a href=\"editor.php\">". get_string("editorsettings") ."</a> - <font size=1>".
+    $configdata .= "<font size=+1>&nbsp;</font><a href=\"editor.php\">". get_string("editorsettings") ."</a> - <font size=\"1\">".
                     get_string("adminhelpeditorsettings")."</font><br />";
 
     $table->data[] = array("<font size=+1><b><a href=\"configure.php\">".get_string("configuration")."</a></b>", 
                             $configdata);
 
 
-    $userdata = "<font size=+1>&nbsp;</font><a href=\"auth.php\">".get_string("authentication")."</a> - <font size=1>".
+    $userdata = "<font size=+1>&nbsp;</font><a href=\"auth.php\">".get_string("authentication")."</a> - <font size=\"1\">".
                  get_string("adminhelpauthentication")."</font><br />";
-    $userdata .= "<font size=+1>&nbsp;</font><a href=\"user.php\">".get_string("edituser")."</a> - <font size=1>".
+    $userdata .= "<font size=+1>&nbsp;</font><a href=\"user.php\">".get_string("edituser")."</a> - <font size=\"1\">".
                  get_string("adminhelpedituser")."</font><br />";
     $userdata .= "<font size=+1>&nbsp;</font><a href=\"$CFG->wwwroot/$CFG->admin/user.php?newuser=true\">".
-                 get_string("addnewuser")."</a> - <font size=1>".
+                 get_string("addnewuser")."</a> - <font size=\"1\">".
                  get_string("adminhelpaddnewuser")."</font><br />";
     $userdata .= "<font size=+1>&nbsp;</font><a href=\"$CFG->wwwroot/$CFG->admin/uploaduser.php\">".
-                 get_string("uploadusers")."</a> - <font size=1>".
+                 get_string("uploadusers")."</a> - <font size=\"1\">".
                  get_string("adminhelpuploadusers")."</font><br />";
 
-    $userdata .= "<hr><font size=+1>&nbsp;</font><a href=\"enrol.php\">".get_string("enrolments")."</a> - <font size=1>".
+    $userdata .= "<hr /><font size=+1>&nbsp;</font><a href=\"enrol.php\">".get_string("enrolments")."</a> - <font size=\"1\">".
                  get_string("adminhelpenrolments")."</font><br />";
-    $userdata .= "<font size=+1>&nbsp;</font><a href=\"../course/index.php?edit=off\">".get_string("assignstudents")."</a> - <font size=1>".
+    $userdata .= "<font size=+1>&nbsp;</font><a href=\"../course/index.php?edit=off\">".get_string("assignstudents")."</a> - <font size=\"1\">".
                  get_string("adminhelpassignstudents")."</font><br />";
 
-    $userdata .= "<font size=+1>&nbsp;</font><a href=\"../course/index.php?edit=on\">".get_string("assignteachers")."</a> - <font size=1>".
+    $userdata .= "<font size=+1>&nbsp;</font><a href=\"../course/index.php?edit=on\">".get_string("assignteachers")."</a> - <font size=\"1\">".
                  get_string("adminhelpassignteachers").
-                 " <img src=\"../pix/t/user.gif\" height=11 width=11></font><br />";
-    $userdata .= "<font size=+1>&nbsp;</font><a href=\"creators.php\">".get_string("assigncreators")."</a> - <font size=1>".
+                 " <img src=\"../pix/t/user.gif\" height=\"11\" width=\"11\"></font><br />";
+    $userdata .= "<font size=+1>&nbsp;</font><a href=\"creators.php\">".get_string("assigncreators")."</a> - <font size=\"1\">".
                  get_string("adminhelpassigncreators")."</font><br />";
-    $userdata .= "<font size=+1>&nbsp;</font><a href=\"admin.php\">".get_string("assignadmins")."</a> - <font size=1>".
+    $userdata .= "<font size=+1>&nbsp;</font><a href=\"admin.php\">".get_string("assignadmins")."</a> - <font size=\"1\">".
                  get_string("adminhelpassignadmins")."</font><br />";
 
     $table->data[] = array("<font size=+1><b><a href=\"users.php\">".get_string("users")."</a></b>", $userdata);
@@ -355,20 +355,20 @@
                      "<a href=\"../doc/?frame=release.html\">$CFG->release</a> ($CFG->version)<br />".
                      "Copyright &copy; 1999-2004 Martin Dougiamas<br />".
                      "<a href=\"../doc/?frame=licence.html\">GNU Public License</a>";
-    echo "<center><p><font size=1>$copyrighttext</font></p></center>";
+    echo "<center><p><font size=\"1\">$copyrighttext</font></p></center>";
     //////////////////////////////////////////////////////////////////////////////////////////////////
 
 
-    echo "<table border=0 align=center width=100%><tr>";
-    echo "<td align=center width=33%>";
+    echo "<table border=\"0\" align=\"center\" width=\"100%\"><tr>";
+    echo "<td align=\"center\" width=\"33%\">";
     print_single_button("$CFG->wwwroot/doc", NULL, get_string("documentation"));
     echo "</td>";
 
-    echo "<td align=center width=33%>";
+    echo "<td align=\"center\" width=\"33%\">";
     print_single_button("phpinfo.php", NULL, get_string("phpinfo"));
     echo "</td>";
 
-    echo "<td align=center width=33%>";
+    echo "<td align=\"center\" width=\"33%\">";
     print_single_button("register.php", NULL, get_string("registration"));
     echo "</td>";
     echo "<tr></table>";

diff --git a/admin/lang.php b/admin/lang.php
index 4c4e25a7dc2b04027b2ba2686e4e5a32ad7e25bd..80e8efe13228684b6905404b7e0c5bfa2a7b1f67 100644 (file)
--- a/admin/lang.php
+++ b/admin/lang.php
@@ -25,14 +25,14 @@
 
     switch ($mode) {
         case "missing":
-            $navigation = "<A HREF=\"lang.php\">$strlanguage</A> -> $strmissingstrings";
+            $navigation = "<a href=\"lang.php\">$strlanguage</a> -> $strmissingstrings";
             $title = $strmissingstrings;
             $button = '<form target="'.$CFG->framename.'" method="get" action="'.$CFG->wwwroot.'/admin/lang.php">'.
                       '<input type="hidden" name="mode" value="compare" />'.
                       '<input type="submit" value="'.$strcomparelanguage.'" /></form>';
             break;
         case "compare":
-            $navigation = "<A HREF=\"lang.php\">$strlanguage</A> -> $strcomparelanguage";
+            $navigation = "<a href=\"lang.php\">$strlanguage</a> -> $strcomparelanguage";
             $title = $strcomparelanguage;
             $button = '<form target="'.$CFG->framename.'" method="get" action="'.$CFG->wwwroot.'/admin/lang.php">'.
                       '<input type="hidden" name="mode" value="missing" />'.
@@ -55,14 +55,14 @@
     if (!$mode) {
         $currlang = current_language();
         $langs = get_list_of_languages();
-        echo "<table align=center><tr><td align=\"right\">";
+        echo "<table align=\"center\"><tr><td align=\"right\">";
         echo "<b>$strcurrentlanguage:</b>";
         echo "</td><td>";
         echo popup_form ("$CFG->wwwroot/admin/lang.php?lang=", $langs, "chooselang", $currlang, "", "", "", true);
         echo "</td></tr></table>";
         print_heading("<a href=\"lang.php?mode=missing\">$strmissingstrings</a>");
         print_heading("<a href=\"lang.php?mode=compare\">$strcomparelanguage</a>");
-        echo "<center><hr noshade size=1>";
+        echo "<center><hr noshade size=\"1\">";
         $options["lang"] = $currentlang;
         print_single_button("http://moodle.org/download/lang/", $options, get_string("latestlanguagepack"));
         echo "</center>";
@@ -91,7 +91,7 @@
         foreach ($stringfiles as $file) {
             if (!file_exists("$langdir/$file")) {
                 if (!touch("$langdir/$file")) {
-                    echo "<p><font color=red>".get_string("filemissing", "", "$langdir/$file")."</font></p>";
+                    echo "<p><font color=\"red\">".get_string("filemissing", "", "$langdir/$file")."</font></p>";
                     continue;
                 }
             }
@@ -128,7 +128,7 @@
     
         foreach ($files as $filekey => $file) {    // check all the help files.
             if (!file_exists("$langdir/help/$file")) {
-                echo "<p><font color=red>".get_string("filemissing", "", "$langdir/help/$file")."</font></p>";
+                echo "<p><font color=\"red\">".get_string("filemissing", "", "$langdir/help/$file")."</font></p>";
                 $somethingfound = true;
                 continue;
             }
@@ -139,7 +139,7 @@
         }
         foreach ($files as $filekey => $file) {    // check all the docs files.
             if (!file_exists("$langdir/docs/$file")) {
-                echo "<P><FONT COLOR=red>".get_string("filemissing", "", "$langdir/docs/$file")."</FONT></P>";
+                echo "<p><font color=\"red\">".get_string("filemissing", "", "$langdir/docs/$file")."</font></p>";
                 $somethingfound = true;
                 continue;
             }
@@ -182,7 +182,7 @@
 
         if (!file_exists("$langdir/$currentfile")) {
             if (!touch("$langdir/$currentfile")) {
-                echo "<p><font color=red>".get_string("filemissing", "", "$langdir/$currentfile")."</font></p>";
+                echo "<p><font color=\"red\">".get_string("filemissing", "", "$langdir/$currentfile")."</font></p>";
                 continue;
             }
         }
@@ -193,7 +193,7 @@
             fclose($f);
         } else {
             $editable = false;
-            echo "<p><font size=1>".get_string("makeeditable", "", "$langdir/$currentfile")."</font></p>";
+            echo "<p><font size=\"1\">".get_string("makeeditable", "", "$langdir/$currentfile")."</font></p>";
         }
         error_reporting(7);
 
@@ -215,7 +215,7 @@
         if ($editable) {
             echo "<form name=\"$currentfile\" action=\"lang.php\" method=\"post\">";
         }
-        echo "<table width=\"100%\" cellpadding=2 cellspacing=3 border=0>";
+        echo "<table width=\"100%\" cellpadding=\"2\" cellspacing=\"3\" border=\"0\">";
         foreach ($enstring as $key => $envalue) {
             $envalue = nl2br(htmlspecialchars($envalue));
             $envalue = preg_replace('/(\$a\-\&gt;[a-zA-Z0-9]*|\$a)/', '<b>$0</b>', $envalue);  // Make variables bold. 
@@ -223,8 +223,8 @@
             $envalue = str_replace("\\","",$envalue);              // Delete all slashes
 
             echo "\n\n<tr>";
-            echo "<td dir=ltr lang=en width=20% bgcolor=\"$THEME->cellheading\" nowrap valign=top>$key</td>\n";
-            echo "<td dir=ltr lang=en width=40% bgcolor=\"$THEME->cellheading\" valign=top>$envalue</td>\n";
+            echo "<td dir=\"ltr\" lang=\"en\" width=\"20%\" bgcolor=\"$THEME->cellheading\" nowrap=\"nowrap\" valign=\"top\">$key</td>\n";
+            echo "<td dir=\"ltr\" lang=\"en\" width=\"40%\" bgcolor=\"$THEME->cellheading\" valign=\"top\">$envalue</td>\n";
 
             $value = $string[$key];
             $value = str_replace("\r","",$value);              // Bad character caused by Windows
@@ -239,7 +239,7 @@
             $cellcolour = $value ? $THEME->cellcontent: $THEME->highlight;
 
             if ($editable) {
-                echo "<td width=40% bgcolor=\"$cellcolour\" valign=top>\n";
+                echo "<td width=\"40%\" bgcolor=\"$cellcolour\" valign=\"top\">\n";
                 if (isset($string[$key])) {
                     $valuelen = strlen($value);
                 } else {
@@ -255,14 +255,14 @@
                     }
                     echo "<input type=\"text\" name=\"string-$key\" value=\"$value\" size=\"$cols\"></td>";
                 }
-                echo "</TD>\n";
+                echo "</td>\n";
 
             } else {
-                echo "<td width=40% bgcolor=\"$cellcolour\" valign=top>$value</td>\n";
+                echo "<td width=\"40%\" bgcolor=\"$cellcolour\" valign=\"top\">$value</td>\n";
             }
         }
         if ($editable) {
-            echo "<tr><td colspan=2>&nbsp;<td><br />";
+            echo "<tr><td colspan=\"2\">&nbsp;<td><br />";
             echo "    <input type=\"hidden\" name=\"currentfile\" value=\"$currentfile\">";
             echo "    <input type=\"hidden\" name=\"mode\" value=\"compare\">";
             echo "    <input type=\"submit\" name=\"update\" value=\"".get_string("savechanges").": $currentfile\">";

diff --git a/admin/modules.php b/admin/modules.php
index 6b151372dd71e3109b2500929b08e83e1cabcc6f..434744dc8dbb7a871a76e97848f01927a0554396 100644 (file)
--- a/admin/modules.php
+++ b/admin/modules.php
@@ -160,7 +160,7 @@
 
     foreach ($modulebyname as $modulename => $module) {
 
-        $icon = "<img src=\"$modpixpath/$module->name/icon.gif\" hspace=10 height=16 width=16 border=0>";
+        $icon = "<img src=\"$modpixpath/$module->name/icon.gif\" hspace=\"10\" height=\"16\" width=\"16\" border=\"0\">";
 
         $delete = "<a href=\"modules.php?delete=$module->name\">$strdelete</a>";
 
@@ -174,11 +174,11 @@
 
         if ($module->visible) {
             $visible = "<a href=\"modules.php?hide=$module->name\" title=\"$strhide\">".
-                       "<img src=\"$pixpath/i/hide.gif\" align=\"absmiddle\" height=16 width=16 border=0></a>";
+                       "<img src=\"$pixpath/i/hide.gif\" align=\"absmiddle\" height=\"16\" width=\"16\" border=\"0\"></a>";
             $class = "";
         } else {
             $visible = "<a href=\"modules.php?show=$module->name\" title=\"$strshow\">".
-                       "<img src=\"$pixpath/i/show.gif\" align=\"absmiddle\" height=16 width=16 border=0></a>";
+                       "<img src=\"$pixpath/i/show.gif\" align=\"absmiddle\" height=\"16\" width=\"16\" border=\"0\"></a>";
             $class = "class=\"dimmed_text\"";
         }
         if ($module->name == "forum") {

diff --git a/admin/register.php b/admin/register.php
index ba2c3faf8b660b9df0aa9b1f33aa25490e37d82c..a4938edfc75a0a68046294ec25c853c77dba7671 100644 (file)
--- a/admin/register.php
+++ b/admin/register.php
@@ -49,11 +49,11 @@
 
     print_simple_box_start("center", "", "$THEME->cellheading");
 
-    echo "<form name=\"form\" action=\"http://moodle.org/register/\" method=post>\n";
-    echo "<table cellpadding=9 border=0>\n";
+    echo "<form name=\"form\" action=\"http://moodle.org/register/\" method=\"post\">\n";
+    echo "<table cellpadding=\"9\" border=\"0\">\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>Moodle URL:</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>Moodle URL:</td>\n";
     echo "<td><p>$CFG->wwwroot</td>\n";
     echo "<!-- The following hidden variables are to help prevent fake entries being sent. -->\n";
     echo "<!-- Together they form a key.  If any of these change between updates then the entry  -->\n";
@@ -65,27 +65,27 @@
     echo "<input type=\"hidden\" name=\"lang\" value=\"".current_language()."\">\n";
     echo "</td></tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("currentversion").":</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("currentversion").":</td>\n";
     echo "<td><p>$CFG->release ($CFG->version)</td>\n";
     echo "<input type=\"hidden\" name=\"version\" value=\"$CFG->version\">\n";
     echo "<input type=\"hidden\" name=\"release\" value=\"$CFG->release\">\n";
     echo "</td></tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("fullsitename").":</td>\n";
-    echo "<td><p><input size=50 type=\"text\" name=\"sitename\" value=\"$site->fullname\"></td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("fullsitename").":</td>\n";
+    echo "<td><p><input size=\"50\" type=\"text\" name=\"sitename\" value=\"$site->fullname\"></td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("country").":</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("country").":</td>\n";
     echo "<td><p>";
     choose_from_menu (get_list_of_countries(), "country", $admin->country, get_string("selectacountry")."...", "", "");
     echo "</td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p><a href=\"http://moodle.org/sites\" title=\"See the current list of sites\" target=_blank>".get_string("publicdirectory")."</a>:</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p><a href=\"http://moodle.org/sites\" title=\"See the current list of sites\" target=_blank>".get_string("publicdirectory")."</a>:</td>\n";
     echo "<td><p>";
     $options[0] = get_string("publicdirectory0");
     $options[1] = get_string("publicdirectory1");
@@ -95,22 +95,22 @@
     echo "</td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td colspan=2><hr size=1 noshade>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td colspan=\"2\"><hr size=\"1\" noshade>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("administrator").":</td>\n";
-    echo "<td><p><input size=50 type=\"text\" name=\"adminname\" value=\"".fullname($admin, true)."\"></td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("administrator").":</td>\n";
+    echo "<td><p><input size=\"50\" type=\"text\" name=\"adminname\" value=\"".fullname($admin, true)."\"></td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("email").":</td>\n";
-    echo "<td><p><input size=50 type=\"text\" name=\"adminemail\" value=\"$admin->email\"></td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("email").":</td>\n";
+    echo "<td><p><input size=\"50\" type=\"text\" name=\"adminemail\" value=\"$admin->email\"></td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right><p>".get_string("registrationemail")."</a>:</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\"><p>".get_string("registrationemail")."</a>:</td>\n";
     echo "<td><p>\n";
     $options[0] = get_string("registrationno");
     $options[1] = get_string("registrationyes");
@@ -119,8 +119,8 @@
     echo "</td>\n";
     echo "</tr>\n";
 
-    echo "<tr valign=top>\n";
-    echo "<td align=right>&nbsp;</td>\n";
+    echo "<tr valign=\"top\">\n";
+    echo "<td align=\"right\">&nbsp;</td>\n";
     echo "<td><p><input type=\"submit\" value=\"".get_string("registrationsend")."\"></td>\n";
     echo "</tr>\n";
 

diff --git a/admin/site.html b/admin/site.html
index 9199b65d72f85bd744714eb93280e7f22e53a681..21c58ecf64d838377215ace70145657687f31ed0 100644 (file)
--- a/admin/site.html
+++ b/admin/site.html
@@ -1,20 +1,20 @@
 <form method="post" action="site.php" name="form" <?php echo $onsubmit ?>>
-<table cellpadding=9 cellspacing=0 >
-<tr valign=top>
-    <td align=right><P><?php print_string("fullsitename") ?>:</td>
+<table cellpadding="9" cellspacing="0" >
+<tr valign="top">
+    <td align="right"><p><?php print_string("fullsitename") ?>:</td>
     <td><input type="text" name="fullname" size="50" maxlength="254" value="<?php p($form->fullname) ?>">
     <?php if (isset($err["fullname"])) formerr($err["fullname"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><P><?php print_string("shortsitename") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php print_string("shortsitename") ?>:</td>
     <td><input type="text" name="shortname" size="15" maxlength="15" value="<?php p($form->shortname) ?>">
     <?php if (isset($err["shortname"])) formerr($err["shortname"]); ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><P><?php print_string("frontpagedescription") ?>:</P><br />
-     <font size=1>
+<tr valign="top">
+    <td align="right"><p><?php print_string("frontpagedescription") ?>:</p><br />
+     <font size="1">
      <?PHP
         helpbutton("writing", get_string("helpwriting"), "moodle", true, true);
         echo "<br />";
@@ -29,8 +29,8 @@
     </td>
     <td><?php  print_textarea($usehtmleditor, 20, 50, 600, 400, "summary", $form->summary); ?></td>
 </tr>
-<tr valign=top>
-        <td align=right><P><?php print_string("frontpageformat") ?>:</td>
+<tr valign="top">
+        <td align="right"><p><?php print_string("frontpageformat") ?>:</td>
         <td><?php 
            $options = array("0"  =>  get_string("frontpagenews"),
                             "1"  =>  get_string("frontpagecourselist"),
@@ -45,8 +45,8 @@
         ?>
         </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");
@@ -65,8 +65,8 @@
         ?>
         </td>
 </tr>
-<tr valign=top>
-    <td align="right"><P><?php  print_string("sitesection") ?>:</td>
+<tr valign="top">
+    <td align="right"><p><?php  print_string("sitesection") ?>:</td>
     <td><?php
     unset($choices);
     $choices["0"] = get_string("no");
@@ -75,29 +75,29 @@
     ?>
     </td>
 </tr>
-<tr valign=top>
-    <td align=right><p><?php print_string("wordforteacher") ?>:</td>
+<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>
+<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>
+<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>
+<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"]); ?>

diff --git a/admin/timezone.php b/admin/timezone.php
index 17d4bfcdc7ef58c4a8b27779f8b1302ed550e95b..1fb3740f91b61549a4bc36e05be35485b645ab0f 100644 (file)
--- a/admin/timezone.php
+++ b/admin/timezone.php
@@ -47,10 +47,10 @@
         }
     }
 
-    echo "<center><form action=timezone.php method=get>";
+    echo "<center><form action=\"timezone\".php method=\"get\">";
     echo "$strusers ($strall): ";
     choose_from_menu ($timezones, "zone", $user->timezone, get_string("serverlocaltime"), "", "99");
-    echo "<input type=submit value=\"$strsavechanges\">";
+    echo "<input type=\"submit\" value=\"$strsavechanges\">";
     echo "</form>";
 
     print_footer();

diff --git a/admin/uploaduser.php b/admin/uploaduser.php
index 4575e8e6a5fd1e616aa31f5f74be9eca91a8825f..6817b73158cc505af42c3c04ef3fb71b7a48370f 100755 (executable)
--- a/admin/uploaduser.php
+++ b/admin/uploaduser.php
@@ -258,7 +258,7 @@
     echo '<center>';
     echo '<form method="post" enctype="multipart/form-data" action="uploaduser.php">'.
          $strchoose.':<input type="hidden" name="MAX_FILE_SIZE" value="'.$maxuploadsize.'">'.
-         '<input type="file" name="userfile" size=30>'.
+         '<input type="file" name="userfile" size="30">'.
          '<input type="submit" value="'.$struploadusers.'">'.
          '</form></br>';
     echo '</center>';

diff --git a/admin/user.php b/admin/user.php
index 8ac5b4d364d87af795d0953171da0146188e6f3f..c926b5f4c15b0d392576885da5794f89b435a07e 100644 (file)
--- a/admin/user.php
+++ b/admin/user.php
@@ -323,10 +323,10 @@
                                         $confirmbutton);
             }
 
-            echo "<table align=center cellpadding=10><tr><td>";
-            echo "<form action=user.php method=post>";
-            echo "<input type=text name=search value=\"$search\" size=20>";
-            echo "<input type=submit value=\"$strsearch\">";
+            echo "<table align=\"center\" cellpadding=\"10\"><tr><td>";
+            echo "<form action=\"user.php\" method=\"post\">";
+            echo "<input type=\"text\" name=\"search\" value=\"$search\" size=\"20\">";
+            echo "<input type=\"submit\" value=\"$strsearch\">";
             if ($search) {
                 echo "<input type=\"button\" onclick=\"document.location='user.php';\" value=\"$strshowallusers\">";
             }

diff --git a/admin/users.php b/admin/users.php
index a7b29f1d415f7bac51a0f90bf3530b8209ce6adb..01f962714d4e22f73a2c8ba239fffc715c5e9331 100644 (file)
--- a/admin/users.php
+++ b/admin/users.php
@@ -39,7 +39,7 @@
     $table->data[] = array("<b><a href=\"../course/index.php?edit=off\">".get_string("assignstudents")."</a></b>",
                            get_string("adminhelpassignstudents"));
     $table->data[] = array("<b><a href=\"../course/index.php?edit=on\">".get_string("assignteachers")."</a></b>",
-                           get_string("adminhelpassignteachers")." <img src=\"../pix/t/user.gif\" height=11 width=11>");
+                           get_string("adminhelpassignteachers")." <img src=\"../pix/t/user.gif\" height=\"11\" width=\"11\">");
     $table->data[] = array("<b><a href=\"creators.php\">".get_string("assigncreators")."</a></b>",
                            get_string("adminhelpassigncreators"));
     $table->data[] = array("<b><a href=\"admin.php\">".get_string("assignadmins")."</a></b>",