Folders created when not necessary

Bug #1708935 reported by Fioravanti Diego
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rapid Photo Downloader
New
Undecided
Unassigned

Bug Description

If I have pictures from two (or more) different days on my sd card, suppose today and yesterday, and I use a costume template for the folder structure, something like "<Image date (YYYY)>/<Image date (MM)>/<Image date (DD)>_Stuff/RAW", then downloading only the pictures from today force the creation of the folders even with the yesterday date, clearly empty. Is not a huge problem but it gets quite annoying if one needs to download 3-4 days of pictures with different names. I think that the preferred behaviour should be that the folders are created using only the metadata from the selected pictures not all the pictures available.

I am using rapid photo downloader version 0.9.1

description: updated
Revision history for this message
Damon Lynch (dlynch3) wrote :

The folders are created so they can be previewed in the download subfolders in the destination panel. They are deleted at program exit if not downloaded to.

Changed in rapid:
status: New → Won't Fix
Revision history for this message
Fioravanti Diego (dfioravanti) wrote :

I have just double checked, is not the case. Attached you can find some screenshots to illustrate the problem, I downloaded only one picture with date 06/08/2017 (DD/MM/YYYY) but since on the SD card there are some pictures from the 02/08/2017 the folder 02_Test is created, with inside an empty folder called RAW.

The problem exists because I create a "static" folder inside the last "dynamic" folder. As I wrote above I wrote above, I am using "<Image date (YYYY)>/<Image date (MM)>/<Image date (DD)>_Test/RAW" as dynamic path, so every folder always contains a RAW folder. It probably messes up with some check. I tried without the RAW folder and it indeed creates only the correct folder.

I believe that is the "cleanTempDirsForScanId" function that misses the directory because is not present in the "self.temp_dirs_by_scan_id" dictionary, but I am not sure since I cannot find where the folders are actually created and if the empty ones are added to self.temp_dirs_by_scan_id for future deletion.

I realize is a bit of an edge case but it would be really helpful to get it fixed. Thank you for your time

Revision history for this message
Damon Lynch (dlynch3) wrote :

Hi Fioravanti,

Sorry for the delay in my reply. I've been extremely busy.

The code to generate preview subfolders is folderspreview.py. You can investigate the code in there to see if you can locate a problem.

Thanks.

Changed in rapid:
status: Won't Fix → New
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.