Bug 395790 - Rename with nested tags breaks due to | bar character
Summary: Rename with nested tags breaks due to | bar character
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: AdvancedRename-metadata (show other bugs)
Version: 5.9.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-23 14:09 UTC by vilkmail
Modified: 2022-02-01 06:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0


Attachments
Calibre setup dialog with save ebook options to disk. (587.12 KB, image/png)
2018-06-24 08:37 UTC, caulier.gilles
Details

Note You need to log in before you can comment on or make changes to this bug.
Description vilkmail 2018-06-23 14:09:46 UTC
When renaming a file using "[file] - [meta:Xmp.digiKam.TagsList]" nested tags are separated by a bar | character, which is not a valid filename character in Windows 10. The rename fails at this point.

Example tags: Location|Paris
Comment 1 Maik Qualmann 2018-06-24 07:02:08 UTC
All these characters are not allowed in Windows file names.

? * < >  , \ + : = / " ; [ ] |

What should we do?
1. Convert all characters under Linux and Windows? With which character replaced?
2. Check the file system and convert only FAT or NTFS these characters?

I would imagine that under Linux certain users use these characters.

Maik
Comment 2 caulier.gilles 2018-06-24 08:37:28 UTC
Created attachment 113533 [details]
Calibre setup dialog with save ebook options to disk.
Comment 3 Maik Qualmann 2018-06-24 11:35:14 UTC
Git commit 33234dc80d3162823e569b0f0bf67d96321e0dc9 by Maik Qualmann.
Committed on 24/06/2018 at 11:33.
Pushed by mqualmann into branch 'master'.

replace invalid characters under a Windows file system when renaming
Related: bug 391312
FIXED-IN: 6.0.0

M  +2    -1    NEWS
M  +11   -1    core/utilities/advancedrename/advancedrenamemanager.cpp

https://commits.kde.org/digikam/33234dc80d3162823e569b0f0bf67d96321e0dc9