"me" to 'you', made attempt to be more carfull for the sake of Tyler's sanity.

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

diff --git a/src/options_highlight.php b/src/options_highlight.php
index 44335ba9493560d81939f630dc878b3e3a732a78..3cc2ea7d14b8ab446267d24d11dc79f127b99255 100644 (file)
--- a/src/options_highlight.php
+++ b/src/options_highlight.php
@@ -13,147 +13,147 @@
    session_start();
 
    if (!isset($strings_php))
-      include("../functions/strings.php");
+      include('../functions/strings.php');
    if (!isset($config_php))
-      include("../config/config.php");
+      include('../config/config.php');
    if (!isset($page_header_php))
-      include("../functions/page_header.php");
+      include('../functions/page_header.php');
    if (!isset($display_messages_php))
-      include("../functions/display_messages.php");
+      include('../functions/display_messages.php');
    if (!isset($imap_php))
-      include("../functions/imap.php");
+      include('../functions/imap.php');
    if (!isset($array_php))
-      include("../functions/array.php");
+      include('../functions/array.php');
    if (!isset($i18n_php))
-      include("../functions/i18n.php");
+      include('../functions/i18n.php');
    if (!isset($plugin_php))
-      include("../functions/plugin.php");
+      include('../functions/plugin.php');
 
    if (! isset($action))
        $action = '';
    if (! isset($message_highlight_list))
        $message_highlight_list = array();
 
-   if ($action == "delete" && isset($theid)) {
+   if ($action == 'delete' && isset($theid)) {
       removePref($data_dir, $username, "highlight$theid");
-   } else if ($action == "save") {
+   } else if ($action == 'save') {
       if (!$theid) $theid = 0;
-      $identname = ereg_replace(",", " ", $identname);
+      $identname = ereg_replace(',', ' ', $identname);
       $identname = str_replace("\\\\", "\\", $identname);
-      $identname = str_replace("\\\"", "\"", $identname);
-      $identname = str_replace("\"", """, $identname);
+      $identname = str_replace("\\\"", '"', $identname);
+      $identname = str_replace('"', '"', $identname);
       if ($color_type == 1) $newcolor = $newcolor_choose;
       else $newcolor = $newcolor_input;
  
-      $newcolor = ereg_replace(",", "", $newcolor);
-      $newcolor = ereg_replace("#", "", $newcolor);
+      $newcolor = ereg_replace(',', '', $newcolor);
+      $newcolor = ereg_replace('#', '', $newcolor);
       $newcolor = "$newcolor";
-      $value = ereg_replace(",", " ", $value);
+      $value = ereg_replace(',', ' ', $value);
       $value = str_replace("\\\\", "\\", $value);
-      $value = str_replace("\\\"", "\"", $value);
-      $value = str_replace("\"", """, $value);
+      $value = str_replace("\\\"", '"', $value);
+      $value = str_replace('"', '"', $value);
 
-      setPref($data_dir, $username, "highlight$theid", $identname.",".$newcolor.",".$value.",".$match_type);
-      $message_highlight_list[$theid]["name"] = $identname;
-      $message_highlight_list[$theid]["color"] = $newcolor;
-      $message_highlight_list[$theid]["value"] = $value;
-      $message_highlight_list[$theid]["match_type"] = $match_type;
+      setPref($data_dir, $username, "highlight$theid", $identname.','.$newcolor.','.$value.','.$match_type);
+      $message_highlight_list[$theid]['name'] = $identname;
+      $message_highlight_list[$theid]['color'] = $newcolor;
+      $message_highlight_list[$theid]['value'] = $value;
+      $message_highlight_list[$theid]['match_type'] = $match_type;
    } 
-   include("../src/load_prefs.php");
-   displayPageHeader($color, "None");
+   include('../src/load_prefs.php');
+   displayPageHeader($color, 'None');
 ?>
    <br>
-   <table width=95% align=center border=0 cellpadding=2 cellspacing=0><tr><td bgcolor="<?php echo $color[0] ?>">
-      <center><b><?php echo _("Options") . " - " . _("Message Highlighting"); ?></b></center>
+   <table width="95%" align="center" border="0" cellpadding="2" cellspacing="0"><tr><td bgcolor="<?php echo $color[0] ?>">
+      <center><b><?php echo _("Options") . ' - ' . _("Message Highlighting"); ?></b></center>
    </td></tr></table>
 
 <?php
-   echo "<br><center>[<a href=\"options_highlight.php?action=add\">" . _("New") . "</a>]";
-   echo " - [<a href=\"options.php\">"._("Done")."</a>]</center><br>\n";
+   echo '<br><center>[<a href="options_highlight.php?action=add">' . _("New") . '</a>]';
+   echo ' - [<a href="options.php">'._("Done").'</a>]</center><br>'."\n";
    if (count($message_highlight_list) >= 1) {
-      echo "<table border=0 cellpadding=3 cellspacing=0 align=center width=80%>\n";
+      echo '<table border="0" cellpadding="3" cellspacing="0" align="center" width="80%">'."\n";
       for ($i=0; $i < count($message_highlight_list); $i++) {
          echo "<tr>\n";
-         echo "   <td width=1% bgcolor=" . $color[4] . ">\n";
+         echo '   <td width="1%" bgcolor="' . $color[4] . "\">\n";
          echo "<nobr><small>[<a href=\"options_highlight.php?action=edit&theid=$i\">" . _("Edit") . "</a>]&nbsp;[<a href=\"options_highlight.php?action=delete&theid=$i\">"._("Delete")."</a>]</small></nobr>\n";
-         echo "   </td>";
-         echo "   <td bgcolor=" . $message_highlight_list[$i]["color"] . ">\n";
-         echo "      " . $message_highlight_list[$i]["name"];
+         echo '   </td>';
+         echo '   <td bgcolor="' . $message_highlight_list[$i]['color'] . "\">\n";
+         echo '      ' . $message_highlight_list[$i]['name'];
          echo "   </td>\n";
-         echo "   <td bgcolor=" . $message_highlight_list[$i]["color"] . ">\n";
-         echo "      ".$message_highlight_list[$i]["match_type"]." = " . $message_highlight_list[$i]["value"];
+         echo '   <td bgcolor="' . $message_highlight_list[$i]['color'] . "\">\n";
+         echo '      '.$message_highlight_list[$i]['match_type'].' = ' . $message_highlight_list[$i]['value'];
          echo "   </td>\n";
          echo "</tr>\n";
       }
       echo "</table>\n";
       echo "<br>\n";
    } else {
-      echo "<center>" . _("No highlighting is defined") . "</center><br>\n";
+      echo '<center>' . _("No highlighting is defined") . "</center><br>\n";
       echo "<br>\n";
    }
-   if ($action == "edit" || $action == "add") {
+   if ($action == 'edit' || $action == 'add') {
       if (!isset($theid)) $theid = count($message_highlight_list);
           $message_highlight_list[$theid] = array();
  
-      $color_list[0] = "4444aa";
-      $color_list[1] = "44aa44";
-      $color_list[2] = "aaaa44";
-      $color_list[3] = "44aaaa";
-      $color_list[4] = "aa44aa";
-      $color_list[5] = "aaaaff";
-      $color_list[6] = "aaffaa";
-      $color_list[7] = "ffffaa";
-      $color_list[8] = "aaffff";
-      $color_list[9] = "ffaaff";
-      $color_list[10] = "aaaaaa";
-      $color_list[11] = "bfbfbf";
-      $color_list[12] = "dfdfdf";
-      $color_list[13] = "ffffff";               
+      $color_list[0] = '4444aa';
+      $color_list[1] = '44aa44';
+      $color_list[2] = 'aaaa44';
+      $color_list[3] = '44aaaa';
+      $color_list[4] = 'aa44aa';
+      $color_list[5] = 'aaaaff';
+      $color_list[6] = 'aaffaa';
+      $color_list[7] = 'ffffaa';
+      $color_list[8] = 'aaffff';
+      $color_list[9] = 'ffaaff';
+      $color_list[10] = 'aaaaaa';
+      $color_list[11] = 'bfbfbf';
+      $color_list[12] = 'dfdfdf';
+      $color_list[13] = 'ffffff';               
       
-      $selected_input = "";
+      $selected_input = '';
       
       for ($i=0; $i < 14; $i++) {
-         ${"selected".$i} = "";
+         ${"selected".$i} = '';
       }
-      if (isset($message_highlight_list[$theid]["color"])) {
+      if (isset($message_highlight_list[$theid]['color'])) {
          for ($i=0; $i < 14; $i++) {
-            if ($color_list[$i] == $message_highlight_list[$theid]["color"]) {
-               $selected_choose = " checked";
-               ${"selected".$i} = " selected";
+            if ($color_list[$i] == $message_highlight_list[$theid]['color']) {
+               $selected_choose = ' checked';
+               ${"selected".$i} = ' selected';
                continue;
             }
             }
       }
-      if (!isset($message_highlight_list[$theid]["color"]))
-         $selected_choose = " checked";
+      if (!isset($message_highlight_list[$theid]['color']))
+         $selected_choose = ' checked';
       else if (!isset($selected_choose))
-         $selected_input = " checked";
+         $selected_input = ' checked';
  
-      echo "<form name=f action=\"options_highlight.php\">\n";
-      echo "<input type=\"hidden\" value=\"save\" name=\"action\">\n";
-      echo "<input type=\"hidden\" value=\"$theid\" name=\"theid\">\n";
-      echo "<table width=80% align=center cellpadding=3 cellspacing=0 border=0>\n";
+      echo '<form name="f" action="options_highlight.php">' . "\n";
+      echo '<input type="hidden" value="save" name="action">' . "\n";
+      echo '<input type="hidden" value="'.$theid.'" name="theid">' . "\n";
+      echo '<table width="80%" align="center" cellpadding="3" cellspacing="0" border="0">' . "\n";
       echo "   <tr bgcolor=\"$color[0]\">\n";
-      echo "      <td align=right width=25%><b>\n";
+      echo "      <td align=\"right\" width=\"25%\"><b>\n";
       echo _("Identifying name") . ":";
-      echo "      </b></td>\n";
-      echo "      <td width=75%>\n";
-      if (isset($message_highlight_list[$theid]["name"]))
-          $disp = $message_highlight_list[$theid]["name"];
+      echo '      </b></td>' . "\n";
+      echo '      <td width="75%">' . "\n";
+      if (isset($message_highlight_list[$theid]['name']))
+          $disp = $message_highlight_list[$theid]['name'];
       else
-          $disp = "";
+          $disp = '';
       $disp = str_replace("\\\\", "\\", $disp);
-      $disp = str_replace("\\\"", "\"", $disp);
-      $disp = str_replace("\"", "&quot;", $disp);
+      $disp = str_replace("\\\"", '"', $disp);
+      $disp = str_replace('"', '&quot;', $disp);
       echo "         <input type=\"text\" value=\"".$disp."\" name=\"identname\">";
       echo "      </td>\n";
       echo "   </tr>\n";
-      echo "   <tr><td><small><small>&nbsp;</small></small></td></tr>\n";
+      echo '   <tr><td><small><small>&nbsp;</small></small></td></tr>' . "\n";
       echo "   <tr bgcolor=\"$color[0]\">\n";
-      echo "      <td align=right width=25%><b>\n";
-      echo _("Color") . ":";
+      echo '      <td align="right" width="25%"><b>' . "\n";
+      echo _("Color") . ':';
       echo "      </b></td>\n";
-      echo "      <td width=75%>\n";
+      echo '      <td width="75%">' . "\n";
       echo "         <input type=\"radio\" name=color_type value=1$selected_choose> &nbsp;<select name=newcolor_choose>\n";
       echo "            <option value=\"$color_list[0]\"$selected0>" . _("Dark Blue") . "\n";
       echo "            <option value=\"$color_list[1]\"$selected1>" . _("Dark Green") . "\n";
@@ -172,41 +172,41 @@
       echo "         </select><br>\n";
       echo "         <input type=\"radio\" name=color_type value=2$selected_input> &nbsp;". _("Other:") ."<input type=\"text\" value=\"";
       if ($selected_input) echo $message_highlight_list[$theid]["color"];
-      echo "\" name=\"newcolor_input\" size=7> "._("Ex: 63aa7f")."<br>\n";
+      echo '" name="newcolor_input" size="7"> '._("Ex: 63aa7f")."<br>\n";
       echo "      </td>\n";
       echo "   </tr>\n";
-      echo "   <tr><td><small><small>&nbsp;</small></small></td></tr>\n";
+      echo '   <tr><td><small><small>&nbsp;</small></small></td></tr>' . "\n";
       echo "   <tr bgcolor=\"$color[0]\">\n";
-      echo "      <td align=right width=25%><b>\n";
-      echo _("Match") . ":";
+      echo '      <td align="right" width="25%"><b>' . "\n";
+      echo _("Match") . ':';
       echo "      </b></td>\n";
-      echo "      <td width=75%>\n";
+      echo '      <td width="75%">' . "\n";
       echo "         <select name=match_type>\n";
-      if (isset($message_highlight_list[$theid]["match_type"]) && $message_highlight_list[$theid]["match_type"] == "from")    echo "            <option value=\"from\" selected>From\n";
+      if (isset($message_highlight_list[$theid]['match_type']) && $message_highlight_list[$theid]['match_type'] == 'from')    echo "            <option value=\"from\" selected>From\n";
       else                                                         echo "            <option value=\"from\">From\n";
-      if (isset($message_highlight_list[$theid]["match_type"]) && $message_highlight_list[$theid]["match_type"] == "to")      echo "            <option value=\"to\" selected>To\n";
+      if (isset($message_highlight_list[$theid]['match_type']) && $message_highlight_list[$theid]['match_type'] == 'to')      echo "            <option value=\"to\" selected>To\n";
       else                                                         echo "            <option value=\"to\">To\n";
-      if (isset($message_highlight_list[$theid]["match_type"]) && $message_highlight_list[$theid]["match_type"] == "cc")      echo "            <option value=\"cc\" selected>Cc\n";
+      if (isset($message_highlight_list[$theid]['match_type']) && $message_highlight_list[$theid]['match_type'] == 'cc')      echo "            <option value=\"cc\" selected>Cc\n";
       else                                                         echo "            <option value=\"cc\">Cc\n";
-      if (isset($message_highlight_list[$theid]["match_type"]) && $message_highlight_list[$theid]["match_type"] == "to_cc")   echo "            <option value=\"to_cc\" selected>To or Cc\n";
+      if (isset($message_highlight_list[$theid]['match_type']) && $message_highlight_list[$theid]['match_type'] == 'to_cc')   echo "            <option value=\"to_cc\" selected>To or Cc\n";
       else                                                         echo "            <option value=\"to_cc\">To or Cc\n";
-      if (isset($message_highlight_list[$theid]["match_type"]) && $message_highlight_list[$theid]["match_type"] == "subject") echo "            <option value=\"subject\" selected>Subject\n";
+      if (isset($message_highlight_list[$theid]['match_type']) && $message_highlight_list[$theid]['match_type'] == 'subject') echo "            <option value=\"subject\" selected>Subject\n";
       else                                                         echo "            <option value=\"subject\">Subject\n";
       echo "         </select>\n";
-      if (isset($message_highlight_list[$theid]["value"]))
-          $disp = $message_highlight_list[$theid]["value"];
+      if (isset($message_highlight_list[$theid]['value']))
+          $disp = $message_highlight_list[$theid]['value'];
       else
           $disp = '';
       $disp = str_replace("\\\\", "\\", $disp);
-      $disp = str_replace("\\\"", "\"", $disp);
-      $disp = str_replace("\"", "&quot;", $disp);
-      echo "         <nobr><input type=\"text\" value=\"".$disp."\" name=\"value\">";
+      $disp = str_replace("\\\"", '"', $disp);
+      $disp = str_replace('"', '&quot;', $disp);
+      echo '         <nobr><input type="text" value="'.$disp.'" name="value">';
       echo "        <nobr></td>\n";
       echo "   </tr>\n";
       echo "</table>\n";
-      echo "<center><input type=\"submit\" value=\"" . _("Submit") . "\"></center>\n";
+      echo '<center><input type="submit" value="' . _("Submit") . "\"></center>\n";
       echo "</form>\n";
-      do_hook("options_highlight_bottom");
+      do_hook('options_highlight_bottom');
    } 
 ?>
 </body></html>

diff --git a/src/options_order.php b/src/options_order.php
index 66aedee83ac55e0c6d114a79b47c7f70e65770ad..ffa70ac17f5672f95dcdf7cbe5be7586cb47c09a 100644 (file)
--- a/src/options_order.php
+++ b/src/options_order.php
@@ -13,30 +13,30 @@
    session_start();
 
    if (!isset($strings_php))
-      include("../functions/strings.php");
+      include('../functions/strings.php');
    if (!isset($config_php))
-      include("../config/config.php");
+      include('../config/config.php');
    if (!isset($page_header_php))
-      include("../functions/page_header.php");
+      include('../functions/page_header.php');
    if (!isset($display_messages_php))
-      include("../functions/display_messages.php");
+      include('../functions/display_messages.php');
    if (!isset($imap_php))
-      include("../functions/imap.php");
+      include('../functions/imap.php');
    if (!isset($array_php))
-      include("../functions/array.php");
+      include('../functions/array.php');
    if (!isset($i18n_php))
-      include("../functions/i18n.php");
+      include('../functions/i18n.php');
    if (!isset($plugin_php))
-      include("../functions/plugin.php");
+      include('../functions/plugin.php');
 
 
-   if (! isset($action)) { $action = ""; }
-   if ($action == "delete" && isset($theid)) {
+   if (! isset($action)) { $action = ''; }
+   if ($action == 'delete' && isset($theid)) {
       removePref($data_dir, $username, "highlight$theid");
-   } else if ($action == "save") {
+   } else if ($action == 'save') {
    } 
-   include("../src/load_prefs.php");
-   displayPageHeader($color, "None");
+   include('../src/load_prefs.php');
+   displayPageHeader($color, 'None');
 ?>
    <br>
    <table width=95% align=center border=0 cellpadding=2 cellspacing=0><tr><td bgcolor="<?php echo $color[0] ?>">
@@ -53,19 +53,19 @@
    $available[5] = _("Flags");
    $available[6] = _("Size");
    
-   if (! isset($method)) { $method = ""; }
+   if (! isset($method)) { $method = ''; }
 
-   if ($method == "up" && $num > 1) {
+   if ($method == 'up' && $num > 1) {
       $prev = $num-1;
       $tmp = $index_order[$prev];
       $index_order[$prev] = $index_order[$num];
       $index_order[$num] = $tmp;
-   } else if ($method == "down" && $num < count($index_order)) {
+   } else if ($method == 'down' && $num < count($index_order)) {
       $next = $num++;
       $tmp = $index_order[$next];
       $index_order[$next] = $index_order[$num];
       $index_order[$num] = $tmp;
-   } else if ($method == "remove" && $num) {
+   } else if ($method == 'remove' && $num) {
       for ($i=1; $i < 8; $i++) {
          removePref($data_dir, $username, "order$i"); 
       }
@@ -78,9 +78,9 @@
       $index_order = array();
       $index_order = $new_ary;
       if (count($index_order) < 1) {
-         include "../src/load_prefs.php";
+         include ('../src/load_prefs.php');
       }
-   } else if ($method == "add" && $add) {
+   } else if ($method == 'add' && $add) {
       $index_order[count($index_order)+1] = $add;
    }
 
@@ -89,38 +89,38 @@
          setPref($data_dir, $username, "order$i", $index_order[$i]);
       }
    }
-   echo "<center>";
-   echo "<table cellspacing=0 cellpadding=0 border=0 width=65%><tr><td>\n";
+   echo '<center>';
+   echo '<table cellspacing="0" cellpadding="0" border="0" width="65%"><tr><td>' . "\n";
    echo _("The index order is the order that the columns are arranged in the message index.  You can add, remove, and move columns around to customize them to fit your needs.");
-   echo "</td></tr></table></center><br>";
+   echo '</td></tr></table></center><br>';
 
    if (count($index_order))
    {
-      echo "<center>";
-      echo "<table cellspacing=0 cellpadding=0 border=0>\n";
+      echo '<center>';
+      echo '<table cellspacing="0" cellpadding="0" border="0">' . "\n";
       for ($i=1; $i <= count($index_order); $i++) {
          $tmp = $index_order[$i];
-         echo "<tr>";
+         echo '<tr>';
          echo "<td><small><a href=\"options_order.php?method=up&num=$i\">". _("up") ."</a></small></td>\n";
-         echo "<td><small>&nbsp;|&nbsp;</small></td>\n";
+         echo '<td><small>&nbsp;|&nbsp;</small></td>' . "\n";
          echo "<td><small><a href=\"options_order.php?method=down&num=$i\">". _("down") . "</a></small></td>\n";
-         echo "<td><small>&nbsp;|&nbsp;</small></td>\n";
-         echo "<td>";
+         echo '<td><small>&nbsp;|&nbsp;</small></td>' . "\n";
+         echo '<td>';
          // Always show the subject
          if ($tmp != 4)
-            echo "<small><a href=\"options_order.php?method=remove&num=$i\">" . _("remove") . "</a></small>";
+            echo "<small><a href=\"options_order.php?method=remove&num=$i\">" . _("remove") . '</a></small>';
          echo "</td>\n";
-         echo "<td><small>&nbsp;-&nbsp;</small></td>\n";
-         echo "<td>" . $available[$tmp] . "</td>\n";
+         echo '<td><small>&nbsp;-&nbsp;</small></td>' . "\n";
+         echo '<td>' . $available[$tmp] . "</td>\n";
          echo "</tr>\n";
       }
       echo "</table>\n";
-      echo "</center>";
+      echo '</center>';
    }
    
    if (count($index_order) != count($available)) {
-   echo "<center><form name=f method=post action=options_order.php>";
-   echo "<select name=add>";
+   echo '<center><form name="f" method="post" action="options_order.php">';
+   echo '<select name="add">';
    for ($i=1; $i <= count($available); $i++) {
       $found = false;
       for ($j=1; $j <= count($index_order); $j++) {
@@ -129,16 +129,16 @@
          }
       }
       if (!$found) {
-         echo "<option value=$i>$available[$i]</option>";
+         echo "<option value=\"$i\">$available[$i]</option>";
       }
    }
-   echo "</select>";
-   echo "<input type=hidden value=add name=method>";
-   echo "<input type=submit value=\""._("Add")."\" name=submit>";
-   echo "</form></center>";
+   echo '</select>';
+   echo '<input type="hidden" value="add" name="method">';
+   echo '<input type="submit" value="'._("Add").'" name="submit">';
+   echo '</form></center>';
    }
 
-   echo "<br><center><a href=\"../src/options.php\">" . _("Return to options page") . "</a></center>";
+   echo '<br><center><a href="../src/options.php">' . _("Return to options page") . '</a></center>';
 
 ?>
    </td></tr></table>

diff --git a/src/options_personal.php b/src/options_personal.php
index 7db053d03cfe14e58b2bb2a1c0fc571209023bd6..415f997f3e41b41c1cd57ec1c334099f2596cab2 100644 (file)
--- a/src/options_personal.php
+++ b/src/options_personal.php
@@ -13,28 +13,28 @@
    session_start();
 
    if (!isset($strings_php))
-      include("../functions/strings.php");
+      include('../functions/strings.php');
    if (!isset($config_php))
-      include("../config/config.php");
+      include('../config/config.php');
    if (!isset($page_header_php))
-      include("../functions/page_header.php");
+      include('../functions/page_header.php');
    if (!isset($display_messages_php))
-      include("../functions/display_messages.php");
+      include('../functions/display_messages.php');
    if (!isset($imap_php))
-      include("../functions/imap.php");
+      include('../functions/imap.php');
    if (!isset($array_php))
-      include("../functions/array.php");
+      include('../functions/array.php');
    if (!isset($i18n_php))
-      include("../functions/i18n.php");
+      include('../functions/i18n.php');
    if (!isset($plugin_php))
-      include("../functions/plugin.php");
+      include('../functions/plugin.php');
 
-   include("../src/load_prefs.php");
-   displayPageHeader($color, "None");
+   include('../src/load_prefs.php');
+   displayPageHeader($color, 'None');
 
-   $fullname = getPref($data_dir, $username, "full_name");
-   $replyto = getPref($data_dir, $username, "reply_to");
-   $email_address  = getPref($data_dir, $username, "email_address"); 
+   $fullname = getPref($data_dir, $username, 'full_name');
+   $replyto = getPref($data_dir, $username, 'reply_to');
+   $email_address  = getPref($data_dir, $username, 'email_address'); 
 
 ?>
    <br>
@@ -67,13 +67,13 @@
             </td><td>
 <?php
    if ($use_signature == true)
-      echo "<input type=checkbox value=\"1\" name=usesignature checked>&nbsp;&nbsp;" . _("Use a signature") . "?<BR>";
+      echo '<input type=checkbox value="1" name=usesignature checked>&nbsp;&nbsp;' . _("Use a signature") . "?<BR>";
    else {
-      echo "<input type=checkbox value=\"1\" name=usesignature>&nbsp;&nbsp;";
+      echo '<input type=checkbox value="1" name=usesignature>&nbsp;&nbsp;';
       echo _("Use a signature?");
-      echo "<BR>";
+      echo '<BR>';
    } 
-   echo "\n<textarea name=signature_edit rows=5 cols=50>$signature_abs</textarea><br>";
+   echo "\n<textarea name=\"signature_edit\" rows=\"5\" cols=\"50\">$signature_abs</textarea><br>";
 ?>
             </td>
          </tr>
@@ -86,5 +86,5 @@
          </tr>
       </table>   
    </form>
-   <?php do_hook("options_personal_bottom"); ?>
+   <?php do_hook('options_personal_bottom'); ?>
 </body></html>