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 584923 - better metadata comment syncing
better metadata comment syncing
Status: RESOLVED OBSOLETE
Product: gthumb
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks: 583464
 
 
Reported: 2009-06-05 12:58 UTC by Michael Chudobiak
Modified: 2009-09-10 20:06 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Better metadata comment syncing (97.76 KB, patch)
2009-07-23 22:05 UTC, Marc Pavot 		none Details | Review

Description Michael Chudobiak 2009-06-05 12:58:14 UTC 
In trunk, the metadata comment / keyword tags are only read in when you click on the image.

We should do something more graceful, without slowing things down. The metadata comments should always be shown.

The XML files should probably store a timestamp showing when the metadata was last read from exiv2, and automatically read the metadata if the image file mtime is more recent.

- Mike
Comment 1 Marc Pavot 2009-06-08 18:01:26 UTC 
Hi Mike,

I totally agree with that. In addition to this, we should keep a cache XML file also for files where comments are empty to avoid useless reads.

I will try to do it asap.

Regards,
Marc
Comment 2 Michael Chudobiak 2009-06-16 19:19:19 UTC 
The XML files should probably also cache the exif datetime info, to speed up the sort-by-by-exif-datetime mode in the browser.

- Mike
Comment 3 Marc Pavot 2009-07-23 22:05:38 UTC 
Created attachment 139116 [details] [review]
Better metadata comment syncing

Hi,

This patch does what is described in this bug.
It also renames Comments files/methods to ImageMetada as it is now wider than comments.

It also simplify the inferface of metadata methods.

Marc
Comment 4 Michael Chudobiak 2009-09-10 20:06:47 UTC 
Closing as obsolete, due to the work in ext branch.