More fixes due to R2L implementation (bugs were posted to squirrel-devel)

The bug in listcommands was an extra <tr> tag.
The bug in read_body.php was a misplaced do_hook (line #793).

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@3071 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/plugins/listcommands/setup.php b/plugins/listcommands/setup.php
index a5fe59570cde56fa92a0ef1565c1198b128cec0e..b33ababf379b78584e942becbfe76e57d11eb02c 100644 (file)
--- a/plugins/listcommands/setup.php
+++ b/plugins/listcommands/setup.php
@@ -110,7 +110,6 @@ function plugin_listcommands_menu() {
     }
 
     if (count($output) > 0) {
-        echo "<tr>";
         echo html_tag( 'tr',
                     html_tag( 'td', str_replace(' ', '&nbsp;', _("Mailing List:")), 'right', $color[0]) .
                     html_tag( 'td',

diff --git a/src/read_body.php b/src/read_body.php
index 8e38c507e8e82fa4dbb966ba745d89a342d7b2d1..07e06960b0e01846534afe713c9612696cfa2e4a 100644 (file)
--- a/src/read_body.php
+++ b/src/read_body.php
@@ -786,13 +786,12 @@ echo '</small></td>' .
     ' </tr>';
 
 /** from **/
-echo html_tag( 'tr', "\n" .
-            html_tag( 'td', _("From:"), 'right', $color[0], 'valign="top"' ) .
-            html_tag( 'td',
-                '<b>' . $from_name . '</b>&nbsp;' . "\n" ,
-            'left', $color[0] )
-       ) . "\n";
-       do_hook("read_body_after_from");
+echo html_tag( 'tr') . "\n" .
+    html_tag( 'td', _("From:"), 'right', $color[0], 'valign="top"' ) .
+    html_tag( 'td', '', 'left', $color[0] ) .
+        '<b>' . $from_name . '</b>&nbsp;&nbsp;';
+   do_hook("read_body_after_from");
+echo "&nbsp;\n" . '</td></tr>';
 /** date **/
 echo html_tag( 'tr', "\n" .
             html_tag( 'td', _("Date:"), 'right', $color[0], 'valign="top"' ) .