Re: Small Question in imap_mailbox.php function..

On Mon, May 26, 2008 at 3:37 PM, Michael Peddemors
<[email protected]> wrote:
> Should it not be:
>
> $ret = boolean_hook_function('special_mailbox',strtolower($box),1);

You're saying you want the strtolower applied?  It may have originally
been an oversight, but OTOH, I'd say it's actually more correct and
sometimes useful to pass any data to a plugin from the core in its
most raw form, and in this case, that means your plugin has to
remember to take care of any details such as applying strtolower.

Cheers,

 Paul

> In the function below...
>
>
> /**
>  * Defines special mailboxes: given a mailbox name, it checks if this is a
>  * "special" one: INBOX, Trash, Sent or Draft.
>  */
> function isSpecialMailbox( $box ) {
>    global $trash_folder, $sent_folder, $draft_folder,
>           $move_to_trash, $move_to_sent, $save_as_draft;
>
>    $ret = ( (strtolower($box) == 'inbox') ||
>             isTrashMailbox($box) || isSentMailbox($box) ||
> isDraftMailbox($box) );
>
>    if ( !$ret ) {
>        $ret = boolean_hook_function('special_mailbox',$box,1);
>    }
>    return $ret;
> }
>

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
-----
squirrelmail-devel mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: [email protected]
List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel