Bug 300848

Summary:	"Reread Metadata From Image" is broken
Product:	[Applications] digikam	Reporter:	Frédéric COIFFIER <frederic.coiffier>
Component:	Maintenance-Metadata	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	caulier.gilles, frederic.coiffier, linux
Priority:	NOR
Version:	2.6.0
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed In:	2.7.0
Sentry Crash Report:

Description Frédéric COIFFIER 2012-05-30 07:07:27 UTC

The problem exists only in recent Git revisions (2.5.0 worked).
commit 597da399b75546f2c89726fdaa4d06fbd874c2cb

When EXIF fields of an image are modified by an external applications, they cannot be reloaded by the "Reread Metadata From Image" function.

Reproducible: Always

Steps to Reproduce:
1. Import an image in Digikam
2. Modify the EXIF with an external applications like :
exiftool -E -overwrite_original_in_place -UserComment=test -Title=test test.jpg
3. Verify the EXIF with exitool 

exiftool test.jpg | grep test
File Name                       : test.jpg
User Comment                    : test
Title                           : test
4. Reread metadata with Digikam

Actual Results:  
=> EXIF information aren't updated

Expected Results:  
EXIF information in Digikam should be updated

Comment 1 Michael Holtermann 2012-06-09 10:00:08 UTC

Same here with digiKam 2.6.0at  KDE 4.8.3 (Ubuntu 12.04, with PPA from philip5).

Today I've updated to new digikam from 2.5, and most of my Tags are missing. The database contains only 67 Tags from about 300 Tags before. I'll fill another issue for that.

But now, if I try to recreate the meta data, none of the options in the "Albums" and "Image" menus work.

BTW - with 2.6 there is the new dialog "Maintenance" that is able to rewrite tags to images. May I suggest to move the menu items that *reads* the metadata to the database to that dialog, too? IMHO this would be more consistent.

Regards,
Michael

Comment 2 Johannes Wienke 2012-06-24 18:59:02 UTC

I can also confirm this at 2.6.

Comment 3 Frédéric COIFFIER 2012-06-28 09:30:35 UTC

Patch like this seems to correct the problem :


diff --git a/digikam/views/digikamview.cpp b/digikam/views/digikamview.cpp
index e461f99..b0b9bad 100644
--- a/digikam/views/digikamview.cpp
+++ b/digikam/views/digikamview.cpp
@@ -1352,13 +1352,15 @@ void DigikamView::slotAlbumReadMetadata()
 void DigikamView::slotImageWriteMetadata()
 {
     ImageInfoList selected = d->iconView->selectedImageInfos();
-    new MetadataSynchronizer(selected, MetadataSynchronizer::WriteFromDatabaseToFile);
+    MetadataSynchronizer* tool = new MetadataSynchronizer(selected, MetadataSynchronizer::WriteFromDatabaseToFile);
+    tool->start();
 }
 
 void DigikamView::slotImageReadMetadata()
 {
     ImageInfoList selected = d->iconView->selectedImageInfos();
-    new MetadataSynchronizer(selected, MetadataSynchronizer::ReadFromFileToDatabase);
+    MetadataSynchronizer* tool = new MetadataSynchronizer(selected, MetadataSynchronizer::ReadFromFileToDatabase);
+    tool->start();
 }
 
 // ----------------------------------------------------------------


Is there a core developper to confirm ?

Comment 4 caulier.gilles 2012-06-28 09:40:42 UTC

Fixed in git master with commit :

http://commits.kde.org/digikam/1feef3d7a7f93a0cc921e9caa7c0fee1f37e6143

Gilles Caulier