0xCCF4 / PhotoSort

A tool to rename/move/copy/hardlink/symlink and sort photos and videos by its EXIF date.
GNU General Public License v3.0
5 stars 1 forks source link

[Feature request] Output if files are not processed by photoSort #48

Open roykrikke opened 5 days ago

roykrikke commented 5 days ago

Two options to implement

1: All files from --source-dir which e.g. text files or images or videos which do not have creation date information will be placed in a directory called unknown without file name change. By doing this you can be sure that the --source-dir directory can be safely deleted after the successful process completion because all files from the --source-dir directory have a copy in the --target-dir directory. Flag --unknown This flag can only be used with flag --move-mode move

2: All files from --source-dir which e.g. text files or images or videos which do not have creation date information will be listed as a warning on the CLI. E.g. 024-10-17T18:35:34Z INFO photo_sort] Unknown target file: "/media/testing/sort/test.txt"

0xCCF4 commented 4 days ago

Great idea.

Let me know, if I should implement this or you would like to try it. (#45)

roykrikke commented 4 days ago

Would be great if you could implement for now. Then I can start using the new feature. On top of this I will start my learning journey and start contributing at a later time.

0xCCF4 commented 3 days ago

Tasks:

0xCCF4 commented 3 days ago

I've implemented your suggestion with the unknown directory in such a way that with --unknown <FILE_FORMAT> a custom format string can be set for unknown files. Does this work for you? If not, just reopen this issue.

roykrikke commented 2 days ago

Thanks for implementing this feature.

You implemented: Show a warning if a file does not contain date information I think it would also be helping if a file is not processed because it is an unknown target file (not match the specified extensions list) to show: 024-10-17T18:35:34Z INFO photo_sort] Unknown target file: "/media/testing/input/test.txt"

You wrote: _I've implemented your suggestion with the unknown directory in such a way that with --unknown a custom format string can be set for unknown files._ This works great. I would be helping other to update help from:

--unknown <UNKNOWN_FILE_FORMAT>  The target file format for files that do not match the specified extensions list.
                                       If not present files that do not match the extension list are ignored, hence not moved,
                                       copied etc. See the `file_format` for an extensive description of possible format values

to (update proposal):

--unknown <UNKNOWN_FILE_FORMAT>  The target file format for files that do not match the specified extensions list.
                                       If not present files that do not match the extension list are ignored, hence not moved,
                                       copied etc. See the `file_format` for an extensive description of possible format values.
                                       E.g. with using --unknown "/unknown/{:name}.{ext}" all files not matching the specified extensions list
                                       will be moved to unknown folder in -t, --target-dir <TARGET_DIR>