Closed Bug 1505191 Opened 6 years ago Closed 6 years ago

Ensure reordering bookmarks bumps the parent folder's last modified time

Categories

(Toolkit :: Places, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: lina, Assigned: lina)

References

Details

Attachments

(1 file)

The old bookmarks engine uses `reorder` to order children, as does sort by name. This doesn't bump the folder's last modified time, though. While that _should_ be okay for Sync, since we'll also call `PlacesSyncUtils.bookmarks.update` for newer remote folders, it's a good idea to do this, anyway. Otherwise, Sync might think a newer local order is older.
MozReview-Commit-ID: i7HJR44kjf
Pushed by kcambridge@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6925b3d4dbd0 Ensure `Bookmarks.reorder` bumps the last modified time. r=mak
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: