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

Commit

Permalink
Merge pull request #3620 from brave/clear-bookmark-history
Browse files Browse the repository at this point in the history
clear the lastAccessedTime for all tags when clearing history
  • Loading branch information
bbondy authored Aug 31, 2016
2 parents 58ce528 + 30e1993 commit d51c09b
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions app/sessionStore.js
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,12 @@ module.exports.cleanAppData = (data, isShutdown) => {
if (data.sites) {
const clearHistory = isShutdown && getSetting(settings.SHUTDOWN_CLEAR_HISTORY) === true
if (clearHistory) {
// TODO - this should the history methods from siteUtils
data.sites = data.sites.filter((site) => site && site.tags && site.tags.length)
data.sites = data.sites.map((site) => {
delete site.lastAccessedTime
return site
})
}
}
if (data.downloads) {
Expand Down

0 comments on commit d51c09b

Please sign in to comment.