Bug 301486

Summary: additional empty queue when batch processing images using toolbar button
Product: [Applications] digikam Reporter: Danni Coy <danni.coy>
Component: BatchQueueManager-WorkflowAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 2.6.0   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: http://commits.kde.org/digikam/801a15da71c25c0559cc052898cfea93767052e2 Version Fixed In: 2.8.0
Sentry Crash Report:

Description Danni Coy 2012-06-09 02:46:10 UTC 
When using the toolbar button to batch process images digikam adds these images to a second queue leaving an empty queue which stops image processing until it is dealt with.

Reproducible: Always

Steps to Reproduce:
1. select some images, press the "Batch Queue Manager" button on the toolbar
2.
3.
Actual Results:  
images are added to a new queue (#2) leaving queue #1 with no images and no operations.

Expected Results:  
if the first queue has no images and no operations the images should have been added to this queue instead.
Comment 1 Marcel Wiesweg 2012-07-15 18:42:06 UTC 
In addition to fixing this bug, I have a question for general discussion:
Assume there is a non-empty queue. Should images with the default action be added to that queue, or to a new one? Same question put differently: Should "Add to New Queue" or "Add to current queue" be the default action?
Comment 2 Marcel Wiesweg 2012-07-16 20:31:51 UTC 
Maybe items are added to the current queue as long as this one is not started, and add a new queue when the current one is running?
Of course, there's still the actions to add the current queue or Add to new queue; it's just about "doing the right thing" per default.
Comment 3 Marcel Wiesweg 2012-07-16 20:42:27 UTC 
Git commit 801a15da71c25c0559cc052898cfea93767052e2 by Marcel Wiesweg.
Committed on 16/07/2012 at 22:41.
Pushed by mwiesweg into branch 'master'.

Reuse an unused, empty queue when adding first items.
TODO: Choosing proper default action, see bug report discussion

M  +2    -1    NEWS
M  +5    -3    digikam/items/imageviewutilities.cpp
M  +19   -0    utilities/queuemanager/main/queuemgrwindow.cpp
M  +2    -0    utilities/queuemanager/main/queuemgrwindow.h

http://commits.kde.org/digikam/801a15da71c25c0559cc052898cfea93767052e2