[imp] Virtual inbox

[Adrian Chadd]

On Sat, Apr 08, 2006, Jan Schneider wrote:

> >I want to desactived this one ?
> >
> >How can I make it ?
> 
> You can't.

Would you consider making this a configurable option?

Reasoning: its a nifty feature but I don't want to turn it on until
relevant user documentation has been written and the support
people have been trained on it.

Oh, and just in case noone's noticed:

--- imp/lib/Search.php  (revision 986)
+++ imp/lib/Search.php  (working copy)
@@ -441,6 +441,12 @@
      */
     function isVINBOXFolder($id = null)
     {
+       /* [ahc] This code was returning true if we're comparing null to null!
+        * This then affected whether we showed the purge deleted links.
+        */
+       if ($GLOBALS['prefs']->getValue('vinbox_id') == null) {
+               return false;
+       }
         $id = (is_null($id)) ? $this->_id : $this->_strip($id);
         return ($id == $GLOBALS['prefs']->getValue('vinbox_id'));
     }

This fixes a problem I've been having where the "purge deleted" link disappeared
on our UW-IMAP server but not for the Dovecot server. I have no idea why,
but if vinbox_id is null then I'd assume no virtual inbox was created.

And for the person who asked:

edit imp/lib/Search.php, find 'function createVINBOXFolder() and
just stick a return; in as the first line in the function.
I'm about to do this on the production servers as I don't want the
functionality enabled just yet.




Adrian