Summary: | importer fails if renaming pattern contains folder separators | ||
---|---|---|---|
Product: | [Applications] gwenview | Reporter: | thePanz <thepanz> |
Component: | importer | Assignee: | Gwenview Bugs <gwenview-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nate |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/gwenview/0eacf80b9331a0156327c7df586af802276abd44 | Version Fixed In: | 19.12.0 |
Sentry Crash Report: |
Description
thePanz
2010-05-10 16:48:59 UTC
This is caused by the '/' you added in the renaming pattern. This is currently not supported (but it should be). Dear Bug Submitter, This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond. Thank you for helping us make KDE software even better for everyone! Dear Bug Submitter, This is a reminder that this bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? This bug will be moved back to REPORTED Status for manual review later, which may take a while. If you are able to, please lend us a hand. Thank you for helping us make KDE software even better for everyone! Git commit 0eacf80b9331a0156327c7df586af802276abd44 by Nate Graham, on behalf of Bart De Vries. Committed on 31/10/2019 at 20:23. Pushed by ngraham into branch 'master'. Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater). Summary: The importer will (recursively) create additional subfolders if the autoformatter pattern contains slashes. Add additional unit tests to importertest. Related: bug 336039 FIXED-IN: 19.12.0 Test Plan: Added additional unit tests to check the creation of subfolders when slashes are present in the autoformatter pattern. Reviewers: #gwenview, ngraham Reviewed By: #gwenview, ngraham Subscribers: ngraham Tags: #gwenview Differential Revision: https://phabricator.kde.org/D25016 M +13 -1 importer/importer.cpp M +21 -0 tests/auto/importertest.cpp https://commits.kde.org/gwenview/0eacf80b9331a0156327c7df586af802276abd44 |