Bug 197641 - Can't delete exif x-default caption once set
Summary: Can't delete exif x-default caption once set
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Metadata-Exif (show other bugs)
Version: 1.0.0
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-23 19:49 UTC by Salvatore Brigaglia
Modified: 2017-08-13 07:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 1.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Salvatore Brigaglia 2009-06-23 19:49:08 UTC
Version:           1.0.0-beta2 (rev.: 984662) (using 4.2.90 (KDE 4.2.90 (KDE 4.3 Beta2)) "release 138", KDE:KDE4:Factory:Desktop / openSUSE_Factory)
Compiler:          gcc
OS:                Linux (i686) release 2.6.30-rc8-6-pae

Try setting a caption to an image. Then try to delete it whit the red "minus" above the caption textbox. It does not delete the text from the metadata of the image...
svn r985213
Comment 1 caulier.gilles 2009-06-23 21:32:36 UTC
Exif, Iptc and XMp comments tags are removed. What do you want to expect ?

Gilles Caulier
Comment 2 Andi Clemens 2009-06-23 21:50:18 UTC
Gilles,

this might be the same problem I had? You were not able to delete captions a few days ago?

Andi
Comment 3 Mikolaj Machowski 2009-06-23 21:50:51 UTC
Unfortunately has to confirm for rev. 984917.

After removing of captions with '-' sign comments are visible in metadata IPTC and XMP tabs.

Also visible in output of exiv2 -p {i,x} or exiftool.
Comment 4 caulier.gilles 2009-06-23 22:08:03 UTC
SVN commit 985963 by cgilles:

fix broken signal/slots order
CCBUGS: 197641


 M  +11 -6     captionsedit.cpp  
 M  +0 -7      imagedescedittab.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=985963
Comment 5 caulier.gilles 2009-06-23 22:08:43 UTC
Try revision #985963

Gilles Caulier
Comment 6 caulier.gilles 2009-06-24 06:10:39 UTC
I forget the most important : use libkexiv2 from trunk where several bugs are fixed...

Gilles
Comment 7 caulier.gilles 2009-06-24 20:46:17 UTC
Marcel,

Since your last changes in ImageComments class, it's impossible to assign x-default comment to an image. I'm sure to have tested this point before...

Gilles Caulier
Comment 8 Mikolaj Machowski 2009-06-24 22:08:14 UTC
For me it works in other way (rev.: 986298M):

I can assign x-default, it is written in digiKam caption, XMP, IPTC but not EXIF.

Also I can use second language but:
1) it is not saved with check in list of languages, cannot edit it later
2) it is saved to XMP
3) if I try third language, it replaces second language, not becomes third
Comment 9 Marcel Wiesweg 2009-06-25 19:50:12 UTC
SVN commit 987220 by mwiesweg:

Dont drop newly inserted comments...
CCBUG: 197641

 M  +1 -1      imagecomments.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=987220
Comment 10 Marcel Wiesweg 2009-06-25 20:11:05 UTC
There is still a problem, in my tests the map passed to MetadataHub::setComments contains ( "x-default" -> QString() ) instead of the value set. The empty comment is then properly stored in the db.
Comment 11 caulier.gilles 2009-06-25 23:36:53 UTC
SVN commit 987354 by cgilles:

unforget to read last captions list from editor when apply button is pressed
BUGS: 197641


 M  +1 -0      imagedescedittab.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=987354