Bug 386785 - Change needed for separator in ignore list (currently blank space)
Summary: Change needed for separator in ignore list (currently blank space)
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Setup-Database (show other bugs)
Version: 5.7.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-12 18:22 UTC by Greg4b
Modified: 2017-11-17 19:33 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.8.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Greg4b 2017-11-12 18:22:13 UTC
Since the current separator of the ignore list is a blank space, foldernames with a space within, cannot be ignored at Version 5.7 running a Windows Machine.

See also https://plus.google.com/112590585164020446705/posts/UmWqK7ae6Vh - Gilles Caulier proposed to change the separator eg. to ";"
Comment 1 Greg4b 2017-11-12 18:25:25 UTC
(In reply to Greg4b from comment #0)
> Since the current separator of the ignore list is a blank space, foldernames
> with a space within, cannot be ignored at Version 5.7 running a Windows
> Machine.
> 
> See also https://plus.google.com/112590585164020446705/posts/UmWqK7ae6Vh -
> Gilles Caulier proposed to change the separator eg. to ";"

Also a folder named "#recycle" is currently also not recognized to be ignored...
Comment 2 Maik Qualmann 2017-11-14 11:40:45 UTC
We would also change the behavior for the mime types. Unless we write an extra function for the ignored directories. I would suggest we support quotes.

Maik
Comment 3 Maik Qualmann 2017-11-17 19:33:57 UTC
Git commit 7336cd2ca9d66a398b5f7fd4098d2ef6f06d1ebe by Maik Qualmann.
Committed on 17/11/2017 at 19:32.
Pushed by mqualmann into branch 'master'.

use only semicolon as separator for the ignored directories
FIXED-IN: 5.8.0

M  +2    -1    NEWS
M  +10   -10   libs/database/coredb/coredb.cpp
M  +2    -2    utilities/setup/setupdatabase.cpp
M  +22   -6    utilities/setup/setuputils.cpp
M  +3    -1    utilities/setup/setuputils.h

https://commits.kde.org/digikam/7336cd2ca9d66a398b5f7fd4098d2ef6f06d1ebe