Bug 89720

Summary:	batch convert from tif to jpg creates thumbnail and main jpeg with wrong names.
Product:	[Applications] digikam	Reporter:	Richard Taylor <rjt-kde-bugs>
Component:	Plugin-Bqm-Convert	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	caulier.gilles
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:	7.6.0
Sentry Crash Report:

Description Richard Taylor 2004-09-17 19:09:03 UTC

Version:           0.1.0-cvs (using KDE 3.3.0, Gentoo)
Compiler:          gcc version 3.3.3 20040412 (Gentoo Linux 3.3.3-r6, ssp-3.3.2-2, pie-8.7.6)
OS:                Linux (i686) release 2.6.8-gentoo-r3

When batch converting from tif images to jpeg images the conversion produces two jpeg files. filename.jpg.0 and filename.jpg.1. The .0 file is the correct image and the .1 file is a thumbnail. 

Because the new jpeg images do not have the correct final extension to the filename digikam does not show them, which is rather confusing.

I guess that the conversion is picking up the thumbnail that is embedded in the tif by the camera.

Comment 1 Richard Taylor 2004-09-21 23:52:11 UTC

The problem is because of the way that ImageMagik convert handles images with embedded thumbnails. The following simple patch fixes the problem for me. I guess that someone else should check that it does not break other peoples conversions.

diff -u -3 -p -r1.18 convertimagesdialog.cpp
--- kipi-plugins/batchprocessimages/convertimagesdialog.cpp     21 Sep 2004 10:33:15 -0000  1.18
+++ kipi-plugins/batchprocessimages/convertimagesdialog.cpp     21 Sep 2004 21:49:33 -0000
@@ -323,7 +323,7 @@ QString ConvertImagesDialog::makeProcess

     *proc << "-verbose";

-    *proc << item->pathSrc();
+    *proc << item->pathSrc() + "[0]";

     if ( !albumDest.isNull() )   // No preview mode !
        {


Richard

Comment 2 caulier.gilles 2004-09-22 10:43:14 UTC

This patch fix the problem. Thanks
I have also patched all makeProcess() methods for all others BatchProcessImages plugins.