115088 - Saving "Web Page, HTML only" or "Text File" still creates directory for images & stuff

Status: VERIFIED DUPLICATE of bug 116418

(Core Graveyard :: File Handling, defect)

Description

[Boris Zbarsky [:bzbarsky]]

BUILD: 2001-12-13-08

STEPS TO REPRODUCE:

1) Hit Ctrl-s
2) Select "Web Page, HTML Only" from the dropdown
3) Click OK

EXPECTED RESULTS:

HTML is saved to the file name in the textbox (foo.html)

ACTUAL RESULTS:

HTML is saved to foo.html and a directory called foo_files is created.  The
directory is empty.

Comment 1

[sairuh (rarely reading bugmail)]

would this be the same as bug 115132?

Comment 2

[sairuh (rarely reading bugmail)]

also see this happening when i save using the "Text File" option.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Adam's comment from bug 115132:

If you want to save HTML only, you should pass a nsnull data path into
nsIWebBrowserPersist::SaveDocument. Otherwise it creates that directory to save
any files that may be linked to the document.

Comment 4

[Arthur]

If you choose "HTML only" I'd expect that it mozilla would leave the source the
way it is. But at the moment, all image links get changed to local files (which
don't get downloaded) too. That's a bit annoying if you're just fixing a typo in
your webpage via saving it with mozilla, correcting it and loading it up again.

Comment 5

[Adam Lock]

This is the JS source code for the saving HTML:

http://lxr.mozilla.org/seamonkey/source/xpfe/components/ucth/resources/helperAppDldProgress.js#403
I have a couple of points to make:

1. Pass null instead of filesFolder if you don't want the document "fixed up".
2. There is no need for the code to create this folder - webbrowserpersist will
create it for you. Currently webbrowserpersist always creates it, but it could
be changed to be on demand at some later stage.

Comment 6

[Diego Biurrun]

On build 2001122108 bug 115132 does not show up anymore, but when I save HTML
only the directory gets created and populated with images.  There is no
difference between "Web page, complete" and "Web page, HTML only".

Comment 7

[Andrew Hagen]

Worksforme on Windows 98, Mozilla 0.9.7 milestone build.

Comment 8

[sairuh (rarely reading bugmail)]

ack, no longer have the "text file" option --see bug 117491.

anyhow, marking a dup of bug 116418, per bz's comments over there.

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

Comment 9

[sairuh (rarely reading bugmail)]

mass verification of duplicate bugs: to find all bugspam pertaining to this, set
your search string to "DuplicateBugsBelongInZahadum".

if you think this particular bug is *not* a duplicate, please provide a
compelling reason, as well as check a recent *trunk* build (on the appropriate
platform[s]), before reopening.