Storage API docs reference undefined "depth" property

VERIFIED FIXED

Status

Cloud Services
Server: Sync
--
trivial
VERIFIED FIXED
7 years ago
6 years ago

People

(Reporter: rfkelly, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

(Reporter)

Description

7 years ago
Reviewing docs at:

   http://docs.services.mozilla.com/storage/apis-1.1.htm#apis

The entry for:

   DELETE https://server/pathname/version/username/storage/collection

States that the deletions can be ordered "by depth, then by sortindex".

What's "depth"?  It's not defined elsewhere on the page.
Heh, wow. Depth is a really old field that we used to have in the early days to try to do trees on the server side. Tarek, is this index even supported in python? I seriously doubt it, and we probably need to pull the line.
(Reporter)

Comment 2

7 years ago
The only reference to "depthindex" in server-storage code is the following comment in the testcases:

        # "sort"
        #   'oldest' - Orders by modification date (oldest first)
        #   'newest' - Orders by modification date (newest first)
        #   'index' - Orders by the sortindex (ordered lists)
        #   'depthindex' - Orders by depth, then by sortindex (ordered trees)

        # sort is used only if limit is used.
        # check this with toby
It's not implemented, we can nuke that doc line, thanks for the catch Ryan
(Reporter)

Comment 4

7 years ago
References to "depth" removed in commits:
http://hg.mozilla.org/services/docs/rev/603a3a0b21d3
http://hg.mozilla.org/services/server-storage/rev/c6bd42742071
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.