[dev] [PATCH] Wholesome Linkage

Chuck Hagenbuch chuck at horde.org
Mon Jul 5 08:58:42 PDT 2004


Quoting Andrew Coleman <mercury at appisolutions.net>:

> 1 order linked attachments
>  Hold the fuss, just the code
>  *extra docs available upon request

I've cleaned up and committed this code. A few things:

To note for the future:
* please read horde/docs/CODING_STANDARDS, and look at the differences between
what you submitted and what I committed.

To do before the bounty can be paid:
* linked attachments need to work when uploaded attachments aren't already
stored in the VFS.
* need a way for administrators to turn linked attachments on/off. Ideal would
be a preference which says "always send links, never send links, or let me
choose". This would then only show the compose screen option if the user wants
to choose, and has the benefit of letting the administrator lock in any 
option.
* looks like you missed sending the patch to imp/config/prefs.php.dist

Nice to haves (bounty not dependent on these):
* Admin interface for administrators to clean out attachments.

I think that's all - let me know if you have any questions. Thanks!

-chuck

--
"Regard my poor demoralized mule!" - Juan Valdez


More information about the dev mailing list