[imp] IMP 4.1 + UW-IMAP

[Simon L Jackson]

Thanks to Adrian Chadd, I now have basic connectivity to a UW IMAP server.

I also found the following issues:

In imp/lib/Session.php 1.74.2.25, there seem to be some possible errors 
when using manual configurations.

At lines 244 & 246, I suspect that ['imp']['imap_config'] should be 
['imp']['imap_server'].

At around lines 296-306, some code seems to be in the wrong block.

For convenience/noise, the differences might be:

# diff --context=2 /usr/local/horde/imp/lib/Session.php.orig 
/usr/local/horde/imp/lib/Session.php
*** /usr/local/horde/imp/lib/Session.php.orig   2006-03-29 
05:52:20.000000000 +1100
--- /usr/local/horde/imp/lib/Session.php        2006-05-04 
12:39:32.000000000 +1000
***************
*** 242,248 ****
              /* Check for manual configuration. */
              if (!empty($ptr['imap_config'])) {
!                 $_SESSION['imp']['imap_config']['children'] = 
$ptr['imap_config']['children'];
                  $_SESSION['imp']['namespace'] = 
$ptr['imap_config']['namespace'];
!                 $_SESSION['imp']['imap_config']['search_charset'] = 
$ptr['imap_config']['search_charset'];
              } else {
                  require_once IMP_BASE . '/lib/IMAP/Client.php';
--- 242,248 ----
              /* Check for manual configuration. */
              if (!empty($ptr['imap_config'])) {
!                 $_SESSION['imp']['imap_server']['children'] = 
true;//$ptr['imap_config']['children'];
                  $_SESSION['imp']['namespace'] = 
$ptr['imap_config']['namespace'];
!                 $_SESSION['imp']['imap_server']['search_charset'] = 
$ptr['imap_config']['search_charset'];
              } else {
                  require_once IMP_BASE . '/lib/IMAP/Client.php';
***************
*** 293,308 ****
                      $imapclient->logout();
                  }

!                 /* Initialize the 'showunsub' value. */
!                 $_SESSION['imp']['showunsub'] = false;

!                 /* Initialize the IMP_Tree object.  This needs to be done
!                  * before initializing IMP_Search::. */
!                 require_once IMP_BASE . '/lib/IMAP/Tree.php';
!                 IMP_IMAP::singleton();

-                 /* Set up search information for the session. */
-                 $GLOBALS['imp_search']->sessionSetup();
-             }
          } else {
              $_SESSION['imp']['namespace'] = null;
--- 293,309 ----
                      $imapclient->logout();
                  }
+             }

!             /* Initialize the 'showunsub' value. */
!             $_SESSION['imp']['showunsub'] = false;

!             /* Initialize the IMP_Tree object.  This needs to be done
!              * before initializing IMP_Search::. */
!             require_once IMP_BASE . '/lib/IMAP/Tree.php';
!             IMP_IMAP::singleton();
!
!             /* Set up search information for the session. */
!             $GLOBALS['imp_search']->sessionSetup();

          } else {
              $_SESSION['imp']['namespace'] = null;

This change seemed to make things much better.

A final issue is that the search folder list works ok, but the tree on 
the sidebar only has the first level folders, i.e. there are no sub-folders.

I suspect that this has been dealt with already, if someone could point 
be to it.

Cheers

-- 
Simon L Jackson
Carringbush.Net

+-
Carringbush.Net
Hosting * Development * Advice

Web: www.carringbush.net <http://www.carringbush.net/>
Email: simon.jackson at carringbush.net <mailto:simon.jackson at carringbush.net>

Office: +61 3 9411 4400
Support: +61 3 9411 4444
Fax: +61 3 9411 4499

Level 1
15 Bedford Street
Collingwood VIC 3066
Australia

P.O. Box 1464
Collingwood VIC 3066
Australia
+-