From b4510d5333f990991b1abc3ccbe3c05c3eae0c01 Mon Sep 17 00:00:00 2001
From: gustav_delius <gustav_delius>
Date: Sun, 12 Sep 2004 22:23:51 +0000
Subject: [PATCH] closed some input tags

---
 blocks/course_list/config.html       |  2 +-
 blocks/online_users/config.html      |  2 +-
 mod/assignment/submissions.php       | 16 ++++----
 mod/attendance/add.php               | 20 +++++-----
 mod/attendance/config.html           |  8 ++--
 mod/attendance/mod.html              | 20 +++++-----
 mod/attendance/teacheredit.php       |  6 +--
 mod/attendance/viewall.php           |  2 +-
 mod/attendance/viewweek.php          |  2 +-
 mod/chat/config.html                 | 16 ++++----
 mod/chat/gui_header/chatinput.php    |  8 ++--
 mod/chat/gui_header_js/chatinput.php | 10 ++---
 mod/chat/gui_push_js/chatinput.php   |  8 ++--
 mod/chat/gui_sockets/chatinput.php   |  2 +-
 mod/chat/gui_text/index.php          | 10 ++---
 mod/chat/lib.php                     |  2 +-
 mod/chat/mod.html                    | 20 +++++-----
 mod/choice/mod.html                  | 54 +++++++++++++--------------
 mod/dialogue/dialogues.php           |  4 +-
 mod/dialogue/locallib.php            | 10 ++---
 mod/dialogue/mod.html                | 18 ++++-----
 mod/dialogue/view.php                |  6 +--
 mod/exercise/assessments.php         | 16 ++++----
 mod/exercise/locallib.php            | 10 ++---
 mod/exercise/mod.html                | 22 +++++------
 mod/exercise/submissions.php         | 10 ++---
 mod/exercise/view.php                |  6 +--
 mod/forum/config.html                |  8 ++--
 mod/forum/lib.php                    | 16 ++++----
 mod/forum/mod.html                   | 56 ++++++++++++++--------------
 mod/forum/post.html                  | 24 ++++++------
 mod/forum/prune.html                 |  8 ++--
 32 files changed, 211 insertions(+), 211 deletions(-)

diff --git a/blocks/course_list/config.html b/blocks/course_list/config.html
index a9aa62babe..d6f1a3e5c5 100644
--- a/blocks/course_list/config.html
+++ b/blocks/course_list/config.html
@@ -24,7 +24,7 @@
 </tr>
 <tr>
     <td colspan="3" align="center">
-    <input type="submit" value="<?php print_string("savechanges") ?>"></td>
+    <input type="submit" value="<?php print_string("savechanges") ?>" /></td>
 </tr>
 </table>
 
diff --git a/blocks/online_users/config.html b/blocks/online_users/config.html
index 3ba2c952a1..85fa3872f2 100644
--- a/blocks/online_users/config.html
+++ b/blocks/online_users/config.html
@@ -17,7 +17,7 @@
 </tr>
 <tr>
     <td colspan="3" align="center">
-	<input type="submit" value="<?php print_string("savechanges") ?>"></td>
+	<input type="submit" value="<?php print_string("savechanges") ?>" /></td>
 </tr>
 </table>
 
diff --git a/mod/assignment/submissions.php b/mod/assignment/submissions.php
index 4371ffe4c6..26c922915f 100644
--- a/mod/assignment/submissions.php
+++ b/mod/assignment/submissions.php
@@ -168,10 +168,10 @@
     if ($allowedtograde) {
         echo '<form action="submissions.php" method="post">';
         echo "<center>";
-        echo "<input type=hidden name=sort value=\"$sort\">";
-        echo "<input type=hidden name=timenow value=\"$timenow\">";
-        echo "<input type=hidden name=id value=\"$assignment->id\">";
-        echo "<input type=submit value=\"$strsaveallfeedback\">";
+        echo "<input type=hidden name=sort value=\"$sort\" />";
+        echo "<input type=hidden name=timenow value=\"$timenow\" />";
+        echo "<input type=hidden name=id value=\"$assignment->id\" />";
+        echo "<input type=submit value=\"$strsaveallfeedback\" />";
         echo "</center>";
     }
     
@@ -186,10 +186,10 @@
 
     if ($allowedtograde) {
         echo "<center>";
-        echo "<input type=hidden name=sort value=\"$sort\">";
-        echo "<input type=hidden name=timenow value=\"$timenow\">";
-        echo "<input type=hidden name=id value=\"$assignment->id\">";
-        echo "<input type=submit value=\"$strsaveallfeedback\">";
+        echo "<input type=hidden name=sort value=\"$sort\" />";
+        echo "<input type=hidden name=timenow value=\"$timenow\" />";
+        echo "<input type=hidden name=id value=\"$assignment->id\" />";
+        echo "<input type=submit value=\"$strsaveallfeedback\" />";
         echo "</center>";
         echo "</form>";
     }
diff --git a/mod/attendance/add.php b/mod/attendance/add.php
index 8ed49418c2..2b924f0731 100644
--- a/mod/attendance/add.php
+++ b/mod/attendance/add.php
@@ -183,19 +183,19 @@ $form->hours = isset($CFG->attendance_default_hours)?$CFG->attendance_default_ho
     <td align=right><p><b><?php print_string("choosedays", "attendance") ?>:</b></p></td>
     <td colspan="3">
     <?php print_string("sunday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="sun" >
+    <input type="checkbox" name="sun" />
     <?php print_string("monday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="mon" "checked">
+    <input type="checkbox" name="mon" checked="checked" />
     <?php print_string("tuesday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="tue" "checked">
+    <input type="checkbox" name="tue" checked="checked" />
     <?php print_string("wednesday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="wed" "checked">
+    <input type="checkbox" name="wed" checked="checked" />
     <?php print_string("thursday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="thu" "checked">
+    <input type="checkbox" name="thu" checked="checked" />
     <?php print_string("friday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="fri" "checked">
+    <input type="checkbox" name="fri" checked="checked" />
     <?php print_string("saturday","attendance"); echo ":"; ?>
-    <input type="checkbox" name="sat" >
+    <input type="checkbox" name="sat" />
 <?php helpbutton("choosedays", get_string("choosedays","attendance"), "attendance");?>
     </td>
 </tr>
@@ -210,7 +210,7 @@ $form->hours = isset($CFG->attendance_default_hours)?$CFG->attendance_default_ho
         choose_from_menu($options, "dynsection", "", "");
         helpbutton("dynsection", get_string("dynamicsectionmulti","attendance"), "attendance");
 ?>
-<!--      <input type="checkbox" name="dynsection" <?php echo !empty($form->dynsection) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="dynsection" <?php echo !empty($form->dynsection) ? 'checked' : '' ?> /> -->
 </td>
 </tr>
 <tr valign=top>
@@ -225,7 +225,7 @@ $form->hours = isset($CFG->attendance_default_hours)?$CFG->attendance_default_ho
 ?>
 
 
-<!--      <input type="checkbox" name="autoattend" <?php echo !empty($form->autoattend) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="autoattend" <?php echo !empty($form->autoattend) ? 'checked' : '' ?> /> -->
     </td>
 </tr>
 <?php // starting with 2 to allow for the nothing value in choose_from_menu to be the default of 1
@@ -248,7 +248,7 @@ for ($i=2;$i<=24;$i++){ $opt[$i] = $i; } ?>
         helpbutton("grade", get_string("gradevalue","attendance"), "attendance");
 ?>
 
-<!--      <input type="checkbox" name="grade" <?php echo !empty($form->grade) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="grade" <?php echo !empty($form->grade) ? 'checked' : '' ?> /> -->
     </td>
 </tr>
 <?php // starting with 2 to allow for the nothing value in choose_from_menu to be the default of 1
diff --git a/mod/attendance/config.html b/mod/attendance/config.html
index 3948df19cd..a31d9b209f 100644
--- a/mod/attendance/config.html
+++ b/mod/attendance/config.html
@@ -57,9 +57,9 @@
 	    if ($CFG->attendance_default_student_status == 1) {$r2c="checked";}
 	    elseif ($CFG->attendance_default_student_status == 2) {$r3c="checked";}
 	    else {$r1c="checked";}
-      $radio1="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"0\" ".$r1c.">";
- 	    $radio2="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"1\" ".$r2c.">";
-      $radio3="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"2\" ".$r3c.">";  	
+      $radio1="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"0\" ".$r1c." />";
+ 	    $radio2="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"1\" ".$r2c." />";
+      $radio3="<input type=\"radio\" name=\"attendance_default_student_status\" value=\"2\" ".$r3c." />";  	
       echo "<table><tr><td align=\"left\" nowrap class=\"generaltablecell\">".
         $radio1.$P."</td>\n";
       echo "<td align=\"left\" nowrap class=\"generaltablecell\">".
@@ -109,7 +109,7 @@
 
 <tr>
     <td colspan=3 align=center>
-	<input type="submit" value="<?php print_string("savechanges") ?>"></td>
+	<input type="submit" value="<?php print_string("savechanges") ?>" /></td>
 </tr>
 </table>
 
diff --git a/mod/attendance/mod.html b/mod/attendance/mod.html
index 76e76009f9..596f332f22 100755
--- a/mod/attendance/mod.html
+++ b/mod/attendance/mod.html
@@ -41,7 +41,7 @@ if (empty($form->id)) {
 <tr valign=top>
     <td align=right><p><b><?php print_string("name") ?>:</b></p></td>
     <td colspan="3">
-        <input type="text" name="name" size=60 value="<?php p($form->notes)?>">
+        <input type="text" name="name" size=60 value="<?php p($form->notes)?>" />
     </td>
 </tr>
 
@@ -59,7 +59,7 @@ if (empty($form->id)) {
         choose_from_menu($options, "dynsection", "$form->dynsection", "");
         helpbutton("dynsection", get_string("dynamicsection","attendance"), "attendance");
 ?>
-<!--      <input type="checkbox" name="dynsection" <?php echo !empty($form->dynsection) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="dynsection" <?php echo !empty($form->dynsection) ? 'checked' : '' ?> /> -->
     </TD>
 </tr>
 <tr valign=top>
@@ -74,7 +74,7 @@ if (empty($form->id)) {
         helpbutton("autoattend", get_string("autoattend","attendance"), "attendance");
 ?>
 
-<!--      <input type="checkbox" name="autoattend" <?php echo !empty($form->autoattend) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="autoattend" <?php echo !empty($form->autoattend) ? 'checked' : '' ?> /> -->
     </TD>
 </tr>
 <?php // starting with 2 to allow for the nothing value in choose_from_menu to be the default of 1
@@ -95,7 +95,7 @@ for ($i=2;$i<=24;$i++){ $opt[$i] = $i; } ?>
         choose_from_menu($options, "grade", "$form->grade", "");
         helpbutton("grade", get_string("gradevalue","attendance"), "attendance");
 ?>
-<!--      <input type="checkbox" name="grade" <?php echo !empty($form->grade) ? 'checked' : '' ?> > -->
+<!--      <input type="checkbox" name="grade" <?php echo !empty($form->grade) ? 'checked' : '' ?> /> -->
     </TD>
 </tr>
 <?php // starting with 2 to allow for the nothing value in choose_from_menu to be the default of 1
@@ -178,17 +178,17 @@ if ($form->hours >1) {
     if ($rollstatus==1) {$r2c="checked";}
     elseif ($rollstatus==2) {$r3c="checked";}
     else {$r1c="checked";}
-    $radio1="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"0\" ".$r1c.">";
-	  $radio2="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"1\" ".$r2c.">";
-    $radio3="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"2\" ".$r3c.">";  	
+    $radio1="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"0\" ".$r1c." />";
+	  $radio2="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"1\" ".$r2c." />";
+    $radio3="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"2\" ".$r3c." />";  	
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-left: 1px dotted; border-top: 1px solid;\">".$radio1."</td>\n";
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-top: 1px solid;\">".$radio2."</td>\n";
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-top: 1px solid;\">".$radio3."</td>\n";
 	} // for loop
     echo "</tr>\n";
-//      $radio1="<input type=\"radio\" name=\"student_".$student->id."\" value=\"0\" checked>";
-//	    $radio2="<input type=\"radio\" name=\"student_".$student->id."\" value=\"1\">";
-//      $radio3="<input type=\"radio\" name=\"student_".$student->id."\" value=\"2\">";
+//      $radio1="<input type=\"radio\" name=\"student_".$student->id."\" value=\"0\" checked />";
+//	    $radio2="<input type=\"radio\" name=\"student_".$student->id."\" value=\"1\" />";
+//      $radio3="<input type=\"radio\" name=\"student_".$student->id."\" value=\"2\" />";
 //      $table->data[$i]=array($student->lastname, $student->firstname,
 //	      $student->idnumber, $radio1,$radio2,$radio3);
 //      $i++;
diff --git a/mod/attendance/teacheredit.php b/mod/attendance/teacheredit.php
index 6f308ceeeb..338895b372 100755
--- a/mod/attendance/teacheredit.php
+++ b/mod/attendance/teacheredit.php
@@ -182,9 +182,9 @@ if ($form->hours >1) {
     if ($rollstatus==1) {$r2c="checked";}
     elseif ($rollstatus==2) {$r3c="checked";}
     else {$r1c="checked";}
-    $radio1="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"0\" ".$r1c.">";
-	  $radio2="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"1\" ".$r2c.">";
-    $radio3="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"2\" ".$r3c.">";  	
+    $radio1="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"0\" ".$r1c." />";
+	  $radio2="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"1\" ".$r2c." />";
+    $radio3="<input type=\"radio\" name=\"student_".$student->id."_".$j."\" value=\"2\" ".$r3c." />";  	
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-left: 1px dotted; border-top: 1px solid;\">".$radio1."</td>\n";
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-top: 1px solid;\">".$radio2."</td>\n";
     echo "<td align=\"left\" nowrap class=\"generaltablecell\" style=\"border-top: 1px solid;\">".$radio3."</td>\n";
diff --git a/mod/attendance/viewall.php b/mod/attendance/viewall.php
index d2a2025a90..e17b3a4aca 100644
--- a/mod/attendance/viewall.php
+++ b/mod/attendance/viewall.php
@@ -442,7 +442,7 @@ if ($onepage) {$page++; echo "<br /> <br />\n"; }
   if (($numhours-4) > 255) {
     echo "<form><input type=\"button\" value=\"".get_string("downloadexcelfull", "attendance").
     "\" onclick=\"alert('Sorry, you have more than 251 days on this report.  This will not fit into an Excel Spreadsheet. ".
-    " Please try downloading the report week by week instead.')\"></form>";
+    " Please try downloading the report week by week instead.')\" /></form>";
   } else {
     $options["id"] = "$course->id";
     $options["download"] = "xls";
diff --git a/mod/attendance/viewweek.php b/mod/attendance/viewweek.php
index e5880a3837..9c9311afd8 100644
--- a/mod/attendance/viewweek.php
+++ b/mod/attendance/viewweek.php
@@ -449,7 +449,7 @@ if ($onepage) {$page++; echo "<br /> <br />\n"; }
   if (($numhours-4) > 255) {
     echo "<form><input type=\"button\" value=\"".get_string("downloadexcelfull", "attendance").
     "\" onclick=\"alert('Sorry, you have more than 251 days on this report.  This will not fit into an Excel Spreadsheet. ".
-    " Please try downloading the report week by week instead.')\"></form>";
+    " Please try downloading the report week by week instead.')\" /></form>";
   } else {
     $options["id"] = "$id";
     $options["download"] = "xls";
diff --git a/mod/chat/config.html b/mod/chat/config.html
index a8e5c5408d..c713b724d5 100644
--- a/mod/chat/config.html
+++ b/mod/chat/config.html
@@ -27,7 +27,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_refresh_userlist:</td>
 	<td>
-    <input name="chat_refresh_userlist" type="text" size="5" value="<?php p($CFG->chat_refresh_userlist) ?>">
+    <input name="chat_refresh_userlist" type="text" size="5" value="<?php p($CFG->chat_refresh_userlist) ?>" />
     </td>
     <td>
     <?php print_string("configrefreshuserlist", "chat") ?>
@@ -36,7 +36,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_old_ping:</td>
 	<td>
-    <input name="chat_old_ping" type="text" size="5" value="<?php p($CFG->chat_old_ping) ?>">
+    <input name="chat_old_ping" type="text" size="5" value="<?php p($CFG->chat_old_ping) ?>" />
     </td>
     <td>
     <?php print_string("configoldping", "chat") ?>
@@ -53,7 +53,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_refresh_room:</td>
 	<td>
-    <input name="chat_refresh_room" type="text" size="5" value="<?php p($CFG->chat_refresh_room) ?>">
+    <input name="chat_refresh_room" type="text" size="5" value="<?php p($CFG->chat_refresh_room) ?>" />
     </td>
     <td>
     <?php print_string("configrefreshroom", "chat") ?>
@@ -70,7 +70,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_serverhost:</td>
 	<td>
-    <input name="chat_serverhost" type="text" size="20" value="<?php p($CFG->chat_serverhost) ?>">
+    <input name="chat_serverhost" type="text" size="20" value="<?php p($CFG->chat_serverhost) ?>" />
     </td>
     <td>
     <?php print_string("configserverhost", "chat") ?>
@@ -79,7 +79,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_serverip:</td>
 	<td>
-    <input name="chat_serverip" type="text" size="16" value="<?php p($CFG->chat_serverip) ?>">
+    <input name="chat_serverip" type="text" size="16" value="<?php p($CFG->chat_serverip) ?>" />
     </td>
     <td>
     <?php print_string("configserverip", "chat") ?>
@@ -88,7 +88,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_serverport:</td>
 	<td>
-    <input name="chat_serverport" type="text" size="5" value="<?php p($CFG->chat_serverport) ?>">
+    <input name="chat_serverport" type="text" size="5" value="<?php p($CFG->chat_serverport) ?>" />
     </td>
     <td>
     <?php print_string("configserverport", "chat") ?>
@@ -97,7 +97,7 @@
 <tr valign="top">
 	<td align="right"><p>chat_servermax:</td>
 	<td>
-    <input name="chat_servermax" type="text" size="5" value="<?php p($CFG->chat_servermax) ?>">
+    <input name="chat_servermax" type="text" size="5" value="<?php p($CFG->chat_servermax) ?>" />
     </td>
     <td>
     <?php print_string("configservermax", "chat") ?>
@@ -106,7 +106,7 @@
 
 <tr>
     <td colspan="3" align="center">
-	<input type="submit" value="<?php print_string("savechanges") ?>"></td>
+	<input type="submit" value="<?php print_string("savechanges") ?>" /></td>
 </tr>
 </table>
 
diff --git a/mod/chat/gui_header/chatinput.php b/mod/chat/gui_header/chatinput.php
index 9d65bccb4d..83338b5927 100644
--- a/mod/chat/gui_header/chatinput.php
+++ b/mod/chat/gui_header/chatinput.php
@@ -12,10 +12,10 @@ if ($arsc_my = arsc_getdatafromsid($arsc_sid))
   echo $arsc_parameters["htmlhead_msginput"];
   ?>
     <form action="../shared/chatins.php" method="POST" name="f">
-     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>">
-     <input type="hidden" name="arsc_chatversion" value="header">
-     <input type="text" name="arsc_message" size="50" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>">
-     <input type="submit" value="<?php echo $arsc_lang["sendmessage"]; ?>">
+     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>" />
+     <input type="hidden" name="arsc_chatversion" value="header" />
+     <input type="text" name="arsc_message" size="50" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>" />
+     <input type="submit" value="<?php echo $arsc_lang["sendmessage"]; ?>" />
     </form>
    </body>
   </html>
diff --git a/mod/chat/gui_header_js/chatinput.php b/mod/chat/gui_header_js/chatinput.php
index c9e7b3514b..4c2461e42b 100644
--- a/mod/chat/gui_header_js/chatinput.php
+++ b/mod/chat/gui_header_js/chatinput.php
@@ -47,16 +47,16 @@ function empty_field_and_submit() {
 
 <form action="../insert.php" method="GET" target="empty" name="f" 
       OnSubmit="return empty_field_and_submit()">
-&gt;&gt;<input type="text" name="chat_message" size="60" value="<?php echo $chat_pretext; ?>">
+&gt;&gt;<input type="text" name="chat_message" size="60" value="<?php echo $chat_pretext; ?>" />
 <?php helpbutton("chatting", get_string("helpchatting", "chat"), "chat", true, false); ?>
 </form>
 
 <form action="../insert.php" method="GET" target="empty" name="fdummy" 
       OnSubmit="return empty_field_and_submit()">
-    <input type="hidden" name="chat_sid" value="<?php echo $chat_sid ?>">
-    <input type="hidden" name="groupid" value="<?php echo $groupid ?>">
-    <input type="hidden" name="chat_version" value="header_js">
-    <input type="hidden" name="chat_message">
+    <input type="hidden" name="chat_sid" value="<?php echo $chat_sid ?>" />
+    <input type="hidden" name="groupid" value="<?php echo $groupid ?>" />
+    <input type="hidden" name="chat_version" value="header_js" />
+    <input type="hidden" name="chat_message" />
 </form>
 
 </body>
diff --git a/mod/chat/gui_push_js/chatinput.php b/mod/chat/gui_push_js/chatinput.php
index ac790d5ba9..605a8e1179 100644
--- a/mod/chat/gui_push_js/chatinput.php
+++ b/mod/chat/gui_push_js/chatinput.php
@@ -12,12 +12,12 @@ if ($arsc_my = arsc_getdatafromsid($arsc_sid))
   echo $arsc_parameters["htmlhead_msginput_js"];
   ?>
     <form action="../shared/chatins.php" method="GET" target="empty" name="f" OnSubmit="return empty_field_and_submit()">
-     <input type="text" name="arsc_message" size="50" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>">
+     <input type="text" name="arsc_message" size="50" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>" />
     </form>
     <form action="../shared/chatins.php" method="GET" target="empty" name="fdummy" OnSubmit="return empty_field_and_submit()">
-     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>">
-     <input type="hidden" name="arsc_chatversion" value="header_js">
-     <input type="hidden" name="arsc_message">
+     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>" />
+     <input type="hidden" name="arsc_chatversion" value="header_js" />
+     <input type="hidden" name="arsc_message" />
     </form>
    </body>
   </html>
diff --git a/mod/chat/gui_sockets/chatinput.php b/mod/chat/gui_sockets/chatinput.php
index ae69549e1c..1b896ca277 100644
--- a/mod/chat/gui_sockets/chatinput.php
+++ b/mod/chat/gui_sockets/chatinput.php
@@ -67,7 +67,7 @@ function reloadusers() {
 <form action="<?php echo "http://$CFG->chat_serverhost:$CFG->chat_serverport"; ?>" method="GET" target="empty" name="f" onsubmit="return empty_field_and_submit()">
 -->
 <form action="../insert.php" method="get" target="empty" id="inputform" onsubmit="return empty_field_and_submit();">
-&gt;&gt; <input type="text" name="chat_message" size="60" value="<?php echo $chat_pretext; ?>">
+&gt;&gt; <input type="text" name="chat_message" size="60" value="<?php echo $chat_pretext; ?>" />
 <?php helpbutton("chatting", get_string("helpchatting", "chat"), "chat", true, false); ?>
 </form>
 
diff --git a/mod/chat/gui_text/index.php b/mod/chat/gui_text/index.php
index 00b65b5615..403b2c824d 100644
--- a/mod/chat/gui_text/index.php
+++ b/mod/chat/gui_text/index.php
@@ -65,11 +65,11 @@ if ($arsc_my = arsc_getdatafromsid($arsc_sid))
    <body>
     <form action="../shared/chatins.php" method="POST">
      <a href="index.php?arsc_sid=<?php echo $arsc_sid; ?>&arsc_lastid=<?php echo $arsc_lastid; ?>"><?php echo $arsc_lang["refreshmessages"]; ?></a>
-     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>">
-     <input type="hidden" name="arsc_lastid" value="<?php echo $arsc_lastid; ?>">
-     <input type="hidden" name="arsc_chatversion" value="text">
-     <input type="text" name="arsc_message" size="30" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>">
-     <input type="submit" value="<?php echo $arsc_lang["sendmessage"]; ?>">
+     <input type="hidden" name="arsc_sid" value="<?php echo $arsc_sid; ?>" />
+     <input type="hidden" name="arsc_lastid" value="<?php echo $arsc_lastid; ?>" />
+     <input type="hidden" name="arsc_chatversion" value="text" />
+     <input type="text" name="arsc_message" size="30" maxlength="<?php echo $arsc_parameters["input_maxsize"]; ?>" value="<?php echo $arsc_pretext; ?>" />
+     <input type="submit" value="<?php echo $arsc_lang["sendmessage"]; ?>" />
      &nbsp;&nbsp;&nbsp;<a href="../logout.php?arsc_sid=<?php echo $arsc_sid; ?>"><?php echo $arsc_lang["leave"]; ?></a>
     </form>
     <?php
diff --git a/mod/chat/lib.php b/mod/chat/lib.php
index 130414d194..fb213a64ac 100644
--- a/mod/chat/lib.php
+++ b/mod/chat/lib.php
@@ -592,7 +592,7 @@ function chat_display_version($version, $browser)
   ?>
   <tr>
    <td valign="top">
-    <input type="radio" name="chat_chatversion" value="<?php echo $version; ?>"<?php echo $checked; ?>>
+    <input type="radio" name="chat_chatversion" value="<?php echo $version; ?>"<?php echo $checked; ?> />
    </td>
    <td valign="top" align="left">
     <font face="Arial" size="2">
diff --git a/mod/chat/mod.html b/mod/chat/mod.html
index 70b3d4bdd1..bee6a1c871 100644
--- a/mod/chat/mod.html
+++ b/mod/chat/mod.html
@@ -24,7 +24,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("chatname", "chat")?>:</b></p></td>
     <td>
-        <input type="text" name="name" size="30" value="<?php p($form->name) ?>">
+        <input type="text" name="name" size="30" value="<?php p($form->name) ?>" />
     </td>
 </tr>
 <tr valign="top">
@@ -99,14 +99,14 @@
 
 </table>
 <center>
-<input type="hidden" name="course"     value="<?php  p($form->course) ?>">
-<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>">
-<input type="hidden" name="section"       value="<?php  p($form->section) ?>">
-<input type="hidden" name="module"     value="<?php  p($form->module) ?>">
-<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>">
-<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>">
-<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>">
-<input type="submit" value="<?php  print_string("savechanges") ?>">
-<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>">
+<input type="hidden" name="course"     value="<?php  p($form->course) ?>" />
+<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>" />
+<input type="hidden" name="section"       value="<?php  p($form->section) ?>" />
+<input type="hidden" name="module"     value="<?php  p($form->module) ?>" />
+<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>" />
+<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>" />
+<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>" />
+<input type="submit" value="<?php  print_string("savechanges") ?>" />
+<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>" />
 </center>
 </form>
diff --git a/mod/choice/mod.html b/mod/choice/mod.html
index 6ee2a07803..0f7fe0b627 100644
--- a/mod/choice/mod.html
+++ b/mod/choice/mod.html
@@ -56,7 +56,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choicename","choice") ?>:</b></p></td>
     <td>
-        <input type="text" name="name" size="30" value="<?php p($form->name) ?>">
+        <input type="text" name="name" size="30" value="<?php p($form->name) ?>" />
     </td>
 </tr>
 
@@ -101,7 +101,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","1") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer1" size="60" value="<?php p($form->answer1) ?>">
+        <input type="text" name="answer1" size="60" value="<?php p($form->answer1) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
@@ -109,7 +109,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","2") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer2" size="60" value="<?php p($form->answer2) ?>">
+        <input type="text" name="answer2" size="60" value="<?php p($form->answer2) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
@@ -117,28 +117,28 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","3") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer3" size="60" value="<?php p($form->answer3) ?>">
+        <input type="text" name="answer3" size="60" value="<?php p($form->answer3) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","4") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer4" size="60" value="<?php p($form->answer4) ?>">
+        <input type="text" name="answer4" size="60" value="<?php p($form->answer4) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","5") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer5" size="60" value="<?php p($form->answer5) ?>">
+        <input type="text" name="answer5" size="60" value="<?php p($form->answer5) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
 <tr valign="top">
     <td align="right"><p><b><?php print_string("choice","choice","6") ?>:</b></p></td>
     <td>
-        <input type="text" name="answer6" size="60" value="<?php p($form->answer6) ?>">
+        <input type="text" name="answer6" size="60" value="<?php p($form->answer6) ?>" />
         <?php helpbutton("options", get_string("modulenameplural", "choice"), "choice") ?>
     </td>
 </tr>
@@ -150,7 +150,7 @@
             var timeitems = ['openday','openmonth','openyear','openhour', 'openminute', 'closeday','closemonth','closeyear','closehour','closeminute'];
         </script>
 
-        <input name="timerestrict" type="checkbox" value="1" onclick="return lockoptions('form','timerestrict', timeitems)" <?php if ($form->timerestrict) echo "checked"; ?> >
+        <input name="timerestrict" type="checkbox" value="1" onclick="return lockoptions('form','timerestrict', timeitems)" <?php if ($form->timerestrict) echo "checked"; ?> />
     <?php
         helpbutton("timerestrict", get_string("timerestrict","choice"), "choice");
      ?>
@@ -173,11 +173,11 @@
         print_time_selector("openhour", "openminute", $form->timeopen);
      ?>
      
-                    <input type="hidden" name="hopenday" value="0">
-                    <input type="hidden" name="hopenmonth" value="0">
-                    <input type="hidden" name="hopenyear" value="0">
-                    <input type="hidden" name="hopenhour" value="0">
-                    <input type="hidden" name="hopenminute" value="0">
+                    <input type="hidden" name="hopenday" value="0" />
+                    <input type="hidden" name="hopenmonth" value="0" />
+                    <input type="hidden" name="hopenyear" value="0" />
+                    <input type="hidden" name="hopenhour" value="0" />
+                    <input type="hidden" name="hopenminute" value="0" />
                 </td>
             </tr>
             <tr>
@@ -192,11 +192,11 @@
         print_time_selector("closehour", "closeminute", $form->timeclose);
      ?>
      
-                    <input type="hidden" name="hcloseday" value="0">
-                    <input type="hidden" name="hclosemonth" value="0">
-                    <input type="hidden" name="hcloseyear" value="0">
-                    <input type="hidden" name="hclosehour" value="0">
-                    <input type="hidden" name="hcloseminute" value="0">
+                    <input type="hidden" name="hcloseday" value="0" />
+                    <input type="hidden" name="hclosemonth" value="0" />
+                    <input type="hidden" name="hcloseyear" value="0" />
+                    <input type="hidden" name="hclosehour" value="0" />
+                    <input type="hidden" name="hcloseminute" value="0" />
 
     <?php
         if (! $form->timerestrict) {
@@ -264,14 +264,14 @@
 </table>
 
 <center>
-<input type="hidden" name="course"     value="<?php p($form->course) ?>">
-<input type="hidden" name="coursemodule"  value="<?php p($form->coursemodule) ?>">
-<input type="hidden" name="section"       value="<?php p($form->section) ?>">
-<input type="hidden" name="module"     value="<?php p($form->module) ?>">
-<input type="hidden" name="modulename" value="<?php p($form->modulename) ?>">
-<input type="hidden" name="instance"   value="<?php p($form->instance) ?>">
-<input type="hidden" name="mode"       value="<?php p($form->mode) ?>">
-<input type="submit" value="<?php print_string("savechanges") ?>">
-<input type="submit" name="cancel" value="<?php print_string("cancel") ?>">
+<input type="hidden" name="course"     value="<?php p($form->course) ?>" />
+<input type="hidden" name="coursemodule"  value="<?php p($form->coursemodule) ?>" />
+<input type="hidden" name="section"       value="<?php p($form->section) ?>" />
+<input type="hidden" name="module"     value="<?php p($form->module) ?>" />
+<input type="hidden" name="modulename" value="<?php p($form->modulename) ?>" />
+<input type="hidden" name="instance"   value="<?php p($form->instance) ?>" />
+<input type="hidden" name="mode"       value="<?php p($form->mode) ?>" />
+<input type="submit" value="<?php print_string("savechanges") ?>" />
+<input type="submit" name="cancel" value="<?php print_string("cancel") ?>" />
 </center>
 </form>
diff --git a/mod/dialogue/dialogues.php b/mod/dialogue/dialogues.php
index 236d21516a..8abaca4ea8 100644
--- a/mod/dialogue/dialogues.php
+++ b/mod/dialogue/dialogues.php
@@ -110,9 +110,9 @@
         echo "<table align=\"center\"border=\"1\" width=\"60%\">\n";
         echo "<tr><td align=\"right\"><b>".get_string("subject", "dialogue")."</b></td>";
         echo "<td><input type=\"text\" size=\"50\" maxsize=\"100\" name=\"subject\" 
-                value=\"\"></td></tr>\n";
+                value=\"\" /></td></tr>\n";
         echo "<tr><td colspan=\"2\" align=\"center\"><input type=\"submit\" value=\"".
-            get_string("addsubject", "dialogue")."\"></td></tr>\n";
+            get_string("addsubject", "dialogue")."\" /></td></tr>\n";
         echo "</table></form>\n";
     }
     
diff --git a/mod/dialogue/locallib.php b/mod/dialogue/locallib.php
index bcc84cfc61..c5dc7f79dd 100644
--- a/mod/dialogue/locallib.php
+++ b/mod/dialogue/locallib.php
@@ -417,8 +417,8 @@ function dialogue_list_conversations_self($dialogue) {
     
     echo "<form name=\"replies\" method=\"post\" action=\"dialogues.php\">\n";
     echo "<input type=\"hidden\" name=\"action\" value=\"insertentries\"/>\n";
-    echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\"/>\n";
-    echo "<input type=\"hidden\" name=\"pane\" value=\"1\"/>\n";
+    echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\" />\n";
+    echo "<input type=\"hidden\" name=\"pane\" value=\"1\" />\n";
 
     // list the conversations requiring a resonse from this user in full
     if ($conversations = dialogue_get_conversations($dialogue, $USER, "lastid != $USER->id AND closed = 0")) {
@@ -531,8 +531,8 @@ function dialogue_print_conversation($dialogue, $conversation) {
     
     echo "<form name=\"replies\" method=\"post\" action=\"dialogues.php\">\n";
     echo "<input type=\"hidden\" name=\"action\" value=\"insertentries\"/>\n";
-    echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\"/>\n";
-    echo "<input type=\"hidden\" name=\"pane\" value=\"2\"/>\n";
+    echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\" />\n";
+    echo "<input type=\"hidden\" name=\"pane\" value=\"2\" />\n";
 
     $showbutton = true;
     print_simple_box_start("center", "", $THEME->cellcontent2);
@@ -597,7 +597,7 @@ function dialogue_print_conversation($dialogue, $conversation) {
     print_simple_box_end();
     if ($showbutton) {
         echo "<hr />\n";
-        echo "<br /><input type=\"submit\" value=\"".get_string("addmynewentry", "dialogue")."\">\n";
+        echo "<br /><input type=\"submit\" value=\"".get_string("addmynewentry", "dialogue")."\" />\n";
     }
     echo "</form>\n";
 }
diff --git a/mod/dialogue/mod.html b/mod/dialogue/mod.html
index 68f9a666b0..94c82bb9bb 100644
--- a/mod/dialogue/mod.html
+++ b/mod/dialogue/mod.html
@@ -95,14 +95,14 @@ print_heading_with_help(get_string("furtherinformation", "dialogue"), "info", "d
 
 </table>
 <center>
-<input type="hidden" name="course"     value="<?php p($form->course) ?>"/>
-<input type="hidden" name="coursemodule"  value="<?php p($form->coursemodule) ?>"/>
-<input type="hidden" name="section"       value="<?php p($form->section) ?>"/>
-<input type="hidden" name="module"     value="<?php p($form->module) ?>"/>
-<input type="hidden" name="modulename" value="<?php p($form->modulename) ?>"/>
-<input type="hidden" name="instance"   value="<?php p($form->instance) ?>"/>
-<input type="hidden" name="mode"       value="<?php p($form->mode) ?>"/>
-<input type="submit" value="<?php print_string("savechanges") ?>"/>
-<input type="submit" name="cancel" value="<?php print_string("cancel") ?>">
+<input type="hidden" name="course"     value="<?php p($form->course) ?>" />
+<input type="hidden" name="coursemodule"  value="<?php p($form->coursemodule) ?>" />
+<input type="hidden" name="section"       value="<?php p($form->section) ?>" />
+<input type="hidden" name="module"     value="<?php p($form->module) ?>" />
+<input type="hidden" name="modulename" value="<?php p($form->modulename) ?>" />
+<input type="hidden" name="instance"   value="<?php p($form->instance) ?>" />
+<input type="hidden" name="mode"       value="<?php p($form->mode) ?>" />
+<input type="submit" value="<?php print_string("savechanges") ?>" />
+<input type="submit" name="cancel" value="<?php print_string("cancel") ?>" />
 </center>
 </form>
diff --git a/mod/dialogue/view.php b/mod/dialogue/view.php
index 56e9d36a0a..10b983566d 100644
--- a/mod/dialogue/view.php
+++ b/mod/dialogue/view.php
@@ -137,8 +137,8 @@
                 if ($names = dialogue_get_available_users($dialogue)) {
                     print_simple_box_start("center");
                     echo "<form name=\"startform\" method=\"post\" action=\"dialogues.php\">\n";
-                    echo "<input type=\"hidden\" name=\"id\"value=\"$cm->id\"/>\n";
-                    echo "<input type=\"hidden\" name=\"action\" value=\"openconversation\"/>\n";
+                    echo "<input type=\"hidden\" name=\"id\"value=\"$cm->id\"/ />\n";
+                    echo "<input type=\"hidden\" name=\"action\" value=\"openconversation\"/ />\n";
                     echo "<table align=\"center\" border=\"0\"><tr>\n";
                     echo "<td align=\"right\"><b>".get_string("openadialoguewith", "dialogue").
                         " : </b></td>\n";
@@ -148,7 +148,7 @@
                     echo "</td></tr>\n";
                     echo "<tr><td align=\"right\"><b>".get_string("subject", "dialogue")." : </b></td>\n";
                     echo "<td><input type=\"text\" size=\"50\" maxsize=\"100\" name=\"subject\" 
-                        value=\"\"></td></tr>\n";
+                        value=\"\" /></td></tr>\n";
                     echo "<tr><td colspan=\"2\" align=\"center\" valign=\"top\"><i>".
                         get_string("typefirstentry", "dialogue")."</i></td></tr>\n";
                     echo "<tr><td valign=\"top\" align=\"right\">\n";
diff --git a/mod/exercise/assessments.php b/mod/exercise/assessments.php
index c831344032..52e26c4a1e 100644
--- a/mod/exercise/assessments.php
+++ b/mod/exercise/assessments.php
@@ -81,10 +81,10 @@
         print_heading(get_string("amend", "exercise")." ".get_string("gradeforstudentsassessment", 
                     "exercise", $course->student));
         echo "<form name=\"amendgrade\" method=\"post\" action=\"assessments.php\">\n";
-        echo "<input type=\"hidden\" name=\"aid\" value=\"$_GET[aid]\">\n";
-        echo "<input type=\"hidden\" name=\"action\" value=\"updategradinggrade\">\n";
-        echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\">\n";
-        echo "<table width=\"50%\" align=\"center\" border=\"1\">\n";
+        echo "<input type=\"hidden\" name=\"aid\" value=\"$_GET[aid]\" />\n";
+        echo "<input type=\"hidden\" name=\"action\" value=\"updategradinggrade\" />\n";
+        echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\" />\n";
+        echo "<table width=\"50%\" align=\"center\" border=\"1\" />\n";
         echo "<tr><td align=\"right\"><b>".get_string("gradeforstudentsassessment", "exercise", 
                 $course->student)." :</td><td>\n";
         // set up coment scale
@@ -263,8 +263,8 @@
         print_heading_with_help(get_string("editingassessmentelements", "exercise"), "elements", "exercise");
         ?>
         <form name="form" method="post" action="assessments.php">
-        <input type="hidden" name="id" value="<?php echo $cm->id ?>">
-        <input type="hidden" name="action" value="insertelements">
+        <input type="hidden" name="id" value="<?php echo $cm->id ?>" />
+        <input type="hidden" name="action" value="insertelements" />
         <center><table cellpadding="5" border="1">
         <?php
         
@@ -436,8 +436,8 @@
 
         ?>
         </table><br />
-        <input type="submit" value="<?php  print_string("savechanges") ?>">
-        <input type="submit" name="cancel" value="<?php  print_string("cancel") ?>">
+        <input type="submit" value="<?php  print_string("savechanges") ?>" />
+        <input type="submit" name="cancel" value="<?php  print_string("cancel") ?>" />
         </center>
 
         </form>
diff --git a/mod/exercise/locallib.php b/mod/exercise/locallib.php
index dca2313ad6..eb7e12799d 100644
--- a/mod/exercise/locallib.php
+++ b/mod/exercise/locallib.php
@@ -2015,13 +2015,13 @@ function exercise_print_assessment_form($exercise, $assessment = false, $allowch
         if (isteacher($course->id)) { 
             // ...show two buttons...to resubmit or not to resubmit
             echo "<input type=\"button\" value=\"".get_string("studentnotallowed", "exercise", $course->student)."\" 
-                onclick=\"document.assessmentform.submit();\">\n";
+                onclick=\"document.assessmentform.submit();\" />\n";
             echo "<input type=\"button\" value=\"".get_string("studentallowedtoresubmit", "exercise", $course->student)."\" 
-                onclick=\"document.assessmentform.resubmit.value='1';document.assessmentform.submit();\">\n";
+                onclick=\"document.assessmentform.resubmit.value='1';document.assessmentform.submit();\" />\n";
             }
         else {
             // ... show save button
-            echo "<input type=\"submit\" value=\"".get_string("savemyassessment", "exercise")."\">\n";
+            echo "<input type=\"submit\" value=\"".get_string("savemyassessment", "exercise")."\" />\n";
             }
         }
     echo "</center></form>\n";
@@ -2782,9 +2782,9 @@ function exercise_print_teacher_assessment_form($exercise, $assessment, $submiss
     // ...and close the table and show two buttons...to resubmit or not to resubmit
     echo "</table>\n";
     echo "<br /><input type=\"button\" value=\"".get_string("studentnotallowed", "exercise", $course->student)."\" 
-        onclick=\"document.assessmentform.submit();\">\n";
+        onclick=\"document.assessmentform.submit();\" />\n";
     echo "<input type=\"button\" value=\"".get_string("studentallowedtoresubmit", "exercise", $course->student)."\" 
-        onclick=\"document.assessmentform.resubmit.value='1';document.assessmentform.submit();\">\n";
+        onclick=\"document.assessmentform.resubmit.value='1';document.assessmentform.submit();\" />\n";
     echo "</center></form>\n";
 }
 
diff --git a/mod/exercise/mod.html b/mod/exercise/mod.html
index c330ef37f0..304d89c7be 100644
--- a/mod/exercise/mod.html
+++ b/mod/exercise/mod.html
@@ -46,7 +46,7 @@
 <tr valign="top">
     <td align="right" width=\"50%\"><p><b><?php  print_string("title", "exercise") ?>:</b></p></td>
     <td width=\"50%\">
-        <input type="text" name="name" size="60" value="<?php  p($form->name) ?>">
+        <input type="text" name="name" size="60" value="<?php  p($form->name) ?>" />
     </td>
 </tr>
 
@@ -144,7 +144,7 @@
 <tr>
     <td align="right"><p><b><?php  print_string("password"); ?>:</b></p></td>
     <td>
-        <input type="text" name="password" size="10" value=""> <?php echo " (".get_string("leavetokeep").")"; ?>
+        <input type="text" name="password" size="10" value="" /> <?php echo " (".get_string("leavetokeep").")"; ?>
 		<?php helpbutton("password", get_string("password"), "exercise"); ?>
     </td>
 </tr>
@@ -212,15 +212,15 @@
 </table>
 <br />
 <center>
-<input type="hidden" name="course"     value="<?php  p($form->course) ?>">
-<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>">
-<input type="hidden" name="section"       value="<?php  p($form->section) ?>">
-<input type="hidden" name="module"     value="<?php  p($form->module) ?>">
-<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>">
-<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>">
-<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>">
-<input type="submit" value="<?php  print_string("savechanges") ?>">
-<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>">
+<input type="hidden" name="course"     value="<?php  p($form->course) ?>" />
+<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>" />
+<input type="hidden" name="section"       value="<?php  p($form->section) ?>" />
+<input type="hidden" name="module"     value="<?php  p($form->module) ?>" />
+<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>" />
+<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>" />
+<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>" />
+<input type="submit" value="<?php  print_string("savechanges") ?>" />
+<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>" />
 <p>
 <p><?php notify(get_string("noteonassessmentelements", "exercise")); ?>
 
diff --git a/mod/exercise/submissions.php b/mod/exercise/submissions.php
index d440f3c51e..09c7ba899d 100644
--- a/mod/exercise/submissions.php
+++ b/mod/exercise/submissions.php
@@ -69,9 +69,9 @@
         print_heading(get_string("amendtitle", "exercise"));
         ?>
         <form name="amendtitleform" action="submissions.php" method="post">
-        <input type="hidden" name="action" value="adminupdatetitle">
-        <input type="hidden" name="id" value="<?php echo $cm->id ?>">
-        <input type="hidden" name="sid" value="<?php echo $_REQUEST['sid'] ?>">
+        <input type="hidden" name="action" value="adminupdatetitle" />
+        <input type="hidden" name="id" value="<?php echo $cm->id ?>" />
+        <input type="hidden" name="sid" value="<?php echo $_REQUEST['sid'] ?>" />
         <center>
         <table celpadding="5" border="1">
         <?php
@@ -80,9 +80,9 @@
         echo "<tr valign=\"top\">\n";
         echo "    <td align=\"right\"><p><b>". get_string("title", "exercise").":</b></p></td>\n";
         echo "    <td>\n";
-        echo "        <input type=\"text\" name=\"title\" size=\"60\" maxlength=\"100\" value=\"$submission->title\">\n";
+        echo "        <input type=\"text\" name=\"title\" size=\"60\" maxlength=\"100\" value=\"$submission->title\" />\n";
         echo "    </td></tr></table>\n";
-        echo "<input type=\"submit\" value=\"".get_string("amendtitle", "exercise")."\">\n";
+        echo "<input type=\"submit\" value=\"".get_string("amendtitle", "exercise")."\" />\n";
         echo "</center></form>\n";
 
         }
diff --git a/mod/exercise/view.php b/mod/exercise/view.php
index b8b96f878b..d54a9c5001 100644
--- a/mod/exercise/view.php
+++ b/mod/exercise/view.php
@@ -251,7 +251,7 @@
 			if (!$correctpass) {
 				print_simple_box_start("center");
 				echo "<form name=\"password\" method=\"post\" action=\"view.php\">\n";
-				echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\">\n";
+				echo "<input type=\"hidden\" name=\"id\" value=\"$cm->id\" />\n";
 				echo "<table cellpadding=\"7px\">";
 				if (isset($_POST['userpassword'])) {
 					echo "<tr align=\"center\" style='color:#DF041E;'><td>".get_string("wrongpassword", "exercise").
@@ -260,13 +260,13 @@
 				echo "<tr align=\"center\"><td>".get_string("passwordprotectedexercise", "exercise", $exercise->name).
                     "</td></tr>";
 				echo "<tr align=\"center\"><td>".get_string("enterpassword", "exercise").
-                    " <input type=\"password\" name=\"userpassword\"></td></tr>";
+                    " <input type=\"password\" name=\"userpassword\" /></td></tr>";
 						
 				echo "<tr align=\"center\"><td>";
 				echo "<input type=\"button\" value=\"".get_string("cancel").
                     "\" onclick=\"parent.location='../../course/view.php?id=$course->id';\">  ";
 				echo "<input type=\"button\" value=\"".get_string("continue").
-                    "\" onclick=\"document.password.submit();\">";
+                    "\" onclick=\"document.password.submit();\" />";
 				echo "</td></tr></table>";
 				print_simple_box_end();
 				exit();
diff --git a/mod/forum/config.html b/mod/forum/config.html
index e9bed36260..f4a92a5b0b 100644
--- a/mod/forum/config.html
+++ b/mod/forum/config.html
@@ -27,7 +27,7 @@
 <tr valign="top">
     <td align="right"><p>forum_shortpost:</td>
     <td>
-    <input name="forum_shortpost" type="text" size="5" value="<?php p($CFG->forum_shortpost) ?>">
+    <input name="forum_shortpost" type="text" size="5" value="<?php p($CFG->forum_shortpost) ?>" />
     </td>
     <td>
     <?php print_string("configshortpost", "forum") ?>
@@ -36,7 +36,7 @@
 <tr valign="top">
     <td align="right"><p>forum_longpost:</td>
     <td>
-    <input name="forum_longpost" type="text" size="5" value="<?php p($CFG->forum_longpost) ?>">
+    <input name="forum_longpost" type="text" size="5" value="<?php p($CFG->forum_longpost) ?>" />
     </td>
     <td>
     <?php print_string("configlongpost", "forum") ?>
@@ -45,7 +45,7 @@
 <tr valign="top">
     <td align="right"><p>forum_manydiscussions:</td>
     <td>
-    <input name="forum_manydiscussions" type="text" size="5" value="<?php p($CFG->forum_manydiscussions) ?>">
+    <input name="forum_manydiscussions" type="text" size="5" value="<?php p($CFG->forum_manydiscussions) ?>" />
     </td>
     <td>
     <?php print_string("configmanydiscussions", "forum") ?>
@@ -95,7 +95,7 @@
 
 <tr>
     <td colspan="3" align="center">
-    <input type="submit" value="<?php print_string("savechanges") ?>"></td>
+    <input type="submit" value="<?php print_string("savechanges") ?>" /></td>
 </tr>
 </table>
 
diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index d5316330d4..d4884f5790 100644
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -1815,20 +1815,20 @@ function forum_print_search_form($course, $search="", $return=false, $type="") {
         $output = "<table border=\"0\" cellpadding=\"0\" cellspacing=\"0\"><tr><td nowrap=\"nowrap\">";
         $output .= "<form name=\"search\" action=\"$CFG->wwwroot/mod/forum/search.php\">";
         $output .= "<font size=\"-1\">";
-        $output .= "<input name=\"search\" type=\"text\" size=\"15\" value=\"$search\">";
-        $output .= "<input value=\"".get_string("searchforums", "forum")."\" type=\"submit\">";
+        $output .= "<input name=\"search\" type=\"text\" size=\"15\" value=\"$search\" />";
+        $output .= "<input value=\"".get_string("searchforums", "forum")."\" type=\"submit\" />";
         $output .= "</font>";
-        $output .= "<input name=\"id\" type=\"hidden\" value=\"$course->id\">";
+        $output .= "<input name=\"id\" type=\"hidden\" value=\"$course->id\" />";
         $output .= "</form>";
         $output .= "</td></tr></table>";
     } else {
         $output = "<table border=\"0\" cellpadding=\"10\" cellspacing=\"0\"><tr><td align=\"center\">";
         $output .= "<form name=\"search\" action=\"$CFG->wwwroot/mod/forum/search.php\">";
         $output .= "<font size=\"-1\">";
-        $output .= "<input name=\"search\" type=\"text\" size=\"15\" value=\"$search\"><br />";
-        $output .= "<input value=\"".get_string("searchforums", "forum")."\" type=\"submit\">";
+        $output .= "<input name=\"search\" type=\"text\" size=\"15\" value=\"$search\" /><br />";
+        $output .= "<input value=\"".get_string("searchforums", "forum")."\" type=\"submit\" />";
         $output .= "</font>";
-        $output .= "<input name=\"id\" type=\"hidden\" value=\"$course->id\">";
+        $output .= "<input name=\"id\" type=\"hidden\" value=\"$course->id\" />";
         $output .= "</form>";
         $output .= "</td></tr></table>";
     }
@@ -2580,7 +2580,7 @@ function forum_print_discussion($course, $forum, $discussion, $post, $mode, $can
             $ratings->allow = ($forum->assessed != 2 or isteacher($course->id));
 
             echo "<form name=\"form\" method=\"post\" action=\"rate.php\">";
-            echo "<input type=\"hidden\" name=\"id\" value=\"$course->id\">";
+            echo "<input type=\"hidden\" name=\"id\" value=\"$course->id\" />";
         }
     }
 
@@ -2613,7 +2613,7 @@ function forum_print_discussion($course, $forum, $discussion, $post, $mode, $can
     }
 
     if ($ratingsmenuused) {
-        echo "<center><input type=\"submit\" value=\"".get_string("sendinratings", "forum")."\">";
+        echo "<center><input type=\"submit\" value=\"".get_string("sendinratings", "forum")."\" />";
         if ($forum->scale < 0) {
             if ($scale = get_record("scale", "id", abs($forum->scale))) {
                 print_scale_menu_helpbutton($course->id, $scale );
diff --git a/mod/forum/mod.html b/mod/forum/mod.html
index b2ef0e43ea..cb20d73231 100644
--- a/mod/forum/mod.html
+++ b/mod/forum/mod.html
@@ -45,7 +45,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("forumname", "forum")?>:</b></p></td>
     <td>
-        <input type="text" name="name" size="30" value="<?php p($form->name) ?>">
+        <input type="text" name="name" size="30" value="<?php p($form->name) ?>" />
     </td>
 </tr>
 <tr valign="top">
@@ -60,10 +60,10 @@
        }
        if ($form->type == "news") {
            print_string("namenews", "forum");
-           echo "<input type=\"hidden\" name=\"type\" value=\"news\">";
+           echo "<input type=\"hidden\" name=\"type\" value=\"news\" />";
        } else if ($form->type == "social") {
            print_string("namesocial", "forum");
-           echo "<input type=\"hidden\" name=\"type\" value=\"social\">";
+           echo "<input type=\"hidden\" name=\"type\" value=\"social\" />";
        } else {
            choose_from_menu($FORUM_TYPES, "type", $form->type, "");
            helpbutton("forumtype", get_string("forumtype", "forum"), "forum");
@@ -180,7 +180,7 @@
             if ($form->assessed) {
                 echo " checked ";
             }
-            echo ">";
+            echo " />";
             echo get_string("ratingsuse", "forum").":";
             echo "<br />";
 
@@ -209,7 +209,7 @@
                 $form->ratingtime = 1;
                 echo " checked ";
             }
-            echo ">";
+            echo " />";
 
             print_string("ratingtime", "forum");
             echo "<table align=\"left\"><tr><td align=\"right\" nowrap=\"nowrap\">";
@@ -222,20 +222,20 @@
             print_time_selector("finishhour", "finishminute", $form->assesstimefinish);
             echo "<br />";
             echo "</td></tr></table>";
-            echo "<input type=\"hidden\" name=\"hratingtime\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hassessed\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hassesspublic\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hscale\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hstartday\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hstartmonth\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hstartyear\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hstarthour\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hstartminute\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hfinishday\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hfinishmonth\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hfinishyear\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hfinishhour\" value=\"0\">";
-            echo "<input type=\"hidden\" name=\"hfinishminute\" value=\"0\">";
+            echo "<input type=\"hidden\" name=\"hratingtime\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hassessed\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hassesspublic\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hscale\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hstartday\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hstartmonth\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hstartyear\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hstarthour\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hstartminute\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hfinishday\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hfinishmonth\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hfinishyear\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hfinishhour\" value=\"0\" />";
+            echo "<input type=\"hidden\" name=\"hfinishminute\" value=\"0\" />";
 
             echo "<script>";
             echo "lockoptions('form','userating', subitemsall);";
@@ -251,14 +251,14 @@
 </tr>
 </table>
 <center>
-<input type="hidden" name="course"     value="<?php  p($form->course) ?>">
-<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>">
-<input type="hidden" name="section"       value="<?php  p($form->section) ?>">
-<input type="hidden" name="module"     value="<?php  p($form->module) ?>">
-<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>">
-<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>">
-<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>">
-<input type="submit" value="<?php  print_string("savechanges") ?>">
-<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>">
+<input type="hidden" name="course"     value="<?php  p($form->course) ?>" />
+<input type="hidden" name="coursemodule"  value="<?php  p($form->coursemodule) ?>" />
+<input type="hidden" name="section"       value="<?php  p($form->section) ?>" />
+<input type="hidden" name="module"     value="<?php  p($form->module) ?>" />
+<input type="hidden" name="modulename" value="<?php  p($form->modulename) ?>" />
+<input type="hidden" name="instance"   value="<?php  p($form->instance) ?>" />
+<input type="hidden" name="mode"       value="<?php  p($form->mode) ?>" />
+<input type="submit" value="<?php  print_string("savechanges") ?>" />
+<input type="submit" name="cancel" value="<?php  print_string("cancel") ?>" />
 </center>
 </form>
diff --git a/mod/forum/post.html b/mod/forum/post.html
index 1eaf1a2202..6a3bdd8d4b 100644
--- a/mod/forum/post.html
+++ b/mod/forum/post.html
@@ -3,7 +3,7 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("subject", "forum"); ?>:</b></p></td>
     <td>
-        <input type="text" name="subject" size="60" value="<?php p($post->subject) ?>">
+        <input type="text" name="subject" size="60" value="<?php p($post->subject) ?>" />
     </td>
 </tr>
 <tr valign="top">
@@ -45,7 +45,7 @@
     <?php 
         if ($usehtmleditor) {   /// Trying this out for a while
             print_string('formathtml');
-            echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'">';
+            echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'" />';
         } else {
             choose_from_menu(format_text_menu(), "format", $post->format, "");
         }
@@ -89,8 +89,8 @@
     <td align="right"><p><b><?php print_string("attachment", "forum") ?>:<br />(<?php print_string("optional") ?>)&nbsp;</b></p></td>
     <td>
     <?php $maxbytes = get_max_upload_file_size($CFG->maxbytes, $course->maxbytes, $forum->maxbytes); ?>
-    <input type="hidden" name="MAX_FILE_SIZE" value="<?php echo $maxbytes ?>">
-    <input type="file" name="attachment" size="40"> 
+    <input type="hidden" name="MAX_FILE_SIZE" value="<?php echo $maxbytes ?>" />
+    <input type="file" name="attachment" size="40" /> 
     <?php 
        helpbutton("attachment", get_string("attachment", "forum"), "forum"); 
        print_string("maxsize", "", display_size($maxbytes)); 
@@ -103,14 +103,14 @@
 
 <tr>
     <td align="center" colspan="2">
-    <input type="hidden" name="course"     value="<?php p($post->course) ?>">
-    <input type="hidden" name="forum"      value="<?php p($post->forum) ?>">
-    <input type="hidden" name="discussion" value="<?php p($post->discussion) ?>">
-    <input type="hidden" name="parent"     value="<?php p($post->parent) ?>">
-    <input type="hidden" name="userid"     value="<?php p($post->userid) ?>">
-    <input type="hidden" name="groupid"    value="<?php p($post->groupid) ?>">
-    <input type="hidden" name="edit"       value="<?php p($post->edit) ?>">
-    <input type="submit" value="<?php p(($post->edit) ? get_string('savechanges') : get_string('posttoforum', 'forum')); ?>">
+    <input type="hidden" name="course"     value="<?php p($post->course) ?>" />
+    <input type="hidden" name="forum"      value="<?php p($post->forum) ?>" />
+    <input type="hidden" name="discussion" value="<?php p($post->discussion) ?>" />
+    <input type="hidden" name="parent"     value="<?php p($post->parent) ?>" />
+    <input type="hidden" name="userid"     value="<?php p($post->userid) ?>" />
+    <input type="hidden" name="groupid"    value="<?php p($post->groupid) ?>" />
+    <input type="hidden" name="edit"       value="<?php p($post->edit) ?>" />
+    <input type="submit" value="<?php p(($post->edit) ? get_string('savechanges') : get_string('posttoforum', 'forum')); ?>" />
     </td>
 
 </tr>
diff --git a/mod/forum/prune.html b/mod/forum/prune.html
index 291bb82fa8..0a59f0aa9d 100644
--- a/mod/forum/prune.html
+++ b/mod/forum/prune.html
@@ -3,16 +3,16 @@
 <tr valign="top">
     <td align="right"><p><b><?php print_string("discussionname", "forum"); ?>:</b></p></td>
     <td>
-        <input type="text" name="name" size="60" value="<?php p($post->subject) ?>">
+        <input type="text" name="name" size="60" value="<?php p($post->subject) ?>" />
     </td>
 </tr>
 
 
 <tr>
     <td align="center" colspan="2">
-    <input type="hidden" name="prune"       value="<?php p($prune) ?>">
-    <input type="hidden" name="confirm"       value="<?php p($prune) ?>">
-    <input type="submit" value="<?php print_string('prune', 'forum'); ?>">
+    <input type="hidden" name="prune"       value="<?php p($prune) ?>" />
+    <input type="hidden" name="confirm"       value="<?php p($prune) ?>" />
+    <input type="submit" value="<?php print_string('prune', 'forum'); ?>" />
     </td>
 
 </tr>
-- 
2.39.5