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

QueryInterface to ContainerResultNode for folders instead of QueryResultNode

RESOLVED FIXED

Status

Cloud Services
Firefox Sync: Backend
P5
normal
RESOLVED FIXED
8 years ago
3 months ago

People

(Reporter: Mardak, Unassigned, Mentored)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sync:bookmarks][good first bug][lang=js])

(Reporter)

Description

8 years ago
Not sure how this affects things though..

Updated

8 years ago
Target Milestone: --- → Future
Priority: -- → P5
Whiteboard: [sync:bookmarks]
Target Milestone: Future → ---
or use PlacesUtils.asContainer() or .asQuery utils.  Regarding how this affects things, it depends if Sync needs any of the properties exposed by QueryResultNode and not by ContainerResultNode.
Probably we do something unsafe :)

(As you can see, I'm triaging old bugsā€¦)
Whiteboard: [sync:bookmarks] → [sync:bookmarks][good first bug][mentor=rnewman][lang=js]
(Assignee)

Updated

3 years ago
Mentor: rnewman@mozilla.com
Whiteboard: [sync:bookmarks][good first bug][mentor=rnewman][lang=js] → [sync:bookmarks][good first bug][lang=js]
Hello all. I would like to know how to work on this bug. Which files to create and/or modify etc. Thanks.
Flags: needinfo?(rnewman)
Hi Leni! Thanks for your interest, but we've already removed this from Sync, so I think we can close the bug.
Status: NEW → RESOLVED
Last Resolved: 3 months ago
Flags: needinfo?(rnewman)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.