If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

startup-with-no-profiles and choosing IE / Opera or Safari doesn't doesn't import favorites / bookmarks

VERIFIED FIXED

Status

()

Firefox
Bookmarks & History
VERIFIED FIXED
11 years ago
8 years ago

People

(Reporter: (not reading, please use seth@sspitzer.org instead), Assigned: (not reading, please use seth@sspitzer.org instead))

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

 
startup-with-no-profiles and choosing IE doesn't doesn't import IE favorites

although manually importing IE favorites works.

additionally, startup-with-no-profiles and Netscape / Seamonkey after we've enable places acts differently that it does in Firefox 2.

Specifically, there is no "Get Bookmarks Add-ons" bookmark, and there is no "Mozilla Firefox" folder.

but, the seamonkey bookmarks are imported, as we copy over the bookmarks.html file, and then on startup, we'll import that into places.
Summary: startup-with-no-profiles and choosing IE doesn → startup-with-no-profiles and choosing IE doesn't doesn't import IE favorites
I think what is going on is that after we copy the favorites, we still blow away all the bookmarks on startup because browser.places.importBookmarksHTML is set to true by default.

I think if we are doing a migration and we call CopyBookmarks() or CopyFavorites() [for IE] with PR_TRUE for aReplace, we need to set browser.places.importBookmarksHTML to false so that on restart, we don't re-import bookmarks.html and blow away what we just migrated.
Assignee: nobody → sspitzer
Created attachment 265403 [details] [diff] [review]
we need to fix ie, opera and safari

the other importers copy bookmarks into bookmarks.html, which we will then import.
Attachment #265403 - Flags: review?(dietrich)
the spin off bug to log is restoring the fx 2 behavior of adding the "Get Bookmarks Add-ons" (and other bookmarks, like the "Mozilla Firefox" folder and its bookmarks?) on startup-with-no-profiles import.
Status: NEW → ASSIGNED
Comment on attachment 265403 [details] [diff] [review]
we need to fix ie, opera and safari

thanks seth, r=me.
Attachment #265403 - Flags: review?(dietrich) → review+
OS: Windows XP → All
Hardware: PC → All
Summary: startup-with-no-profiles and choosing IE doesn't doesn't import IE favorites → startup-with-no-profiles and choosing IE / Opera or Safari doesn't doesn't import favorites / bookmarks
Depends on: 381315
verifying this on opera will require a fix to bug #381315 first.  

(I locally tested the opera part of my fix by commenting out the code that causes nsINIParser to crash.)
fixed

Checking in nsIEProfileMigrator.cpp;
/cvsroot/mozilla/browser/components/migration/src/nsIEProfileMigrator.cpp,v  <--
  nsIEProfileMigrator.cpp
new revision: 1.57; previous revision: 1.56
done
Checking in nsOperaProfileMigrator.cpp;
/cvsroot/mozilla/browser/components/migration/src/nsOperaProfileMigrator.cpp,v
<--  nsOperaProfileMigrator.cpp
new revision: 1.62; previous revision: 1.61
done
Checking in nsSafariProfileMigrator.cpp;
/cvsroot/mozilla/browser/components/migration/src/nsSafariProfileMigrator.cpp,v
 <--  nsSafariProfileMigrator.cpp
new revision: 1.38; previous revision: 1.37
done
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
bsmedberg has fixed bug #381315, so it is now possible to verify the fix for opera.
Verified fixed for Opera and Internet Explorer with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a5pre) Gecko/20070525 Minefield/3.0a5pre ID:2007052512 [cairo] / no crash on opera and also bookmarks imported with IE 7
Also verified with safari on Mac build from 2007052505
Status: RESOLVED → VERIFIED
> the spin off bug to log is restoring the fx 2 behavior of adding the "Get
> Bookmarks Add-ons" (and other bookmarks, like the "Mozilla Firefox" folder and
> its bookmarks?) on startup-with-no-profiles import.

see bug #38667
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.