Need to see if your shared folder is taking up space on your dropbox 👨‍💻? Find out how to check here.

Forum Discussion

pyraxic's avatar
pyraxic
Helpful | Level 6
8 years ago
Solved

Dropbox API not returning all files

I am using Django and trying to get the list of all files. My dropbox contains 4149 files but it only returns 3998. Upon checking it seems that the most recent files uploaded to Dropbox isn't showing up in the list. Here is my code.

 

    dbx = dropbox.Dropbox(oauth_result.access_token)
    metadata = dbx.files_list_folder("", recursive=True)
    flist = []
    if metadata.has_more == True:
        m1 = metadata.entries
        cur = metadata.cursor
        for i in m1:
            if isinstance(i, dropbox.files.FileMetadata):
                flist.append([i.name])
        m2 = dbx.files_list_folder_continue(cur)
        while m2.has_more == True:
            for i in m2.entries:
                if isinstance(i, dropbox.files.FileMetadata):
                    flist.append([i.name])
            cur = m2.cursor
            m2 = dbx.files_list_folder_continue(cur)
  • I tried running the code you posted and was able to reproduce the issue of it dropping the last page. Here's a fixed version:

     

    dbx = dropbox.Dropbox(oauth_result.access_token)
    result = dbx.files_list_folder("", recursive=True)
    file_list = []
    
    def process_entries(entries):
        for entry in entries:
            if isinstance(entry, dropbox.files.FileMetadata):
                file_list.append([entry.name])
    
    process_entries(result.entries)
    
    while result.has_more:
        result = dbx.files_list_folder_continue(result.cursor)
    
        process_entries(result.entries)
    
    print(len(file_list))
    

9 Replies

Replies have been turned off for this discussion
  • Greg-DB's avatar
    Greg-DB
    Icon for Dropbox Community Moderator rankDropbox Community Moderator
    8 years ago
    For reference, where/how are you getting the 4149 number from?

    Does it include folders by any chance? In your code, you're only picking out files. Folders would be `FolderMetadata`.

    Also, note that it looks like your code wouldn't list any files if the account happens to only have one page worth of entries. I.e., it is possible for the first response's `has_more` to be `False`, in which case you wouldn't get to your `for` loop.
  • pyraxic's avatar
    pyraxic
    Helpful | Level 6
    8 years ago

    I am getting 4149 from dropbox web gui. I only have one folder called "/Camera". I selected all files and it shows that there are 4149 files in that folder. Even if I try to use the code

     metadata = dbx.files_list_folder("/Camera", recursive=True)

     It still shows 3998 files.

  • Greg-DB's avatar
    Greg-DB
    Icon for Dropbox Community Moderator rankDropbox Community Moderator
    8 years ago
    Thanks! I just took another look at your code, and it looks like it actually won't process the last page of results. That is, the last call to `files_list_folder_continue` will cause `m2.has_more` to be `False`, which will terminate your `while` loop before that final `m2.entries` is processed. You'll just need to restructure the code to avoid that. (E.g., only put `files_list_folder_continue` at the top of `while` loop, or something to that effect.)
  • pyraxic's avatar
    pyraxic
    Helpful | Level 6
    8 years ago

    Hi Greg,

     

    files_list_folders_continue() is already outside of the while loop.

     

    m2 = dbx.files_list_folder_continue(cur)
            while m2.has_more == True:
                for i in m2.entries:
                    if isinstance(i, dropbox.files.FileMetadata):
                        flist.append(i.name)
                cur = m2.cursor
                m2 = dbx.files_list_folder_continue(cur)
  • Greg-DB's avatar
    Greg-DB
    Icon for Dropbox Community Moderator rankDropbox Community Moderator
    8 years ago
    I'm referring to the second call to files_list_folder_continue at the end of the while loop.
  • pyraxic's avatar
    pyraxic
    Helpful | Level 6
    8 years ago

    Even putting it outside of while loop didn't make any difference.

     

            m2 = dbx.files_list_folder_continue(cur)
            while m2.has_more == True:
                for i in m2.entries:
                    if isinstance(i, dropbox.files.FileMetadata):
                        flist.append(i.name)
                cur = m2.cursor
            m2 = dbx.files_list_folder_continue(cur)
        return render(request, 'accounts/dropbox.html', {'flist': flist})
  • pyraxic's avatar
    pyraxic
    Helpful | Level 6
    8 years ago

    I even changed that to outside of loop but same results. It doesn't seem to be getting to the while loop at all.

  • Greg-DB's avatar
    Greg-DB
    Icon for Dropbox Community Moderator rankDropbox Community Moderator
    8 years ago

    I tried running the code you posted and was able to reproduce the issue of it dropping the last page. Here's a fixed version:

     

    dbx = dropbox.Dropbox(oauth_result.access_token)
    result = dbx.files_list_folder("", recursive=True)
    file_list = []
    
    def process_entries(entries):
        for entry in entries:
            if isinstance(entry, dropbox.files.FileMetadata):
                file_list.append([entry.name])
    
    process_entries(result.entries)
    
    while result.has_more:
        result = dbx.files_list_folder_continue(result.cursor)
    
        process_entries(result.entries)
    
    print(len(file_list))
    

About Dropbox API Support & Feedback

Node avatar for Dropbox API Support & Feedback
Find help with the Dropbox API from other developers.

The Dropbox Community team is active from Monday to Friday. We try to respond to you as soon as we can, usually within 2 hours.

If you need more help you can view your support options (expected response time for an email or ticket is 24 hours), or contact us on X, Facebook or Instagram.

For more info on available support options for your Dropbox plan, see this article.

If you found the answer to your question in this Community thread, please 'like' the post to say thanks and to let us know it was useful!