[dev] [cvs] commit: imp mailbox.php imp/lib Message.php

[Jan Schneider]

Zitat von "Kevin M. Myer" <kevin_myer at iu13.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> The Virtual Trash folder is nothing but a search result, so this should
>> work. OTOH using in_vfolder should make it safe to remove that extra
>> check.
>
> Not sure if its worth anything but one thing I noticed is that vfolder
> IDs show up in the URL prefaced by **, but that in some cases, if
> thismailbox was present as a paremeter in the URL, the value was
> actually a URL-encoded **search (%2A%2Asearch).  Since IMP_SEARCH_MBOX
> is defined as **search, the $mbox == IMP_SEARCH_MBOX would never be
> true if $mbox was the URL-encoded value, right?  Which would cause that

No, unless it is double encoded. PHP automatically decodes urlencoded 
parameters.

> Also, I don't know if it matters or not but the URL for the Empty Trash
> icon pulls in the preference value for the trash folder, even if you
> are using Virtual Trash.  I hacked IMP.php to use the Trash vfolder id

But that is only to have all values for this rather long expression at 
hand. It is only used if necessary.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/