[dev] Re: [patch] to fix imp cancel message with attachments problem

Jan Schneider jan at horde.org
Sun Oct 24 08:21:48 PDT 2004


Zitat von Rich Bartell <rwb at bartellonline.com>:

> On Sat, 23 Oct 2004, Rich Bartell wrote:
>
> <snip>
>> Upon digging into the code, I noticed a difference in how the 
>> javascript code for the cancel function was created between those 
>> composing in a popup window (which worked) and those composing in 
>> the same window (which didn't).
> <snip>
>
> Actually, I typed too soon on this - while the popup window version
> doesn't give the false message sent confirmation when "Cancel Message" is
> clicked (because the confirmation option only applies when the compose in
> a new window is not selected), it does fail to delete the temporary file
> with the attachment, because of a type now also corrected by the attached
> revised version of the patch . . .

Nice catch. Committed, thanks.

Jan.

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


More information about the dev mailing list