You might see that the Dropbox Community team have been busy working on some major updates to the Community itself! So, here is some info on what’s changed, what’s staying the same and what you can expect from the Dropbox Community overall.
Forum Discussion
ctubbs
5 years agoDropbox Staff
Python TypeError using Dropbox API move_v2function
Hi, I am trying to use the API to batch rename file. I had specific questions are around the move_v2 function:
I’m running dropbox.files.move_v2(params), where the args are from_path and to_pa...
LRomano
Explorer | Level 3
I see, so you need the entire path for each file you iterate over.
In which case, if I'm looking to change all names of all files (which have a certain extension, say .mp4 or .png), is there a way to extract just the folder path so I can change the name of each file as a modification of its original?
Thanks :)
Edit:
Error was
process_entries(result.entries)
File "test.py", line 25, in process_entries
result = dbx.files_move(from_path=original, to_path=modified)
File "/opt/anaconda3/lib/python3.7/site-packages/dropbox/base.py", line 2398, in files_move
None,
File "/opt/anaconda3/lib/python3.7/site-packages/dropbox/dropbox.py", line 338, in request
user_message_locale)
dropbox.exceptions.ApiError: ApiError('ffb95a528b5ad351d54e7d1722f6baae', RelocationError('from_lookup', LookupError('not_found', None)))
Greg-DB
5 years agoDropbox Staff
LRomano Thanks for sharing the error. That 'from_lookup/not_found' error does indicate that the issue is that the 'from_path' value doesn't point to anything. That does seem to be because you specified just the name and not the path, as discussed.
The Metadata.path_lower value is just a string containing the path for the item, which might look something like: "/folder/another folder/file.ext".
Since that's just a string, you can operate on it however you need. For instance, you could split it on "/" and take the parts you want, modify them, and build a new path string. Or, you could use a library or module like pathlib to work with it.
- LRomano5 years agoExplorer | Level 3
Fantastic. I used rsplit with '/' and to pull the lower_path apart giving me the file's directory and the file name.
Then I was able to add some text to the beginning of the file name, concat the whole thing together to create a new_path and it works perfectly!
Thanks so much for your help. Just FYI - code below if you want it (I had a spark DF with 1000s of rows of unique data that I needed to add to the beginning of each file):
result = dbx.files_list_folder(root_path, recursive=True) index = 0 def process_entries(entries): for entry in entries: if isinstance(entry, dropbox.files.FileMetadata): path = entry.path_lower.rsplit('/', 1)[0]+'/' file = entry.path_lower.rsplit('/', 1)[1] path_original = path+file global index code = sparkDF.collect()[index][2] path_new = path+code+'_'+file print(path_original) print(path_new) move = dbx.files_move(from_path=path_original, to_path=path_new) index += 1 process_entries(result.entries) while result.has_more: result = dbx.files_list_folder_continue(result.cursor) process_entries(result.entries)
- LRomano5 years agoExplorer | Level 3
Hmmmm, actually, it seems to be running ok, but I've had to put in another if statement on has_more = TRUE
Even with the while statement, for some reason it was running the function twice. Any idea why?
result = dbx.files_list_folder(root_path, recursive=True) full_list = [] index = 0 def process_entries(entries): for entry in entries: if result.has_more: if isinstance(entry, dropbox.files.FileMetadata): file = entry.path_lower.rsplit('/', 1)[1] if file.endswith(('.mp4','.png','.mov','.gif','.jpg')): path = entry.path_lower.rsplit('/', 1)[0]+'/' path_original = path+file global index code = sparkDF.collect()[index][2] file_new = code+'_'+file path_new = path+file_new sublist = [code,file_new,path_new] print(sublist) print(result.has_more) full_list.append(sublist) move = dbx.files_move(from_path=path_original, to_path=path_new) index += 1 process_entries(result.entries) while result.has_more: result = dbx.files_list_folder_continue(result.cursor) process_entries(result.entries)
- Greg-DB5 years agoDropbox Staff
I'm not sure off hand what the issue may be, but calling `if result.has_more` for each `entry` doesn't look like what you'd want to do. Once you're inspecting a particular entry, it shouldn't matter if there is another page of entries to retrieve (since you've already retrieved the entry you're operating on).
I recommend stepping through with a debugger to see where the code execution is going and why, and what the variables look like at any point in time.
Perhaps what you're seeing is the new entries created by your own move operations? You may need to implement some way to avoid operating on files that you've already moved, or maybe retrieve the full file listing before you start moving files, depending on exactly what you want your program to do and how.
- LRomano5 years agoExplorer | Level 3
Hmmmmm, yes that makes sense actually, that the move operation would in a sense 'reset' the file in terms of where the cursor is, and thus might get included in a future operation.
Problem is the full file list is 160k total files across a number of directories, and even counting them all takes ~8 mins.
I think a workaround is to include an if statement to exclue any file that beginswith('str'), as all the strings that I'm appending to files begin with 'zas', but it's not exactly elegant. :)
About Dropbox API Support & Feedback
Find help with the Dropbox API from other developers.
5,888 PostsLatest Activity: 24 hours agoIf 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 or Facebook.
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!