1591790 - Return bookmark paths in PlacesUtils.bookmarks.fetch

Status: RESOLVED FIXED

(Toolkit :: Places, enhancement, P3)

Description

[fluks]

PlacesUtils.bookmarks.fetch should also return paths of a bookmark to make it possible to implement a feature in Library window, and maybe in bookmarks sidebar too, to open enclosing folder of a bookmark.

See also this comment.

Comment 1

[Marco Bonardo [:mak]]

yes, the idea was to have an includePath option added to the options object, and if it's set the query can use RECURSIVE to fetch the path, that may be an array of { title, guid} objects, probably starting from the root and arriving to the parent.

Comment 2

[Robert Townley]

Any progress on this?

Comment 3

[Mark Banner (:standard8)]

(In reply to Robert Townley from comment #2)

Any progress on this?

Generally you should be able to assume that if there's been no comments on a bug recently (and nothing in any attachments), then there's been no progress.

Comment 4

[Lebar]

Attachment: Bug 1591790 - Return bookmark paths in fetch. r=mak,Standard8

Comment 5

[Lebar]

Attachment: Bug 1591790 - Test for returning bookmark path in bookmarks.fetch. r?mak

Depends on D110786

Comment 6

[Marco Bonardo [:mak]]

Atm we're a bit overloaded with the Proton work, but I'll try to look at this soon. I added Mark as a first-come-first-serve reviewer, though he's out of office this week... Anyway, thank you for you contribution and hopefully we can take it very soon.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f897c6bb3c38
Return bookmark paths in fetch. r=Standard8

Comment 8

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/f897c6bb3c38