58782 - no alert when I insert an image that doesn't exist, get an alert on send

Status: VERIFIED DUPLICATE of bug 58646

(MailNews Core :: Composition, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

to reproduce:

html compose a new message
insert an image, say foo.gif where foo.gif does not exist.
no alert from the insert image dialog.

send the message.  upon send, you'll get an alert saying:

"http://foo.gif does not exist, do you still want to send?"

this error can still happen if we add and an alert to the image insert dialog,
but it seems less likely.

nbaca, do you remember the exact alert text?

Comment 1

[Ninoschka Baca]

It states "There was a problem including the file foo.gif. Would you like to
continue sending the message without this file?"

Comment 2

[Jean-Francois Ducarroz]

reassign to cmanske to add an alert if user select an image that doean't exist.

Comment 3

[Charles Manske]

The problem is we don't want to try to test for images that may be remote, but
user isn't online. 

Comment 4

[Charles Manske]

We don't want to test for image existence because of the remote image problem.
But we should show the "broken image" icon when user types in a local filename
that is bad, so this is really a dup of the image icon problem: 58646

*** This bug has been marked as a duplicate of 58646 ***

Comment 5

[Sheela Ravindran]

Seth,
4.7 puts a broken image at the location of where you insert a non-existant file
4.7 then puts up an alert when sending, similar to one 6.0 puts up.
The missing item in 6.0 is the broken image when inserting a non-existant file,
6.0 shows nothing.  Seth in this bug were you asking for an alert or will the
broken image be OK?

If a broken image will do, this can be verified as a dup of 58646?  If you're
asking for an alert, then please comment so we can add your comment to the other
bug or reopen this one.

Comment 6

[Charles Manske]

I don't think we want an alert, so dupping to 58646 is fine with me.

Comment 7

[Sheela Ravindran]

Based on comments below marking this bug as a dup.