Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExportToDatabase doesn't respect metadata when specifying file output location #2580

Closed
bethac07 opened this issue Feb 7, 2017 · 4 comments

Comments

@bethac07
Copy link
Member

bethac07 commented Feb 7, 2017

In ExportToDatabase, at least with the 'SQLite' option, if you use a piece of metadata in the 'Ouput file location' box (either as 'Elsewhere' or one of the subfolder options), it doesn't actually use the metadata that you give it- ie if all of the images in my image set have an extracted piece of metadata called 'SubfolderName' with a value of 'Subfolder1' , and I set my 'Output file location' as Default Output Subfolder -> 'SubfolderName', the .db is created in the Default Output Folder, not Default Output Folder/Subfolder1. Subfolders DO otherwise work fine as long as it's just a text value and not a piece of metadata.

I can pretty easily understand why this might be (CP isn't going to assume all images have the same metadata, so it wants a universal place to put the images), but if the metadata isn't used we shouldn't let it be specified.

@braymp
Copy link
Contributor

braymp commented Feb 7, 2017

Duplicate of #1276?

@bethac07
Copy link
Member Author

bethac07 commented Feb 7, 2017

Related but not sure if duplicate- this is for SQLite, which should be being written at the beginning not in post_run (I think). I think just removing Metadata based specification from that module entirely is probably the best solution.

@bethac07
Copy link
Member Author

bethac07 commented Feb 10, 2017

Ability to specify locations based on metadata will be removed in 3.0

@mcquin
Copy link
Contributor

mcquin commented Apr 19, 2017

Closing in favor of #2702

@mcquin mcquin closed this as completed Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants