906532 - Composer 2.20 corrupts HTML in ways 2.19 didn't

Status: RESOLVED WORKSFORME

(SeaMonkey :: Composer, defect)

Description

[Jon Roland]

Attachment: dub-dom.html

User Agent: Mozilla/5.0 (X11; Linux i686; rv:23.0) Gecko/20100101 Firefox/23.0 SeaMonkey/2.20 (Beta/Release)
Build ID: 20130803195701

Steps to reproduce:

Opened page http://constitution.org/proppriv/e-domain/dub-dom.html for edit, made changes, and saved.


Actual results:

1. The javascript that produces the ads was altered to convert absolute URLs into relative URLs with the creation of supporting files and a directory that should not be created. The scripts only work by accessing the remote site.
2. Several of the HTML codes were replaced with garbage, specifically  » and —
3. This behavior was not observed with the previous version 2.19, so something got broken. It was observed several versions back, in late 2011.


Expected results:

Editing the body of the text should have been the only changes made. All the existing HTML should have been left alone.

Comment 1

[therube]

I'm not seeing any change?

I took this line
> <big>Dubious Domain<br>

And changed it to
> <big>Dubious Domain219<br>
in SeaMonkey 2.19 & saved the page as html.

And changed it to
> <big>Dubious Domain220<br>
in SeaMonkey 2.20 & saved the page as html.

Comparing the two pages, the only difference is /219/220/.

Comment 2

[Phoenix]

Opened page in current trunk version, changed a couple of letters, saved it, observed differences:
1) » and — in original file became > and - respectively, so problem is currently fixed
2) Some minor attributes rearrangements in td tag, nothing critical, setting close date to somewhere close to 2.23 release

Comment 3

[Jon Roland]

All of the problems don't show up by just loading the file from the local drive and saving it again using the Composer. It is only after making certain changes and saving it that one gets a save that alters the main file and creates several support files that are definitely not desired. I am attaching the corrupted file and the support files in the subdirectory dub-dom-bad_files with files donatre.html and show_ads.js. In that directory is a subdirectory donate_data with the file blue-small.gif

Comment 4

[Jon Roland]

Attachment: dub-dom-bad.html

Comment 5

[Jon Roland]

Attachment: donate.html

Comment 6

[Jon Roland]

Attachment: show_ads.js

Comment 7

[Jon Roland]

Attachment: blue-small.gif

Comment 8

[Jon Roland]

The intended use is to download an HTML file from the remote site using ftp or a method of saving that does not alter links or create support files, because the intent is to edit it and ftp it back up to the remote server for viewing, not viewing from the local drive except in the composer window. That means support files will only appear if their links are absolute rather than relative, although those should work.

The main desire is that the adsense ads not be altered, but continue to work before and after editing other parts of the page and saving the result to be ftp'd up to the server again.

Comment 9

[Phoenix]

Check this build - http://ftp.mozilla.org/pub/mozilla.org/seamonkey/nightly/latest-comm-central-trunk/seamonkey-2.23a1.en-US.linux-i686.tar.bz2 and see if problem is present there

Comment 10

[Jon Roland]

Tried  http://ftp.mozilla.org/pub/mozilla.org/seamonkey/nightly/latest-comm-central-trunk/seamonkey-2.23a1.en-US.linux-i686.tar.bz2 and the problem remains, both for altering external links and for the special characters.

Comment 11

[Jon Roland]

Some target release needs to be tested on a file containing EVERY HTML character code. That should be standard QA procedure.

Comment 12

[Phoenix]

Resolved per whiteboard