mod/glossary: HTML fixes

Fixed HTML errors causing parser errors in Firefox when in developer debug
mode.

Author: Matt Clarkson <mattc@catalyst.net.nz>
Committer: Matt Clarkson <mattc@catalyst.net.nz>

diff --git a/mod/glossary/lib.php b/mod/glossary/lib.php
index 5cad095fcd586f529c8b8a92bf47c877e9d5c217..64e6dc3e41a8ae7772d0446c1ac3c34e03f62db4 100644 (file)
--- a/mod/glossary/lib.php
+++ b/mod/glossary/lib.php
@@ -1476,7 +1476,7 @@ function glossary_print_comment($course, $cm, $glossary, $entry, $comment) {
     echo '<div class="boxaligncenter">';
     echo '<table class="glossarycomment" cellspacing="0">';
     echo '<tr valign="top">';
-    echo '<tr><td class="left picture">';
+    echo '<td class="left picture">';
     print_user_picture($user->id, $course->id, $user->picture);
     echo '</td>';
     echo '<td class="entryheader">';

diff --git a/mod/glossary/view.php b/mod/glossary/view.php
index ddd34ef285fb6c190e8b2482521601a5ffd425c5..7eae7475753ba5eabbaf22b9fb91dbec45fe0f2d 100644 (file)
--- a/mod/glossary/view.php
+++ b/mod/glossary/view.php
@@ -407,6 +407,7 @@
                 $ratings->allow = true;
             }
             $formsent = 1;
+
             echo "<form method=\"post\" action=\"rate.php\">";
             echo "<div>";
             echo "<input type=\"hidden\" name=\"id\" value=\"$course->id\" />";
@@ -496,15 +497,16 @@
                 print_scale_menu_helpbutton($course->id, $scale );
             }
         }
-        echo "</div>";
-        echo "</div>";
+        echo "</div>";    
+    }
+    if ($entriesshown ) {
         echo "</form>";
     } else if (!empty($formsent)) {
         // close the form properly if not  used
         echo "</div>";
         echo "</form>";  
     }
-
+    
     if ( $paging ) {
         echo '<hr />';
         echo '<div class="paging">';