[dev] Attachments broken?

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at bigworm.curecanti.org>:

> Quoting Marc Jauvin <marc at r4l.com>:
>
>> OK, I can see from the CVS log that this has been changed on purpous...
>> this is fine, except for one thing: if you don't have read permission
>> on the file you  upload or if the file does not actually exists, you
>> won't notice and the mail will be sent anyway (without warning).
>>
>> So if we're gonna keep this, we'd need some form of verification BEFORE
>> it's too late and the email gets delivered.
>
> Try what I just committed.

The attach button shouldn't go completely. You can't set a description 
anymore now, or change the disposition, or preview the attachment after 
attaching it anymore, which I personally used a lot to verify that I'm 
really sending the correct file.

Jan.

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