Bug 372159 - Add option to run only the selected queue in BQM
Summary: Add option to run only the selected queue in BQM
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: BatchQueueManager-Workflow (show other bugs)
Version: 5.2.0
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-06 21:39 UTC by Simon
Modified: 2017-01-06 18:09 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.4.0


Attachments
Add option to run just current queue (8.93 KB, patch)
2017-01-06 15:21 UTC, Simon
Details
Add option to run just current queue (increased rc version counter) (9.20 KB, patch)
2017-01-06 17:48 UTC, Simon
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Simon 2016-11-06 21:39:00 UTC
Every time one clicks the BQM button in the toolbar a new queue is created and not deleted after closing. So it quickly happens that several queues are open at the same time. In this position it is impossible run only the currently selected queue. One has to cumbersomely select every single queue except the desired one to delete them (adding a shortcut to the "del" key would be nice as well).

I would propose to have two possibilities in place of the current "Run": "Run all" that does the same as it does now and "Run selected" to only run the currently selected queue.
Comment 1 Simon 2017-01-06 15:21:26 UTC
Created attachment 103232 [details]
Add option to run just current queue

The proposed tag adds the buttons "Run" and "Run all" instead of just "Run". Clicking "Run" processes the currently selected queue, clicking "Run all" processes all queue (previous behaviour).

It works as intended for me, could someone test this and give their opinion whether the used terminology is intuitive.
Comment 2 caulier.gilles 2017-01-06 17:42:43 UTC
Comment on attachment 103232 [details]
Add option to run just current queue

Patch is fine for me to commit before 5.4.0.

Note you patch .rc file without to increase the version ID on top of file. Please fix it.

Gilles Caulier
Comment 3 Simon 2017-01-06 17:48:25 UTC
Created attachment 103236 [details]
Add option to run just current queue (increased rc version counter)

Done.
Note to self: Read comments.
Comment 4 caulier.gilles 2017-01-06 17:51:20 UTC
Comment on attachment 103236 [details]
Add option to run just current queue (increased rc version counter)

perfect. let's go...
Gilles Caulier
Comment 5 Simon 2017-01-06 18:09:05 UTC
Git commit d79e40fdbdbd90ff33c11475d575f20f33112bb9 by Simon Frei.
Committed on 06/01/2017 at 18:06.
Pushed by sfrei into branch 'master'.

batch queue manager: add option to run current queue only
FIXED-IN: 5.4.0

M  +2    -1    NEWS
M  +71   -8    utilities/queuemanager/main/queuemgrwindow.cpp
M  +1    -0    utilities/queuemanager/main/queuemgrwindow.h
M  +3    -0    utilities/queuemanager/main/queuemgrwindow_p.h
M  +3    -1    utilities/queuemanager/main/queuemgrwindowui.rc
M  +7    -2    utilities/queuemanager/views/queuepool.cpp
M  +1    -0    utilities/queuemanager/views/queuepool.h

https://commits.kde.org/digikam/d79e40fdbdbd90ff33c11475d575f20f33112bb9