Parcourir la source

updated documentation, fixed bugs with displaying multiple addrs

Luke Ehresman il y a 25 ans
Parent
commit
4f343ff7a2
6 fichiers modifiés avec 19 ajouts et 12 suppressions
  1. 2 5
      BUG
  2. 2 0
      ChangeLog
  3. 7 3
      INSTALL
  4. 2 2
      functions/imap_messages.php
  5. 5 2
      functions/mailbox_display.php
  6. 1 0
      src/read_body.php

+ 2 - 5
BUG

@@ -1,12 +1,9 @@
 Known BUGS to be fixed before 0.4pre1:
 Known BUGS to be fixed before 0.4pre1:
 
 
-  -    When more than one line of CCs or TOs is there, it doesn't have
+--SQUASHED--
+(lme)  When more than one line of CCs or TOs is there, it doesn't have
        the (more) or (less) option that used to be there.. only displays
        the (more) or (less) option that used to be there.. only displays
 		 the first line.
 		 the first line.
-  -    Check about optomizing Delete.. it takes quite a while when deleting
-       a large number of messages.
-
---SQUASHED--
 (lme)  When reading through 7 unread messages, every time I would read
 (lme)  When reading through 7 unread messages, every time I would read
        a message and click on "Message List" to go back, they would stay
        a message and click on "Message List" to go back, they would stay
 		 marked "unread".  The caching was doing too good of a job.  (:
 		 marked "unread".  The caching was doing too good of a job.  (:

+ 2 - 0
ChangeLog

@@ -1,5 +1,7 @@
 Version 0.4pre1 -- Development
 Version 0.4pre1 -- Development
 ------------------------------
 ------------------------------
+- Fixed all bugs listed in BUG
+- When inside the Sent folder, it displays "To" instead of "From" 
 - Added ability to go to Next and Previous message while reading a message
 - Added ability to go to Next and Previous message while reading a message
 - Caching of the message headers in mailbox (much faster)
 - Caching of the message headers in mailbox (much faster)
 - Added a preference that allows users to customize how many messages
 - Added a preference that allows users to customize how many messages

+ 7 - 3
INSTALL

@@ -124,9 +124,13 @@ b. Setting up directories
 
 
 c. Setting up SquirrelMail
 c. Setting up SquirrelMail
 
 
-  All configuration directives you need to worry about in SquirrelMail
-  is in the file config/config.php in you SquirrelMail directory. This
-  file is pretty well commented.
+  There are two ways to configure Squirrelmail.  In the config/ directory,
+  there is a perl script called conf.pl that will aid you in the
+  configuration process.  This is the most recommended way of handling
+  the config.
+
+  You can also copy the config/config_default.php file to config.php
+  and edit that manually.
 
 
 4. RUNNING SQUIRRELMAIL
 4. RUNNING SQUIRRELMAIL
 -----------------------
 -----------------------

+ 2 - 2
functions/imap_messages.php

@@ -247,7 +247,7 @@
             $pos = 0;
             $pos = 0;
             $header["CC"][$pos] = trim(substr($read[$i], 4));
             $header["CC"][$pos] = trim(substr($read[$i], 4));
             $i++;
             $i++;
-            while ((substr($read[$i], 0, 1) == " ") && (trim($read[$i]) != "")) {
+				while (((substr($read[$i], 0, 1) == "\t") || (substr($read[$i], 0, 1) == " ")) && (trim($read[$i]) != "")) {
                $pos++;
                $pos++;
                $header["CC"][$pos] = trim($read[$i]);
                $header["CC"][$pos] = trim($read[$i]);
                $i++;
                $i++;
@@ -258,7 +258,7 @@
             $pos = 0;
             $pos = 0;
             $header["TO"][$pos] = trim(substr($read[$i], 4));
             $header["TO"][$pos] = trim(substr($read[$i], 4));
             $i++;
             $i++;
-            while ((substr($read[$i], 0, 1) == " ")  && (trim($read[$i]) != "")){
+				while (((substr($read[$i], 0, 1) == "\t") || (substr($read[$i], 0, 1) == " ")) && (trim($read[$i]) != "")) {
                $pos++;
                $pos++;
                $header["TO"][$pos] = trim($read[$i]);
                $header["TO"][$pos] = trim($read[$i]);
                $i++;
                $i++;

+ 5 - 2
functions/mailbox_display.php

@@ -23,13 +23,14 @@
       
       
       if ($msg["FLAG_FLAGGED"] == true) { $flag = "<font color=$color[2]>"; $flag_end = "</font>"; }
       if ($msg["FLAG_FLAGGED"] == true) { $flag = "<font color=$color[2]>"; $flag_end = "</font>"; }
       if ($msg["FLAG_SEEN"] == false) { $bold = "<b>"; $bold_end = "</b>"; }
       if ($msg["FLAG_SEEN"] == false) { $bold = "<b>"; $bold_end = "</b>"; }
-      if ($msg["FLAG_ANSWERED"] == true) { $ans = "&nbsp;[A]"; }
 		if ($mailbox == $sent_folder) { $italic = "<i>"; $italic_end = "</i>"; }
 		if ($mailbox == $sent_folder) { $italic = "<i>"; $italic_end = "</i>"; }
       
       
       echo "   <td width=1% align=center><input type=checkbox name=\"msg[$t]\" value=".$msg["ID"]."></TD>\n";
       echo "   <td width=1% align=center><input type=checkbox name=\"msg[$t]\" value=".$msg["ID"]."></TD>\n";
       echo "   <td width=30%>$italic$bold$flag$senderName$flag_end$bold_end$italic_end</td>\n";
       echo "   <td width=30%>$italic$bold$flag$senderName$flag_end$bold_end$italic_end</td>\n";
       echo "   <td nowrap width=1%><center>$bold$flag".$msg["DATE_STRING"]."$flag_end$bold_end</center></td>\n";
       echo "   <td nowrap width=1%><center>$bold$flag".$msg["DATE_STRING"]."$flag_end$bold_end</center></td>\n";
-      echo "   <td width=%>$bold<a href=\"read_body.php?mailbox=$urlMailbox&passed_id=".$msg["ID"]."&startMessage=$startMessage&show_more=0\">$flag$subject$flag_end</a>$ans$bold_end</td>\n";
+		if ($msg["FLAG_ANSWERED"] == true) echo "   <td width=1%><b><small>A</small></b></td>";
+		else	echo "   <td width=1%>&nbsp;</td>";
+      echo "   <td width=%>$bold<a href=\"read_body.php?mailbox=$urlMailbox&passed_id=".$msg["ID"]."&startMessage=$startMessage&show_more=0\">$flag$subject$flag_end</a>$bold_end</td>\n";
 
 
       echo "</tr>\n";
       echo "</tr>\n";
    }
    }
@@ -232,6 +233,7 @@
          echo "   <A HREF=\"right_main.php?newsort=0&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/down_pointer.gif\" BORDER=0></A></TD>\n";
          echo "   <A HREF=\"right_main.php?newsort=0&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/down_pointer.gif\" BORDER=0></A></TD>\n";
       else
       else
          echo "   <A HREF=\"right_main.php?newsort=0&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/sort_none.gif\" BORDER=0></A></TD>\n";
          echo "   <A HREF=\"right_main.php?newsort=0&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/sort_none.gif\" BORDER=0></A></TD>\n";
+      echo "   <TD WIDTH=1%>&nbsp;</TD>\n";
       /** SUBJECT HEADER **/
       /** SUBJECT HEADER **/
       echo "   <TD WIDTH=%><B>". _("Subject") ."</B>\n";
       echo "   <TD WIDTH=%><B>". _("Subject") ."</B>\n";
       if ($sort == 4)
       if ($sort == 4)
@@ -240,6 +242,7 @@
          echo "   <A HREF=\"right_main.php?newsort=4&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/down_pointer.gif\" BORDER=0></A></TD>\n";
          echo "   <A HREF=\"right_main.php?newsort=4&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/down_pointer.gif\" BORDER=0></A></TD>\n";
       else
       else
          echo "   <A HREF=\"right_main.php?newsort=5&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/sort_none.gif\" BORDER=0></A></TD>\n";
          echo "   <A HREF=\"right_main.php?newsort=5&startMessage=1&mailbox=$urlMailbox\" TARGET=\"right\"><IMG SRC=\"../images/sort_none.gif\" BORDER=0></A></TD>\n";
+
       echo "</TR>";
       echo "</TR>";
 
 
       
       

+ 1 - 0
src/read_body.php

@@ -96,6 +96,7 @@
    $to_ary = $message["HEADER"]["TO"];
    $to_ary = $message["HEADER"]["TO"];
    while ($i < count($to_ary)) {
    while ($i < count($to_ary)) {
       $to_ary[$i] = htmlspecialchars($to_ary[$i]);
       $to_ary[$i] = htmlspecialchars($to_ary[$i]);
+
       if ($to_string)
       if ($to_string)
          $to_string = "$to_string<BR>$to_ary[$i]";
          $to_string = "$to_string<BR>$to_ary[$i]";
       else
       else