262988 - Importing bookmark with missing CR/LF fails to import the whole file

Status: RESOLVED INCOMPLETE

(SeaMonkey :: Bookmarks & History, defect)

Description

[SiKing]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a4) Gecko/20040927
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a4) Gecko/20040927

I have a bookmarks file, which was generated in NN4.8 under Linux. When I moved
this file to a Windows machine, the CR/LF pairs were messed up - it was moved as
a binary file.
When I tried to Import this file as a bookmark file, the import failed. Only the
first bookmark was actually imported. When I corrected the file, inserting the
correct Enters throughout, the file imported correctly.
Since the bookmarks file is supposed to be an XML file, it should be
independednt of placement of CR/LF. This is irrelenvant of the fact that at the
top of the bookmakrs file it says that the user should not edit the file. :)
Therefore I consider this to be a problem.

Reproducible: Always
Steps to Reproduce:
1. Generate a sample bookmarks file, that has as a minimum of two bookmarks,
with missing CR/LF - essentially all one line.
2. Import the bookmarks file from step 1.
3. Notice the missing second (and subsequent) bookmakrs.
Actual Results:  
All bookmarks after the first one are missing.

Expected Results:  
Imported all the bookmakrs that were actually in the file.

about:buildconfig

Build platform
target
i586-pc-msvc

Build tools
Compiler 	Version 	Compiler flags
$(CYGWIN_WRAPPER) cl 	12.00.8804 	-TC -nologo -W3 -Gy -Fd$(PDBFILE)
$(CYGWIN_WRAPPER) cl 	12.00.8804 	-TP -nologo -W3 -Gy -Fd$(PDBFILE)
-I/usr/X11R6/include

Configure arguments
--without-system-jpg --without-system-zlib --enable-extensions=default,tasks
--enable-crypto --disable-auto-deps --disable-debug --enable-optimize

Comment 1

[Gervase Markham [:gerv]]

This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

Comment 2

[Gervase Markham [:gerv]]

This bug has been automatically resolved after a period of inactivity (see above
comment). If anyone thinks this is incorrect, they should feel free to reopen it.

Comment 3

[SiKing]

I think it may be too late :( but I did manage to reproduce this in the latest
Mozilla. No additional info that I can offer, other than what is stated above.

Comment 4

[SiKing]

OK I do have something:
I suspect, but I am not certain, this bug to be related to
https://bugzilla.mozilla.org/show_bug.cgi?id=265447

Comment 5

[Ray Booysen]

Reassigning as per Bug #32644

Comment 6

[Serge Gautherie (:sgautherie)]

Can you reproduce with SeaMonkey v1.1.9 ?

Comment 7

[Jens Hatlak (:InvisibleSmiley)]

bookmarks.html is neither XML nor arbitrary HTML, it's a defined format which is written in a certain way (internal rules) and only supposed to be read according to the same rules. Suggesting to WONTFIX.

Comment 8

[Philip Chee]

No reply from the original reporter in two years. In addition, on trunk we are now using Places Bookmarks and the import code is different so it's unclear if this bug still happens.

Comment 9

[SiKing]

I am the original reporter - no reply from the development team in two years. :P

I think Philip is correct: the bookmarks are now handled completely different, and I feel that this is no longer an issue. Feel free to close.

Thank You.