[imp] Virtual inbox

[Michael M Slusarz]

Quoting Adrian Chadd <adrian at creative.net.au>:

> 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?

Sure.  Patch?

> 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.

This was a bug that has already been fixed for IMP 4.1.1.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]