The default bug view has changed. See this FAQ.

backup bookmark file

RESOLVED FIXED in Firefox1.0beta

Status

()

Firefox
Bookmarks & History
P2
enhancement
RESOLVED FIXED
14 years ago
11 years ago

People

(Reporter: sheng han, Assigned: vlad)

Tracking

({fixed-aviary1.0})

unspecified
Firefox1.0beta
fixed-aviary1.0
Points:
---
Bug Flags:
blocking-aviary1.0PR +
blocking-aviary1.0 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [eta 04-08-02])

Attachments

(1 attachment, 1 obsolete attachment)

9.04 KB, patch
Ben Goodger (use ben at mozilla dot org for email)
: review+
Ben Goodger (use ben at mozilla dot org for email)
: approval-aviary+
Details | Diff | Splinter Review
(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.4) Gecko/20030701
Build Identifier: Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.4) Gecko/20030701

I use Firebird on PC/Windows platform. I have a problem when windows system
crash (blue screen), the next time I invoke Firebird, the bookmark file sometime
gone.

I wonder if its possible to place a backup file before 'FB' is up? This backup
file will just contain the bookmarks from last saved session, don't need to
syncronize with current using one. And while 'FB' is closed, do nothing to
backup file.  Next time 'FB' is called, remove old backup with new bookmark....



Reproducible: Sometimes

Steps to Reproduce:
1.
2.
3.

Actual Results:  
lost bookmark file...

Expected Results:  
that i can recover my bookmark from not too long ago when windows F$$K up

please....i've lost my bookmark for a couple of times.....

Comment 1

14 years ago
Quite agree on this bug. If it had journaling backup and rollback feature, it'd
be better, for Mozilla bookmark is a kind of personal database.

Updated

14 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All
*** Bug 220252 has been marked as a duplicate of this bug. ***
*** Bug 220252 has been marked as a duplicate of this bug. ***
*** Bug 221432 has been marked as a duplicate of this bug. ***
*** Bug 221721 has been marked as a duplicate of this bug. ***

Comment 6

14 years ago
Hi,

I suggest to add a feature to Mozilla / Firebird that allows to create backups
of files such as bookmarks.html, Personal toolbar, mail prefs, news prefs, etc.

How about adding a menu entry to the "Edit / Preferences..." menu underneath the
"Navigator" branch called "Settings backup"?

This should contain tick boxes of what somebody wants to backup, to where and
how often (time interval) this should happen. Also a restore feature should be
introduced with a selection option what and which backup should be restored.

This may solve some of the issues where an OS freezes or crashes unrecoverably.

--Christian
(Reporter)

Comment 7

14 years ago
FYI,
I discovered a plugin in the Firebird extentions page:
(http://texturizer.net/firebird/extensions/#bookmarkbackup)

I am using it, it will save the bookmark to your specified folder every day with
the weekday as a loop. So I end up with 7 bookmark backups in that folder, which
is quite acceptable for me. I guess people who need the backup feature can use
this before the actual native solution would came out.
*** Bug 227734 has been marked as a duplicate of this bug. ***
*** Bug 227968 has been marked as a duplicate of this bug. ***

Comment 10

14 years ago
*** Bug 228985 has been marked as a duplicate of this bug. ***
Targeting for Pierre. Enough people are encountering mysterious bookmarks loss
that having a backup is useful. 
Priority: -- → P3
Target Milestone: --- → Firefox1.0beta
brendan says it's worthwhile investigating if there's a platform pattern here
... perhaps file rename/move operations aren't atomic on windows, etc. 
*** Bug 229276 has been marked as a duplicate of this bug. ***
The Bookmarks Synchronizer extension is also a valuable tool that allows for
off-site backup of bookmarks using the FTP protocol. It can also export
bookmarks in XML format, and the latest versions have quite advanced functionality.

http://www.extensionsmirror.nl/index.php?showtopic=15
Flags: blocking1.0+
->danm

do a bugzilla query on "lost bookmarks" to find a number of bugs that backing up
the bookmark file would work around. 

First of all, just check to see if bookmark file is being held open when it
should have been closed...

A safe approach in general might be to make a copy of the bookmarks.html file
for backup purposes after a successful write of the bookmarks file. Many of the
"lost bookmarks" bugs result in bookmarks.html either being REMOVED or being
truncated to 0. In either case - when the browser starts instead of replacing
the bookmarks file with the DEFAULT one, the BACKUP file if present should be
used. This may hide the bug completely from users. 
Assignee: p_ch → danm.moz
Flags: blocking-aviary1.0RC1+
Priority: P3 → P2
*** Bug 249731 has been marked as a duplicate of this bug. ***
*** Bug 250302 has been marked as a duplicate of this bug. ***
Assignee: danm.moz → bugs
Created attachment 154340 [details] [diff] [review]
simple patch

not sure if we want to do this - not sure what the use is, but here it is
Assignee: bugs → vladimir

Comment 19

13 years ago
who would be good reviewers?  danm, sspitzer, dveditz...  lets get some eyes on
this and decide..  
(Assignee)

Comment 20

13 years ago
Patch needs to be updated due to nsSafeOutputStream changes (bug 252053).  Ben
and I also talked about making the parser catch a truncated file by catching a
missing "</HTML>" -- should be done either later today or by monday at the latest.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Whiteboard: [eta 04-08-02]
(Assignee)

Comment 21

13 years ago
wrong button.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 22

13 years ago
Created attachment 155020 [details] [diff] [review]
bookmarks-save-backup-0.patch

Updated patch.	Tries to save a backup copy of the previous successful write,
instead of right after the latest write -- if the problems we're seeing are
caused by fs corruption/lossage, then doing that copy afterwards won't help us
any.  The patch then ends up saving a backup after any change but before the
next Flush or if the save timer fires, whichever comes first.  It also moves
the scheduling-related timer-disabling into the timer callback itself, so we
still get the periodic saves even though scheduling/etc. is disabled
(Assignee)

Updated

13 years ago
Attachment #154340 - Attachment is obsolete: true
(Assignee)

Updated

13 years ago
Attachment #155020 - Flags: review?(bugs)
Attachment #155020 - Flags: approval-aviary?
Comment on attachment 155020 [details] [diff] [review]
bookmarks-save-backup-0.patch

>-    // bookmark timers and scheduling are disabled for 1.0, until
>-    // they can be fixed (bug 253478)
>-#if 0
>     /* timer initialization */
>     busyResource = nsnull;
> 
>     if (!mTimer)
>     {
>         busySchedule = PR_FALSE;
>         mTimer = do_CreateInstance("@mozilla.org/timer;1", &rv);
>         NS_ASSERTION(NS_SUCCEEDED(rv), "unable to create a timer");
>         if (NS_FAILED(rv)) return rv;
>         mTimer->InitWithFuncCallback(nsBookmarksService::FireTimer, this, BOOKMARK_TIMEOUT, 
>                                      nsITimer::TYPE_REPEATING_SLACK);
>         // Note: don't addref "this" as we'll cancel the timer in the nsBookmarkService destructor
>     }
>-#endif

did you mean to remove this if 0? 

otherwise, r/a=ben@mozilla.org
Attachment #155020 - Flags: review?(bugs)
Attachment #155020 - Flags: review+
Attachment #155020 - Flags: approval-aviary?
Attachment #155020 - Flags: approval-aviary+
(Assignee)

Comment 24

13 years ago
(In reply to comment #23)

> did you mean to remove this if 0? 

Yeah, I did, though I guess I meant to change the comment above it as well --
turns out I need that timer to do periodic saves of bookmarks. :)
(Assignee)

Comment 25

13 years ago
in branch/trunk.
Status: REOPENED → RESOLVED
Last Resolved: 13 years ago13 years ago
Resolution: --- → FIXED

Comment 26

13 years ago
On my system the bookmarks.bak file doesn't seem to get updated when
bookmarks.html is changed and Firefox restarted, it simply stays the same as it
was when first created (according to nsIFile.idl copyTo can fail if the target
file already exists).

Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7) Gecko/20040809 Firefox/0.9.1+

Comment 27

13 years ago
setting fixed-aviary1.0 for bugfixes checked into branch, for searching purposes.
Keywords: fixed-aviary1.0
*** Bug 262228 has been marked as a duplicate of this bug. ***
*** Bug 276169 has been marked as a duplicate of this bug. ***
*** Bug 276169 has been marked as a duplicate of this bug. ***
*** Bug 276169 has been marked as a duplicate of this bug. ***
*** Bug 276169 has been marked as a duplicate of this bug. ***

Comment 33

12 years ago
*** Bug 308237 has been marked as a duplicate of this bug. ***
*** Bug 311160 has been marked as a duplicate of this bug. ***

Comment 35

12 years ago
*** Bug 314424 has been marked as a duplicate of this bug. ***

Comment 36

12 years ago
*** Bug 315102 has been marked as a duplicate of this bug. ***
*** Bug 329102 has been marked as a duplicate of this bug. ***
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → bookmarks
You need to log in before you can comment on or make changes to this bug.