Bug 392195 - Import an existing bundle gives incorrect error message
Summary: Import an existing bundle gives incorrect error message
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Resource Management (show other bugs)
Version: 4.0
Platform: Debian stable Linux
: NOR normal
Target Milestone: ---
Assignee: Halla Rempt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-22 20:45 UTC by Ahab Greybeard
Modified: 2021-03-26 13:20 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahab Greybeard 2018-03-22 20:45:54 UTC
With the Krita 4.0.0 appimage, if you try to import a bundle that has just been created or has just been imported, an error message says that it does not exist.
If you create a bundle then close krita, you can restart it then import the bundle to get two active copies shown. After that you get the 'does not exist' error message if you try to import it again.
After another restart, there is only one bundle showing as active.
This seems to be a minor logical/presentation problem problem.
Comment 1 Scott Petrovic 2018-03-22 20:51:26 UTC
What are you trying to achieve by doing this? What were the results you were expecting by importing a bundle that you just created?

I am not sure if you are talking about multiple bugs, or just one bug in this ticket. Try to keep one bug per ticket otherwise it gets really confusing to understand and solve.
Comment 2 Ahab Greybeard 2018-03-22 20:56:55 UTC
I was expecting it to say that the bundle was already in krita. It is confusing for anyone who is trying to create bundles and test/check them.

I would agree that it has no practical purpose overall but it is a possible cause of confusion since the error message is incorrect.
Comment 3 Ahab Greybeard 2018-03-22 20:58:33 UTC
Please feel free to reject the report for whatever reason.
Comment 4 Halla Rempt 2018-03-28 14:05:18 UTC
I can confirm the issue, and krita should handle this more gracefully.
Comment 5 Halla Rempt 2021-03-26 13:20:48 UTC
Git commit fe3d2f4ac0217b338fd10e6a319250ba0facbf13 by Halla Rempt.
Committed on 26/03/2021 at 13:20.
Pushed by rempt into branch 'master'.

Improve adding an existing bundle

Instead of getting into a loop asking for a different name, offer
to overwrite or not.

M  +14   -3    libs/resources/KisResourceLocator.cpp
M  +2    -1    libs/resources/KisResourceLocator.h
M  +2    -1    libs/resources/KisStorageModel.cpp
M  +6    -13   plugins/extensions/resourcemanager/dlg_bundle_manager.cpp

https://invent.kde.org/graphics/krita/commit/fe3d2f4ac0217b338fd10e6a319250ba0facbf13