After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 505125 - Rotating a picture rotates all versions of that picture
Rotating a picture rotates all versions of that picture
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: Editing
0.4.x
Other All
: Normal minor
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2007-12-22 22:30 UTC by Niclas Andersson
Modified: 2008-03-21 09:23 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
rotate-only-default-version.patch (1.26 KB, patch)
2007-12-23 09:56 UTC, Maxxer 		none Details | Review
rotate-only-default-version.patch (1.48 KB, patch)
2007-12-23 10:45 UTC, Maxxer 		committed Details | Review

Description Niclas Andersson 2007-12-22 22:30:16 UTC 
When rotating a picture, all versions of the picture are rotated.

To reproduce:
 1) Create a new Version of a picture.
 2) Somehow rotate the picture.
 3) Notice that all versions (both the newly created version and the original file) gets rotated and new "last modified" timestamps on disk.

Expected beahaviour:
 Only the currently selected version should be rotated. The other versions should not be touched at all.



Other information:
Running F-Spot 0.4.0 on Ubuntu Gutsy. No "copy on import" activated.
Thanks!
Comment 1 Maxxer 2007-12-23 09:56:52 UTC 
Created attachment 101493 [details] [review]
rotate-only-default-version.patch

quick fix for that.
Comment 2 Maxxer 2007-12-23 10:45:31 UTC 
Created attachment 101494 [details] [review]
rotate-only-default-version.patch

simplified a bit
Comment 3 Niclas Andersson 2007-12-23 16:02:02 UTC 
I've applied the second patch to the Ubuntu 7.10 package (F-Spot version 0.4.0), and it works for me. Only the currently selected version gets rotated for me with that patch applied. Thanks!
Comment 4 Stephane Delcroix 2008-03-21 09:23:40 UTC 
fixed in r3773