[dev] [commits] Horde branch master updated. 273ee9009dbfb88d925bcf459648270291cab48c

[Michael M Slusarz]

Quoting "Michael J. Rubinsky" <mrubinsk at horde.org>:

> commit 3a6dae7dab4c44fce7e283bd0c0ed4489808851a
> Author: Michael J Rubinsky <mrubinsk at horde.org>
> Date:   Mon May 23 18:05:00 2011 -0400
>
>     Bug: 10050 Don't remove tmp image file at request shutdown.
>
>     Commit 9fd6579fa7b1e1a920601c3f9517c1fd18a84c0e broke the existing
>     behavior of not removing the temporary file. This restores that
>     functionality. Not sure if this is the final fix for this, as this
>     causes tmp image files to build up in the temporary directory. However,
>     this was the previous behavior and should be the admin's job to perform
>     gc on temp directory periodically anyway.
>
>  turba/lib/Object.php |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

If data *must* persist beyond a single access, we should never be  
using temp files.  We must use some sort of permanent storage solution  
instead (e.g. VFS; see., e.g., compose attachment data in IMP).   
Anything else is attempting to use temp files in a manner not intended  
in the original design.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]