Bug 300848 - "Reread Metadata From Image" is broken
Summary: "Reread Metadata From Image" is broken
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Maintenance-Metadata (show other bugs)
Version: 2.6.0
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-30 07:07 UTC by Frédéric COIFFIER
Modified: 2017-08-13 15:43 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.7.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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