[dev] Re: [cvs] commit: imp compose.php mailbox.php

Jan Schneider jan@horde.org
Fri, 31 May 2002 23:56:38 +0200


Zitat von Michael M Slusarz <slusarz@bigworm.colorado.edu>:

> Quoting Jan Schneider <jan@horde.org>:
> 
> | That's wrong. 'mailbox' has been '**search', but 'thismailbox' has
> | correctly
> | been the folder name in the search results.
> |  
> 
> Here's what I get when I click on the "From:" field after doing a search
> in 
> INBOX using revision 2.359:
> 
> javascript:open_compose_win
> ('popup=1&to=&cc=&bcc=&msg=&subject=&actionID=124&mailbox=%2A%
> 2Asearch&index=4687&thismailbox=%2A%2Asearch');
> 
> Here's what I get in revision 2.360 (after my patch):
> 
> javascript:open_compose_win
> ('popup=1&to=&cc=&bcc=&msg=&subject=&actionID=124&mailbox=INBOX&index=4687&t
> hismailbox=%2A%2Asearch');
> 
> As you see, in both cases, thismailbox has not been set correctly - this
> is 
> why I set mailbox to be the location of the search message.  Am I doing 
> something wrong?

Yes, this doesn't happen in mailbox.php but in IMP::composeLink() and
IMP::getThisMailbox(). If you look at the links behind the subject column
you'll see that it actually works. You only have to fix the link that's been
generated for the from column if you set $conf['mailbox']['from_link'] to
'compose'.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft