[krita] [Bug 446279] Resource files that failed to import stay in the resource folder (shouldn't they be removed?)
https://bugs.kde.org/show_bug.cgi?id=446279 Tiar changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/c76519a76 |hics/krita/commit/0ef02e5e4 |452f6d7cdd2eba0911861f3ad88 |3bc1ac6d03c0110379b9c169672 |dd6e|165b --- Comment #5 from Tiar --- Git commit 0ef02e5e43bc1ac6d03c0110379b9c169672165b by Agata Cacko. Committed on 22/02/2022 at 22:51. Pushed by tymond into branch 'krita/5.0'. Don't leave incorrect resource files after loading Before this commit, KisFolderStorage wouldn't check if the resource can be loaded after importing, so it didn't even know that maybe it imported an incorrect file which is later staying in the resource folder unnecessarily. This commit adds checking for whether the resource can be loaded immediately after copying it to the folder, and then removes it if the resource file is incorrect. (cherry picked from commit c76519a76452f6d7cdd2eba0911861f3ad88dd6e) M +7-0libs/resources/KisFolderStorage.cpp https://invent.kde.org/graphics/krita/commit/0ef02e5e43bc1ac6d03c0110379b9c169672165b -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 446279] Resource files that failed to import stay in the resource folder (shouldn't they be removed?)
https://bugs.kde.org/show_bug.cgi?id=446279 Tiar changed: What|Removed |Added Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/c76519a76 ||452f6d7cdd2eba0911861f3ad88 ||dd6e Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #4 from Tiar --- Git commit c76519a76452f6d7cdd2eba0911861f3ad88dd6e by Agata Cacko. Committed on 08/01/2022 at 21:38. Pushed by tymond into branch 'master'. Don't leave incorrect resource files after loading Before this commit, KisFolderStorage wouldn't check if the resource can be loaded after importing, so it didn't even know that maybe it imported an incorrect file which is later staying in the resource folder unnecessarily. This commit adds checking for whether the resource can be loaded immediately after copying it to the folder, and then removes it if the resource file is incorrect. M +7-0libs/resources/KisFolderStorage.cpp https://invent.kde.org/graphics/krita/commit/c76519a76452f6d7cdd2eba0911861f3ad88dd6e -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 446279] Resource files that failed to import stay in the resource folder (shouldn't they be removed?)
https://bugs.kde.org/show_bug.cgi?id=446279 --- Comment #3 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/graphics/krita/-/merge_requests/1284 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 446279] Resource files that failed to import stay in the resource folder (shouldn't they be removed?)
https://bugs.kde.org/show_bug.cgi?id=446279 Halla Rempt changed: What|Removed |Added Ever confirmed|0 |1 Assignee|krita-bugs-n...@kde.org |tamtamy.tym...@gmail.com CC||ha...@valdyas.org Status|REPORTED|ASSIGNED --- Comment #2 from Halla Rempt --- Yes, they should removed. Please implement this? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 446279] Resource files that failed to import stay in the resource folder (shouldn't they be removed?)
https://bugs.kde.org/show_bug.cgi?id=446279 wolthera changed: What|Removed |Added CC||griffinval...@gmail.com --- Comment #1 from wolthera --- Yes, they should be removed if possible. -- You are receiving this mail because: You are watching all bug changes.