Changeset fa4cd20 in github

Timestamp:

Aug 4, 2006 8:10:34 AM (7 years ago)

Author:

thomascube <thomas@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

42000a5

Parents:

c4e7e4f

Message:

Several bugfixes; see CHANGELOG for details

Files:

• CHANGELOG (modified) (1 diff)
• config/main.inc.php.dist (modified) (1 diff)
• program/include/main.inc (modified) (3 diffs)
• program/include/rcube_imap.inc (modified) (16 diffs)
• program/js/app.js (modified) (1 diff)
• program/steps/settings/save_identity.inc (modified) (1 diff)

CHANGELOG

@@ -1 +1 @@
 CHANGELOG RoundCube Webmail
 ---------------------------
+
+2006/08/04 (thomasb)
+----------
+- Fixed Bug in saving identities (Ticket #1483915)
+- Set folder name in window title (Bug #1483919)
+- Don't add imap_root to INBOX path (Bug #1483816)
+- Attempt to create default folders only after login
+- Avoid usage of $CONFIG in rcube_imap class
+
 
 2006/07/30 (thomasb)

config/main.inc.php.dist

@@ -138 +138 @@
 $rcmail_config['trash_mbox'] = 'Trash';
 
-// display these folders separately in the mailbox list
+// display these folders separately in the mailbox list.
+// these folders will automatically be created if they do not exist
 $rcmail_config['default_imap_folders'] = array('INBOX', 'Drafts', 'Sent', 'Junk', 'Trash');
 

program/include/main.inc

@@ -206 +206 @@
     $IMAP->set_caching(TRUE);
 
-  if (is_array($CONFIG['default_imap_folders']))
-    $IMAP->set_default_mailboxes($CONFIG['default_imap_folders']);
-
   // set pagesize from config
   if (isset($CONFIG['pagesize']))
@@ -224 +221 @@
   if (!empty($CONFIG['imap_root']))
     $IMAP->set_rootdir($CONFIG['imap_root']);
+
+  if (is_array($CONFIG['default_imap_folders']))
+    $IMAP->set_default_mailboxes($CONFIG['default_imap_folders']);
 
   if (!empty($_SESSION['mbox']))
@@ -511 +511 @@
     $_SESSION['password']  = encrypt_passwd($pass);
 
-    // force reloading complete list of subscribed mailboxes    
+    // force reloading complete list of subscribed mailboxes
+    rcmail_set_imap_prop();
     $IMAP->clear_cache('mailboxes');
+    $IMAP->create_default_folders();
 
     return TRUE;

program/include/rcube_imap.inc

@@ -37 +37 @@
  * @package    RoundCube Webmail
  * @author     Thomas Bruederli <roundcube@gmail.com>
- * @version    1.30
+ * @version    1.31
  * @link       http://ilohamail.org
  */
@@ -53 +53 @@
   var $delimiter = NULL;
   var $caching_enabled = FALSE;
-  var $default_folders = array('inbox', 'drafts', 'sent', 'junk', 'trash');
+  var $default_folders = array('INBOX');
+  var $default_folders_lc = array('inbox');
   var $cache = array();
   var $cache_keys = array();  
@@ -209 +210 @@
     if (is_array($arr))
       {
-      $this->default_folders = array();
-      
-      // add mailbox names lower case
-      foreach ($arr as $mbox_row)
-        $this->default_folders[] = strtolower($mbox_row);
-      
+      $this->default_folders = $arr;
+      $this->default_folders_lc = array();
+
       // add inbox if not included
-      if (!in_array('inbox', $this->default_folders))
-        array_unshift($arr, 'inbox');
+      if (!in_array_nocase('INBOX', $this->default_folders))
+        array_unshift($this->default_folders, 'INBOX');
+
+      // create a second list with lower cased names
+      foreach ($this->default_folders as $mbox)
+        $this->default_folders_lc[] = strtolower($mbox);
       }
     }
@@ -334 +336 @@
       }
 
+    // INBOX should always be available
+    if (!in_array_nocase('INBOX', $a_out))
+      array_unshift($a_out, 'INBOX');
+
     // sort mailboxes
     $a_out = $this->_sort_mailbox_list($a_out);
@@ -362 +368 @@
     if (!is_array($a_folders) || !sizeof($a_folders))
       $a_folders = array();
-
-    // create Default folders if they do not exist
-    global $CONFIG;
-    foreach ($CONFIG['default_imap_folders'] as $folder)
-      {
-      if (!in_array_nocase($folder, $a_folders))
-        {
-        $this->create_mailbox($folder, TRUE);
-        $this->subscribe($folder);
-        }
-      }
-
-    $a_folders = iil_C_ListSubscribed($this->conn, $this->_mod_mailbox($root), $filter);
-    $a_mailbox_cache = array();
 
     // write mailboxlist to cache
@@ -1156 +1148 @@
 
 
-  // return an array with all folders available in IMAP server
+  /**
+   * Get a list of all folders available on the IMAP server
+   * 
+   * @param string IMAP root dir
+   * @return array Inbdexed array with folder names 
+   */
   function list_unsubscribed($root='')
     {
@@ -1198 +1195 @@
 
 
-  // subscribe to a specific mailbox(es)
+  /**
+   * subscribe to a specific mailbox(es)
+   */ 
   function subscribe($mbox_name, $mode='subscribe')
     {
@@ -1211 +1210 @@
 
 
-  // unsubscribe mailboxes
+  /**
+   * unsubscribe mailboxes
+   */
   function unsubscribe($mbox_name)
     {
@@ -1224 +1225 @@
 
 
-  // create a new mailbox on the server and register it in local cache
+  /**
+   * create a new mailbox on the server and register it in local cache
+   */
   function create_mailbox($name, $subscribe=FALSE)
     {
@@ -1239 +1242 @@
     $abs_name = $this->_mod_mailbox($name_enc);
     $a_mailbox_cache = $this->get_cache('mailboxes');
-        
-    if (strlen($abs_name) && (!is_array($a_mailbox_cache) || !in_array($abs_name, $a_mailbox_cache)))
+
+    if (strlen($abs_name) && (!is_array($a_mailbox_cache) || !in_array_nocase($abs_name, $a_mailbox_cache)))
       $result = iil_C_CreateFolder($this->conn, $abs_name);
 
-    // update mailboxlist cache
-    if ($result && $subscribe)
+    // try to subscribe it
+    if ($subscribe)
       $this->subscribe($name_enc);
 
@@ -1251 +1254 @@
 
 
-  // set a new name to an existing mailbox
+  /**
+   * set a new name to an existing mailbox
+   */
   function rename_mailbox($mbox_name, $new_name)
     {
@@ -1280 +1285 @@
 
 
-  // remove mailboxes from server
+  /**
+   * remove mailboxes from server
+   */
   function delete_mailbox($mbox_name)
     {
@@ -1297 +1304 @@
         // unsubscribe mailbox before deleting
         iil_C_UnSubscribe($this->conn, $mailbox);
-        
+
         // send delete command to server
         $result = iil_C_DeleteFolder($this->conn, $mailbox);
@@ -1314 +1321 @@
     }
 
+
+  /**
+   * Create all folders specified as default
+   */
+  function create_default_folders()
+    {
+    $a_folders = iil_C_ListMailboxes($this->conn, $this->_mod_mailbox(''), '*');
+    $a_subscribed = iil_C_ListSubscribed($this->conn, $this->_mod_mailbox(''), '*');
+    
+    // create default folders if they do not exist
+    foreach ($this->default_folders as $folder)
+      {
+      $abs_name = $this->_mod_mailbox($folder);
+      if (!in_array_nocase($abs_name, $a_subscribed))
+        {
+        if (!in_array_nocase($abs_name, $a_folders))
+          $this->create_mailbox($folder, TRUE);
+        else
+          $this->subscribe($folder);
+        }
+      }
+    }
 
 
@@ -1831 +1860 @@
   function _mod_mailbox($mbox_name, $mode='in')
     {
-    if ((!empty($this->root_ns) && $this->root_ns == $mbox_name) || ($mbox_name == 'INBOX' && $mode == 'in'))
+    if ((!empty($this->root_ns) && $this->root_ns == $mbox_name) || $mbox_name == 'INBOX')
       return $mbox_name;
 
@@ -1853 +1882 @@
       if ($folder{0}=='.')
         continue;
-        
-      if (($p = array_search(strtolower($folder), $this->default_folders))!==FALSE)
+
+      if (($p = array_search(strtolower($folder), $this->default_folders_lc))!==FALSE)
         $a_defaults[$p] = $folder;
       else

program/js/app.js

@@ -3213 +3213 @@
         }
       }
+      
+    // also update mailbox name in window title
+    if (document.title)
+      {
+      var doc_title = String(document.title);
+      var reg = new RegExp(this.env.mailbox.toLowerCase(), 'i');
+      if (this.env.mailbox && doc_title.match(reg))
+        document.title = doc_title.replace(reg, mbox).replace(/^\([0-9]+\)\s+/i, '');
+      }
     
     this.env.mailbox = mbox;

program/steps/settings/save_identity.inc

@@ -62 +62 @@
     }
        
-  if ($updated)
+  if ($updated && !empty($_POST['_standard']))
     {
     show_message('successfullysaved', 'confirmation');