[Bug 118501] New: exifs lost when modifying the images

classic Classic list List threaded Threaded
31 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Bugzilla from ach@mpe.mpg.de
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From ach mpe mpg de  2006-02-08 01:31 -------
[ this comment was triggered by http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=349450 ]

Hmm, was this fixed after 0.8.1? There no note when/why
it was set resolved.  I can reproduce it with a write
protected image in 0.8.1:

cp jpeg-with-exif.jpg readonly.jpg
chmod 444 readonly.jpg

start digikam imageeditor with readonly.jpg
RMB -> rotate -> 90 degree
save and exit IE

Now EXIF tab of properties is empty for readonly.jpg :(

Doing the same with original jpeg-with-exif.jpg
and the EXIF tab is still there.


Achim
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Bugzilla from ach@mpe.mpg.de
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         
ach mpe mpg de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
         Resolution|WORKSFORME                  |



------- Additional Comments From ach mpe mpg de  2006-02-08 02:50 -------
I've build 0.8.2-svn from stable trunk and EXIF infos
are still deleted for a readonly picture.   Here the
output digikam writes:

digikam: Saving to :/home/ttt/Pictures/Test-Test-Test/HUK6qb.tmp (jpeg)
digikam: WARNING: Error in opening output file
digikam: WARNING: Failed to open file: /home/ttt/Pictures/Test-Test-Test/HUK6qb.tmp
digikam: Dirty: /Test-Test-Test
digikam: Found JPEGLossless plugin

Achim
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Volker Christian-3
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From volker.christian fh-hagenberg at  2006-02-28 20:28 -------
I create a jpeg by use of ufraw from a cr2-raw file, copy the exif information from the cr2 file to the jpeg file with

   exiftool -overwrite_original -tagsfromfile %f.cr2 -all:all -ext jpg

If i open the file in digikam and try to save it again, i get the output below.
Bad, because all exif-info is lost in the saved file.

I can provide you with example-files also. Just ask!

Please help!
Volker

digikam: /media/MOBILE_HD/Albums/Home/2006-02-27/img_6562.jpg : JPEG file identi
fied
digikam: Reading JPEG metadata: APP1 (size=16916)
digikam: Reading JPEG metadata: APP1 (size=2820)
digikam: mimetypes=image/x-portable-bitmap image/x-pcx image/x-portable-greymap
image/x-xbm image/x-targa image/png image/x-portable-pixmap image/jpeg image/x-x
pm image/x-eps image/x-bmp image/jp2 image/x-rgb image/tiff
digikam: mimetypes=image/x-portable-bitmap image/x-pcx image/x-portable-greymap
image/x-xbm image/x-targa image/png image/x-portable-pixmap image/jpeg image/x-x
pm image/x-eps image/x-bmp image/jp2 image/x-rgb image/tiff
digikam: Saving to :/media/MOBILE_HD/Albums/Home/2006-02-27/YxM1Qa.tmp (jpeg)
digikam: Dirty: /Home/2006-02-27
digikam: Writing JPEG metadata: APP1: DATA=[68 74 74 70 3a 2f 2f 6e 73 2e 61 64
6f 62 65 2e 63 6f 6d 2f 78 61 70 2f 31 2e 30 2f 00 3c 3f 78 70 61 63 6b 65 74 20
 62 65 67 69 6e 3d 27 ef bb bf 27 20 69 64 3d 27 57 35 4d 30 4d 70 43 65 68 ...]
digikam: Dirty: /
digikam: renaming to /media/MOBILE_HD/Albums/Home/2006-02-27/img_6562-2.jpg
Error: Makernote entry 1358 lies outside of the IFD memory buffer.
Warning: Failed to read Makernote, rc = 6
digikam: Dirty: /
digikam: Dirty: /Home/2006-02-27
digikam: Dirty: /
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Gilles Caulier
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From caulier.gilles free fr  2006-03-03 10:59 -------
SVN commit 515268 by cgilles:

digikam from trunk : Metadata support using Exiv2 :

- New image properties sidebar tab named "Metadata" instead old "Exif". This area include :

* Standard Exif tags viewer.
* MarkerNote Exif tags viewer.
* IPTC records viewer.

- New capability to copy metadata in clipboard like text.
- New capability to print metadata.
- Tags name use the "user Friendly" conversion from Exiv2 instead the internal name provided by old Exif viewer based on libkexif.
- Capability to read metadata from CRW files (Canon RAW files)
- New class DMetadata to load/save metadata without loading image data. Actually JPEG, CRW, and PNG files are supported. About PNG (Exif and IPTC raw profiles generated by ImageMagick are supported.

To support new file formats (like NEF, MRW, TIFF, DNG, etc), new image file parsers must be added to Exiv2 library.

IMPORTANT:

- Exiv2 do not support yet gettext for i18n rules. All informations in metadata viewers aren't yet i18n (tags name, tags values, and tags descriptions)
- Any tag names use internal Exiv2 name, not the user friendly text transformations. This point must be fixed in Exiv2 libs.

To 0.9.0, these tools are just metadata readers. Writting capabilities (metadata editors) will added later 0.9.0. The files in B.K.O directly or indirectly relevant of this commits are listed below :

*Pending:

103255
106103
115764
111560

*Partially fixed:

91812
96459
109253
110598
118501        

* To check before closing:

122264

* Fixed (can be closed):

103489
121371
105670
109319

CCMAIL: digikam-devel kde org,  Andreas Huggel <ahuggel gmx net>

CCBUGS: 103255, 106103, 115764, 111560, 91812, 96459, 109253, 110598, 118501, 122264, 103489, 121371, 105670, 109319


 M  +2 -1      libs/Makefile.am  
 M  +5 -4      libs/dimg/Makefile.am  
 M  +1 -1      libs/dimg/dimg.cpp  
 M  +1 -1      libs/dimg/dimg.h  
 M  +19 -1     libs/dimg/dimgloader.cpp  
 M  +4 -2      libs/dimg/dimgloader.h  
 M  +4 -61     libs/dimg/loaders/jpegloader.cpp  
 M  +5 -29     libs/dimg/loaders/pngloader.cpp  
 M  +2 -1      libs/dimg/loaders/rawloader.cpp  
 A             libs/dmetadata (directory)  
 A             libs/dmetadata/Makefile.am  
 A             libs/dmetadata/dmetadata.cpp   [License: GPL]
 A             libs/dmetadata/dmetadata.h   [License: GPL]
 A             libs/dmetadata/loaders (directory)  
 A             libs/dmetadata/loaders/Makefile.am  
 A             libs/dmetadata/loaders/dmetaloader.cpp   [License: GPL]
 A             libs/dmetadata/loaders/dmetaloader.h   [License: GPL]
 A             libs/dmetadata/loaders/jpegmetaloader.cpp   [License: GPL]
 A             libs/dmetadata/loaders/jpegmetaloader.h   [License: GPL]
 A             libs/dmetadata/loaders/pngmetaloader.cpp   [License: GPL]
 A             libs/dmetadata/loaders/pngmetaloader.h   [License: GPL]
 A             libs/dmetadata/loaders/rawmetaloader.cpp   [License: GPL]
 A             libs/dmetadata/loaders/rawmetaloader.h   [License: GPL]
 A             libs/dmetadata/loaders/tiffmetaloader.cpp   [License: GPL]
 A             libs/dmetadata/loaders/tiffmetaloader.h   [License: GPL]
 M  +3 -2      libs/imageproperties/Makefile.am  
 M  +82 -323   libs/imageproperties/imagepropertiesexiftab.cpp  
 M  +12 -25    libs/imageproperties/imagepropertiesexiftab.h  
 M  +19 -18    libs/imageproperties/imagepropertiessidebar.cpp  
 M  +10 -9     libs/imageproperties/imagepropertiessidebar.h  
 M  +27 -26    libs/imageproperties/imagepropertiessidebarcamgui.cpp  
 M  +2 -1      libs/imageproperties/imagepropertiessidebarcamgui.h  
 M  +16 -15    libs/imageproperties/imagepropertiessidebardb.cpp  
 M  +3 -1      libs/widgets/Makefile.am  
 A             libs/widgets/metadata (directory)  
 A             libs/widgets/metadata/Makefile.am  
 A             libs/widgets/metadata/exifwidget.cpp   [License: GPL]
 A             libs/widgets/metadata/exifwidget.h   [License: GPL]
 A             libs/widgets/metadata/iptcwidget.cpp   [License: GPL]
 A             libs/widgets/metadata/iptcwidget.h   [License: GPL]
 A             libs/widgets/metadata/makernotewidget.cpp   [License: GPL]
 A             libs/widgets/metadata/makernotewidget.h   [License: GPL]
 A             libs/widgets/metadata/mdkeylistviewitem.cpp   [License: GPL]
 A             libs/widgets/metadata/mdkeylistviewitem.h   [License: GPL]
 A             libs/widgets/metadata/metadatalistview.cpp   [License: GPL]
 A             libs/widgets/metadata/metadatalistview.h   [License: GPL]
 A             libs/widgets/metadata/metadatalistviewitem.cpp   [License: GPL]
 A             libs/widgets/metadata/metadatalistviewitem.h   [License: GPL]
 A             libs/widgets/metadata/metadatawidget.cpp   [License: GPL]
 A             libs/widgets/metadata/metadatawidget.h   [License: GPL]
 M  +24 -1     utilities/cameragui/cameraui.cpp
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Bugzilla from bugs.kde.org@nanonanonano.net
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From bugs.kde.org nanonanonano net  2006-04-01 00:19 -------
Much of this discussion has centred around read-only files, but I see this bug when I import pictures from the camera and let digikam auto-rotate them. When the images are brought in to my "Incoming" folder, portrait photos are automagically rotated and lose their EXIF data but landscape photos still have their EXIF data intact.

Unfortunately, I only just discovered it so my last few hundred portrait photos do not have any EXIF data in them, including the date they were taken :( (i.e. this bug means loss of data making it serious not just mildly annoying)

Good to see some progress noted against this bug, but it is unclear as to whether the svn trunk now has this fixed or not.

(digikam and plugins fom Debian etch 0.8.1-3)
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Gilles Caulier
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From caulier.gilles free fr  2006-04-03 14:05 -------
Tested indeep with current trunk implementation. The problem do not exist in image editor and camera gui.

No test yet done with current implemention of stable branch.  Please give me a feedback.

Gilles Caulier
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

John Green-4
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From john thegreens co uk  2006-04-24 21:42 -------
I also get this after moving to ubuntu dapper package 0.8.1-4ubuntu1. Auto-rotated pictures lose exif after import. This did not happen in the previous version (sorry I do not exactly but would have been with breezy KDE 3.5.2 )
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Bugzilla from ach@mpe.mpg.de
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         
ach mpe mpg de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ach mpe mpg de
             Status|UNCONFIRMED                 |NEW
      everconfirmed|0                           |1



------- Additional Comments From ach mpe mpg de  2006-04-27 00:54 -------
Confirmed with 0.8.2-beta1 on Kubuntu/Dapper _when_ the image is write
protected as given in debian bug:

 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=349450

Reproduce:

o chmod 444 image-with-exif-info.jpg
o load this image into IE
o Transform -> rotate 90 degree
o save it

==> rotated image is silently overriden, despite the 444 protection.
(silently == no err/info dialog, no output to console).   The exif info
is gone.

Achim
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Bugzilla from ach@mpe.mpg.de
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         




------- Additional Comments From ach mpe mpg de  2006-04-27 02:21 -------
#23 From Stuart Prescott
#23 From John Green

Hi Stuart, John

this is a similar but different problem.  See:

       http://bugs.kde.org/show_bug.cgi?id=126326
       [ https://launchpad.net/distros/ubuntu/+source/digikam/+bug/34462 ]

Achim
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Pierre Habouzit-4
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         
madcoder debian org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pkg-kde-bugs-
                   |                            |fwd lists alioth debian org
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 118501] exifs lost when modifying the images

Marcel Wiesweg
In reply to this post by jdd@dodin.org
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118501         
marcel.wiesweg gmx de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From marcel.wiesweg gmx de  2006-04-30 19:54 -------
SVN commit 535907 by mwiesweg:

digikam stable branch:
Add write permission when copying permissions of original file to temp file
so that std::ofstream of temp file can open the file.

BUG: 118501


 M  +4 -1      imlibinterface.cpp  


--- branches/stable/extragear/graphics/digikam/utilities/imageeditor/imlibinterface.cpp #535906:535907
 @ -1200,7 +1200,10  @
     // file saved. now preserve the permissions
     if (d->filePermissions != 0)
     {
-        ::chmod(QFile::encodeName(saveFile), d->filePermissions);
+        // Add the user write permission.
+        // There is a problem with lost Exif info on read only files,
+        // and there won't be sophisticated handling for this in the 0.8 branch any more.
+        ::chmod(QFile::encodeName(saveFile), d->filePermissions | S_IWRITE);
     }
 
     return true;
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
12