Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
clear old pinned items stored in wrong position
Browse files Browse the repository at this point in the history
fix #12941
  • Loading branch information
cezaraugusto committed Feb 1, 2018
1 parent 94e5e15 commit 4d4d193
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions app/sessionStore.js
Original file line number Diff line number Diff line change
Expand Up @@ -847,6 +847,16 @@ module.exports.runPreMigrations = (data) => {
if (data.cache) {
delete data.cache.bookmarkLocation
}

// pinned top sites were stored in the wrong position in 0.19.x
// allowing duplicated items. See #12941
// in this case eliminate pinned items so they can be properly
// populated in their own indexes
if (data.about.newtab.pinnedTopSites) {
// Empty array is currently set to include default pinned sites
// which we avoid given the user already have a profile
data.about.newtab.pinnedTopSites = [null]
}
}
}

Expand Down

0 comments on commit 4d4d193

Please sign in to comment.